[PATCH 2/5] firmware: zynqmp: Load config overlay for core0 to pmufw

Stefan Herbrechtsmeier stefan.herbrechtsmeier-oss at weidmueller.com
Sat Jul 16 13:17:44 CEST 2022



Am 15.07.2022 um 18:34 schrieb Michal Simek:
> 
> 
> On 7/15/22 18:13, Stefan Herbrechtsmeier wrote:
>> Am 15.07.2022 um 11:39 schrieb Ashok Reddy Soma:
>>> Try loading pmufw config overlay for core0, if it doesn't return any
>>> error it means pmufw is accepting nodes for other IP's. Otherwise dont
>>> try to load config object for any other IP, just return from
>>> zynqmp_pmufw_node function.
>>>
>>> Signed-off-by: Ashok Reddy Soma <ashok.reddy.soma at xilinx.com>
>>> ---
>>>
>>>   drivers/firmware/firmware-zynqmp.c | 14 ++++++++++++++
>>>   1 file changed, 14 insertions(+)
>>>
>>> diff --git a/drivers/firmware/firmware-zynqmp.c 
>>> b/drivers/firmware/firmware-zynqmp.c
>>> index 34d9b47003..288151533e 100644
>>> --- a/drivers/firmware/firmware-zynqmp.c
>>> +++ b/drivers/firmware/firmware-zynqmp.c
>>> @@ -68,8 +68,13 @@ int zynqmp_pmufw_config_close(void)
>>>       return 0;
>>>   }
>>> +static bool config_enabled;
>>> +
>>
>> Please move the variable inside the function.
> 
> How can this work? When you move it to zynqmp_pmufw_node() then won't be 
> visible in zynqmp_power_probe() and vice-versa.

If you reuse the zynqmp_pmufw_node function in zynqmp_power_probe 
function you can check the id parameter to update the config_enabled 
variable in zynqmp_pmufw_node.


More information about the U-Boot mailing list