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 05:10:54PM +0000, Charles Keepax wrote:
> 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.
> 
Agreed, but we would have to think about it more before jumping into
it. There would be several possible solutions. Adding new sysfs files
might be one. Another might be "scale" attributes or similar. Or get rid
of the sysfs ABI entirely and use something similar to iio. Or go further
and create a hwmon->iio bridge and use iio to report high resolution
information.

Guenter



[Index of Archives]     [LM Sensors]     [Linux Sound]     [ALSA Users]     [ALSA Devel]     [Linux Audio Users]     [Linux Media]     [Kernel]     [Gimp]     [Yosemite News]     [Linux Media]

  Powered by Linux