From: Hollis Blanchard <hollis_blanchard@mentor.com>
To: Scott Wood <scottwood@freescale.com>
Cc: Hunter Cobbs <hunter.cobbs@gmail.com>,
devicetree-discuss@lists.ozlabs.org,
linuxppc-dev@lists.ozlabs.org
Subject: Re: "status" property checks
Date: Fri, 08 Jan 2010 11:45:28 -0800 [thread overview]
Message-ID: <1262979928.31871.81.camel@localhost.localdomain> (raw)
In-Reply-To: <4B478747.8070009@freescale.com>
On Fri, 2010-01-08 at 13:28 -0600, Scott Wood wrote:
> Hollis Blanchard wrote:
> > On Thu, 2010-01-07 at 20:35 -0600, Hunter Cobbs wrote:
> >> I think that is definitely a solution. It does centralize the testing
> >> for this particular issue. The only thing question I have is if its
> >> really better to have the upper level do the check. Shouldn't the
> >> driver itself handle the hardware and device node status?
> >=20
> > Practically speaking, all drivers doing the checks today just return
> > -ENODEV. They don't try to do anything to "handle" the situation.
> >=20
> > The definition of the status property implies it's outside of software'=
s
> > control, for example:
> > "disabled"
> > "Indicates that the device is not presently operational, but it
> > might become operational in the future (for example, something
> > is not plugged in, or switched off)."
> >=20
> > If a device is "not operational" in this sense, I don't think there's
> > anything for a device driver to do.
>=20
> I could see situations where there is some software action that could=20
> enable the device (e.g. multiple devices sharing pins, where only one=20
> can be active at a time) -- but it's likely to not be the driver itself=20
> that knows how to do that.
>=20
> If the need arises, there could be a mechanism where the enabling entity=20
> can tell the platform bus that it has enabled a previously-disabled=20
> device, overriding the status in the device tree (and likewise if it=20
> wants take down a device that was previously enabled).
OK, that makes sense to me. I'll put together a patch for the original
idea, and the enable/disable part can come later as needed.
--=20
Hollis Blanchard
Mentor Graphics, Embedded Systems Division
next prev parent reply other threads:[~2010-01-08 19:45 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-01-07 23:07 "status" property checks Hollis Blanchard
2010-01-08 2:35 ` Hunter Cobbs
2010-01-08 18:34 ` Hollis Blanchard
2010-01-08 19:28 ` Scott Wood
2010-01-08 19:45 ` Hollis Blanchard [this message]
2010-01-08 23:46 ` David Gibson
2010-01-08 23:58 ` Hollis Blanchard
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=1262979928.31871.81.camel@localhost.localdomain \
--to=hollis_blanchard@mentor.com \
--cc=devicetree-discuss@lists.ozlabs.org \
--cc=hunter.cobbs@gmail.com \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=scottwood@freescale.com \
/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).