Re: [PATCH] sensors-detect: Refactor AMD CPU sensor detection

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Andreas,

On Wed, 6 Apr 2011 16:00:26 +0200, Andreas Herrmann wrote:
> On Wed, Apr 06, 2011 at 03:17:36PM +0200, Jean Delvare wrote:
> > Except for the Family 10h case which is more complex, AMD CPU sensor
> > detection is straightforward, so we don't really need a dedicated
> > function for each CPU family. By passing the expected F3 PCI device ID
> > as a parameter, we can use the same function for all families.
> > 
> > Objection anyone? This could be seen as moving some of the detection
> > code into @cpu_ids itself, but OTOH it saves quite a few lines of
> > redundant code.
> 
> No objections -- of cours, it's a good idea.
> 
> BTW, detection for "AMD Family 15h thermal sensors" can also be used
> for f15h_power (once the driver is accepted/integrated).

Yes and no. A similar logic can be used, but the PCI device ID is
different. Will you send a patch, or should I add it myself?

-- 
Jean Delvare

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


[Index of Archives]     [Linux Kernel]     [Linux Hardware Monitoring]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux