| 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
| ||
|
Date: Tue, 9 Sep 2014 01:37:22 +0000
From: "Li.Xiubo@...escale.com" <Li.Xiubo@...escale.com>
To: Guenter Roeck <linux@...ck-us.net>
CC: "wim@...ana.be" <wim@...ana.be>,
"mpa@...gutronix.de" <mpa@...gutronix.de>,
"linux-watchdog@...r.kernel.org" <linux-watchdog@...r.kernel.org>,
"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,
Get it.
Thanks very much.
BRs
Xiubo
> -----Original Message-----
> From: Guenter Roeck [mailto:groeck7@...il.com] On Behalf Of Guenter Roeck
> Sent: Saturday, September 06, 2014 12:07 AM
> To: Xiubo Li-B47053
> Cc: wim@...ana.be; mpa@...gutronix.de; linux-watchdog@...r.kernel.org; linux-
> kernel@...r.kernel.org; devicetree@...r.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@...escale.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@...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-watchdog" in
> > the body of a message to majordomo@...r.kernel.org
> > More majordomo info at http://vger.kernel.org/majordomo-info.html
--
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/
Powered by blists - more mailing lists