diff mbox

[U-Boot,v2,1/3] am335x_evm: add support for BeagleBone Black DT name

Message ID 1363184418-23957-2-git-send-email-trini@ti.com
State Changes Requested
Delegated to: Tom Rini
Headers show

Commit Message

Tom Rini March 13, 2013, 2:20 p.m. UTC 
From: Koen Kooi <koen@dominion.thruhere.net>

Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
Acked-by: Peter Korsgaard <jacmet@sunsite.dk>
---
 include/configs/am335x_evm.h |    2 ++
 1 file changed, 2 insertions(+)

Comments

Nishanth Menon March 13, 2013, 2:35 p.m. UTC | #1 
On 10:20-20130313, Tom Rini wrote:
> From: Koen Kooi <koen@dominion.thruhere.net>
> 
> Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
> Acked-by: Peter Korsgaard <jacmet@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?
>  		"if test $board_name = A33515BB; then " \
>  			"setenv fdtfile am335x-evm.dtb; fi; " \
>  		"if test $board_name = A335X_SK; then " \
> -- 
> 1.7.9.5
> 
> _______________________________________________
> U-Boot mailing list
> U-Boot@lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
Koen Kooi March 13, 2013, 2:57 p.m. UTC | #2 
Op 13 mrt. 2013, om 15:35 heeft Nishanth Menon <nm@ti.com> het volgende geschreven:

> On 10:20-20130313, Tom Rini wrote:
>> From: Koen Kooi <koen@dominion.thruhere.net>
>> 
>> Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
>> Acked-by: Peter Korsgaard <jacmet@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

Koen
Nishanth Menon March 13, 2013, 3:02 p.m. UTC | #3 
On Wed, Mar 13, 2013 at 9:57 AM, Koen Kooi <koen@dominion.thruhere.net> wrote:
>
> Op 13 mrt. 2013, om 15:35 heeft Nishanth Menon <nm@ti.com> het volgende geschreven:
>
>> On 10:20-20130313, Tom Rini wrote:
>>> From: Koen Kooi <koen@dominion.thruhere.net>
>>>
>>> Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
>>> Acked-by: Peter Korsgaard <jacmet@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
???
If we need to fix the kernel naming, lets fix it instead of having to
deal with people asking - which dtb should i use for beaglebone black?
Regards,
Nishanth Menon
Koen Kooi March 13, 2013, 3:05 p.m. UTC | #4 
Op 13 mrt. 2013, om 16:02 heeft Nishanth Menon <nm@ti.com> het volgende geschreven:

> On Wed, Mar 13, 2013 at 9:57 AM, Koen Kooi <koen@dominion.thruhere.net> wrote:
>> 
>> Op 13 mrt. 2013, om 15:35 heeft Nishanth Menon <nm@ti.com> het volgende geschreven:
>> 
>>> On 10:20-20130313, Tom Rini wrote:
>>>> From: Koen Kooi <koen@dominion.thruhere.net>
>>>> 
>>>> Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
>>>> Acked-by: Peter Korsgaard <jacmet@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.
Nishanth Menon March 13, 2013, 3:07 p.m. UTC | #5 
On 16:05-20130313, Koen Kooi wrote:
> 
> Op 13 mrt. 2013, om 16:02 heeft Nishanth Menon <nm@ti.com> het volgende geschreven:
> 
> > On Wed, Mar 13, 2013 at 9:57 AM, Koen Kooi <koen@dominion.thruhere.net> wrote:
> >> 
> >> Op 13 mrt. 2013, om 15:35 heeft Nishanth Menon <nm@ti.com> het volgende geschreven:
> >> 
> >>> On 10:20-20130313, Tom Rini wrote:
> >>>> From: Koen Kooi <koen@dominion.thruhere.net>
> >>>> 
> >>>> Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
> >>>> Acked-by: Peter Korsgaard <jacmet@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.
Koen Kooi March 13, 2013, 7:02 p.m. UTC | #6 
Op 13 mrt. 2013, om 16:07 heeft Nishanth Menon <nm@ti.com> het volgende geschreven:

> On 16:05-20130313, Koen Kooi wrote:
>> 
>> Op 13 mrt. 2013, om 16:02 heeft Nishanth Menon <nm@ti.com> het volgende geschreven:
>> 
>>> On Wed, Mar 13, 2013 at 9:57 AM, Koen Kooi <koen@dominion.thruhere.net> wrote:
>>>> 
>>>> Op 13 mrt. 2013, om 15:35 heeft Nishanth Menon <nm@ti.com> het volgende geschreven:
>>>> 
>>>>> On 10:20-20130313, Tom Rini wrote:
>>>>>> From: Koen Kooi <koen@dominion.thruhere.net>
>>>>>> 
>>>>>> Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
>>>>>> Acked-by: Peter Korsgaard <jacmet@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 :)
Nishanth Menon March 13, 2013, 7:26 p.m. UTC | #7 
On 20:02-20130313, Koen Kooi wrote:
> 
> Op 13 mrt. 2013, om 16:07 heeft Nishanth Menon <nm@ti.com> het volgende geschreven:
> 
> > On 16:05-20130313, Koen Kooi wrote:
> >> 
> >> Op 13 mrt. 2013, om 16:02 heeft Nishanth Menon <nm@ti.com> het volgende geschreven:
> >> 
> >>> On Wed, Mar 13, 2013 at 9:57 AM, Koen Kooi <koen@dominion.thruhere.net> wrote:
> >>>> 
> >>>> Op 13 mrt. 2013, om 15:35 heeft Nishanth Menon <nm@ti.com> het volgende geschreven:
> >>>> 
> >>>>> On 10:20-20130313, Tom Rini wrote:
> >>>>>> From: Koen Kooi <koen@dominion.thruhere.net>
> >>>>>> 
> >>>>>> Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
> >>>>>> Acked-by: Peter Korsgaard <jacmet@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 :)
:) I will leave it at EEPROM entry defining what the file name should be
and if it is not same as product's official name, guess it is some one's
future explanation burden ;)
Matt Porter March 13, 2013, 8:19 p.m. UTC | #8 
On Wed, Mar 13, 2013 at 08:02:26PM +0100, Koen Kooi wrote:
> 
> Op 13 mrt. 2013, om 16:07 heeft Nishanth Menon <nm@ti.com> het volgende geschreven:
> 
> > On 16:05-20130313, Koen Kooi wrote:
> >> 
> >> Op 13 mrt. 2013, om 16:02 heeft Nishanth Menon <nm@ti.com> het volgende geschreven:
> >> 
> >>> On Wed, Mar 13, 2013 at 9:57 AM, Koen Kooi <koen@dominion.thruhere.net> wrote:
> >>>> 
> >>>> Op 13 mrt. 2013, om 15:35 heeft Nishanth Menon <nm@ti.com> het volgende geschreven:
> >>>> 
> >>>>> On 10:20-20130313, Tom Rini wrote:
> >>>>>> From: Koen Kooi <koen@dominion.thruhere.net>
> >>>>>> 
> >>>>>> Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
> >>>>>> Acked-by: Peter Korsgaard <jacmet@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
Koen Kooi March 14, 2013, 6:53 a.m. UTC | #9 
Op 13 mrt. 2013, om 21:19 heeft Matt Porter <mporter@ti.com> het volgende geschreven:

> On Wed, Mar 13, 2013 at 08:02:26PM +0100, Koen Kooi wrote:
>> 
>> Op 13 mrt. 2013, om 16:07 heeft Nishanth Menon <nm@ti.com> het volgende geschreven:
>> 
>>> On 16:05-20130313, Koen Kooi wrote:
>>>> 
>>>> Op 13 mrt. 2013, om 16:02 heeft Nishanth Menon <nm@ti.com> het volgende geschreven:
>>>> 
>>>>> On Wed, Mar 13, 2013 at 9:57 AM, Koen Kooi <koen@dominion.thruhere.net> wrote:
>>>>>> 
>>>>>> Op 13 mrt. 2013, om 15:35 heeft Nishanth Menon <nm@ti.com> het volgende geschreven:
>>>>>> 
>>>>>>> On 10:20-20130313, Tom Rini wrote:
>>>>>>>> From: Koen Kooi <koen@dominion.thruhere.net>
>>>>>>>> 
>>>>>>>> Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
>>>>>>>> Acked-by: Peter Korsgaard <jacmet@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.

It is stupid, but that's what you get for renaming boards. The real problem with this is that it is a flag day, the kernel and u-boot need to change both at the same time. And after that you'll have a mismatch if you use different versions.

Anyway, patches welcome for the vendor kernel to rename it.

regards,

Koen
Matt Porter March 14, 2013, 3:07 p.m. UTC | #10 
On Thu, Mar 14, 2013 at 07:53:07AM +0100, Koen Kooi wrote:
> 
> Op 13 mrt. 2013, om 21:19 heeft Matt Porter <mporter@ti.com> het volgende geschreven:
> 
> > On Wed, Mar 13, 2013 at 08:02:26PM +0100, Koen Kooi wrote:
> >> 
> >> Op 13 mrt. 2013, om 16:07 heeft Nishanth Menon <nm@ti.com> het volgende geschreven:
> >> 
> >>> On 16:05-20130313, Koen Kooi wrote:
> >>>> 
> >>>> Op 13 mrt. 2013, om 16:02 heeft Nishanth Menon <nm@ti.com> het volgende geschreven:
> >>>> 
> >>>>> On Wed, Mar 13, 2013 at 9:57 AM, Koen Kooi <koen@dominion.thruhere.net> wrote:
> >>>>>> 
> >>>>>> Op 13 mrt. 2013, om 15:35 heeft Nishanth Menon <nm@ti.com> het volgende geschreven:
> >>>>>> 
> >>>>>>> On 10:20-20130313, Tom Rini wrote:
> >>>>>>>> From: Koen Kooi <koen@dominion.thruhere.net>
> >>>>>>>> 
> >>>>>>>> Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
> >>>>>>>> Acked-by: Peter Korsgaard <jacmet@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.
> 
> It is stupid, but that's what you get for renaming boards. The real problem with this is that it is a flag day, the kernel and u-boot need to change both at the same time. And after that you'll have a mismatch if you use different versions.
> 
> Anyway, patches welcome for the vendor kernel to rename it.

The pull request is https://github.com/beagleboard/kernel/pull/33

Should be easy to keep those in sync as you control the s/w going
out on all the new BeagleBones. The type of people that have early
boards can all handle s/w changes like this otherwise they shouldn't
be in the alpha test group.

Anything else that prevents us from fixing this fdtfile hunk now?

-Matt
diff mbox

Patch

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; " \
 		"if test $board_name = A33515BB; then " \
 			"setenv fdtfile am335x-evm.dtb; fi; " \
 		"if test $board_name = A335X_SK; then " \