[PATCH] Fix detection of odd memory configurations on sunxi
Jernej Škrabec
jernej.skrabec at gmail.com
Mon Sep 8 16:11:41 CEST 2025
Dne sobota, 16. avgust 2025 ob 14:18:44 Srednjeevropski poletni čas je Rudi Horn napisal(a):
> Hi Jernej,
>
> thanks for the comments on the patch.
>
> > I don't see a benefit for public function here, since it's unlikely
> to be > used anywhere else. Just mark it static. Compiler can optimize
> it better.
>
> My motivation for this was that the implementation relies on subtracting
> `sizeof(u32)` from the address. This is an implementation detail that relies
> on the internals of `mctl_mem_matches_base`, and so I wanted to place
> it in the same file as `mctl_mem_matches_base`, preventing it from being
> static. It will only be called twice, so I don't think optimization is
> too much
> of an issue. However I don't feel strongly on this if you would still prefer
> it in the same file.
>
> Once this is resolved, I will submit another patch with all of the other
> comments addressed.
Ah, right. You can leave as it is.
Best regards,
Jernej
>
> Thanks,
> Rudi
>
>
More information about the U-Boot
mailing list