On Wed, Jul 13, 2022 at 11:20:43AM +0530, Satya Priya Kakitapalli (Temp) wrote:
> On 7/12/2022 6:17 PM, Lee Jones wrote:
> > On Mon, 11 Jul 2022, Satya Priya Kakitapalli (Temp) wrote:
> > 
> > > On 7/4/2022 6:19 PM, Lee Jones wrote:
> > > > On Mon, 04 Jul 2022, Satya Priya Kakitapalli (Temp) wrote:
> > > > 
> > > > > On 7/1/2022 2:42 PM, Lee Jones wrote:
> > > > > > On Fri, 01 Jul 2022, Satya Priya Kakitapalli (Temp) wrote:
> > > > > > 
> > > > > > > On 7/1/2022 1:24 PM, Lee Jones wrote:
> > > > > > > > On Fri, 01 Jul 2022, Satya Priya Kakitapalli (Temp) wrote:
> > > > > > > > 
> > > > > > > > > On 6/30/2022 4:04 PM, Lee Jones wrote:
> > > > > > > > > > On Thu, 30 Jun 2022, Satya Priya Kakitapalli (Temp) wrote:
> > > > > > > > > > 
> > > > > > > > > > > On 6/29/2022 8:48 PM, Lee Jones wrote:
> > > > > > > > > > > > On Wed, 29 Jun 2022, Satya Priya Kakitapalli (Temp) wrote:

Please delete unneeded context from mails when replying.  Doing this
makes it much easier to find your reply in the message, helping ensure
it won't be missed by people scrolling through the irrelevant quoted
material.

> > That's not to say that the v9 submission was the right way to go
> > either.  Everything in relation to:
> > 
> >    i2c_add_driver(&pm8008_regulators_driver);
> > 
> > ... should be moved into the Regulator driver itself.

> Mark/Stephen,
> 
> 
> Could you please share your inputs on this approach?

I don't have particularly strong opinions.  It does seem like the chip
should be visible as being a single entity in DT if it's a single
physical thing in the schematic and board.