linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Mark Brown <broonie@kernel.org>
To: Guenter Roeck <linux@roeck-us.net>
Cc: Jean Delvare <khali@linux-fr.org>,
	LM Sensors <lm-sensors@lm-sensors.org>,
	linux-kernel@vger.kernel.org, Liam Girdwood <lgirdwood@gmail.com>,
	Wei Ni <wni@nvidia.com>
Subject: Re: lm90 driver no longer working on PCs in 3.13
Date: Sun, 26 Jan 2014 23:37:03 +0000	[thread overview]
Message-ID: <20140126233703.GO11727@sirena.org.uk> (raw)
In-Reply-To: <52E585E8.4000100@roeck-us.net>

[-- Attachment #1: Type: text/plain, Size: 1427 bytes --]

On Sun, Jan 26, 2014 at 02:02:16PM -0800, Guenter Roeck wrote:
> On 01/26/2014 01:53 PM, Mark Brown wrote:

As previously mentioned please fix your mailer to word wrap at less than
80 columns.

> >Oh, magic.  Can you please file a bug with or otherwise talk to them
> >asking them to turn that off (or contribute code to make it work)?  Feel
> >free to add me as a CC when you do so.

> That means they would have to stop supporting anything that _does_
> need regulators, which doesn't make much sense to me.

> I think the bug is on your side, not theirs.

Regardless of how we deal with this with the code that Ubuntu actually
have in the kernel releases you highlighted has isn't intended to work
well with what they're doing.  Like I said in another mail ACPI telling
the regulator core that everything can be stubbed would be better but
with what's there right now my immediate recommendation would be to turn
off the config option since we know having it on can cause problems.  No
matter what we do in mainline it's not going to change already released
kernels and they can do this right now.

I'm open to surprise but I'm relatively confident that there aren't x86
systems running a mainline kernel without out of tree code that would be
affected (which would be required to set up supplies to do anything
useful).  Anything using dummy regulators only will get equivalent
behaviour by turning off the API completely.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

  reply	other threads:[~2014-01-26 23:37 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-01-26 19:28 lm90 driver no longer working on PCs in 3.13 Guenter Roeck
2014-01-26 20:13 ` Jean Delvare
2014-01-26 20:42   ` Mark Brown
2014-01-26 20:44   ` Guenter Roeck
2014-01-26 20:49     ` Jean Delvare
2014-01-26 21:22       ` Mark Brown
2014-01-26 21:40         ` Guenter Roeck
2014-01-26 21:53           ` Mark Brown
2014-01-26 22:02             ` Guenter Roeck
2014-01-26 23:37               ` Mark Brown [this message]
2014-01-26 21:44         ` Jean Delvare
2014-01-26 23:15           ` Mark Brown
2014-01-27 10:10             ` Geert Uytterhoeven
2014-01-27 13:18               ` Mark Brown
2014-01-26 21:50       ` Guenter Roeck
2014-01-26 22:04         ` [lm-sensors] " Guenter Roeck
2014-01-26 23:51           ` Mark Brown
2014-01-27  4:16             ` Guenter Roeck
2014-01-27 10:34               ` Mark Brown
2014-01-27 17:19             ` Stephen Warren
2014-01-27 18:50               ` Guenter Roeck
2014-01-27 22:04                 ` Jean Delvare
2014-01-27 23:41                   ` Guenter Roeck
2014-01-27 23:58                     ` Mark Brown
2014-01-28  2:33                       ` Guenter Roeck
2014-01-28 10:24                         ` Mark Brown
2014-01-28 12:34                   ` Mark Brown
2014-01-27 22:36                 ` Mark Brown
2014-01-26 20:52     ` Mark Brown
2014-01-26 21:47       ` Guenter Roeck
2014-01-26 22:01         ` Mark Brown

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=20140126233703.GO11727@sirena.org.uk \
    --to=broonie@kernel.org \
    --cc=khali@linux-fr.org \
    --cc=lgirdwood@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@roeck-us.net \
    --cc=lm-sensors@lm-sensors.org \
    --cc=wni@nvidia.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).