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

[Nirujogi, Pratap]

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