lkml.org 
[lkml]   [2014]   [Sep]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [PATCHv2] watchdog: imx2_wdt: Convert to use regmap framework's endianness method.
Date
Hi,

Get it.

Thanks very much.

BRs
Xiubo




> -----Original Message-----
> From: Guenter Roeck [mailto:groeck7@gmail.com] On Behalf Of Guenter Roeck
> Sent: Saturday, September 06, 2014 12:07 AM
> To: Xiubo Li-B47053
> Cc: wim@iguana.be; mpa@pengutronix.de; linux-watchdog@vger.kernel.org; linux-
> kernel@vger.kernel.org; devicetree@vger.kernel.org
> Subject: Re: [PATCHv2] watchdog: imx2_wdt: Convert to use regmap framework's
> endianness method.
>
> On Fri, Sep 05, 2014 at 07:11:31AM +0000, Li.Xiubo@freescale.com wrote:
> > Hi,
> >
> > Ping :)
> >
> Wim usually picks up patches shortly before the next release window opens.
>
> It is in my watchdog-next branch, so it is not forgotten. I'll send a pull
> request to Wim after -rc5.
>
> Wim, I am wondering - should I ask Stephen to add my watchdog-next branch to
> linux-next ? I know it isn't really perfect, and there is no guarantee that
> you will accept the patches in my branch, but it would expose the pending
> patches to earlier visibility in -next.
>
> Thanks,
> Guenter
>
> > Thanks,
> >
> > BRs
> > Xiubo
> >
> >
> > > -----Original Message-----
> > > From: Xiubo Li [mailto:Li.Xiubo@freescale.com]
> > > Sent: Monday, August 25, 2014 3:55 PM
> > > To: wim@iguana.be; linux@roeck-us.net; mpa@pengutronix.de; linux-
> > > watchdog@vger.kernel.org
> > > Cc: linux-kernel@vger.kernel.org; devicetree@vger.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@freescale.com>
> > > Cc: Markus Pargmann <mpa@pengutronix.de>
> > > Reviewed-by: Guenter Roeck <linux@roeck-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-watchdog" in
> > the body of a message to majordomo@vger.kernel.org
> > More majordomo info at http://vger.kernel.org/majordomo-info.html


\
 
 \ /
  Last update: 2014-09-09 04:21    [W:1.013 / U:0.176 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site