All of lore.kernel.org
 help / color / mirror / Atom feed
From: yani.ioannou@gmail.com (Yani Ioannou)
To: lm-sensors@vger.kernel.org
Subject: [lm-sensors] Re: [RFC PATCH 2.6.12-rc3-mm2 1/2] i2c: new sysfs
Date: Mon, 23 May 2005 05:37:40 +0000	[thread overview]
Message-ID: <2538186705052220372c2d6162@mail.gmail.com> (raw)
In-Reply-To: <20050503034928.GC4977@jupiter.solarsys.private>

Hi Mark,

> I thought I would finish the userspace/libsensors modifications first,
> then I could shake out any interface problems.  As I wrote earlier:
...
> I was thinking of passing the class device ID string as a parameter to
> hwmon_device_register() instead of pulling it out of dev->bus_id.  If I
> were to re-submit the patch for inclusion (w/ the changes suggested by
> Greg)... what would you use for the class device ID string for bmcsensors?

err... I don't know...not bus id :-). bmcsensors works using the
kernel IPMI messaging interface, and that really isn't a bus at
all...its more like a messaging protocol, hence the need in my mind to
remove all the i2c references.

Jean was explaining to me today that this patch just adds the sysfs
clsss and drivers would still be i2c based, I was under the mistaken
impression it was standalone, but as your e-mail stated that is yet to
be implemented.

I fear that bmcsensors really needs the full separation from i2c, how
realistic would it be to go ahead and try something like that at this
stage (I can help out)? It would seem to me that basically everything
under drivers/i2c/chips should be moved into drivers/hwmon and perhaps
drivers/i2c/bus to drivers/i2c (although the latter isn't necessary,
it just makes sense to me).

> If we can have such a patch included on the condition that it may still
> change (due to interface / naming conventions) then I will rework it
> and submit it on Wed or Thu.  Maybe we can keep it in -mm for an extended
> period until I finish the libsensors stuff.  Would that work for you?

Yes, I need the dynamic sysfs stuff that will be in -mm anyway, I so
that's perfectly fine, I expect all of this to be in testing for a
while.

> Mark (hacking on his house instead of his computer) Hoffman

Breaking things takes on all new meaning with that type of hacking, be
careful ;-)

Thanks,
Yani

  parent reply	other threads:[~2005-05-23  5:37 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-05-19  6:25 [RFC PATCH 2.6.12-rc3-mm2 1/2] i2c: new sysfs class "hwmon" Mark M. Hoffman
2005-05-19  6:25 ` Greg KH
2005-05-21 15:22 ` [lm-sensors] Re: [RFC PATCH 2.6.12-rc3-mm2 1/2] i2c: new sysfs Yani Ioannou
2005-05-23  4:01 ` Mark M. Hoffman
2005-05-23  5:37 ` Yani Ioannou [this message]
2005-05-23 14:48 ` [lm-sensors] Re: [RFC PATCH 2.6.12-rc3-mm2 1/2] i2c: new sysfs class Jean Delvare
2005-05-23 16:28 ` [lm-sensors] Re: [RFC PATCH 2.6.12-rc3-mm2 1/2] i2c: new sysfs Yani Ioannou
2005-05-25  3:42 ` Yani Ioannou
2005-05-26  5:38 ` Mark M. Hoffman
2005-05-26  5:55 ` Mark M. Hoffman
2005-05-26  6:18 ` Yani Ioannou

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=2538186705052220372c2d6162@mail.gmail.com \
    --to=yani.ioannou@gmail.com \
    --cc=lm-sensors@vger.kernel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.