Re: [PATCH] char: Add Dell Systems Management Base driver

[Pavel Machek]

Hi!

Please use enter key every 70 chars or so...

> >    > This patch adds the Dell Systems Management Base driver.
> > 
> >    You keep posting this driver without explaining/showing how it's used.
> >    Could you perhaps give some more details here please?
> 
> Here's some more information on the driver and the systems that it supports.  Because the hardware interfaces on those systems and the Dell systems management software that access the interfaces are proprietary, I can't provide specifications for the interfaces or source code for the software.
> 

So... instead of providing nice & standard interface, you invent very ugly interface
that allows your usermode app to band on SMI directly -- because you don't want
to tell us how to drive SMI.

No, sorry. Provide something with standard interface (like lm_sensors use). You'll have to
tell us how to control the hardware. Its temperature sensor; I don't see how that is worh calling
"proprietary"...
				Pavel
-- 
64 bytes from 195.113.31.123: icmp_seq=28 ttl=51 time=448769.1 ms         

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/