From: Bjorn Andersson <bjorn.andersson@linaro.org>
To: Lee Jones <lee.jones@linaro.org>
Cc: Rob Herring <robh+dt@kernel.org>,
Mark Rutland <mark.rutland@arm.com>,
Jonathan Cameron <jic23@kernel.org>,
Hartmut Knaack <knaack.h@gmx.de>,
Lars-Peter Clausen <lars@metafoo.de>,
Peter Meerwald-Stadler <pmeerw@pmeerw.net>,
Richard Purdie <rpurdie@rpsys.net>,
Jacek Anaszewski <j.anaszewski@samsung.com>,
Pavel Machek <pavel@ucw.cz>, Jingoo Han <jingoohan1@gmail.com>,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-iio@vger.kernel.org, linux-leds@vger.kernel.org,
Bjorn Andersson <bjorn.andersson@sonymobile.com>
Subject: Re: [PATCH v4 2/5] mfd: lm3533: Support initialization from Device Tree
Date: Wed, 11 Jan 2017 11:55:21 -0800 [thread overview]
Message-ID: <20170111195521.GB10531@minitux> (raw)
In-Reply-To: <20170109083638.GB2155@dell>
On Mon 09 Jan 00:36 PST 2017, Lee Jones wrote:
> On Fri, 06 Jan 2017, Bjorn Andersson wrote:
>
> > On Fri 06 Jan 01:53 PST 2017, Lee Jones wrote:
> >
> > > On Thu, 05 Jan 2017, Bjorn Andersson wrote:
> > >
> > > > On Wed 04 Jan 23:49 PST 2017, Lee Jones wrote:
> > > >
> > > > > On Wed, 04 Jan 2017, Bjorn Andersson wrote:
> > > > >
> > > > > > On Wed 04 Jan 03:54 PST 2017, Lee Jones wrote:
> > > > > >
> > > > > > > On Mon, 26 Dec 2016, Bjorn Andersson wrote:
> > > > > > >
> > > > > > > > From: Bjorn Andersson <bjorn.andersson@sonymobile.com>
> > > > > > > >
> > > > > > > > Implement support for initialization of the lm3533 driver core and
> > > > > > > > probing child devices from Device Tree.
> > > > > > > >
> > > > > >
> > > > > > [..]
> > > > > >
> > > > > > > > @@ -512,6 +514,11 @@ static int lm3533_device_init(struct lm3533 *lm3533)
> > > > > > > > lm3533_device_bl_init(lm3533);
> > > > > > > > lm3533_device_led_init(lm3533);
> > > > > > > >
> > > > > > > > + if (lm3533->dev->of_node) {
> > > > > > > > + of_platform_populate(lm3533->dev->of_node, NULL, NULL,
> > > > > > > > + lm3533->dev);
> > > > > > > > + }
> > > > > > >
> > > > > > > I think it's save to call of_platform_populate(), even if !of_node.
> > > > > > > It will just fail and return an error code, which you are ignoring
> > > > > > > anyway.
> > > > > > >
> > > > > >
> > > > > > I thought so too, but that's apparently how you trigger probing children
> > > > > > of the root node. So we're stuck with a conditional.
> > > > >
> > > > > Ah, so this is to protect against the case where DT is present, but a
> > > > > node for this device is not (or is disabled), so is left unprobed.
> > > > > Then the bind is initiated via I2C? Or something else?
> > > > >
> > > >
> > > > In the event that a new lm3533 is spawned from sysfs we would not have
> > > > platform_data when entering lm3533_device_init() and just bail early.
> > > >
> > > > Therefor, this issue would be limited to the odd case of lm3533 being
> > > > initiated from code (e.g. a board file) on a DT enabled system. In which
> > > > case it will create and probe new devices from the root of the DT.
> > >
> > > Eewww, do we really want to support that?
> > >
> >
> > As long as we support non-DT probing of the driver this is a possible
> > scenario. And with modern ARM being DT-centric I think that if anyone
> > uses this driver with a modern version of the Linux kernel it's likely
> > that they would have this kind of hybrid solution.
> >
> > So, although ugly, I think we should keep this conditional and hope that
> > anyone using the driver will transition to use the DT binding.
>
> Very well, but can you add a comment describing the reason for its
> existence with a view to removing it further down the line?
>
Sounds reasonable, I will prepare an updated patch with this.
Regards,
Bjorn
next prev parent reply other threads:[~2017-01-11 19:55 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-12-26 18:11 [PATCH v4 1/5] devicetree: mfd: Add binding for the TI LM3533 Bjorn Andersson
2016-12-26 18:11 ` [PATCH v4 2/5] mfd: lm3533: Support initialization from Device Tree Bjorn Andersson
2017-01-04 11:54 ` Lee Jones
2017-01-04 19:26 ` Bjorn Andersson
2017-01-05 7:49 ` Lee Jones
2017-01-05 16:30 ` Bjorn Andersson
2017-01-06 9:53 ` Lee Jones
2017-01-06 18:54 ` Bjorn Andersson
2017-01-09 8:36 ` Lee Jones
2017-01-11 19:55 ` Bjorn Andersson [this message]
2016-12-26 18:11 ` [PATCH v4 3/5] backlight: " Bjorn Andersson
2016-12-27 10:46 ` Pavel Machek
2016-12-27 18:23 ` Jingoo Han
2016-12-27 10:49 ` Pavel Machek
2016-12-30 19:50 ` [PATCH v4 1/5] devicetree: mfd: Add binding for the TI LM3533 Jonathan Cameron
2017-01-03 16:56 ` Rob Herring
2017-01-04 11:54 ` Lee Jones
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20170111195521.GB10531@minitux \
--to=bjorn.andersson@linaro.org \
--cc=bjorn.andersson@sonymobile.com \
--cc=devicetree@vger.kernel.org \
--cc=j.anaszewski@samsung.com \
--cc=jic23@kernel.org \
--cc=jingoohan1@gmail.com \
--cc=knaack.h@gmx.de \
--cc=lars@metafoo.de \
--cc=lee.jones@linaro.org \
--cc=linux-iio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-leds@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=pavel@ucw.cz \
--cc=pmeerw@pmeerw.net \
--cc=robh+dt@kernel.org \
--cc=rpurdie@rpsys.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).