public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [PATCH] doc: board: starfive: jh7110 common give build output dir by env not arg
@ 2026-03-05 20:56 E Shattow
  2026-03-05 21:05 ` E Shattow
  2026-03-16  8:26 ` Leo Liang
  0 siblings, 2 replies; 4+ messages in thread
From: E Shattow @ 2026-03-05 20:56 UTC (permalink / raw)
  To: Minda Chen, Hal Feng, Tom Rini, E Shattow; +Cc: u-boot

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
-       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
-- 
2.50.0


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] doc: board: starfive: jh7110 common give build output dir by env not arg
  2026-03-05 20:56 [PATCH] doc: board: starfive: jh7110 common give build output dir by env not arg E Shattow
@ 2026-03-05 21:05 ` E Shattow
  2026-03-05 21:56   ` E Shattow
  2026-03-16  8:26 ` Leo Liang
  1 sibling, 1 reply; 4+ messages in thread
From: E Shattow @ 2026-03-05 21:05 UTC (permalink / raw)
  To: Minda Chen, Hal Feng, Tom Rini; +Cc: u-boot



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.

> -       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

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] doc: board: starfive: jh7110 common give build output dir by env not arg
  2026-03-05 21:05 ` E Shattow
@ 2026-03-05 21:56   ` E Shattow
  0 siblings, 0 replies; 4+ messages in thread
From: E Shattow @ 2026-03-05 21:56 UTC (permalink / raw)
  To: Minda Chen, Hal Feng, Tom Rini; +Cc: u-boot


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

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] doc: board: starfive: jh7110 common give build output dir by env not arg
  2026-03-05 20:56 [PATCH] doc: board: starfive: jh7110 common give build output dir by env not arg E Shattow
  2026-03-05 21:05 ` E Shattow
@ 2026-03-16  8:26 ` Leo Liang
  1 sibling, 0 replies; 4+ messages in thread
From: Leo Liang @ 2026-03-16  8:26 UTC (permalink / raw)
  To: E Shattow; +Cc: Minda Chen, Hal Feng, Tom Rini, u-boot

On Thu, Mar 05, 2026 at 12:56:40PM -0800, 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(-)

Reviewed-by: Leo Yu-Chi Liang <ycliang@andestech.com>

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2026-03-16  8:27 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-03-05 20:56 [PATCH] doc: board: starfive: jh7110 common give build output dir by env not arg E Shattow
2026-03-05 21:05 ` E Shattow
2026-03-05 21:56   ` E Shattow
2026-03-16  8:26 ` Leo Liang

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox