[PATCH v2 2/2] bootstd: Replace bootmethod(s) -> bootmeth(s)
Quentin Schulz
quentin.schulz at cherry.de
Mon Jun 17 11:00:08 CEST 2024
Hi all,
On 6/17/24 8:31 AM, Mattijs Korpershoek wrote:
> Hi Heinrich,
>
> Thank you for your review.
>
> On dim., juin 16, 2024 at 09:38, Heinrich Schuchardt <xypron.glpk at gmx.de> wrote:
>
>> On 6/4/24 17:15, Mattijs Korpershoek wrote:
>>> According to [1], we should use bootmeth when describing the
>>> struct bootmeth:
>>>
>>> """
>>> For version 2, a new naming scheme is used as above:
>>>
>>> - bootdev is used instead of bootdevice, because 'device' is overused,
>>> is everywhere in U-Boot, can be confused with udevice
>>
>> Boot devices are udevices though they don't relate to hardware but to an
>> abstract concept.
>>
>> bootdev is just an abbreviation. This does not make the meaning any clearer.
>
> Per my understanding, the name for this concept is "bootdev", not
> "boot device", see:
>
> https://docs.u-boot.org/en/latest/develop/bootstd.html#introduction
>
>>
>>> - bootmeth - because 'method' is too vanilla, appears 1300 times in
>>> U-Boot
>>> """
>>
>> Avoiding abbreviations like bootdev and bootmeth improved readability.
>
> The above paragraph is quoted from email [1].
> In this email, Simon made the choice to use bootmeth and bootdev
> when pushing the initial implementation.
>
> This patch just corrects the places where the older terminology
> (bootmethod, bootdevice) was still used.
>
The current wording is just incorrect, so it needs to be fixed. We have
two choices: use the struct/abbreviated name (bootdevice -> bootdev;
bootmethod -> bootmeth) or the full name (bootdevice -> boot device;
bootmethod -> boot method).
Heinrich are you suggesting we go for full name instead?
board/sandbox/sandbox.env should be using bootmeth instead as that's the
name of the feature?
Cheers,
Quentin
More information about the U-Boot
mailing list