Re: [PATCH v2 2/2] cache: add SMCCC-backed cache invalidate provider

[Srirangan Madhavan]

On 7/2/26 6:04 AM, Bence Csókás wrote:
> I'm reading the 1.7 H BET0 [1] version of the spec (released
> 2026-04-20), and I see some discrepancies. Not sure if I'm the one
> reading it wrong, or if you had a different version than me.
>
> [1] https://developer.arm.com/documentation/den0028/h/?lang=en
>
> Either way, thanks for posting this patch, I'm also looking forward to
> seeing it get merged!

Hi Bence, 
 

 
 
 

Thanks for the review, and apologies for the delayed response.

I have addressed the declaration ordering, preserved the final BUSY or 
RATE_LIMITED error after retry exhaustion, removed the redundant feature 
query for the mandatory ATTRIBUTES call, retained the full 64-bit X3 
rate-limit value, and removed the routine registration message.

The macro values are already aligned to the same column using tabs, so I
left those unchanged. I also kept the status-to-errno conversion local 
for now because the mapping reflects this provider's handling policy and 
there is currently no other caller that would justify a common SMCCC 
API. Please let me know if this works, else I'll move it.

I've sent out these changes in v3.



--
Regards,
Srirangan