Hi,

On 2/11/26 8:53 AM, Sascha Hauer wrote:
> On Tue, Feb 10, 2026 at 11:13:40PM +0100, Marco Felsch wrote:
>> While specifying the barebox environment within the barebox dts is very
>> helpful for standalone barebox development it's difficult for BSPs which
>> use this development platform and want to use the GPT mechanism.
>> As a result BSPs had to manually delete the nodes via a external
>> provided dts, because we wanted to keep the comfort for the standalone
>> development flow.
>>
>> Lucky commit 86531d4bf7aa ("commands: create createnv command")
>> introduced a convenient helper to create a barebox environemnt on
>> demand exactly for this purpose:
>>
>> | "We want to move away from describing the barebox environment explicitly
>> | in the device tree and instead motivate usage of GPT partitions.."
>>
>> So start with the i.MX8MP-EVK to encourage the use of GPT partitions and
>> to make the BSP integration for these development platforms easier.
> 
> Generally I like this idea, but it breaks the existing users
> environment and doing this sharp cut might be annoying for the phase
> where you switch between different barebox versions which could be quite
> common for development boards.
> 
> Would be great to have a grace period in which we prefer the UUID
> environment over the device tree environment (we might do this already)
> and warn in case we still use the latter. We could stick the warning to
> a device tree property like "warn-deprecated-env" and remove the
> environment once they carry this property for a year or so.
> 
> Just the ideas from the top of my head. Thoughts?

Here's my opinion from a year back:
https://lore.barebox.org/barebox/[email protected]/

I am interested to hear thoughts on it.
Compared to our suggestion, there will be no eventual removal of the node.

Cheers,
Ahmad

> 
> Sascha
> 
>>
>> Signed-off-by: Marco Felsch <[email protected]>
>> ---
>>  arch/arm/dts/imx8mp-evk.dts | 43 -------------------------------------
>>  1 file changed, 43 deletions(-)
>>
>> diff --git a/arch/arm/dts/imx8mp-evk.dts b/arch/arm/dts/imx8mp-evk.dts
>> index e6f8c2d1b482..37798fa99efa 100644
>> --- a/arch/arm/dts/imx8mp-evk.dts
>> +++ b/arch/arm/dts/imx8mp-evk.dts
>> @@ -12,19 +12,6 @@
>>  / {
>>      barebox,deep-probe;
>>  
>> -    chosen {
>> -            environment-sd {
>> -                    compatible = "barebox,environment";
>> -                    device-path = &env_sd2;
>> -                    status = "disabled";
>> -            };
>> -            environment-emmc {
>> -                    compatible = "barebox,environment";
>> -                    device-path = &env_sd3;
>> -                    status = "disabled";
>> -            };
>> -    };
>> -
>>      gpio-leds {
>>              status {
>>                      barebox,default-trigger = "heartbeat";
>> @@ -78,33 +65,3 @@ &usb_dwc3_0 {
>>      usb-role-switch;
>>      status = "okay";
>>  };
>> -
>> -&usdhc2 {
>> -    #address-cells = <1>;
>> -    #size-cells = <1>;
>> -
>> -    partition@0 {
>> -            label = "barebox";
>> -            reg = <0x0 0xe0000>;
>> -    };
>> -
>> -    env_sd2: partition@e0000 {
>> -            label = "barebox-environment";
>> -            reg = <0xe0000 0x20000>;
>> -    };
>> -};
>> -
>> -&usdhc3 {
>> -    #address-cells = <1>;
>> -    #size-cells = <1>;
>> -
>> -    partition@0 {
>> -            label = "barebox";
>> -            reg = <0x0 0xe0000>;
>> -    };
>> -
>> -    env_sd3: partition@e0000 {
>> -            label = "barebox-environment";
>> -            reg = <0xe0000 0x20000>;
>> -    };
>> -};
>> -- 
>> 2.47.3
>>
>>
>>
> 

-- 
Pengutronix e.K.                  |                             |
Steuerwalder Str. 21              | http://www.pengutronix.de/  |
31137 Hildesheim, Germany         | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686  | Fax:   +49-5121-206917-5555 |


Reply via email to