On 8/14/23 15:05, Andre Przywara wrote:
Yes, I will add this to the header file, either defined as 0, or to its
actual address.

Gotcha; my v2 will also assume you've taken care of merging these guys:
+#define SUNXI_CPUX_BASE                        0x09010000
+#define SUNXI_CPUCFG_BASE              0

(As I presume from your comments on 3/3 that it's better to consider "CPUX" simply an updated CPUCFG than to distinguish between them as fundamentally different concepts.)

Yes, but you handle both above explicitly.

No, I was passing a NULL clamp address in those cases too. So I'm noting that this must also be fixed in v2.

Thanks,
Sam

Reply via email to