All of lore.kernel.org
 help / color / mirror / Atom feed
From: Wolfgang Grandegger <wg@grandegger.com>
To: lm-sensors@vger.kernel.org
Subject: Re: [lm-sensors] New style driver for w83781d?
Date: Tue, 12 Aug 2008 09:28:54 +0000	[thread overview]
Message-ID: <48A157D6.5080200@grandegger.com> (raw)
In-Reply-To: <4899A4F7.70201@grandegger.com>

Jean Delvare wrote:
> Hi Wolfgang,
> 
> On Tue, 12 Aug 2008 10:55:58 +0200, Wolfgang Grandegger wrote:
>> We need support for the MON35W42 on an embedded MPC8544 board, which 
>> seems to be compatible with the w83782d. There are more issues, e.g. the 
>> early ISA bus probing of the existing driver hangs the system. Maybe i2c 
>> probing should be done first!?
> 
> If both interfaces are available, we want to use the ISA one, because
> it's much faster, so it must be registered first.
> 
> What could be done though would be disabling the ISA interface
> altogether on architectures where it isn't supported (in particular
> PPC.) This shouldn't be too difficult. If you write a patch doing this,
> I'll be happy to review it. Affected drivers would be w83781d and lm78.

Some PPC have an ISA bus as well:

   http://lxr.linux.no/linux+v2.6.26.2/arch/powerpc/Kconfig#L494

I think selecting the ISA code with '#ifdef CONFIG_ISA' would be the 
right solution?

Wolfgang.

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

  parent reply	other threads:[~2008-08-12  9:28 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-08-06 13:19 [lm-sensors] New style driver for w83781d? Wolfgang Grandegger
2008-08-06 13:40 ` Jean Delvare
2008-08-07  6:49 ` Wolfgang Grandegger
2008-08-07  7:42 ` Jean Delvare
2008-08-12  8:55 ` Wolfgang Grandegger
2008-08-12  8:59 ` Jean Delvare
2008-08-12  9:28 ` Wolfgang Grandegger [this message]
2008-08-12  9:50 ` Jean Delvare
2008-08-13 10:14 ` Wolfgang Grandegger
2008-08-13 13:18 ` Jean Delvare
2008-08-13 13:48 ` Wolfgang Grandegger
2008-08-13 14:05 ` Wolfgang Grandegger
2008-08-13 14:13 ` 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=48A157D6.5080200@grandegger.com \
    --to=wg@grandegger.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.