linux-kernel - RE: [PATCHv2] watchdog: imx2_wdt: Convert to use regmap framework's endianness method.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Fri, 5 Sep 2014 07:11:31 +0000
From:	"Li.Xiubo@...escale.com" <Li.Xiubo@...escale.com>
To:	"wim@...ana.be" <wim@...ana.be>,
	"linux@...ck-us.net" <linux@...ck-us.net>,
	"mpa@...gutronix.de" <mpa@...gutronix.de>,
	"linux-watchdog@...r.kernel.org" <linux-watchdog@...r.kernel.org>
CC:	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>
Subject: RE: [PATCHv2] watchdog: imx2_wdt: Convert to use regmap framework's
 endianness method.

Hi,

Ping :)

Thanks,

BRs
Xiubo


> -----Original Message-----
> From: Xiubo Li [mailto:Li.Xiubo@...escale.com]
> Sent: Monday, August 25, 2014 3:55 PM
> To: wim@...ana.be; linux@...ck-us.net; mpa@...gutronix.de; linux-
> watchdog@...r.kernel.org
> Cc: linux-kernel@...r.kernel.org; devicetree@...r.kernel.org; Xiubo Li-B47053
> Subject: [PATCHv2] watchdog: imx2_wdt: Convert to use regmap framework's
> endianness method.
> 
> Regmap framework will contain some patches to handle endianess, and there
> does not need any parsing of endianness DT properties of individual driver,
> so we can savely remove it here.
> 
> Signed-off-by: Xiubo Li <Li.Xiubo@...escale.com>
> Cc: Markus Pargmann <mpa@...gutronix.de>
> Reviewed-by: Guenter Roeck <linux@...ck-us.net>
> ---
> 
> Changes in v2:
> - Remove useless header file.
> 
> 
> 
> 
>  Documentation/devicetree/bindings/watchdog/fsl-imx-wdt.txt | 4 ++--
>  drivers/watchdog/imx2_wdt.c                                | 9 +--------
>  2 files changed, 3 insertions(+), 10 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/watchdog/fsl-imx-wdt.txt
> b/Documentation/devicetree/bindings/watchdog/fsl-imx-wdt.txt
> index e52ba2d..9e1ebe2 100644
> --- a/Documentation/devicetree/bindings/watchdog/fsl-imx-wdt.txt
> +++ b/Documentation/devicetree/bindings/watchdog/fsl-imx-wdt.txt
> @@ -6,8 +6,8 @@ Required properties:
>  - interrupts : Should contain WDT interrupt
> 
>  Optional property:
> -- big-endian: If present the watchdog device's registers are implemented
> -  in big endian mode, otherwise in little mode.
> +- big-endian: Boolean property, required if all the i.MX WDT registers
> +  are big-endian rather than little-endian.
> 
>  Examples:
> 
> diff --git a/drivers/watchdog/imx2_wdt.c b/drivers/watchdog/imx2_wdt.c
> index 68c3d37..afc8738 100644
> --- a/drivers/watchdog/imx2_wdt.c
> +++ b/drivers/watchdog/imx2_wdt.c
> @@ -28,7 +28,6 @@
>  #include <linux/kernel.h>
>  #include <linux/module.h>
>  #include <linux/moduleparam.h>
> -#include <linux/of_address.h>
>  #include <linux/platform_device.h>
>  #include <linux/regmap.h>
>  #include <linux/timer.h>
> @@ -182,7 +181,7 @@ static struct watchdog_ops imx2_wdt_ops = {
>  	.set_timeout = imx2_wdt_set_timeout,
>  };
> 
> -static struct regmap_config imx2_wdt_regmap_config = {
> +static const struct regmap_config imx2_wdt_regmap_config = {
>  	.reg_bits = 16,
>  	.reg_stride = 2,
>  	.val_bits = 16,
> @@ -191,12 +190,10 @@ static struct regmap_config imx2_wdt_regmap_config = {
> 
>  static int __init imx2_wdt_probe(struct platform_device *pdev)
>  {
> -	struct device_node *np = pdev->dev.of_node;
>  	struct imx2_wdt_device *wdev;
>  	struct watchdog_device *wdog;
>  	struct resource *res;
>  	void __iomem *base;
> -	bool big_endian;
>  	int ret;
>  	u32 val;
> 
> @@ -204,10 +201,6 @@ static int __init imx2_wdt_probe(struct platform_device
> *pdev)
>  	if (!wdev)
>  		return -ENOMEM;
> 
> -	big_endian = of_property_read_bool(np, "big-endian");
> -	if (big_endian)
> -		imx2_wdt_regmap_config.val_format_endian = REGMAP_ENDIAN_BIG;
> -
>  	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
>  	base = devm_ioremap_resource(&pdev->dev, res);
>  	if (IS_ERR(base))
> --
> 1.8.5

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