Re: [PATCH v10 1/2] platform/x86: int3472: Add TPS68470 board data for Intel nvl

From: Sakari Ailus

Date: Wed May 13 2026 - 08:27:04 EST


Hi Arun,

On Wed, Apr 08, 2026 at 04:15:06PM +0530, Arun T wrote:
> The Intel NVL platform uses IPU8 is powered by a TPS68470 PMIC,requiring board
> data to configure the GPIOs and regulators for proper camera sensor operation.

Please rewrap this -- checkpatch.pl would have warned about it.

>
> Signed-off-by: Arun T <arun.t@xxxxxxxxx>
> Reviewed-by: Daniel Scally <dan.scally@xxxxxxxxxxxxxxxx>
> Reviewed-by: Hans de Goede <johannes.goede@xxxxxxxxxxxxxxxx>

...

> @@ -336,10 +418,27 @@ static const struct property_entry msi_p14_ai_evo_gpio_props[] = {
> { }
> };
>
> +static const struct property_entry intel_nvl_gpio_props[] = {
> + PROPERTY_ENTRY_BOOL("daisy-chain-enable"),

After Antti's patches this no longer applies.

Can you rebase the set on my int3472 branch
<URL:https://gitlab.freedesktop.org/linux-media/users/sailus>, please?

With this, I think it'd make sense to rename msi_prestige_ai_evo_gpio_props
and msi_prestige_ai_evo_tps68470_gpio_swnode as
int3472_tps68470_daisy_chain_gpio_swnode and
int3472_tps68470_daisy_chain_gpio_props, respectively, and use it here for
NVL as well.

Can you add a patch for that, to precede these two?

--
Kind regards,

Sakari Ailus