Re: [PATCH v4 3/3] media: i2c: imx471: Add Sony IMX471 image sensor driver

[Tarang Raval]

Hi Kate,

> > > +static const char * const imx471_supply_name[] = {
> > > +       "avdd",
> > > +};
> >
> > Only avdd is defined as a regulator supply.
> >
> > According to the datasheet, are there any additional power rails
> > (e.g. dvdd or iovdd) required by the sensor?
> I only found avdd on X1 Carbon G14, my X9-14 and 15. I only have this
> one regulator to test and ensure it runs.
> According to the datasheet, I found "VNAN, VDIG, and VIF". Should I
> add them to the list?

If I'm not mistaken, avdd corresponds to the sensor's analog supply (vnan),
So it may be better to name it vnan. 
                                                                           
For the other supplies, I don't think we should add them without hardware
validation, as additional regulators can affect the power-up sequence.     

Best Regards,
Tarang