All of lore.kernel.org
 help / color / mirror / Atom feed
From: r.marek@sh.cvut.cz (Rudolf Marek)
To: lm-sensors@vger.kernel.org
Subject: [lm-sensors] W83627EHF driver enhancements
Date: Sat, 01 Apr 2006 19:35:50 +0000	[thread overview]
Message-ID: <442ED616.4070109@sh.cvut.cz> (raw)

Hello,

I'm begining new thread so we are not lost in three past threads.

Let me summarize what we have now:

Me: 23.03.2006 23:08
     I posted the patch on the top of all changes (alarms, volts)
You: 24.03.2006
       You posted the clean up patch and the DC 0 and PWM 1 patch.

You: Re: [lm-sensors] [PATCH] Add fan control support to W83627EHF
      We agreed to have RPM cruise mode with new sysfs files for this, no patch yet.

I really like your cleanup patch but we cannot use it for the final file until we have finished the fans stuff.
There are two ways:

1) split it up to before the fan control patch and after fancontrol patch applied (more work)
2) apply it on the top of fancontrol patch when we are done with it (little extra work, just rip the DC change from the patch,
   plus little tweaking when there is new code in he file, no need to rework whole cleanup)

I propose this steps:

1) I will document that we have 0 for DC and 1 for PWM (patch to kernel doc)
2) You will send me the patch on the top of that one from 23.3 that fixes the DC 0 and PWM 1
   plus the manual stuff, optionaly you may include the new RPM cruise files features. This can be done in one patch because it
   is just "our" internal step.
   When we are done with this I will post the final patch and submit it to Jean for inclusion
3) Now you can send the cleanup patch on the top of previous patch (With the Signed-off-by stuff)

I'm sorry for late responces but I have simply lot of to do.

Thanks,
Regards
Rudolf


             reply	other threads:[~2006-04-01 19:35 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-04-01 19:35 Rudolf Marek [this message]
2006-04-02  5:43 ` [lm-sensors] W83627EHF driver enhancements David Hubbard
2006-04-02  5:46 ` David Hubbard
2006-04-02 10:09 ` Rudolf Marek
2006-04-12  1:04 ` David Hubbard
2006-04-16 21:21 ` Rudolf Marek
2006-05-10 17:07 ` Rudolf Marek
2006-05-10 19:01 ` David Hubbard
2006-05-10 19:12 ` Rudolf Marek
2006-05-10 20:25 ` David Hubbard
2006-05-11 15:25 ` David Hubbard
2006-05-12  6:44 ` Sylvain Jeaugey
2006-05-14 20:55 ` Rudolf Marek
2006-05-15 14:14 ` David Hubbard
2006-05-15 16:47 ` Jean Delvare
2006-05-15 18:37 ` Sylvain Jeaugey
2006-05-15 20:17 ` David Hubbard
2006-05-15 21:11 ` Rudolf Marek
2006-05-20 13:57 ` Rudolf Marek
2006-06-04 18:08 ` Rudolf Marek

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=442ED616.4070109@sh.cvut.cz \
    --to=r.marek@sh.cvut.cz \
    --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.