Re: [PATCH v12] platform/x86: Add AMD ISP platform config for OV05C10

["Nirujogi, Pratap" <pnirujog@amd.com>]

On 5/6/2025 8:53 AM, Ilpo Järvinen wrote:
> Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding.
> 
> 
> On Tue, 6 May 2025, Ilya K wrote:
> 
>>> +#define AMDISP_OV05C10_I2C_ADDR            0x10
>>> +#define AMDISP_OV05C10_PLAT_NAME   "amdisp_ov05c10_platform"
>>> +#define AMDISP_OV05C10_HID         "OMNI5C10"
>>> +#define AMDISP_OV05C10_REMOTE_EP_NAME      "ov05c10_isp_4_1_1"
>>> +#define AMD_ISP_PLAT_DRV_NAME              "amd-isp4"
>>
>> Hey folks, I know v12 might be a bit too late for this one, but I've got
>> another device here (Asus GZ302EA tablet) with a very similar camera
>> setup, but a different sensor (OV13B10), and it looks like this driver
>> just assumes a certain hardcoded configuration... I wonder if it makes
>> sense to reorganize the code so that more sensor configurations can be
>> added without making a separate module? I'd be happy to help with
>> refactoring/testing/etc, if people are interested.
> 
> v12 is not too late, and besides, v9..v12 has happened within 5 days
> which is rather short time (hint to the submitter that there's no need
> to burn patch series version numbers at that speed :-)).
> 
> I'll give folks some time to sort this out if you need to add e.g., some
> driver_data instead.
> 
> --
>   i.
> 
Hi Ilya, Ilpo,

I agree with the suggestion, but how about taking-up the refactoring 
part in a separate patch. Yes this patch focussed on supporting OV05C10 
and even the code review proceeded with this understanding. Refactoring 
now for generic support would require changes that would undo some of 
the recent review feedback (especially related to global variables 
usage). Please let us know what do you think.

Thanks,
Pratap