On Tue, Mar 3, 2015 at 11:25 AM, Lee Jones <[email protected]> wrote:
> On Tue, 03 Mar 2015, Linus Walleij wrote:
>> +Multi-Function Devices (MFD)
>> +
>> +These devices comprise a nexus for heterogeneous hardware blocks
>
> I don't think you meant 'comprise' here. Perhaps 'represent' might
> fit better?
I meant "comprise", even looked up the same word in a Swedish-English
dictionary. I mean those devices really are such heterogeneous things,
not just modeled like such in Linux.
>> +spawning multiple child devices.
>
> Spawning devices is something software will do. Think less about what
> the actions the OS will take order to handle these IPs and more about
> what they are physically. I would prefer the description mentions
> that these chips are single pieces of silicon which contain >1 devices
> with non-unique/varying functionality.
OK.
>> +A typical MFD can be:
>> +
>> +- A mixed signal ASIC on an external bus, sometimes a PMIC (power management
>> + integrated circuit) that is manufactured in a lower technology node (rough
>
> Please capitalise the expanded PMIC abbreviation.
OK.
>> + silicon) that handles analog drivers for things like audio amplifiers, LED
>> + drivers, level shifters, PHY (physical interfaces to things like USB or
>> + ethernet), regulators etc.
>> +
>> +- A range of memory registers containing "miscellaneous system registers"
>> also
>> + known as a system controller "syscon" or any other memory range
>> containing a
>> + mix of unrelated registers.
>
> s/registers/{hardware|devices}/
OK.
>> +Optional properties:
>> +
>> +- compatible : "simple-mfd" - this signifies that the operating system
>> should
>> + spawn child devices for all the subnodes of the MFD device akin to how
>
> This is pretty Linux specific IMHO. Other OSes might not spawn
> seperate devices to handle MFD type h/w. The idea is fine, but please
> generify a little.
OK.
>> +foo@1000 {
>
> Should be 10000.
OK.
>> + compatible = "syscon", "simple-mfd";
>
> Please address your tabbing issues.
OK.
Yours,
Linus Walleij
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
