Re: [PATCH v2 2/2] hwmon: lochnagar: Add Lochnagar 2 hardware monitoring driver

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

 



On Mon, Mar 25, 2019 at 09:46:22AM -0700, Guenter Roeck wrote:
> On Mon, Mar 25, 2019 at 01:16:51PM +0000, Charles Keepax wrote:
> > On Mon, Mar 25, 2019 at 04:17:31AM -0700, Guenter Roeck wrote:
> > > On 3/25/19 4:00 AM, Charles Keepax wrote:
> > > >From: Lucas Tanure <tanureal@xxxxxxxxxxxxxxxxxxxxx>
> > Apologies couldn't find an archive for the hwmon list going
> > back far enough, so just had to copy that in, hopefully your
> > own records go back far enough. I am certainly open to other
> > options such as your other suggest of a newer higher precision
> > file being added.
> > 
> Try lore.kernel.org/linux-hwmon.
> 

Indeed here we go:

https://lore.kernel.org/linux-hwmon/32c30882-2466-a992-158b-870cc8af5b40@xxxxxxxxxxxxxxxxxxxxx/

Apologies for not finding that, over reliance on Google.

> > As regards the averaging, the draw of the audio hardware can be
> > quite spikey and since we want to check the power consumption
> > averaging is greatly preferable. As I mentioned in my previous
> > email the average actually takes place in analog so it helps
> > us get power numbers that include the spikes. Without that one
> > could greatly over/under estimate depending on your luck with
> > the timing of the current measurements.
> > 
> That makes some sense. Maybe add a note to the code describing the
> reason for taking several measurements ?
> 

Can do.

> > We could potentially switch to millivolts rather than microvolts,
> > as our use-cases don't generally need the precision on voltage,
> > if you prefer? But for current we will need to come up with some
> > solution to pass more accurate measurements.
> > 
> I do have some second thoughts about my 2017 comments. From an ABI
> perspective, it is highly undesirable to provide false data on purpose.
> 
> Do you need micro-units for all measurements ? If not, or even if so,
> a simple solution might be to provide currX_input_ua and possibly
> inX_input_uv for those attributes where needed, in addition to the
> default attributes, which would be reported in mV / mA. That would meet
> both the ABI and your requirements.
> 
> There is another possibility which might meet your requirements: If the
> important attribute is power consumption, you might consider providing
> power attributes. Those _are provided in micro-units. That isn't exactly
> as expected, as drivers should only provide power attributes if actually
> reported by the HW, but there is an argument to make that it makes sense
> here. You could then even provide powerX_average and make the number
> of samples indirectly configurable with powerX_average_interval.
> 

Thank you for the suggestions I will investigate them and see
where I get to. The power option does sound tempting as the
ability to configure the number of samples feels like something
that could be handy in the future. But on the flip side adding
high accuracy APIs might be useful for others in the future.

Thanks,
Charles



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux