[RFC v2 PATCH 2/3] dt: device tree bindings for TI's EMIF sdram controller

Olof Johansson olof at lixom.net
Mon Dec 19 11:59:18 EST 2011


Oh wait, when I saw 3/3 I realized the following too:

On Mon, Dec 19, 2011 at 6:05 AM, Aneesh V <aneesh at ti.com> wrote:

> +- phy-type     : string indicating the phy type. Should be one of the
> +  following:
> +
> +  "phy-type-omap4"  : PHY used in OMAP4 family of SoCs
> +
> +  "phy-type-dm81xx" : PHY used in DM81XX family of SoCs

No, again please don't define new string properties when a simple
binary property will do just fine. Just use "phy-type-omap4" or
"phy-type-dm81xx" as an empty property to indicate what phy is used.
Or encode as an integer and list the valid values here.

Every time you add a string to the device tree, the data structure
grows, not to mention the fact that it adds a lot of string
comparisons during setup. Please keep that in mind when defining
bindings.



-Olof



More information about the linux-arm-kernel mailing list