Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1625677

Re: [PATCH 2/2] hwmon: (brcmstb) Add driver for Broadcom STB DPFE

Show all headers | View raw

Hey Guenter,

On 04/18/2017 01:58 PM, Guenter Roeck wrote:
> Hi Markus,
> 
> On Tue, Apr 18, 2017 at 01:17:02PM -0700, Markus Mayer wrote:
>> From: Markus Mayer <mmayer@broadcom.com>
>>
>> This driver allows access to DRAM properties, such as the refresh rate,
>> via the Broadcom STB DDR PHY Front End (DPFE). The refresh rate can be
>> used as indirect indicator of the DRAM temperature.
>>
>> The driver also allows setting of the sampling interval.
>>
>> Signed-off-by: Markus Mayer <mmayer@broadcom.com>
>> ---
> [ ... ]
> 
>> +
>> +static SENSOR_DEVICE_ATTR(dpfe_info, 0444, show_info, NULL, 1000);
>> +static SENSOR_DEVICE_ATTR(dpfe_refresh, 0644, show_refresh, store_refresh,
>> +			  1000);
>> +static SENSOR_DEVICE_ATTR(dpfe_vendor, 0444, show_vendor, NULL, 1000);
>> +static struct attribute *dpfe_attrs[] = {
>> +	&sensor_dev_attr_dpfe_info.dev_attr.attr,
>> +	&sensor_dev_attr_dpfe_refresh.dev_attr.attr,
>> +	&sensor_dev_attr_dpfe_vendor.dev_attr.attr,
>> +	NULL
>> +};
>> +ATTRIBUTE_GROUPS(dpfe);
>> +
> 
> There is not a single standard hwmon attribute. I don't know how
> to classify this driver, and where it should reside, but it is not
> a hwmon driver. 

This is a driver that talks to an embedded CPU running firmware which is
capable of giving various informations about the DRAM chip being
populated, including a temperature trend (hotter or cooler). We thought
initially we would be able to expose the actual temperature, but this in
turn required a lot more knowledge about the DRAM chip that we wish we
knew about. That is sort of where and why this driver was proposed for
hwmon.

Which subsystem do you think would be best for this driver drivers/misc/
or drivers/soc/bcm/brcmstb/ maybe?
-- 
Florian

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 0/2] HWMON driver for Broadcom STB DPFE Markus Mayer <code@mmayer.net> - 2017-04-18 22:30 +0200
  [PATCH 1/2] dt/bindings: Add bindings for Broadcom STB DRAM Sensors Markus Mayer <code@mmayer.net> - 2017-04-18 22:30 +0200
  [PATCH 2/2] hwmon: (brcmstb) Add driver for Broadcom STB DPFE Markus Mayer <code@mmayer.net> - 2017-04-18 22:30 +0200
    Re: [PATCH 2/2] hwmon: (brcmstb) Add driver for Broadcom STB DPFE Guenter Roeck <linux@roeck-us.net> - 2017-04-18 23:00 +0200
      Re: [PATCH 2/2] hwmon: (brcmstb) Add driver for Broadcom STB DPFE Florian Fainelli <f.fainelli@gmail.com> - 2017-04-19 00:40 +0200
        Re: [PATCH 2/2] hwmon: (brcmstb) Add driver for Broadcom STB DPFE Guenter Roeck <linux@roeck-us.net> - 2017-04-19 00:50 +0200
          Re: [PATCH 2/2] hwmon: (brcmstb) Add driver for Broadcom STB DPFE Markus Mayer <code@mmayer.net> - 2017-04-19 02:20 +0200
            Re: [PATCH 2/2] hwmon: (brcmstb) Add driver for Broadcom STB DPFE Florian Fainelli <f.fainelli@gmail.com> - 2017-04-19 02:30 +0200

csiph-web