public inbox for linux-i2c@vger.kernel.org
 help / color / mirror / Atom feed
From: Michael Lawnick <ml.lawnick-Mmb7MZpHnFY@public.gmane.org>
To: Jean Delvare <khali-PUYAD+kWke1g9hUCZPvPmw@public.gmane.org>
Cc: linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH]i2c-dev: add ioctls PROBE and REMOVE
Date: Tue, 07 Apr 2009 14:17:46 +0200	[thread overview]
Message-ID: <49DB446A.2000705@gmx.de> (raw)
In-Reply-To: <20090407121550.7d15bfae-ig7AzVSIIG7kN2dkZ6Wm7A@public.gmane.org>

Hi Jean,

although you are very busy, I hope you can spend a little more time.

Jean Delvare said the following:
> Hi Michael,
> 
> Sorry for the late answer, I am pretty busy with the 2.6.30 merge
> window.
> 
> ... The whole point of the sysfs entries you are adding is to let the
> user instantiate I2C devices (at the device driver model level) for
> physical devices which can _not_ be detected. 

I see, that's _your_ motivation. You want a user 'force' method.

...
> I hear you, "what about hot-plugged devices, they won't be detected"?
> For one thing, these need to be behind a mux or gate, which we don't
> support at the moment, so discussing this now doesn't make much sense.

No, sorry, here your are mistaken.
IMHO you recognize only a part of the problem: Even if separation is
done by a mux (what is not strictly necessary), it will be powered by
main system and its driver (i.e. adapters) loaded on boot. Still clients
may come up when system is already initialized and auto detection on the
sub buses has already completed.

> But at any rate I don't think that the files you are adding in sysfs
> are meant to address this particular issue.

Sorry, then you misunderstood me - it *is* this particular issue I need
to solve.
Please forget the thread about lm75a (vs. lm75), this was just a stone
on my way to a new running kernel. It took it, made a remark and threw
it aside. Nothing more. It is not related to the detect problem.

> If we want to be able to
> re-trigger detection on a given I2C segment (which currently only
> happens at I2C adapter creation time and when a new driver is
> registered) this would be done by a separate sysfs entry (say "detect",
> write 1 to re-trigger all detections on a given segment).

This solution will cause much useless traffic while the system is
already running and doing other hard work :-(

> But really
> I'd rather wait for Rodoflo's work to be merged first, depending on the
> implementation there may or may not be a practical need for this.

AFAICS has Rodolfo's work nothing to do with the problem above, as on
adapter creation an auto detection will (should) be triggered.

-- 
Michael Lawnick

  parent reply	other threads:[~2009-04-07 12:17 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-03-26  8:59 [PATCH]i2c-dev: add ioctls PROBE and REMOVE Michael Lawnick
     [not found] ` <49CB43E0.2050503-Mmb7MZpHnFY@public.gmane.org>
2009-03-27 10:19   ` Michael Lawnick
     [not found]     ` <49CCA82F.90106-Mmb7MZpHnFY@public.gmane.org>
2009-03-27 12:40       ` Jean Delvare
     [not found]         ` <20090327134034.754908b5-ig7AzVSIIG7kN2dkZ6Wm7A@public.gmane.org>
2009-03-27 13:13           ` Michael Lawnick
     [not found]             ` <49CCD0FC.9030500-Mmb7MZpHnFY@public.gmane.org>
2009-03-27 13:57               ` Jean Delvare
     [not found]                 ` <20090327145726.13c99aed-ig7AzVSIIG7kN2dkZ6Wm7A@public.gmane.org>
2009-03-27 14:07                   ` Jean Delvare
     [not found]                     ` <20090327150716.094666d1-ig7AzVSIIG7kN2dkZ6Wm7A@public.gmane.org>
2009-04-02 12:36                       ` Michael Lawnick
     [not found]                         ` <49D4B14B.50405-Mmb7MZpHnFY@public.gmane.org>
2009-04-07 10:15                           ` Jean Delvare
     [not found]                             ` <20090407121550.7d15bfae-ig7AzVSIIG7kN2dkZ6Wm7A@public.gmane.org>
2009-04-07 12:17                               ` Michael Lawnick [this message]
     [not found]                                 ` <49DB446A.2000705-Mmb7MZpHnFY@public.gmane.org>
2009-04-07 13:11                                   ` Jean Delvare

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=49DB446A.2000705@gmx.de \
    --to=ml.lawnick-mmb7mzphnfy@public.gmane.org \
    --cc=khali-PUYAD+kWke1g9hUCZPvPmw@public.gmane.org \
    --cc=linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    /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