| Message ID | 20240614120416.4126896-1-d-gole@ti.com |
|---|---|
| State | Superseded |
| Delegated to: | Tom Rini |
| Headers | show |
| Series | [v3] arm: dts: k3-am625-beagleplay: Add symlinks for tiboot3 and tispl | expand |
Hi Dhruva, On 17:34-20240614, Dhruva Gole wrote: > Add symlinks for both tiboot3.bin and tispl.bin because a user has to > anyway rename these files to get the platform to boot up. > This just makes it more intuitive and convenient. > > Signed-off-by: Dhruva Gole <d-gole@ti.com> > --- > > This is just a RESEND of v1, because the changes in v2 were not > required. Link to previous revision: > https://lore.kernel.org/u-boot/20240614115340.4091211-1-d-gole@ti.com/ Didn't realise that u-boot.img comes by itself from the core framework itself. Thanks for re-sending v1. Reviewed-by: Manorit Chawdhry <m-chawdhry@ti.com> Regards, Manorit > > > arch/arm/dts/k3-am625-beagleplay-u-boot.dtsi | 1 + > arch/arm/dts/k3-am625-r5-beagleplay.dts | 1 + > 2 files changed, 2 insertions(+) > > diff --git a/arch/arm/dts/k3-am625-beagleplay-u-boot.dtsi b/arch/arm/dts/k3-am625-beagleplay-u-boot.dtsi > index fb2032068d1c..bd48af8d186a 100644 > --- a/arch/arm/dts/k3-am625-beagleplay-u-boot.dtsi > +++ b/arch/arm/dts/k3-am625-beagleplay-u-boot.dtsi > @@ -71,6 +71,7 @@ > > ti-spl_unsigned { > filename = "tispl.bin_unsigned"; > + symlink = "tispl.bin"; > pad-byte = <0xff>; > > fit { > diff --git a/arch/arm/dts/k3-am625-r5-beagleplay.dts b/arch/arm/dts/k3-am625-r5-beagleplay.dts > index 9db58f093c8c..da6746e009d5 100644 > --- a/arch/arm/dts/k3-am625-r5-beagleplay.dts > +++ b/arch/arm/dts/k3-am625-r5-beagleplay.dts > @@ -79,6 +79,7 @@ > &binman { > tiboot3-am62x-gp-evm.bin { > filename = "tiboot3-am62x-gp-evm.bin"; > + symlink = "tiboot3.bin"; > ti-secure-rom { > content = <&u_boot_spl_unsigned>, <&ti_fs_gp>, > <&combined_tifs_cfg_gp>, <&combined_dm_cfg_gp>; > > base-commit: 1ebd659cf020843fd8e8ef90d85a66941cbab6ec > -- > 2.34.1 >
On 10:37-20240617, Manorit Chawdhry wrote: > Hi Dhruva, > > On 17:34-20240614, Dhruva Gole wrote: > > Add symlinks for both tiboot3.bin and tispl.bin because a user has to > > anyway rename these files to get the platform to boot up. > > This just makes it more intuitive and convenient. > > > > Signed-off-by: Dhruva Gole <d-gole@ti.com> > > --- > > > > This is just a RESEND of v1, because the changes in v2 were not > > required. Link to previous revision: > > https://lore.kernel.org/u-boot/20240614115340.4091211-1-d-gole@ti.com/ > > Didn't realise that u-boot.img comes by itself from the core framework > itself. Thanks for re-sending v1. > > Reviewed-by: Manorit Chawdhry <m-chawdhry@ti.com> > > Regards, > Manorit > > > > > > > arch/arm/dts/k3-am625-beagleplay-u-boot.dtsi | 1 + > > arch/arm/dts/k3-am625-r5-beagleplay.dts | 1 + > > 2 files changed, 2 insertions(+) > > > > diff --git a/arch/arm/dts/k3-am625-beagleplay-u-boot.dtsi b/arch/arm/dts/k3-am625-beagleplay-u-boot.dtsi > > index fb2032068d1c..bd48af8d186a 100644 > > --- a/arch/arm/dts/k3-am625-beagleplay-u-boot.dtsi > > +++ b/arch/arm/dts/k3-am625-beagleplay-u-boot.dtsi > > @@ -71,6 +71,7 @@ > > > > ti-spl_unsigned { > > filename = "tispl.bin_unsigned"; > > + symlink = "tispl.bin"; > > pad-byte = <0xff>; > > > > fit { > > diff --git a/arch/arm/dts/k3-am625-r5-beagleplay.dts b/arch/arm/dts/k3-am625-r5-beagleplay.dts > > index 9db58f093c8c..da6746e009d5 100644 > > --- a/arch/arm/dts/k3-am625-r5-beagleplay.dts > > +++ b/arch/arm/dts/k3-am625-r5-beagleplay.dts > > @@ -79,6 +79,7 @@ > > &binman { > > tiboot3-am62x-gp-evm.bin { > > filename = "tiboot3-am62x-gp-evm.bin"; > > + symlink = "tiboot3.bin"; > > ti-secure-rom { > > content = <&u_boot_spl_unsigned>, <&ti_fs_gp>, > > <&combined_tifs_cfg_gp>, <&combined_dm_cfg_gp>; > > > > base-commit: 1ebd659cf020843fd8e8ef90d85a66941cbab6ec > > -- > > 2.34.1 > > Do we need any documentation update?
diff --git a/arch/arm/dts/k3-am625-beagleplay-u-boot.dtsi b/arch/arm/dts/k3-am625-beagleplay-u-boot.dtsi index fb2032068d1c..bd48af8d186a 100644 --- a/arch/arm/dts/k3-am625-beagleplay-u-boot.dtsi +++ b/arch/arm/dts/k3-am625-beagleplay-u-boot.dtsi @@ -71,6 +71,7 @@ ti-spl_unsigned { filename = "tispl.bin_unsigned"; + symlink = "tispl.bin"; pad-byte = <0xff>; fit { diff --git a/arch/arm/dts/k3-am625-r5-beagleplay.dts b/arch/arm/dts/k3-am625-r5-beagleplay.dts index 9db58f093c8c..da6746e009d5 100644 --- a/arch/arm/dts/k3-am625-r5-beagleplay.dts +++ b/arch/arm/dts/k3-am625-r5-beagleplay.dts @@ -79,6 +79,7 @@ &binman { tiboot3-am62x-gp-evm.bin { filename = "tiboot3-am62x-gp-evm.bin"; + symlink = "tiboot3.bin"; ti-secure-rom { content = <&u_boot_spl_unsigned>, <&ti_fs_gp>, <&combined_tifs_cfg_gp>, <&combined_dm_cfg_gp>;
Add symlinks for both tiboot3.bin and tispl.bin because a user has to anyway rename these files to get the platform to boot up. This just makes it more intuitive and convenient. Signed-off-by: Dhruva Gole <d-gole@ti.com> --- This is just a RESEND of v1, because the changes in v2 were not required. Link to previous revision: https://lore.kernel.org/u-boot/20240614115340.4091211-1-d-gole@ti.com/ arch/arm/dts/k3-am625-beagleplay-u-boot.dtsi | 1 + arch/arm/dts/k3-am625-r5-beagleplay.dts | 1 + 2 files changed, 2 insertions(+) base-commit: 1ebd659cf020843fd8e8ef90d85a66941cbab6ec