[U-Boot] [PATCH] ARM: rpi: Add CONFIG_ENV_VARS_UBOOT_RUNTIME_CONFIG support

[Stephen Warren]

On 08/25/2015 07:46 PM, Ian Lepore wrote:
> On Tue, 2015-08-25 at 19:34 -0600, Stephen Warren wrote:
>> On 08/21/2015 03:47 AM, Guillaume Gardet wrote:
>>>
>>>
>>> Le 19/08/2015 05:14, Stephen Warren a écrit :
>>>> On 08/18/2015 08:03 AM, Guillaume GARDET wrote:
>>>>> Add CONFIG_ENV_VARS_UBOOT_RUNTIME_CONFIG support to set 'board_rev' and
>>>>> 'board_name' envs.
>>>> That states what the patch does rather than why its useful to do it. Can
>>>> you expand on why it's useful to set these variables?
>>>
>>> Using boot scripts you may need to get the board version / revision
>>> infos, for example to select the right DTB since u-boot DTB names and
>>> kernel DTB files do not match.
>>
>> The fix here isn't to craft all kinds of complex scripts in U-Boot based
>> on a slew of extra variables. Rather, the set of DT files in the kernel
>> should be expanded so that there is one DT per board design, i.e. so the
>> filenames in the kernel match the filenames that U-Boot expects.
> 
> You speak of "the kernel" as if there were only one.

You're right; I should have said "DT file repository" not "kernel" here.
In practice, that's the Linux kernel at present though.

Still, I stand by my assertion that the values U-Boot expects for DTB
filenames make sense, and we should update any/all repositories of DTB
files so there is a complete set of matching files.