Re: [PATCH] regulator: core: use correct device for device supply lookup

[Mark Brown <broonie@opensource.wolfsonmicro.com>]

[-- Attachment #1: Type: text/plain, Size: 1182 bytes --]

On Sat, May 19, 2012 at 10:50:54PM +0530, Laxman Dewangan wrote:

> > static struct regulator_dev *regulator_dev_lookup(struct device *dev,
> >                                                   const char *supply,
> >                                                   int *ret)
> >

> Also in regulator_register we set the of_node as

>        rdev->dev.of_node = config->of_node;

So, here we're just setting it to whatever the driver told us to use
which seems about right.

>         rdev->dev.parent = dev;

And here we're parenting the class device with the real device we were
passed which again seems obvious.

> Passed config->of_node will only be used if we pass the rdev->dev,
> not rdev->dev.parent

What does this mean and how is it related to anything you've said above?
I'm sorry but I really can't make head nor tail of what you're trying to
say here, there's just lots and lots of statements here but I'm
struggling to understand how they are related to each other.

> Am I missing anything here in understanding?

I certainly am.  Please go back to square one: what's the problem you
are seeing here?  Then go forward and step by step relate it to the code
change.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]