All of lore.kernel.org
 help / color / mirror / Atom feed
From: Hans de Goede <j.w.r.degoede@hhs.nl>
To: lm-sensors@vger.kernel.org
Subject: Re: [lm-sensors] Fujitsu-Siemens Scylla (fscscy)
Date: Sun, 22 Jul 2007 09:27:55 +0000	[thread overview]
Message-ID: <46A3231B.1020506@hhs.nl> (raw)
In-Reply-To: <46933578.3010701@miloi.de>

Jean Delvare wrote:
> Hi Titus,
> 
> On Tue, 10 Jul 2007 09:30:00 +0200, Titus wrote:
>> I'm new to this mailing list and I have some questions to the fscscy 
>> module. I have searchted the archives and this subject does not seem to 
>> have been posted already, so I hope I don't bother you with issues 
>> already discussed.
>>
>> - I did not manage to load the fscscy kernel module into kernel 2.6.18 
>> (Debian Etch) and 2.6.21 (current kernel from kernel.org). Although all 
>> seems to be compiled correctly, this kernel module was not built. Is 
>> there any way to fix that?
> 
> The fscscy driver was not ported to Linux 2.6 yet. There was a first
> request one year and a half ago:
> http://lists.lm-sensors.org/pipermail/lm-sensors/2006-February/015319.html
> http://lists.lm-sensors.org/pipermail/lm-sensors/2006-March/015489.html
> 
> Nobody volunteered yet to do the work. I've added your request on our
> wiki/Devices page, but don't hold your breath. The FSC Scylla is an old
> and rare chip, it's unlikely that anyone will volunteer to port it for
> free.
> 

Well actually, I would like to volunteer, as it fits within my current fscher / 
fscpos driver activities. I've done some checking and the fscscy is very much 
like the fscher / fscpos, and it has the tempX_limit registers right were my 
reverse engineering found them in the fscher :) As always if anyone has a 
datasheet for the fscscy, that would be very welcome.

With the possibility to add fscscy support to the driver and with my wish to 
rip out the watchdog support (for now, I might redo it with the official kernel 
api later, esp. if there are requests for it) + the possibility for many other 
cleanups, I'm starting to think that it would be (much) easier to add a new 
fscxxx driver to the kernel, based on my current fscher work, with added fscscy 
support and cleanups.

Jean & Mark, were do you stand with regards to this., I see 2 options:

1) * Many smaller patches with incremental improvements to the fscher
    * essentially making it an fscxxx driver
    * with the current pseudo watchdog support left in for compatibility
    * with other uglies like raw export of status registers left in for
      compatibility, while also exporting the exact same info with _alarm and
      _fault files

2) A new fscxxx driver tackling all the issues at once, based on my current
    fscher work, with some major cleanup and added fscscy support

As I type this, and also remind myself that the current driver has to keep 
carying the tempX_status ugliness, my vote strongly goes to the new driver 
approach. Then we can mark the fscpos and fscher drivers as obsolete for a 
while and remove them eventually.

Titus, would you be willing to test fscscy support for us?

Regards,

Hans

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

  parent reply	other threads:[~2007-07-22  9:27 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-07-10  7:30 [lm-sensors] Fujitsu-Siemens Scylla (fscscy) Titus
2007-07-22  8:33 ` Jean Delvare
2007-07-22  9:27 ` Hans de Goede [this message]
2007-07-22 20:53 ` Jean Delvare
2007-07-23 14:33 ` Hans de Goede
2007-07-24 11:58 ` Jean Delvare
2007-07-24 14:40 ` Hans de Goede
2007-08-12 10:01 ` 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=46A3231B.1020506@hhs.nl \
    --to=j.w.r.degoede@hhs.nl \
    --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.