Re: [PATCH RESEND] media: camss: csiphy: Make CSIPHY status macro cross-platform

From: Vijay Kumar Tumati
Date: Mon Dec 01 2025 - 09:49:08 EST



On 12/1/2025 4:20 AM, Konrad Dybcio wrote:
On 12/1/25 7:25 AM, Hangxiang Ma wrote:
The current value of '0xb0' that represents the offset to the status
registers within the common registers of the CSIPHY has been changed on
the newer SOCs and it requires generalizing the macro using a new
variable 'common_status_offset'. This variable is initialized in the
csiphy_init() function.
"offset" + "common_status_offset" is confusing

Let's maybe add some platform data where we store the actual offset of
the registers in question and pass a csiphy ptr as an argument

Konrad
Hi Konrad, may be I didn't follow correctly. This is consistent with the way we maintain the other SOC specific reg offsets / data in the CSIPHY driver, in csiphy_device_regs, isn't it? I seem to think it's clearer this way for the reader to see all the offsets at one place. No? Thanks.