[PATCH 06/14] common: add generic machine partitions interface

[Ahmad Fatoum]

Hi,

On 14.04.21 12:08, Sascha Hauer wrote:
> On Tue, Apr 13, 2021 at 12:34:54AM +0200, Ahmad Fatoum wrote:
>> Both Fastboot and DFU have their own global variables that allow
>> specifying the partitions that can be flashed via the environment.
>> With the upcoming addition of the USB mass storage gadget, we will need
>> some way to define the partitions there as well.
>>
>> Instead of adding yet another way download method-specific variable,
>> add a generic machine.partitions variable that can be specified on a
>> per-board basis and can be used for all methods.
>>
>> Existing variables will still remain for backwards-compatibility, but
>> when unset, it should fall back to this new parameter. This is done
>> if the follow-up patches.
>>
>> +static int machine_partitions_var_init(void)
>> +{
>> +	struct param_d *param;
>> +
>> +	machine_partitions = file_list_parse("");
>> +	param = dev_add_param_file_list(of_get_machine(), "partitions",
>> +					NULL, NULL, &machine_partitions, NULL);
> 
> Why machine.partitions and not global.something? What's the very good
> reason to open up another namespace for configuration variables?

I'll go with global.system.partitions. That way it's clear it's machine-specific
(system) and it's still within global.*.

Looking forward to any other feedback before I send v2.

Cheers,
Ahmad

> 
> Sascha
> 

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