On Wed, Mar 13, 2013 at 08:02:26PM +0100, Koen Kooi wrote: > > Op 13 mrt. 2013, om 16:07 heeft Nishanth Menon <n...@ti.com> het volgende > geschreven: > > > On 16:05-20130313, Koen Kooi wrote: > >> > >> Op 13 mrt. 2013, om 16:02 heeft Nishanth Menon <n...@ti.com> het volgende > >> geschreven: > >> > >>> On Wed, Mar 13, 2013 at 9:57 AM, Koen Kooi <k...@dominion.thruhere.net> > >>> wrote: > >>>> > >>>> Op 13 mrt. 2013, om 15:35 heeft Nishanth Menon <n...@ti.com> het > >>>> volgende geschreven: > >>>> > >>>>> On 10:20-20130313, Tom Rini wrote: > >>>>>> From: Koen Kooi <k...@dominion.thruhere.net> > >>>>>> > >>>>>> Signed-off-by: Koen Kooi <k...@dominion.thruhere.net> > >>>>>> Acked-by: Peter Korsgaard <jac...@sunsite.dk> > >>>>>> --- > >>>>>> include/configs/am335x_evm.h | 2 ++ > >>>>>> 1 file changed, 2 insertions(+) > >>>>>> > >>>>>> diff --git a/include/configs/am335x_evm.h > >>>>>> b/include/configs/am335x_evm.h > >>>>>> index 33ee2c4..abf4e39 100644 > >>>>>> --- a/include/configs/am335x_evm.h > >>>>>> +++ b/include/configs/am335x_evm.h > >>>>>> @@ -87,6 +87,8 @@ > >>>>>> "findfdt="\ > >>>>>> "if test $board_name = A335BONE; then " \ > >>>>>> "setenv fdtfile am335x-bone.dtb; fi; " \ > >>>>>> + "if test $board_name = A335BNLT; then " \ > >>>>>> + "setenv fdtfile am335x-bonelt.dtb; fi; " \ > >>>>> could we not use am335x-boneblack.dtb instead? > >>>> > >>>> it's bonelt in the kernel, so boneblack would fail to load a dtb > >>> $ git describe > >>> v3.9-rc2 > >>> $ ls arch/arm/boot/dts/*bone*.dts* > >>> arch/arm/boot/dts/am335x-bone.dts > >>> ??? > >> > >> What a surprise, missing stuff in mainline! It's present in the vendor > >> kernel, which is what is shipping with the board. > > No surprise there as marketing names are finalized close to product > > launches. If there is an vendor u-boot, it can remain in sync with what > > ever the vendor kernel is. If it is upstream, we might want to think > > longterm. > > I'm in charge of pushing the am335x-bonelt.dts upstream to Linus and I'd like > to keep everything in sync. I'm not fond of renaming it again, the EEPROM > says BNLT, not BNB :)
That's a horrible reason to confuse users. The EEPROM contents are irrelevant. Users look for something to match the board name, this does not. It's wrong. -Matt _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
