From: Grant Likely <grant.likely@secretlab•ca>
To: Jean Delvare <khali@linux-fr•org>
Cc: Linuxppc-dev@ozlabs•org, lm-sensors@lm-sensors•org,
Detlev Zundel <dzu@denx•de>,
Alan Clucas <alanc@pipstechnology•co.uk>
Subject: Re: [PATCH] Support for DS75 thermal sensor
Date: Wed, 16 Jul 2008 08:08:07 -0600 [thread overview]
Message-ID: <20080716140807.GB24045@secretlab.ca> (raw)
In-Reply-To: <20080716121059.2dbbfebe@hyperion.delvare>
On Wed, Jul 16, 2008 at 12:10:59PM +0200, Jean Delvare wrote:
> On Wed, 16 Jul 2008 11:50:15 +0200, Wolfgang Grandegger wrote:
> > Jean Delvare wrote:
> >
> > Yep, as probing might not be acceptable in some cases, I makes sense to
> > add a property to suppress probing:
>
> It'd rather make no-probing the default if possible. My understanding
> is that all systems using i2c-mpc should have proper platform data.
Total ACK. From my perspective, probing should be off by default because the
typical use case in powerpc land is to trust data in the device tree. Add the
property to turn on probing, not to turn it off. Also, you'll need to
document the semantics of such a property. ie. what exactly does it
mean when the probing property is present and the spi bus node has child
nodes?
g.
next prev parent reply other threads:[~2008-07-16 14:08 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <487331DC.2020601@grandegger.com>
[not found] ` <20080708115319.111226e6@hyperion.delvare>
[not found] ` <4873670C.6080204@grandegger.com>
[not found] ` <20080708152935.7457bc90@hyperion.delvare>
[not found] ` <487754DA.1060207@grandegger.com>
[not found] ` <20080711145613.14380360@hyperion.delvare>
2008-07-16 9:12 ` [lm-sensors] [PATCH] Support for DS75 thermal sensor Wolfgang Grandegger
2008-07-16 9:33 ` Jean Delvare
2008-07-16 9:50 ` Wolfgang Grandegger
2008-07-16 10:10 ` Jean Delvare
2008-07-16 10:23 ` Wolfgang Grandegger
2008-07-16 14:08 ` Grant Likely [this message]
2008-07-16 14:18 ` Jon Smirl
2008-07-16 14:29 ` Jean Delvare
2008-07-17 7:31 ` Wolfgang Grandegger
2008-07-17 7:33 ` Grant Likely
2008-07-17 10:39 ` Wolfgang Grandegger
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=20080716140807.GB24045@secretlab.ca \
--to=grant.likely@secretlab$(echo .)ca \
--cc=Linuxppc-dev@ozlabs$(echo .)org \
--cc=alanc@pipstechnology$(echo .)co.uk \
--cc=dzu@denx$(echo .)de \
--cc=khali@linux-fr$(echo .)org \
--cc=lm-sensors@lm-sensors$(echo .)org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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