[PATCH 3/5] powerpc: Document device nodes for I2C devices.
Kumar Gala
galak at kernel.crashing.org
Fri May 18 02:39:22 EST 2007
On May 17, 2007, at 11:17 AM, Scott Wood wrote:
> Kumar Gala wrote:
>>> diff --git a/Documentation/powerpc/booting-without-of.txt b/
>>> Documentation/powerpc/booting-without-of.txt
>>> index b49ce16..67026ad 100644
>>> --- a/Documentation/powerpc/booting-without-of.txt
>>> +++ b/Documentation/powerpc/booting-without-of.txt
>>> @@ -1257,6 +1257,8 @@ platforms are moved over to use the
>>> flattened- device-tree model.
>>>
>>> e) I2C
>>>
>>> + e1) I2C Controller
>>> +
>>> Required properties :
>>>
>>> - device_type : Should be "i2c"
>>> @@ -1277,6 +1279,10 @@ platforms are moved over to use the
>>> flattened-device-tree model.
>>> a digital filter sampling rate register
>>> - fsl5200-clocking : boolean; if defined, indicated that
>>> this device
>>> uses the FSL 5200 clocking mechanism.
>>> + - #address-cells : should exist and be 1 if I2C devices are
>>> declared
>>> + in the device tree.
>>> + - #size-cells : should exist and be 0 if I2C devices are
>>> declared
>>> + in the device tree.
>> As I've stated before, we need a bus number as well so we can
>> handle things like I2C switches and muxes.
>
> Is this something we handle now? If not, then it's really not
> within the scope of this patchset. If so, how am I breaking it?
We don't handle i2c devices in the dev tree today. If you are going
to propose a solution it should work for all cases that people are
aware of even if linux doesn't support the functionality.
If only some subset of cases are handled what good is the device tree
to a user? They will just have to figure out if their usage is
supported or not and if not find some other solution that works for
them.
- k
More information about the Linuxppc-dev
mailing list