Re: [PATCH v6 3/4] platform: int3472: discrete: con_id vana for Sony IMX471 as power enable

[Sakari Ailus]

Hi Hans, Tarang,

On Thu, Jul 02, 2026 at 08:05:25PM +0200, Hans de Goede wrote:
> > I went through the INT3472 driver and would like to propose a generic     
> > approach that satisfies both sides without per-HID quirks or sensor driver
> > changes.                                                                  
> >                                                                           
> > The problem is:                                                      
> >  - INT3472 standardizes on "avdd" internally                             
> >  - Sony IMX sensor drivers use "vana" per datasheet, and all existing    
> >    Sony DT bindings (imx219, imx290, imx415) already use vana-supply     
> >  - Changing imx471 to "avdd" now will create inconsistency with those    
> >    bindings, or require a rename later
> 
> Ack, as mentioned in my reply to Sakari from 1 minute ago I'm ok
> with sticking with vana for the imx* case,

At least some Sony sensors use "INT" PnP vendor prefix and so telling them
apart from the rest doesn't work at least this way. There could also be
other prefixes as well, they're not all "SONY". Right now there is one with
INT prefix and three with SONY prefix.

If we start having lots of devices with the same quirk, we could also
introduce a pointer to an array of IDs to avoid repeating the same quirk
over and over.

Kate's patch adds two quirks so this could be already considered (and only
one of these IDs is using SONY prefix).

-- 
Regards,

Sakari Ailus