Dear Wolfgang,
Wolfgang Denk wrote:
> Dear Dirk,
>
> In message <49f5c746.6040...@googlemail.com> you wrote:
>> Short status update after recent merges and patch updates. As usual,
>> please correct if anything is wrong or missing.
>
> Thanks for the summary.
>
>> Dirk Behme wrote:
>>> To avoid loosing the overview, here my list of pending OMAP3 patches
>>> ready to be applied. From my point of view there are no open comments on
>>> these which will prevent to apply them. But please correct if I
>>> overlooked anything or add what (patches? comments?) I missed.
>>>
>>> 1. OMAP3: Beagle: Set pinmux conditionally for Rev C boards
>>> http://lists.denx.de/pipermail/u-boot/2009-April/051013.html
>> Mail
>>
>> http://lists.denx.de/pipermail/u-boot/2009-April/051459.html
>>
>> tells us that this should be applied to u-boot-arm next. But I can't
>> find it in gitweb
>>
>> http://git.denx.de/?p=u-boot/u-boot-arm.git;a=shortlog;h=refs/heads/next
>>
>> Did I miss something?
>
> I cannot find it either.
:(
>> Btw.: As initial patch and first update was sent ~1 day after merge
>> window close (Apr 4?) I hoped to have a chance to still get it into
>> master ;(
>
> We don't have to bend much to consider this even a bug fix (it fixes a
> specific board configuration, doesn't it?).
Yes. We got a new board HW revision ('revision C', with some
re-wiring) which needs some minor changes in PinMux to support this
changed wiring. Thanks to a HW identification (resistors on GPIOs) we
are able to auto detect HW revision and configure correct pin mux
automagically. This is done by this patch.
>>> 2. OMAP3: Remove legacy NAND defines
>>> http://lists.denx.de/pipermail/u-boot/2009-April/050882.html
>> Applied to u-boot-arm next. Request to move to master by Wolfgang.
>>
>>> 3. OMAP3: Fix timer handling to 1ms and CONFIG_SYS_HZ to 1000
>>> http://lists.denx.de/pipermail/u-boot/2009-April/051178.html
>>>
>>> 4. OMAP3: Fix changed mmc init command
>>> http://lists.denx.de/pipermail/u-boot/2009-April/051179.html
>> Applied to u-boot-arm master.
>>
>>> 5. OMAP3: Remove unused board-types
>>> http://lists.denx.de/pipermail/u-boot/2009-April/051338.html
>>>
>>> 6. OMAP3: Print correct silicon revision
>>> http://lists.denx.de/pipermail/u-boot/2009-April/051339.html
>> Patches 5 & 6 replaced by update which includes switch to checkboard
>> and print_cpuinfo:
>>
>> OMAP3: Board specific updates (3 patches)
>> http://lists.denx.de/pipermail/u-boot/2009-April/051505.html
>> http://lists.denx.de/pipermail/u-boot/2009-April/051506.html
>> http://lists.denx.de/pipermail/u-boot/2009-April/051507.html
>>
>>> 7. Zoom2 respin II (10 patches)
>> Note: Initial patch series for this was sent 'Wed Apr 1 18:08:38 CEST
>> 2009'
>>
>> http://lists.denx.de/pipermail/u-boot/2009-April/050094.html
>>
>> IMHO this was while open merge window.
>>
>>> http://lists.denx.de/pipermail/u-boot/2009-April/050863.html
>>> http://lists.denx.de/pipermail/u-boot/2009-April/050864.html
>>> http://lists.denx.de/pipermail/u-boot/2009-April/050865.html
>>> http://lists.denx.de/pipermail/u-boot/2009-April/050866.html
>>> http://lists.denx.de/pipermail/u-boot/2009-April/050868.html
>>> http://lists.denx.de/pipermail/u-boot/2009-April/050867.html
>> Applied to u-boot-arm next (see comment about merge window above).
>>
>>> http://lists.denx.de/pipermail/u-boot/2009-April/050869.html
>> Applied to u-boot-arm next (see comment about merge window above).
>>
>>> http://lists.denx.de/pipermail/u-boot/2009-April/050870.html
>>> http://lists.denx.de/pipermail/u-boot/2009-April/050871.html
>> Applied to u-boot-arm next. Request to move to master by Wolfgang.
>
>
> I think all the patches listed above could and should go into master
> now.
>
> Jean-Cristophe, what do you think? Do you have any specific reasons
> for not pulling these into master now?
>
>>> http://lists.denx.de/pipermail/u-boot/2009-April/050872.html
>> For the rest of the Zoom2 patches (#1, #2, #3, #4, #5, #8 & #10)
>>
>> http://lists.denx.de/pipermail/u-boot/2009-April/051383.html
>> http://lists.denx.de/pipermail/u-boot/2009-April/051384.html
>> http://lists.denx.de/pipermail/u-boot/2009-April/051385.html
>> http://lists.denx.de/pipermail/u-boot/2009-April/051386.html
>> http://lists.denx.de/pipermail/u-boot/2009-April/051455.html
>>
>> seem to apply.
>
> I'm not sure what you mean here, Dirk.
I tried to say that for patches #1, #2, #3, #4, #5, #8 & #10 there are
comments in the 5 mails linked above (i.e. the comments in the 5 mails
above apply to the patches #1, #2, #3, #4, #5, #8 & #10).
> I understand that
> Jean-Cristophe requested changes,
Yes.
> and we're waiting for a resubmit,
> right?
Well, before waiting for a resubmit, I would wait for a comment
(discussion?) from the patch author (Tom).
Anyhow, seems we got a comment from Tom
http://lists.denx.de/pipermail/u-boot/2009-April/051605.html
and _now_ we are waiting for a resubmit ;)
> Jean-Cristophe, I really would like to see some progress in this
> area, so I'd appreciate if you could apply the patches listed above
> to master and send me a pull request. Thanks.
Many thanks for helping and best regards
Dirk
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
