[coreboot] [PATCH] v3: Move default mainboard vendor/subsystem from Kconfig to dts
Carl-Daniel Hailfinger
c-d.hailfinger.devel.2006 at gmx.net
Fri Jul 11 14:36:07 CEST 2008
On 11.07.2008 13:59, Uwe Hermann wrote:
> On Fri, Jul 11, 2008 at 03:06:43AM +0200, Carl-Daniel Hailfinger wrote:
>
>>> My goal was to have the same name in the dts which is also used as
>>> variable name in C code. The existing code has a somewhat peculiar
>>> mapping from organic growth over time:
>>> mainboard-vendor -> const char *mainboard_vendor
>>> mainboard-name -> const char *mainboard_part_number
>>> If we can agree on whether the variable should be called
>>> "mainboard_name" or "mainboard_part_number", I'd be willing to prepare a
>>> patch.
>>>
>
> mainboard_name sounds good.
>
Indeed. The only problem is that this sort of conflicts with
CONFIG_MAINBOARD_NAME, but that Kconfig variable is misnamed anyway (it
contains the mainboard directory).
>> OK, here comes a patch which makes mainboard-vendor naming more consistent.
>> mainboard-name naming has been postponed because it's not clear what the
>> real name should be.
>>
>> Generated code is identical to the state before the patch.
>>
>> Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>
>>
>
> Acked-by: Uwe Hermann <uwe at hermann-uwe.de>
>
> Build-tested with a few targets.
>
Thanks, r694.
Regards,
Carl-Daniel
--
http://www.hailfinger.org/
More information about the coreboot
mailing list