Re: [PATCH] doc: board: starfive: jh7110 common give build output dir by env not arg

[E Shattow <e@freeshell.de>]

On 3/5/26 13:05, E Shattow wrote:
> 
> 
> On 3/5/26 12:56, E Shattow wrote:
>> Describe build with output directory as O=<dir> environment variable
>> and not unrelated -O <output sync> command line argument.
>>
>> Fixes: 8304f3226700 ("doc: board: starfive: update jh7110 common description")
>> Signed-off-by: E Shattow <e@freeshell.de>
>> ---
>>  doc/board/starfive/jh7110_common.rst | 10 +++++-----
>>  1 file changed, 5 insertions(+), 5 deletions(-)
>>
>> diff --git a/doc/board/starfive/jh7110_common.rst b/doc/board/starfive/jh7110_common.rst
>> index 77102fcc189..b0b60da0dc3 100644
>> --- a/doc/board/starfive/jh7110_common.rst
>> +++ b/doc/board/starfive/jh7110_common.rst
>> @@ -162,8 +162,8 @@ Build U-Boot
>>         git -C opensbi.git checkout v1.7
>>         # always clean build directory when building OpenSBI due to incomplete
>>         # dependency tracking
>> -       make -C opensbi.git -O opensbi clean
>> -       make -C opensbi.git -O opensbi PLATFORM=generic
>> +       make -C opensbi.git O=opensbi clean
>> +       make -C opensbi.git O=opensbi PLATFORM=generic
>>  
>>  4.  Now build the First Stage BootLoader (U-Boot Secondary Program Loader) and
>>      Second Boot Loader (OpenSBI + U-Boot Main):
>> @@ -171,9 +171,9 @@ Build U-Boot
>>      .. code-block:: console
>>  
>>         git clone https://source.denx.de/u-boot/u-boot.git u-boot.git
>> -       make -C u-boot.git -O u-boot starfive_visionfive2_defconfig
>> +       make -C u-boot.git O=u-boot starfive_visionfive2_defconfig
> 
>>         export OPENSBI=opensbi/build/platform/generic/firmware/fw_dynamic.bin
> 
> Oh and what is this OPENSBI directory ? That's not correct, either.
> 

Replying myself, follow-up patch for the OPENSBI path typo is submitted
at: https://lore.kernel.org/u-boot/20260305214922.97337-1-e@freeshell.de

>> -       make -C u-boot.git -O u-boot
>> +       make -C u-boot.git O=u-boot
>>  
>>      This will generate the U-Boot SPL image object post-processed with StarFive
>>      SPL headers (u-boot/spl/u-boot-spl.bin.normal.out) as well as the FIT image
>> @@ -191,7 +191,7 @@ Build U-Boot
>>           --set-val SPL_DEBUG_UART_BASE 0x10000000 \
>>           --set-val DEBUG_UART_SHIFT 2
>>  
>> -        make -C u-boot.git -O u-boot olddefconfig
>> +        make -C u-boot.git O=u-boot olddefconfig
>>  
>>  Boot description
>>  ----------------
>>
>> base-commit: b26cc03b7cecaabdb306c2f68eebde764161a545
> 
> -E

-E