Re: [PATCH v3 RESEND] media: i2c: Add OV05C10 camera sensor driver

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

On 6/13/2025 8:05 AM, Bryan O'Donoghue wrote:
> Caution: This message originated from an External Source. Use proper 
> caution when opening attachments, clicking links, or responding.
> 
> 
> On 13/06/2025 12:02, Kieran Bingham wrote:
>> Quoting Hao Yao (2025-06-13 05:55:46)
>>> Hi Pratap,
>>>
>>> Thanks for your patch.
>>>
>>> This patch is written for your camera sensor module, which seems very
>>> different from those already applied on Dell laptops (some of "Dell Pro"
>>> series). Looking into the driver, I think this version will break the
>> Have there been existing efforts from Intel to upstream support for that
>> device?
> 
> FWIW +1
> 
> Qualcomm devices - Acer Swift 14 AI, HP OmniBook x14 both use this sensor.
> 
> I'd expect though that aside from OF bindings, regulators and clocks
> that any upstream configuration with the right number of lanes would
> "just work", including this one from AMD.
> 
> That has been the experience picking up OV02E10 and OV02C10 from the
> IPU6 repository where its ACPI binding and repurposing to OF/Qcom.
> 
> So how incompatible could OV05C10 be between different x86/ACPI systems
> ? Less than the gap between x86/ACPI and Arm/OF you'd imagine.
> 
> Getting any OV05C10 driver upstream would be great, we can work from
> there to bridge whatever gap needs be for !AMD.
> 
Hi Bryan, thanks for your support and we will be happy to contribute too 
to bridge the gaps and make the driver as generic as possible to support 
both AMD && !AMD platforms.

Thanks,
Pratap
> ---
> bod