Skip to contentSkip to conversation

U-Boot PXA

GIT PULL Marek Vasut 2 participants 5 messages Aug 28, 2011

lore
Share & tools
lore
#0 lore raw

Dear Albert ARIBAUD,

I prepared a branch for you to pull containing PXA fixes:

Cheers

The following changes since commit 6e8a3951f7656958cefccab55945604a8bbc43e7:

Merge branch 'master' of git://git.denx.de/u-boot (2011-08-28 01:36:27 +0200)

are available in the git repository at:

git://git.denx.de/u-boot-pxa.git for-arm-fix

Marek Vasut (3):
PXA: Fix Lubbock, remove redundant parenthesis
PXA: Fix CSB226, fix monitor length
PXA: vpac270: Remove re-defined CONFIG_SYS_TEXT_BASE

 board/lubbock/flash.c     |    2 +-
 include/configs/csb226.h  |    2 +-
 include/configs/vpac270.h |    1 -
 3 files changed, 2 insertions(+), 3 deletions(-)
#1 lore raw

Dear Marek Vasut,

Quoted 9 lines · In message <[email protected]> you wrote:

I prepared a branch for you to pull containing PXA fixes:

Cheers

The following changes since commit 6e8a3951f7656958cefccab55945604a8bbc43e7:

Merge branch 'master' of git://git.denx.de/u-boot (2011-08-28 01:36:27 +0200)

Quoted 8 lines

are available in the git repository at:

git://git.denx.de/u-boot-pxa.git for-arm-fix

Marek Vasut (3):
PXA: Fix Lubbock, remove redundant parenthesis
PXA: Fix CSB226, fix monitor length
PXA: vpac270: Remove re-defined CONFIG_SYS_TEXT_BASE

Again, what a mess... Why are you selecting just 3 commits from a 6 patches series? This makes no sense.

Either these are separate, independent patches, then submit separately. Or they form a series, with patches depending on each other, then obviously you want to include ALL of them? Ok, # 3 and 4 are for other custodians, but there is still # 6 missing here?

Best regards,

Wolfgang Denk

Signature · 8 lines
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
A Stanford research group advertised for participants in a  study  of
obsessive-compulsive  disorder. They were looking for therapy clients
who had been diagnosed with this disorder. The  response  was  grati-
fying;  they  got  3,000 responses about three days after the ad came
out. All from the same person.
#2 lore raw
Quoted 14 lines · Wolfgang Denk

Dear Marek Vasut,

Quoted 11 lines · In message <[email protected]> you wrote:

I prepared a branch for you to pull containing PXA fixes:

Cheers

The following changes since commit 6e8a3951f7656958cefccab55945604a8bbc43e7:
Merge branch 'master' of git://git.denx.de/u-boot (2011-08-28 01:36:27 +0200)

are available in the git repository at:
git://git.denx.de/u-boot-pxa.git for-arm-fix

Quoted 8 lines
Quoted 4 lines

Marek Vasut (3):
PXA: Fix Lubbock, remove redundant parenthesis
PXA: Fix CSB226, fix monitor length
PXA: vpac270: Remove re-defined CONFIG_SYS_TEXT_BASE

Again, what a mess... Why are you selecting just 3 commits from a 6 patches series? This makes no sense.

Because Albert wanted me to send a pull rq with the PXA fixes. I don't know if he hand-picked them already or not, but if not, here's a pull request.

Quoted 9 lines

Either these are separate, independent patches, then submit separately. Or they form a series, with patches depending on each other, then obviously you want to include ALL of them? Ok, # 3 and 4 are for other custodians, but there is still # 6 missing here?

Best regards,

Wolfgang Denk

Cheers!

#3 lore raw

Dear Marek Vasut,

Quoted 6 lines · In message <[email protected]> you wrote:
Quoted 4 lines

Again, what a mess... Why are you selecting just 3 commits from a 6 patches series? This makes no sense.

Because Albert wanted me to send a pull rq with the PXA fixes. I don't know if he hand-picked them already or not, but if not, here's a pull request.

I think you did not understand what I mean.

My question was why (1) you send a patch series which is obviously not really a set of patches that depend on each other (these should be submitted as separate patches then), and (2) why you did not inclyde the 4th PXA related patch into the pull request.

Best regards,

Wolfgang Denk

Signature · 5 lines
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Artificial Intelligence is the study of how to  make  real  computers
act like the ones in movies.
#4 lore raw
Quoted 10 lines · Wolfgang Denk

Dear Marek Vasut,

Quoted 6 lines · In message <[email protected]> you wrote:
Quoted 2 lines

Again, what a mess... Why are you selecting just 3 commits from a 6 patches series? This makes no sense.

Because Albert wanted me to send a pull rq with the PXA fixes. I don't know if he hand-picked them already or not, but if not, here's a pull request.

Quoted 5 lines

I think you did not understand what I mean.

My question was why (1) you send a patch series which is obviously not really a set of patches that depend on each other (these should be submitted as separate patches then), and

Ok, I get what you mean.

Quoted 2 lines

(2) why you did not
inclyde the 4th PXA related patch into the pull request.

Because that's really something I want to be tested properly. I want to avoid breakage.

Cheers

Quoted 4 lines

Best regards,

Wolfgang Denk