* [U-Boot] [U-boot] Marvell Pull Request
@ 2009-09-21 17:14 Prafulla Wadaskar
2009-09-21 19:15 ` Tom
0 siblings, 1 reply; 12+ messages in thread
From: Prafulla Wadaskar @ 2009-09-21 17:14 UTC (permalink / raw)
To: u-boot
Hi Wolfgang
Please pull
The following changes since commit 3b6a9267f0de7b85d387fa4123d0b58379363447:
Wolfgang Denk (1):
board/flagadm/flash.c: fix compile warning
are available in the git repository at:
git://git.denx.de/u-boot-marvell.git master
Prafulla Wadaskar (2):
Kirkwood: rd6281a: Add kwbimage build support
Kirkwood: mv88f6281gtw_ge: Add kwbimage build support
Simon Kagstrom (1):
Support for the OpenRD base board
MAINTAINERS | 4 +
MAKEALL | 1 +
Makefile | 3 +
board/Marvell/mv88f6281gtw_ge/config.mk | 3 +
board/Marvell/mv88f6281gtw_ge/kwbimage.cfg | 165 +++++++++++++++++++++
board/Marvell/openrd_base/Makefile | 56 +++++++
board/Marvell/openrd_base/config.mk | 33 ++++
board/Marvell/openrd_base/kwbimage.cfg | 168 +++++++++++++++++++++
board/Marvell/openrd_base/openrd_base.c | 160 ++++++++++++++++++++
board/Marvell/openrd_base/openrd_base.h | 46 ++++++
board/Marvell/rd6281a/config.mk | 3 +
board/Marvell/rd6281a/kwbimage.cfg | 167 +++++++++++++++++++++
include/configs/openrd_base.h | 220 ++++++++++++++++++++++++++++
13 files changed, 1029 insertions(+), 0 deletions(-)
create mode 100644 board/Marvell/mv88f6281gtw_ge/kwbimage.cfg
create mode 100644 board/Marvell/openrd_base/Makefile
create mode 100644 board/Marvell/openrd_base/config.mk
create mode 100644 board/Marvell/openrd_base/kwbimage.cfg
create mode 100644 board/Marvell/openrd_base/openrd_base.c
create mode 100644 board/Marvell/openrd_base/openrd_base.h
create mode 100644 board/Marvell/rd6281a/kwbimage.cfg
create mode 100644 include/configs/openrd_base.h
Regards..
Prafulla . . .
^ permalink raw reply [flat|nested] 12+ messages in thread* [U-Boot] [U-boot] Marvell Pull Request
2009-09-21 17:14 [U-Boot] [U-boot] Marvell Pull Request Prafulla Wadaskar
@ 2009-09-21 19:15 ` Tom
2009-09-22 9:28 ` Prafulla Wadaskar
0 siblings, 1 reply; 12+ messages in thread
From: Tom @ 2009-09-21 19:15 UTC (permalink / raw)
To: u-boot
Prafulla Wadaskar wrote:
> Hi Wolfgang
>
> Please pull
Shouldn't this rather get pulled to u-boot-arm?
Tom
>
> The following changes since commit 3b6a9267f0de7b85d387fa4123d0b58379363447:
> Wolfgang Denk (1):
> board/flagadm/flash.c: fix compile warning
>
> are available in the git repository at:
>
> git://git.denx.de/u-boot-marvell.git master
>
> Prafulla Wadaskar (2):
> Kirkwood: rd6281a: Add kwbimage build support
> Kirkwood: mv88f6281gtw_ge: Add kwbimage build support
>
> Simon Kagstrom (1):
> Support for the OpenRD base board
>
> MAINTAINERS | 4 +
> MAKEALL | 1 +
> Makefile | 3 +
> board/Marvell/mv88f6281gtw_ge/config.mk | 3 +
> board/Marvell/mv88f6281gtw_ge/kwbimage.cfg | 165 +++++++++++++++++++++
> board/Marvell/openrd_base/Makefile | 56 +++++++
> board/Marvell/openrd_base/config.mk | 33 ++++
> board/Marvell/openrd_base/kwbimage.cfg | 168 +++++++++++++++++++++
> board/Marvell/openrd_base/openrd_base.c | 160 ++++++++++++++++++++
> board/Marvell/openrd_base/openrd_base.h | 46 ++++++
> board/Marvell/rd6281a/config.mk | 3 +
> board/Marvell/rd6281a/kwbimage.cfg | 167 +++++++++++++++++++++
> include/configs/openrd_base.h | 220 ++++++++++++++++++++++++++++
> 13 files changed, 1029 insertions(+), 0 deletions(-)
> create mode 100644 board/Marvell/mv88f6281gtw_ge/kwbimage.cfg
> create mode 100644 board/Marvell/openrd_base/Makefile
> create mode 100644 board/Marvell/openrd_base/config.mk
> create mode 100644 board/Marvell/openrd_base/kwbimage.cfg
> create mode 100644 board/Marvell/openrd_base/openrd_base.c
> create mode 100644 board/Marvell/openrd_base/openrd_base.h
> create mode 100644 board/Marvell/rd6281a/kwbimage.cfg
> create mode 100644 include/configs/openrd_base.h
>
> Regards..
> Prafulla . . .
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
^ permalink raw reply [flat|nested] 12+ messages in thread
* [U-Boot] [U-boot] Marvell Pull Request
2009-09-21 19:15 ` Tom
@ 2009-09-22 9:28 ` Prafulla Wadaskar
2009-09-22 13:20 ` Tom
2009-09-27 23:51 ` Tom
0 siblings, 2 replies; 12+ messages in thread
From: Prafulla Wadaskar @ 2009-09-22 9:28 UTC (permalink / raw)
To: u-boot
> -----Original Message-----
> From: Tom [mailto:Tom.Rix at windriver.com]
> Sent: Tuesday, September 22, 2009 12:46 AM
> To: Prafulla Wadaskar
> Cc: U-Boot
> Subject: Re: [U-Boot] [U-boot] Marvell Pull Request
>
> Prafulla Wadaskar wrote:
> > Hi Wolfgang
> >
> > Please pull
>
> Shouldn't this rather get pulled to u-boot-arm?
> Tom
Hi Tom,
Yes :-) you are right.
I was bit biased since u-boot-marvell.git was cloned from u-boot.git
Shall I generate new email pull request? Or
Can you please kindly pull it to u-boot-arm?
Regards..
Prafulla . .
>
>
> >
> > The following changes since commit
> 3b6a9267f0de7b85d387fa4123d0b58379363447:
> > Wolfgang Denk (1):
> > board/flagadm/flash.c: fix compile warning
> >
> > are available in the git repository at:
> >
> > git://git.denx.de/u-boot-marvell.git master
> >
> > Prafulla Wadaskar (2):
> > Kirkwood: rd6281a: Add kwbimage build support
> > Kirkwood: mv88f6281gtw_ge: Add kwbimage build support
> >
> > Simon Kagstrom (1):
> > Support for the OpenRD base board
> >
> > MAINTAINERS | 4 +
> > MAKEALL | 1 +
> > Makefile | 3 +
> > board/Marvell/mv88f6281gtw_ge/config.mk | 3 +
> > board/Marvell/mv88f6281gtw_ge/kwbimage.cfg | 165
> +++++++++++++++++++++
> > board/Marvell/openrd_base/Makefile | 56 +++++++
> > board/Marvell/openrd_base/config.mk | 33 ++++
> > board/Marvell/openrd_base/kwbimage.cfg | 168
> +++++++++++++++++++++
> > board/Marvell/openrd_base/openrd_base.c | 160
> ++++++++++++++++++++
> > board/Marvell/openrd_base/openrd_base.h | 46 ++++++
> > board/Marvell/rd6281a/config.mk | 3 +
> > board/Marvell/rd6281a/kwbimage.cfg | 167
> +++++++++++++++++++++
> > include/configs/openrd_base.h | 220
> ++++++++++++++++++++++++++++
> > 13 files changed, 1029 insertions(+), 0 deletions(-)
> > create mode 100644 board/Marvell/mv88f6281gtw_ge/kwbimage.cfg
> > create mode 100644 board/Marvell/openrd_base/Makefile
> > create mode 100644 board/Marvell/openrd_base/config.mk
> > create mode 100644 board/Marvell/openrd_base/kwbimage.cfg
> > create mode 100644 board/Marvell/openrd_base/openrd_base.c
> > create mode 100644 board/Marvell/openrd_base/openrd_base.h
> > create mode 100644 board/Marvell/rd6281a/kwbimage.cfg
> > create mode 100644 include/configs/openrd_base.h
> >
> > Regards..
> > Prafulla . . .
> > _______________________________________________
> > U-Boot mailing list
> > U-Boot at lists.denx.de
> > http://lists.denx.de/mailman/listinfo/u-boot
>
>
^ permalink raw reply [flat|nested] 12+ messages in thread
* [U-Boot] [U-boot] Marvell Pull Request
2009-09-22 9:28 ` Prafulla Wadaskar
@ 2009-09-22 13:20 ` Tom
2009-09-23 6:13 ` Prafulla Wadaskar
2009-09-23 22:12 ` Wolfgang Denk
2009-09-27 23:51 ` Tom
1 sibling, 2 replies; 12+ messages in thread
From: Tom @ 2009-09-22 13:20 UTC (permalink / raw)
To: u-boot
Prafulla Wadaskar wrote:
>
>
>> -----Original Message-----
>> From: Tom [mailto:Tom.Rix at windriver.com]
>> Sent: Tuesday, September 22, 2009 12:46 AM
>> To: Prafulla Wadaskar
>> Cc: U-Boot
>> Subject: Re: [U-Boot] [U-boot] Marvell Pull Request
>>
>> Prafulla Wadaskar wrote:
>>> Hi Wolfgang
>>>
>>> Please pull
>> Shouldn't this rather get pulled to u-boot-arm?
>> Tom
>
> Hi Tom,
> Yes :-) you are right.
> I was bit biased since u-boot-marvell.git was cloned from u-boot.git
> Shall I generate new email pull request? Or
> Can you please kindly pull it to u-boot-arm?
I would like to set up a process where I just cherry picked your changes
directly from marvell/master.
Comparing marvell/master to arm/next
git cherry t-next
+ ceb2d57c2205db5bbd868577f756c74a2568160c
+ 95a4a593b577b6e2f1da2d4b0f5ec86975c33413
+ 84a45d33c2cc261dbd5411f7c2ad45f6003025b6
+ e67af44d0167d8237dd2c2ddf8e301d19ca12914
+ 0413cfecea350000eab5e591a0965c3e3ee0ff00
+ 3b6a9267f0de7b85d387fa4123d0b58379363447
+ ffed6c85ce591cd93e28190153fec421c3b2499c <- pick me
+ c928c19b2563e4977adc0570d06c900ac1e7788c <- pick me
+ 9ef0569a3c8b0d2458bdbce21a5370807e13e20a <- pick me
The others look like they came from u-boot/master
It would be helpful but not required if you merged against arm/master or
arm/next instead of u-boot/master.
If I can assume what is good in marvell/master, we do not need
for you go through the asking for a pulls, they will just happen
when I have time, likely over the weekend.
These changes will accumulate in arm/next.
Less frequently I will merge these to arm/master.
This will happen once you and the other arm maintainers have
given then the ok. I was thinking that this could happen monthly
but I am open for other suggestions.
Tom
>
> Regards..
> Prafulla . .
>
>>
>>> The following changes since commit
>> 3b6a9267f0de7b85d387fa4123d0b58379363447:
>>> Wolfgang Denk (1):
>>> board/flagadm/flash.c: fix compile warning
>>>
>>> are available in the git repository at:
>>>
>>> git://git.denx.de/u-boot-marvell.git master
>>>
>>> Prafulla Wadaskar (2):
>>> Kirkwood: rd6281a: Add kwbimage build support
>>> Kirkwood: mv88f6281gtw_ge: Add kwbimage build support
>>>
>>> Simon Kagstrom (1):
>>> Support for the OpenRD base board
>>>
>>> MAINTAINERS | 4 +
>>> MAKEALL | 1 +
>>> Makefile | 3 +
>>> board/Marvell/mv88f6281gtw_ge/config.mk | 3 +
>>> board/Marvell/mv88f6281gtw_ge/kwbimage.cfg | 165
>> +++++++++++++++++++++
>>> board/Marvell/openrd_base/Makefile | 56 +++++++
>>> board/Marvell/openrd_base/config.mk | 33 ++++
>>> board/Marvell/openrd_base/kwbimage.cfg | 168
>> +++++++++++++++++++++
>>> board/Marvell/openrd_base/openrd_base.c | 160
>> ++++++++++++++++++++
>>> board/Marvell/openrd_base/openrd_base.h | 46 ++++++
>>> board/Marvell/rd6281a/config.mk | 3 +
>>> board/Marvell/rd6281a/kwbimage.cfg | 167
>> +++++++++++++++++++++
>>> include/configs/openrd_base.h | 220
>> ++++++++++++++++++++++++++++
>>> 13 files changed, 1029 insertions(+), 0 deletions(-)
>>> create mode 100644 board/Marvell/mv88f6281gtw_ge/kwbimage.cfg
>>> create mode 100644 board/Marvell/openrd_base/Makefile
>>> create mode 100644 board/Marvell/openrd_base/config.mk
>>> create mode 100644 board/Marvell/openrd_base/kwbimage.cfg
>>> create mode 100644 board/Marvell/openrd_base/openrd_base.c
>>> create mode 100644 board/Marvell/openrd_base/openrd_base.h
>>> create mode 100644 board/Marvell/rd6281a/kwbimage.cfg
>>> create mode 100644 include/configs/openrd_base.h
>>>
>>> Regards..
>>> Prafulla . . .
>>> _______________________________________________
>>> U-Boot mailing list
>>> U-Boot at lists.denx.de
>>> http://lists.denx.de/mailman/listinfo/u-boot
^ permalink raw reply [flat|nested] 12+ messages in thread
* [U-Boot] [U-boot] Marvell Pull Request
2009-09-22 13:20 ` Tom
@ 2009-09-23 6:13 ` Prafulla Wadaskar
2009-09-23 11:23 ` Tom
2009-09-23 22:12 ` Wolfgang Denk
1 sibling, 1 reply; 12+ messages in thread
From: Prafulla Wadaskar @ 2009-09-23 6:13 UTC (permalink / raw)
To: u-boot
> -----Original Message-----
> From: Tom [mailto:Tom.Rix at windriver.com]
> Sent: Tuesday, September 22, 2009 6:50 PM
> To: Prafulla Wadaskar
> Cc: U-Boot; Paulraj, Sandeep; Minkyu Kang; Wolfgang Denk
> Subject: Re: [U-Boot] [U-boot] Marvell Pull Request
>
> Prafulla Wadaskar wrote:
> >
> >
> >> -----Original Message-----
> >> From: Tom [mailto:Tom.Rix at windriver.com]
> >> Sent: Tuesday, September 22, 2009 12:46 AM
> >> To: Prafulla Wadaskar
> >> Cc: U-Boot
> >> Subject: Re: [U-Boot] [U-boot] Marvell Pull Request
> >>
> >> Prafulla Wadaskar wrote:
> >>> Hi Wolfgang
> >>>
> >>> Please pull
> >> Shouldn't this rather get pulled to u-boot-arm?
> >> Tom
> >
> > Hi Tom,
> > Yes :-) you are right.
> > I was bit biased since u-boot-marvell.git was cloned from u-boot.git
> > Shall I generate new email pull request? Or
> > Can you please kindly pull it to u-boot-arm?
>
> I would like to set up a process where I just cherry picked
> your changes
> directly from marvell/master.
>
> Comparing marvell/master to arm/next
>
> git cherry t-next
> + ceb2d57c2205db5bbd868577f756c74a2568160c
> + 95a4a593b577b6e2f1da2d4b0f5ec86975c33413
> + 84a45d33c2cc261dbd5411f7c2ad45f6003025b6
> + e67af44d0167d8237dd2c2ddf8e301d19ca12914
> + 0413cfecea350000eab5e591a0965c3e3ee0ff00
> + 3b6a9267f0de7b85d387fa4123d0b58379363447
> + ffed6c85ce591cd93e28190153fec421c3b2499c <- pick me
> + c928c19b2563e4977adc0570d06c900ac1e7788c <- pick me
> + 9ef0569a3c8b0d2458bdbce21a5370807e13e20a <- pick me
>
> The others look like they came from u-boot/master
>
> It would be helpful but not required if you merged against
> arm/master or
> arm/next instead of u-boot/master.
I think this makes more sense to merge it from arm/master, I will do that, I am thinking of creating a branch "arm" instead of disturbing master, what do you think?
>
> If I can assume what is good in marvell/master, we do not need
> for you go through the asking for a pulls, they will just happen
> when I have time, likely over the weekend.
If I have clean changes in lined with arm/master, then why not to pull?
>
> These changes will accumulate in arm/next.
If this is clear from you that Changes will be accumulated in arm/next then I must rebase a branch on u-boot-marvell.git with arm/next.
> Less frequently I will merge these to arm/master.
> This will happen once you and the other arm maintainers have
> given then the ok. I was thinking that this could happen monthly
> but I am open for other suggestions.
I think monthly will be too long, may be wolfgang can comment better on this :-)
Regards..
Prafulla . .
>
> Tom
>
> >
> > Regards..
> > Prafulla . .
> >
> >>
> >>> The following changes since commit
> >> 3b6a9267f0de7b85d387fa4123d0b58379363447:
> >>> Wolfgang Denk (1):
> >>> board/flagadm/flash.c: fix compile warning
> >>>
> >>> are available in the git repository at:
> >>>
> >>> git://git.denx.de/u-boot-marvell.git master
> >>>
> >>> Prafulla Wadaskar (2):
> >>> Kirkwood: rd6281a: Add kwbimage build support
> >>> Kirkwood: mv88f6281gtw_ge: Add kwbimage build support
> >>>
> >>> Simon Kagstrom (1):
> >>> Support for the OpenRD base board
> >>>
> >>> MAINTAINERS | 4 +
> >>> MAKEALL | 1 +
> >>> Makefile | 3 +
> >>> board/Marvell/mv88f6281gtw_ge/config.mk | 3 +
> >>> board/Marvell/mv88f6281gtw_ge/kwbimage.cfg | 165
> >> +++++++++++++++++++++
> >>> board/Marvell/openrd_base/Makefile | 56 +++++++
> >>> board/Marvell/openrd_base/config.mk | 33 ++++
> >>> board/Marvell/openrd_base/kwbimage.cfg | 168
> >> +++++++++++++++++++++
> >>> board/Marvell/openrd_base/openrd_base.c | 160
> >> ++++++++++++++++++++
> >>> board/Marvell/openrd_base/openrd_base.h | 46 ++++++
> >>> board/Marvell/rd6281a/config.mk | 3 +
> >>> board/Marvell/rd6281a/kwbimage.cfg | 167
> >> +++++++++++++++++++++
> >>> include/configs/openrd_base.h | 220
> >> ++++++++++++++++++++++++++++
> >>> 13 files changed, 1029 insertions(+), 0 deletions(-)
> >>> create mode 100644 board/Marvell/mv88f6281gtw_ge/kwbimage.cfg
> >>> create mode 100644 board/Marvell/openrd_base/Makefile
> >>> create mode 100644 board/Marvell/openrd_base/config.mk
> >>> create mode 100644 board/Marvell/openrd_base/kwbimage.cfg
> >>> create mode 100644 board/Marvell/openrd_base/openrd_base.c
> >>> create mode 100644 board/Marvell/openrd_base/openrd_base.h
> >>> create mode 100644 board/Marvell/rd6281a/kwbimage.cfg
> >>> create mode 100644 include/configs/openrd_base.h
> >>>
> >>> Regards..
> >>> Prafulla . . .
> >>> _______________________________________________
> >>> U-Boot mailing list
> >>> U-Boot at lists.denx.de
> >>> http://lists.denx.de/mailman/listinfo/u-boot
>
>
^ permalink raw reply [flat|nested] 12+ messages in thread
* [U-Boot] [U-boot] Marvell Pull Request
2009-09-23 6:13 ` Prafulla Wadaskar
@ 2009-09-23 11:23 ` Tom
0 siblings, 0 replies; 12+ messages in thread
From: Tom @ 2009-09-23 11:23 UTC (permalink / raw)
To: u-boot
Prafulla Wadaskar wrote:
>
>
>> -----Original Message-----
>> From: Tom [mailto:Tom.Rix at windriver.com]
>> Sent: Tuesday, September 22, 2009 6:50 PM
>> To: Prafulla Wadaskar
>> Cc: U-Boot; Paulraj, Sandeep; Minkyu Kang; Wolfgang Denk
>> Subject: Re: [U-Boot] [U-boot] Marvell Pull Request
>>
>> Prafulla Wadaskar wrote:
>>>
>>>
>>>> -----Original Message-----
>>>> From: Tom [mailto:Tom.Rix at windriver.com]
>>>> Sent: Tuesday, September 22, 2009 12:46 AM
>>>> To: Prafulla Wadaskar
>>>> Cc: U-Boot
>>>> Subject: Re: [U-Boot] [U-boot] Marvell Pull Request
>>>>
>>>> Prafulla Wadaskar wrote:
>>>>> Hi Wolfgang
>>>>>
>>>>> Please pull
>>>> Shouldn't this rather get pulled to u-boot-arm?
>>>> Tom
>>> Hi Tom,
>>> Yes :-) you are right.
>>> I was bit biased since u-boot-marvell.git was cloned from u-boot.git
>>> Shall I generate new email pull request? Or
>>> Can you please kindly pull it to u-boot-arm?
>> I would like to set up a process where I just cherry picked
>> your changes
>> directly from marvell/master.
>>
>> Comparing marvell/master to arm/next
>>
>> git cherry t-next
>> + ceb2d57c2205db5bbd868577f756c74a2568160c
>> + 95a4a593b577b6e2f1da2d4b0f5ec86975c33413
>> + 84a45d33c2cc261dbd5411f7c2ad45f6003025b6
>> + e67af44d0167d8237dd2c2ddf8e301d19ca12914
>> + 0413cfecea350000eab5e591a0965c3e3ee0ff00
>> + 3b6a9267f0de7b85d387fa4123d0b58379363447
>> + ffed6c85ce591cd93e28190153fec421c3b2499c <- pick me
>> + c928c19b2563e4977adc0570d06c900ac1e7788c <- pick me
>> + 9ef0569a3c8b0d2458bdbce21a5370807e13e20a <- pick me
>>
>> The others look like they came from u-boot/master
>>
>> It would be helpful but not required if you merged against
>> arm/master or
>> arm/next instead of u-boot/master.
>
> I think this makes more sense to merge it from arm/master, I will do that, I am thinking of creating a branch "arm" instead of disturbing master, what do you think?
I think keeping the branches simple
'master', 'next' and optionally 'testing' is better.
Locally you can do whatever you want.
I make a compound mirror of u-boot, arm and all the vendor arms to
help me navigate the cherry picking.
>
>> If I can assume what is good in marvell/master, we do not need
>> for you go through the asking for a pulls, they will just happen
>> when I have time, likely over the weekend.
>
> If I have clean changes in lined with arm/master, then why not to pull?
>
Yes that would be ideal.
I like to look over the changes before I merge them in so
it is not going to be that automatic.
>> These changes will accumulate in arm/next.
>
> If this is clear from you that Changes will be accumulated in arm/next then I must rebase a branch on u-boot-marvell.git with arm/next.
>
>> Less frequently I will merge these to arm/master.
>> This will happen once you and the other arm maintainers have
>> given then the ok. I was thinking that this could happen monthly
>> but I am open for other suggestions.
>
> I think monthly will be too long, may be wolfgang can comment better on this :-)
Yup
Tom
^ permalink raw reply [flat|nested] 12+ messages in thread
* [U-Boot] [U-boot] Marvell Pull Request
2009-09-22 13:20 ` Tom
2009-09-23 6:13 ` Prafulla Wadaskar
@ 2009-09-23 22:12 ` Wolfgang Denk
2009-09-24 1:07 ` Tom
1 sibling, 1 reply; 12+ messages in thread
From: Wolfgang Denk @ 2009-09-23 22:12 UTC (permalink / raw)
To: u-boot
Dear Tom,
In message <4AB8CF0B.5070404@windriver.com> you wrote:
>
> I would like to set up a process where I just cherry picked your changes
> directly from marvell/master.
Please pull in the normal case, don't cherry-pick unless really necessary.
> If I can assume what is good in marvell/master, we do not need
> for you go through the asking for a pulls, they will just happen
> when I have time, likely over the weekend.
No, please don't automatize this.
Prafulla, please continue to send pullr equests to Tom, as this is
important information for me, too.
Best regards,
Wolfgang Denk
--
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
Looks clean and obviously correct to me, but then _everything_ I
write always looks obviously correct to me. - Linus Torvalds in
<Pine.LNX.4.10.10012090054360.791-100000@penguin.transmeta.com>
^ permalink raw reply [flat|nested] 12+ messages in thread
* [U-Boot] [U-boot] Marvell Pull Request
2009-09-23 22:12 ` Wolfgang Denk
@ 2009-09-24 1:07 ` Tom
2009-09-24 10:11 ` Wolfgang Denk
0 siblings, 1 reply; 12+ messages in thread
From: Tom @ 2009-09-24 1:07 UTC (permalink / raw)
To: u-boot
Wolfgang Denk wrote:
> Dear Tom,
>
> In message <4AB8CF0B.5070404@windriver.com> you wrote:
>> I would like to set up a process where I just cherry picked your changes
>> directly from marvell/master.
>
> Please pull in the normal case, don't cherry-pick unless really necessary.
>
>> If I can assume what is good in marvell/master, we do not need
>> for you go through the asking for a pulls, they will just happen
>> when I have time, likely over the weekend.
>
> No, please don't automatize this.
>
> Prafulla, please continue to send pullr equests to Tom, as this is
> important information for me, too.
>
>
Ok.
What do you think about the frequency of arm/master monthly updates?
Tom
> Best regards,
>
> Wolfgang Denk
>
^ permalink raw reply [flat|nested] 12+ messages in thread
* [U-Boot] [U-boot] Marvell Pull Request
2009-09-24 1:07 ` Tom
@ 2009-09-24 10:11 ` Wolfgang Denk
2009-09-24 10:23 ` Tom
0 siblings, 1 reply; 12+ messages in thread
From: Wolfgang Denk @ 2009-09-24 10:11 UTC (permalink / raw)
To: u-boot
Dear Tom,
In message <4ABAC668.3040907@windriver.com> you wrote:
>
> What do you think about the frequency of arm/master monthly updates?
Monthly seems to be extremely long. Looking at current activity for
ARM systems, I would expeect to see 2...3 (or more) PRs while the
merge window is open, and about the same during the stabilization
phase. Also, I guess a fixed period is not optimal; doing this as
needed is probably more appropriate.
Best regards,
Wolfgang Denk
--
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 list is only as strong as its weakest link. -- Don Knuth
^ permalink raw reply [flat|nested] 12+ messages in thread
* [U-Boot] [U-boot] Marvell Pull Request
2009-09-24 10:11 ` Wolfgang Denk
@ 2009-09-24 10:23 ` Tom
0 siblings, 0 replies; 12+ messages in thread
From: Tom @ 2009-09-24 10:23 UTC (permalink / raw)
To: u-boot
Wolfgang Denk wrote:
> Dear Tom,
>
> In message <4ABAC668.3040907@windriver.com> you wrote:
>> What do you think about the frequency of arm/master monthly updates?
>
> Monthly seems to be extremely long. Looking at current activity for
> ARM systems, I would expeect to see 2...3 (or more) PRs while the
> merge window is open, and about the same during the stabilization
> phase. Also, I guess a fixed period is not optimal; doing this as
> needed is probably more appropriate.
>
Ok.
Tom
> Best regards,
>
> Wolfgang Denk
>
> --
> 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 list is only as strong as its weakest link. -- Don Knuth
^ permalink raw reply [flat|nested] 12+ messages in thread
* [U-Boot] [U-boot] Marvell Pull Request
2009-09-22 9:28 ` Prafulla Wadaskar
2009-09-22 13:20 ` Tom
@ 2009-09-27 23:51 ` Tom
2009-09-29 1:21 ` Prafulla Wadaskar
1 sibling, 1 reply; 12+ messages in thread
From: Tom @ 2009-09-27 23:51 UTC (permalink / raw)
To: u-boot
Prafulla Wadaskar wrote:
>
>
>> -----Original Message-----
>> From: Tom [mailto:Tom.Rix at windriver.com]
>> Sent: Tuesday, September 22, 2009 12:46 AM
>> To: Prafulla Wadaskar
>> Cc: U-Boot
>> Subject: Re: [U-Boot] [U-boot] Marvell Pull Request
>>
>> Prafulla Wadaskar wrote:
>>> Hi Wolfgang
>>>
>>> Please pull
>> Shouldn't this rather get pulled to u-boot-arm?
>> Tom
>
> Hi Tom,
> Yes :-) you are right.
> I was bit biased since u-boot-marvell.git was cloned from u-boot.git
> Shall I generate new email pull request? Or
> Can you please kindly pull it to u-boot-arm?
>
> Regards..
> Prafulla . .
Prafulla,
The commits have been pulled to arm/next.
openrd_base had this ERR in MAKEALL arm
kirkwood_egiga.c: In function 'kwgbe_init':
kirkwood_egiga.c:445: warning: dereferencing type-punned pointer will
break strict-aliasing rules
kirkwood_egiga.c: In function 'kwgbe_recv':
kirkwood_egiga.c:606: warning: dereferencing type-punned pointer will
break strict-aliasing rules
Please verify the commits have made it into arm/next and help resolve the
error in openrd_base.
Tom
>
>>
>>> The following changes since commit
>> 3b6a9267f0de7b85d387fa4123d0b58379363447:
>>> Wolfgang Denk (1):
>>> board/flagadm/flash.c: fix compile warning
>>>
>>> are available in the git repository at:
>>>
>>> git://git.denx.de/u-boot-marvell.git master
>>>
>>> Prafulla Wadaskar (2):
>>> Kirkwood: rd6281a: Add kwbimage build support
>>> Kirkwood: mv88f6281gtw_ge: Add kwbimage build support
>>>
>>> Simon Kagstrom (1):
>>> Support for the OpenRD base board
>>>
>>> MAINTAINERS | 4 +
>>> MAKEALL | 1 +
>>> Makefile | 3 +
>>> board/Marvell/mv88f6281gtw_ge/config.mk | 3 +
>>> board/Marvell/mv88f6281gtw_ge/kwbimage.cfg | 165
>> +++++++++++++++++++++
>>> board/Marvell/openrd_base/Makefile | 56 +++++++
>>> board/Marvell/openrd_base/config.mk | 33 ++++
>>> board/Marvell/openrd_base/kwbimage.cfg | 168
>> +++++++++++++++++++++
>>> board/Marvell/openrd_base/openrd_base.c | 160
>> ++++++++++++++++++++
>>> board/Marvell/openrd_base/openrd_base.h | 46 ++++++
>>> board/Marvell/rd6281a/config.mk | 3 +
>>> board/Marvell/rd6281a/kwbimage.cfg | 167
>> +++++++++++++++++++++
>>> include/configs/openrd_base.h | 220
>> ++++++++++++++++++++++++++++
>>> 13 files changed, 1029 insertions(+), 0 deletions(-)
>>> create mode 100644 board/Marvell/mv88f6281gtw_ge/kwbimage.cfg
>>> create mode 100644 board/Marvell/openrd_base/Makefile
>>> create mode 100644 board/Marvell/openrd_base/config.mk
>>> create mode 100644 board/Marvell/openrd_base/kwbimage.cfg
>>> create mode 100644 board/Marvell/openrd_base/openrd_base.c
>>> create mode 100644 board/Marvell/openrd_base/openrd_base.h
>>> create mode 100644 board/Marvell/rd6281a/kwbimage.cfg
>>> create mode 100644 include/configs/openrd_base.h
>>>
>>> Regards..
>>> Prafulla . . .
>>> _______________________________________________
>>> U-Boot mailing list
>>> U-Boot at lists.denx.de
>>> http://lists.denx.de/mailman/listinfo/u-boot
^ permalink raw reply [flat|nested] 12+ messages in thread
* [U-Boot] [U-boot] Marvell Pull Request
2009-09-27 23:51 ` Tom
@ 2009-09-29 1:21 ` Prafulla Wadaskar
0 siblings, 0 replies; 12+ messages in thread
From: Prafulla Wadaskar @ 2009-09-29 1:21 UTC (permalink / raw)
To: u-boot
> -----Original Message-----
> From: Tom [mailto:Tom.Rix at windriver.com]
> Sent: Monday, September 28, 2009 5:21 AM
> To: Prafulla Wadaskar
> Cc: U-Boot
> Subject: Re: [U-Boot] [U-boot] Marvell Pull Request
>
> Prafulla Wadaskar wrote:
> >
> >
> >> -----Original Message-----
> >> From: Tom [mailto:Tom.Rix at windriver.com]
> >> Sent: Tuesday, September 22, 2009 12:46 AM
> >> To: Prafulla Wadaskar
> >> Cc: U-Boot
> >> Subject: Re: [U-Boot] [U-boot] Marvell Pull Request
> >>
> >> Prafulla Wadaskar wrote:
> >>> Hi Wolfgang
> >>>
> >>> Please pull
> >> Shouldn't this rather get pulled to u-boot-arm?
> >> Tom
> >
> > Hi Tom,
> > Yes :-) you are right.
> > I was bit biased since u-boot-marvell.git was cloned from u-boot.git
> > Shall I generate new email pull request? Or
> > Can you please kindly pull it to u-boot-arm?
> >
> > Regards..
> > Prafulla . .
>
> Prafulla,
>
> The commits have been pulled to arm/next.
>
> openrd_base had this ERR in MAKEALL arm
>
> kirkwood_egiga.c: In function 'kwgbe_init':
> kirkwood_egiga.c:445: warning: dereferencing type-punned pointer will
> break strict-aliasing rules
> kirkwood_egiga.c: In function 'kwgbe_recv':
> kirkwood_egiga.c:606: warning: dereferencing type-punned pointer will
> break strict-aliasing rules
>
> Please verify the commits have made it into arm/next and help
> resolve the
> error in openrd_base.
Hi Tom
Yes I observed this too,
I didn't observed it earlier, in last release.
I think this is poped up due to gcc clean up patch.
I was in process to fix this,
unfortunately I could not push it up as suggested earlier due to some issues and holidays at our end.
Regards..
Prafulla . .
>
> Tom
>
> >
> >>
> >>> The following changes since commit
> >> 3b6a9267f0de7b85d387fa4123d0b58379363447:
> >>> Wolfgang Denk (1):
> >>> board/flagadm/flash.c: fix compile warning
> >>>
> >>> are available in the git repository at:
> >>>
> >>> git://git.denx.de/u-boot-marvell.git master
> >>>
> >>> Prafulla Wadaskar (2):
> >>> Kirkwood: rd6281a: Add kwbimage build support
> >>> Kirkwood: mv88f6281gtw_ge: Add kwbimage build support
> >>>
> >>> Simon Kagstrom (1):
> >>> Support for the OpenRD base board
> >>>
> >>> MAINTAINERS | 4 +
> >>> MAKEALL | 1 +
> >>> Makefile | 3 +
> >>> board/Marvell/mv88f6281gtw_ge/config.mk | 3 +
> >>> board/Marvell/mv88f6281gtw_ge/kwbimage.cfg | 165
> >> +++++++++++++++++++++
> >>> board/Marvell/openrd_base/Makefile | 56 +++++++
> >>> board/Marvell/openrd_base/config.mk | 33 ++++
> >>> board/Marvell/openrd_base/kwbimage.cfg | 168
> >> +++++++++++++++++++++
> >>> board/Marvell/openrd_base/openrd_base.c | 160
> >> ++++++++++++++++++++
> >>> board/Marvell/openrd_base/openrd_base.h | 46 ++++++
> >>> board/Marvell/rd6281a/config.mk | 3 +
> >>> board/Marvell/rd6281a/kwbimage.cfg | 167
> >> +++++++++++++++++++++
> >>> include/configs/openrd_base.h | 220
> >> ++++++++++++++++++++++++++++
> >>> 13 files changed, 1029 insertions(+), 0 deletions(-)
> >>> create mode 100644 board/Marvell/mv88f6281gtw_ge/kwbimage.cfg
> >>> create mode 100644 board/Marvell/openrd_base/Makefile
> >>> create mode 100644 board/Marvell/openrd_base/config.mk
> >>> create mode 100644 board/Marvell/openrd_base/kwbimage.cfg
> >>> create mode 100644 board/Marvell/openrd_base/openrd_base.c
> >>> create mode 100644 board/Marvell/openrd_base/openrd_base.h
> >>> create mode 100644 board/Marvell/rd6281a/kwbimage.cfg
> >>> create mode 100644 include/configs/openrd_base.h
> >>>
> >>> Regards..
> >>> Prafulla . . .
> >>> _______________________________________________
> >>> U-Boot mailing list
> >>> U-Boot at lists.denx.de
> >>> http://lists.denx.de/mailman/listinfo/u-boot
>
>
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2009-09-29 1:21 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-09-21 17:14 [U-Boot] [U-boot] Marvell Pull Request Prafulla Wadaskar
2009-09-21 19:15 ` Tom
2009-09-22 9:28 ` Prafulla Wadaskar
2009-09-22 13:20 ` Tom
2009-09-23 6:13 ` Prafulla Wadaskar
2009-09-23 11:23 ` Tom
2009-09-23 22:12 ` Wolfgang Denk
2009-09-24 1:07 ` Tom
2009-09-24 10:11 ` Wolfgang Denk
2009-09-24 10:23 ` Tom
2009-09-27 23:51 ` Tom
2009-09-29 1:21 ` Prafulla Wadaskar
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.