[Nouveau] [lm-sensors] hwmon API update

Dave Airlie airlied at gmail.com
Thu Mar 3 01:36:09 PST 2011


On Mon, Feb 14, 2011 at 8:08 AM, Jean Delvare <khali at linux-fr.org> wrote:
> On Sun, 13 Feb 2011 09:16:40 -0800, Guenter Roeck wrote:
>> On Sun, Feb 13, 2011 at 07:18:44AM -0500, Martin Peres wrote:
>> > Hi,
>> >
>> > I am working on power management on the nouveau driver and I need a way
>> > to get data out of and send commands to the i2c drivers from the kernel
>> > space.

Martin,

you probably should have cc'ed Matthew since it was his patch you based this on,
and I think he can provide a good explaination.

to clarify some points,

radeon does probably want something exactly like this, we just haven't gotten to
it completely yet, I'd rather not have two drivers in the kernel for
exact same hardware,
and I believe sharing the hwmon code to do what we want is a good plan since you
don't go around reinventing wheels, but if hwmon/i2c maintainers have
no interest
it leaves with little choice but to implement about 5-10 i2c drivers
again in drm codebase.

Maybe hwmon/i2c maintainers could suggest a cleaner way to implement
what we want,
which I think I can summarize as

a) access to monitored values in-kernel
b) no userspace access to the same values except via sanitised via the driver.

though I'm not following this as closely as I should so I may have
missed something.

Dave.


More information about the Nouveau mailing list