Hi Joel,
On 03/29/2016 02:34 AM, Joel Stanley wrote:
Hello Guenter,
On Tue, Mar 29, 2016 at 7:44 PM, Guenter Roeck <linux@xxxxxxxxxxxx> wrote:
On 03/29/2016 01:50 AM, Joel Stanley wrote:
This adds a of_device_id table so we can probe the driver using the
device tree.
The subject line does not match the driver being modified; it suggests
adm1031
while in reality the adm1275 driver is being modified.
Apologies, that's what I get for writing a patch as I'm walking out the door.
Besides, this should already work, through trivial bindings.
Is there a reason to believe that it does not already work ?
I wasn't aware that we had such a mechanism, thank you. I assume you
referring to bindings/i2c/trival-devices.txt?
Yes, and the associated mechanism.
As background for why I added the bindings; originally this code was
patch one of two. The second added a device tree property to inform
the kernel of the external sense resistor so we could perform scaling
in the kernel.
Add the bindings (all of them) in patch 1, and the code (all of it) in patch 2.
I read the hwmon documentation on why this is not currently done.
However, as the device tree is specific to a board - things like i2c
addresses and which bus they are on - I was wondering if you would
take patches to add such a feature?
Yes. Search for "shunt-resistor", which is already used for some of the
TI devices. Also see Documentation/devicetree/bindings/hwmon/ina2xx.txt.
Guenter
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html