[PATCH 1/4] firmware: zynqmp: Swap addr_hi/low when PM_FPGA_LOAD is called

[Michal Simek]

On 12. 10. 20 5:34, Simon Glass wrote:
> On Wed, 7 Oct 2020 at 07:40, Michal Simek <michal.simek at xilinx.com> wrote:
>>
>> Don't know reason but in regular flow addr_hi/low are swapped in ATF. It
>> means when fpga load is done from EL3 there is a need to swap it for PMUFW
>> to load bitstream.
>>
>> Signed-off-by: Michal Simek <michal.simek at xilinx.com>
>> ---
>>
>>  drivers/firmware/firmware-zynqmp.c | 8 ++++++++
>>  1 file changed, 8 insertions(+)
> 
> Reviewed-by: Simon Glass <sjg at chromium.org>
> 
> Is this a bug?

I have never enable loading fpga from SPLthat's why from u-boot point of
view it is not a bug but adding new functionality.

>From ATF/PMU point of view they should never done it and I would
consider this as a bug there.

Thanks,
Michal