All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jean Delvare <khali@linux-fr.org>
To: lm-sensors@vger.kernel.org
Subject: Re: [lm-sensors] [PATCH 3/6] libsensors4: Don't substitute the isa
Date: Thu, 16 Aug 2007 07:40:51 +0000	[thread overview]
Message-ID: <20070816094051.3106db21@hyperion.delvare> (raw)
In-Reply-To: <20070815174020.055f6ea3@hyperion>

Hi Hans,

On Wed, 15 Aug 2007 18:56:36 +0200, Hans de Goede wrote:
> Jean Delvare wrote:
> > Don't handle the isa bus in sensors_parse_i2cbus_name(). We really
> > don't need to, as there is a single ISA bus, there's no need for
> > substituting anything.
> 
> This one requires deeper knowledge of libsensors then I have.

Bus substitution is how we addressed the fact that i2c bus numbers can
change across reboots. With bus statements in sensors.conf, you can
give an arbitrary (and persistent) bus number to an i2c bus based on
it's name. After that, whenever you refer to that bus number in the
configuration file (in a chip statement), it will mean that bus, even if
the current bus numbering is different. This mechanism is very useful
when you have similar I2C hardware monitoring chips on different buses
and each needs a specific configuration.

This bus substitution mechanism would also presumably be useful for
other bus types where several buses can exist on the system (w1, spi),
but for ISA, there can really be only one isa bus, so there's no risk
of bus number changing across reboot.

Not too sure about the PCI bus. So far I assumed that there was only
one, but it seems that on big systems there can be several.

As a side note, the i2c bus number substitution is only done for the
configuration file at the moment. If you want to specify a chip name on
sensors' command line, you still must use the bus numbers of the
running system. This should probably be changed. I'll create a ticket
for this.

-- 
Jean Delvare

_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

      parent reply	other threads:[~2007-08-16  7:40 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-08-15 15:40 [lm-sensors] [PATCH 3/6] libsensors4: Don't substitute the isa bus Jean Delvare
2007-08-15 16:56 ` [lm-sensors] [PATCH 3/6] libsensors4: Don't substitute the isa Hans de Goede
2007-08-16  7:40 ` Jean Delvare [this message]

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=20070816094051.3106db21@hyperion.delvare \
    --to=khali@linux-fr.org \
    --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.