[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