Re: [PATCH 1/2] ref-manual: improve description of kernel-fitimage variables

[Michael Opdenacker <michael.opdenacker@bootlin.com>]

Hi Sean,

Many thanks for the review!

On 11.05.23 at 20:43, Sean Anderson wrote:
> On 5/11/23 12:11, michael.opdenacker@bootlin.com wrote:
>> [You don't often get email from michael.opdenacker@bootlin.com. Learn why this is important at https://cas5-0-urlprotect.trendmicro.com:443/wis/clicktime/v1/query?url=https%3a%2f%2faka.ms%2fLearnAboutSenderIdentification&umid=518cf34e-efd1-4968-94ed-83b8c8896b6c&auth=d807158c60b7d2502abde8a2fc01f40662980862-ebaf0a6579314efc58145c5fef613fa0b2d5e7d3 ]
>>
>> From: Michael Opdenacker <michael.opdenacker@bootlin.com>
>>
>> - Mentioning that they are related to this class
>>    (for further information about their usage)
>> - Replace "fitImage" by "FIT image"
>> - Minor formating and style fixes
>>
>> Signed-off-by: Michael Opdenacker <michael.opdenacker@bootlin.com>
>> CC: Jamin Lin <jamin_lin@aspeedtech.com>
>> CC: Sean Anderson <sean.anderson@seco.com>
>> ---
>>   documentation/ref-manual/classes.rst   |   4 +-
>>   documentation/ref-manual/variables.rst | 105 ++++++++++++++++---------
>>   2 files changed, 68 insertions(+), 41 deletions(-)
>>
>> diff --git a/documentation/ref-manual/classes.rst b/documentation/ref-manual/classes.rst
>> index 94b6a3ebd7..2f96ffb82f 100644
>> --- a/documentation/ref-manual/classes.rst
>> +++ b/documentation/ref-manual/classes.rst
>> @@ -1582,13 +1582,13 @@ The :ref:`ref-classes-kernel-devicetree` class, which is inherited by the
>>   ===================
>>
>>   The :ref:`ref-classes-kernel-fitimage` class provides support to pack a kernel image,
>> -device trees, a U-boot script, a :term:`Initramfs` bundle and a RAM disk
>> +device trees, a U-boot script, an :term:`Initramfs` bundle and a RAM disk
> If we're being pedantic, it's "U-Boot." All these variables should really be U_BOOT_... too, but that ship has sailed.
Yes, things can be complicated to change once they have been deployed :|
>
> Reviewed-by: Sean Anderson <sean.anderson@seco.com>


Thanks again!
Cheers
Michael.

-- 
Michael Opdenacker, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com