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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <0d801367-da24-4596-83d9-08ccd89ca670@redhat.com>
Date: Wed, 7 May 2025 23:13:18 +0200
From: Hans de Goede <hdegoede@...hat.com>
To: Sakari Ailus <sakari.ailus@....fi>,
 Pratap Nirujogi <pratap.nirujogi@....com>
Cc: W_Armin@....de, ilpo.jarvinen@...ux.intel.com, mario.limonciello@....com,
 platform-driver-x86@...r.kernel.org, linux-kernel@...r.kernel.org,
 benjamin.chan@....com, bin.du@....com, gjorgji.rosikopulos@....com,
 king.li@....com, dantony@....com
Subject: Re: [PATCH v12] platform/x86: Add AMD ISP platform config for OV05C10

Hi Sakari,

On 6-May-25 5:37 PM, Sakari Ailus wrote:
> Hi Pratap,
> 
> On Mon, May 05, 2025 at 01:11:26PM -0400, Pratap Nirujogi wrote:
>> ISP device specific configuration is not available in ACPI. Add
>> swnode graph to configure the missing device properties for the
>> OV05C10 camera device supported on amdisp platform.
>>
>> Add support to create i2c-client dynamically when amdisp i2c
>> adapter is available.
>>
>> Co-developed-by: Benjamin Chan <benjamin.chan@....com>
>> Signed-off-by: Benjamin Chan <benjamin.chan@....com>
>> Reviewed-by: Mario Limonciello <mario.limonciello@....com>
>> Reviewed-by: Hans de Goede <hdegoede@...hat.com>
>> Reviewed-by: Armin Wolf <W_Armin@....de>
>> Signed-off-by: Pratap Nirujogi <pratap.nirujogi@....com>
>> ---

<snip>

>> +/*
>> + * Remote endpoint AMD ISP node definition. No properties defined for
>> + * remote endpoint node for OV05C10.
> 
> How will this scale? Can you use other sensors with this ISP? Although if
> you get little from firmware, there's not much you can do. That being said,
> switching to DisCo for Imaging could be an easier step in this case.

Note I've already talked to AMD about the way the camera setup
is currently being described in ACPI tables is suboptimal and
how they really should use proper ACPI description using e.g.
a _CRS with an I2cSerialBus resource for the sensor.

Although I must admit I did not bring up the ACPI DisCo for imaging
spec as something to also look at for future generations.

Note that there currently is hw shipping using the somewhat
broken ACPI sensor description this glue driver binds to,
so we're stuck with dealing with these ACPI tables as they
are already out there in the wild.

But yes for future hw generations it would be good to have
a better description of the hw in ACPI.

Regards,

Hans