diff mbox

spi: fsl-dspi: Convert to use regmap framework's endianness method.

Message ID 1408348100-28819-1-git-send-email-Li.Xiubo@freescale.com (mailing list archive)
State Accepted
Commit c99428d035908b9c0b8be452f9b091bc5e090256
Headers show

Commit Message

Xiubo Li Aug. 18, 2014, 7:48 a.m. UTC
Signed-off-by: Xiubo Li <Li.Xiubo@freescale.com>
Cc: Chao Fu <b44548@freescale.com>
---

This is depended on the following patches, which has been applied into
Regmap-Tree:
https://lkml.org/lkml/2014/7/15/6
https://lkml.org/lkml/2014/7/15/5
https://lkml.org/lkml/2014/7/15/7




 Documentation/devicetree/bindings/spi/spi-fsl-dspi.txt | 7 ++++++-
 drivers/spi/spi-fsl-dspi.c                             | 3 ---
 2 files changed, 6 insertions(+), 4 deletions(-)

Comments

Chao Fu Aug. 18, 2014, 8:06 a.m. UTC | #1
> -----Original Message-----

> From: Xiubo Li [mailto:Li.Xiubo@freescale.com]

> Sent: 2014?8?18? 15:48

> To: broonie@kernel.org; Fu Chao-B44548

> Cc: devicetree@vger.kernel.org; linux-spi@vger.kernel.org; Xiubo Li-

> B47053

> Subject: [PATCH] spi: fsl-dspi: Convert to use regmap framework's

> endianness method.

> 

> Signed-off-by: Xiubo Li <Li.Xiubo@freescale.com>

> Cc: Chao Fu <b44548@freescale.com>

> ---

> 


Acked-by: Chao Fu <B44548@freescale.com>

Thank you!

 
> This is depended on the following patches, which has been applied into

> Regmap-Tree:

> https://lkml.org/lkml/2014/7/15/6

> https://lkml.org/lkml/2014/7/15/5

> https://lkml.org/lkml/2014/7/15/7

> 

> 

> 

> 

>  Documentation/devicetree/bindings/spi/spi-fsl-dspi.txt | 7 ++++++-

>  drivers/spi/spi-fsl-dspi.c                             | 3 ---

>  2 files changed, 6 insertions(+), 4 deletions(-)

> 

> diff --git a/Documentation/devicetree/bindings/spi/spi-fsl-dspi.txt

> b/Documentation/devicetree/bindings/spi/spi-fsl-dspi.txt

> index 5376de4..cbbe16e 100644

> --- a/Documentation/devicetree/bindings/spi/spi-fsl-dspi.txt

> +++ b/Documentation/devicetree/bindings/spi/spi-fsl-dspi.txt

> @@ -10,7 +10,12 @@ Required properties:

>  - pinctrl-names: must contain a "default" entry.

>  - spi-num-chipselects : the number of the chipselect signals.

>  - bus-num : the slave chip chipselect signal number.

> -- big-endian : if DSPI modudle is big endian, the bool will be set in

> node.

> +

> +Optional property:

> +- big-endian: If present the dspi device's registers are implemented

> +  in big endian mode, otherwise in native mode(same with CPU), for more

> +  detail please see: Documentation/devicetree/bindings/regmap/regmap.txt.

> +

>  Example:

> 

>  dspi0@4002c000 {

> diff --git a/drivers/spi/spi-fsl-dspi.c b/drivers/spi/spi-fsl-dspi.c

> index 5021ddf..ebc4d1f 100644

> --- a/drivers/spi/spi-fsl-dspi.c

> +++ b/drivers/spi/spi-fsl-dspi.c

> @@ -493,9 +493,6 @@ static int dspi_probe(struct platform_device *pdev)

>  	}

> 

>  	dspi_regmap_config.lock_arg = dspi;

> -	dspi_regmap_config.val_format_endian =

> -		of_property_read_bool(np, "big-endian")

> -			? REGMAP_ENDIAN_BIG : REGMAP_ENDIAN_DEFAULT;

>  	dspi->regmap = devm_regmap_init_mmio_clk(&pdev->dev, "dspi", base,

>  						&dspi_regmap_config);

>  	if (IS_ERR(dspi->regmap)) {

> --

> 1.8.5
Mark Brown Aug. 18, 2014, 2:43 p.m. UTC | #2
On Mon, Aug 18, 2014 at 03:48:20PM +0800, Xiubo Li wrote:
> Signed-off-by: Xiubo Li <Li.Xiubo@freescale.com>
> Cc: Chao Fu <b44548@freescale.com>

Applied, thanks.
diff mbox

Patch

diff --git a/Documentation/devicetree/bindings/spi/spi-fsl-dspi.txt b/Documentation/devicetree/bindings/spi/spi-fsl-dspi.txt
index 5376de4..cbbe16e 100644
--- a/Documentation/devicetree/bindings/spi/spi-fsl-dspi.txt
+++ b/Documentation/devicetree/bindings/spi/spi-fsl-dspi.txt
@@ -10,7 +10,12 @@  Required properties:
 - pinctrl-names: must contain a "default" entry.
 - spi-num-chipselects : the number of the chipselect signals.
 - bus-num : the slave chip chipselect signal number.
-- big-endian : if DSPI modudle is big endian, the bool will be set in node.
+
+Optional property:
+- big-endian: If present the dspi device's registers are implemented
+  in big endian mode, otherwise in native mode(same with CPU), for more
+  detail please see: Documentation/devicetree/bindings/regmap/regmap.txt.
+
 Example:
 
 dspi0@4002c000 {
diff --git a/drivers/spi/spi-fsl-dspi.c b/drivers/spi/spi-fsl-dspi.c
index 5021ddf..ebc4d1f 100644
--- a/drivers/spi/spi-fsl-dspi.c
+++ b/drivers/spi/spi-fsl-dspi.c
@@ -493,9 +493,6 @@  static int dspi_probe(struct platform_device *pdev)
 	}
 
 	dspi_regmap_config.lock_arg = dspi;
-	dspi_regmap_config.val_format_endian =
-		of_property_read_bool(np, "big-endian")
-			? REGMAP_ENDIAN_BIG : REGMAP_ENDIAN_DEFAULT;
 	dspi->regmap = devm_regmap_init_mmio_clk(&pdev->dev, "dspi", base,
 						&dspi_regmap_config);
 	if (IS_ERR(dspi->regmap)) {