| Date | Tue, 9 Apr 2024 14:39:34 +0100 | Subject | Re: [PATCH v2 6/8] media: qcom: camss: Add new VFE driver for SM8550 | From | Bryan O'Donoghue <> |
| |
On 20/03/2024 14:11, Depeng Shao wrote: > +#define VFE_IRQ_STATUS(n) ((vfe_is_lite(vfe) ? 0x101C : 0x44) + (n) * 4)
These defines do the right thing for the RDI however
1. Please use 'rdi' not 'n' as the parameter here and 2. Pass 'vfe' explicitly not implicitly
Right now you need a variable named 'vfe' in the scope of the macro but that ought to be an explicit parameter passed to the macro.
--- bod
|