RE: [PATCH] mfd: syscon: binding: Add syscon endianness support.

From: Li.Xiubo@xxxxxxxxxxxxx
Date: Mon Oct 13 2014 - 22:30:15 EST


Hi DT guys,

Could someone help me review or ack this patch?

Thanks very much.

BRs
Xiubo



> -----Original Message-----
> From: Xiubo Li [mailto:Li.Xiubo@xxxxxxxxxxxxx]
> Sent: Thursday, September 18, 2014 3:27 PM
> To: Dong Aisheng-B29396; pankaj.dubey@xxxxxxxxxxx; robh+dt@xxxxxxxxxx;
> pawel.moll@xxxxxxx; mark.rutland@xxxxxxx; ijc+devicetree@xxxxxxxxxxxxxx;
> galak@xxxxxxxxxxxxxx; sameo@xxxxxxxxxxxxxxx; lee.jones@xxxxxxxxxx;
> devicetree@xxxxxxxxxxxxxxx
> Cc: linux-kernel@xxxxxxxxxxxxxxx; Xiubo Li-B47053
> Subject: [PATCH] mfd: syscon: binding: Add syscon endianness support.
>
> For many drivers which will support rich endianness of Devices
> need define DT properties by itself with the binding support.
>
> The regmap core has already support endianness of the following
> case:
> Index Device DT properties needed
> ------------------------------------------
> 1 BE 'big-endian'
> 2 LE 'little-endian'
>
> Please see the regmap binding for more detail information of
> the endianness support.
>
> Signed-off-by: Xiubo Li <Li.Xiubo@xxxxxxxxxxxxx>
> ---
>
>
> This patch hasn't any dependency with Pankaj Dubey's:
> mfd: syscon: Decouple syscon interface from platform devices
>
>
>
> Documentation/devicetree/bindings/mfd/syscon.txt | 17 ++++++++++++++---
> 1 file changed, 14 insertions(+), 3 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/mfd/syscon.txt
> b/Documentation/devicetree/bindings/mfd/syscon.txt
> index fe8150b..7c33178 100644
> --- a/Documentation/devicetree/bindings/mfd/syscon.txt
> +++ b/Documentation/devicetree/bindings/mfd/syscon.txt
> @@ -10,11 +10,22 @@ OS driver) to determine the location of the registers, and
> access the
> registers directly.
>
> Required properties:
> -- compatible: Should contain "syscon".
> -- reg: the register region can be accessed from syscon
> +- compatible: Should contain "syscon".
> +- reg: The register region can be accessed from syscon
>
> -Examples:
> +Optional property:
> +- {big,little}-endian: Boolean property. Please see ../regmap/regmap.txt
> + for more information.
> +
> +Example 1:
> gpr: iomuxc-gpr@020e0000 {
> compatible = "fsl,imx6q-iomuxc-gpr", "syscon";
> reg = <0x020e0000 0x38>;
> };
> +
> +Example 2:
> +dcfg: dcfg@1ee0000 {
> + compatible = "fsl,ls1021a-dcfg", "syscon";
> + reg = <0x0 0x1ee0000 0x0 0x10000>;
> + big-endian;
> +};
> --
> 2.1.0.27.g96db324

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/