Re: [PATCH v3 2/2] Documentation: dt: hwmon: add OF document for LM90

[Wei Ni <wni-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>]

On 09/09/2013 06:57 PM, Ramkumar Ramachandra wrote:
> Wei Ni wrote:
>> diff --git a/Documentation/devicetree/bindings/hwmon/lm90.txt b/Documentation/devicetree/bindings/hwmon/lm90.txt
>> new file mode 100644
>> index 0000000..5570875
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/hwmon/lm90.txt
> 
> While at it, please update and rename ads1015.txt.

This series is for lm90 compatible devices, it seems ads1015 is not
register-compatible with lm90.

> 
>> @@ -0,0 +1,44 @@
>> +* LM90 series thermometer.
>> +
>> +Required node properties:
>> +- compatible: manufacture and chip name, one of
>> +               "<manufacture>,adm1032"
>> +               "<manufacture>,adt7461"
>> +               "<manufacture>,adt7461a"
>> +               "<manufacture>,g781"
>> +               "<manufacture>,lm90"
>> +               "<manufacture>,lm86"
>> +               "<manufacture>,lm89"
>> +               "<manufacture>,lm99"
> 
> More versions of this of different ages are required.

In here, the chip names, such as "lm99", come from lm90.c id_table, the
i2c subsystem want to use it to load driver, so we can add any other names.

> 
>> +               "<manufacture>,max6646"
>> +               "<manufacture>,max6647"
>> +               "<manufacture>,max6649"
>> +               "<manufacture>,max6657"
>> +               "<manufacture>,max6658"
>> +               "<manufacture>,max6659"
>> +               "<manufacture>,max6680"
>> +               "<manufacture>,max6681"
>> +               "<manufacture>,max6695"
>> +               "<manufacture>,max6696"
> 
> SSDNow devices are required.
What's the SSDNow device? It seems the lm90.c doesn't support it.

Thanks.
Wei.

>