From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail.active-venture.com ([67.228.131.205]:65034 "EHLO mail.active-venture.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755056Ab3K0P4p (ORCPT ); Wed, 27 Nov 2013 10:56:45 -0500 Message-ID: <5296163B.5040004@roeck-us.net> Date: Wed, 27 Nov 2013 07:56:43 -0800 From: Guenter Roeck MIME-Version: 1.0 To: Ivan Khoronzhuk , Santosh Shilimkar CC: linux-arm-kernel@lists.infradead.org, mark.rutland@arm.com, wim@iguana.be, nsekhar@ti.com, linux-watchdog@vger.kernel.org, devicetree@vger.kernel.org, Pawel.Moll@arm.com, swarren@wwwdotorg.org, ijc+devicetree@hellion.org.uk, galak@kernel.crashing.org, rob.herring@calxeda.com, linux-kernel@vger.kernel.org, grant.likely@linaro.org, grygorii.strashko@ti.com Subject: Re: [PATCH] watchdog: davinci: rename platform driver to davinci-wdt References: <1385559113-4354-1-git-send-email-ivan.khoronzhuk@ti.com> In-Reply-To: <1385559113-4354-1-git-send-email-ivan.khoronzhuk@ti.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-watchdog-owner@vger.kernel.org List-Id: linux-watchdog@vger.kernel.org On 11/27/2013 05:31 AM, Ivan Khoronzhuk wrote: > As we switch to use the watchdog core which permits more than one > active watchdog in the system, rename platform driver to > "davinci-wdt" to be identifiable. > > Signed-off-by: Ivan Khoronzhuk Reviewed-by: Guenter Roeck > --- > based on: > https://lkml.org/lkml/2013/11/27/164 > > CC: > Sekhar Nori > Guenter Roeck > > arch/arm/mach-davinci/da830.c | 2 +- > arch/arm/mach-davinci/da850.c | 2 +- > arch/arm/mach-davinci/da8xx-dt.c | 2 +- > arch/arm/mach-davinci/devices-da8xx.c | 4 ++-- > arch/arm/mach-davinci/devices.c | 2 +- > arch/arm/mach-davinci/dm355.c | 2 +- > arch/arm/mach-davinci/dm365.c | 2 +- > arch/arm/mach-davinci/dm644x.c | 2 +- > arch/arm/mach-davinci/dm646x.c | 2 +- > drivers/watchdog/davinci_wdt.c | 4 ++-- > 10 files changed, 12 insertions(+), 12 deletions(-) > > diff --git a/arch/arm/mach-davinci/da830.c b/arch/arm/mach-davinci/da830.c > index 0813b51..82c6013 100644 > --- a/arch/arm/mach-davinci/da830.c > +++ b/arch/arm/mach-davinci/da830.c > @@ -385,7 +385,7 @@ static struct clk_lookup da830_clks[] = { > CLK(NULL, "pll0_sysclk7", &pll0_sysclk7), > CLK("i2c_davinci.1", NULL, &i2c0_clk), > CLK(NULL, "timer0", &timerp64_0_clk), > - CLK("watchdog", NULL, &timerp64_1_clk), > + CLK("davinci-wdt", NULL, &timerp64_1_clk), > CLK(NULL, "arm_rom", &arm_rom_clk), > CLK(NULL, "scr0_ss", &scr0_ss_clk), > CLK(NULL, "scr1_ss", &scr1_ss_clk), > diff --git a/arch/arm/mach-davinci/da850.c b/arch/arm/mach-davinci/da850.c > index 352984e..ccb2f58 100644 > --- a/arch/arm/mach-davinci/da850.c > +++ b/arch/arm/mach-davinci/da850.c > @@ -443,7 +443,7 @@ static struct clk_lookup da850_clks[] = { > CLK(NULL, "pll1_sysclk3", &pll1_sysclk3), > CLK("i2c_davinci.1", NULL, &i2c0_clk), > CLK(NULL, "timer0", &timerp64_0_clk), > - CLK("watchdog", NULL, &timerp64_1_clk), > + CLK("davinci-wdt", NULL, &timerp64_1_clk), > CLK(NULL, "arm_rom", &arm_rom_clk), > CLK(NULL, "tpcc0", &tpcc0_clk), > CLK(NULL, "tptc0", &tptc0_clk), > diff --git a/arch/arm/mach-davinci/da8xx-dt.c b/arch/arm/mach-davinci/da8xx-dt.c > index d2bc574..ed19287 100644 > --- a/arch/arm/mach-davinci/da8xx-dt.c > +++ b/arch/arm/mach-davinci/da8xx-dt.c > @@ -32,7 +32,7 @@ static void __init da8xx_init_irq(void) > > static struct of_dev_auxdata da850_auxdata_lookup[] __initdata = { > OF_DEV_AUXDATA("ti,davinci-i2c", 0x01c22000, "i2c_davinci.1", NULL), > - OF_DEV_AUXDATA("ti,davinci-wdt", 0x01c21000, "watchdog", NULL), > + OF_DEV_AUXDATA("ti,davinci-wdt", 0x01c21000, "davinci-wdt", NULL), > OF_DEV_AUXDATA("ti,da830-mmc", 0x01c40000, "da830-mmc.0", NULL), > OF_DEV_AUXDATA("ti,da850-ehrpwm", 0x01f00000, "ehrpwm", NULL), > OF_DEV_AUXDATA("ti,da850-ehrpwm", 0x01f02000, "ehrpwm", NULL), > diff --git a/arch/arm/mach-davinci/devices-da8xx.c b/arch/arm/mach-davinci/devices-da8xx.c > index c46eccb..f9ba74b 100644 > --- a/arch/arm/mach-davinci/devices-da8xx.c > +++ b/arch/arm/mach-davinci/devices-da8xx.c > @@ -389,7 +389,7 @@ static struct resource da8xx_watchdog_resources[] = { > }; > > static struct platform_device da8xx_wdt_device = { > - .name = "watchdog", > + .name = "davinci-wdt", > .id = -1, > .num_resources = ARRAY_SIZE(da8xx_watchdog_resources), > .resource = da8xx_watchdog_resources, > @@ -399,7 +399,7 @@ void da8xx_restart(enum reboot_mode mode, const char *cmd) > { > struct device *dev; > > - dev = bus_find_device_by_name(&platform_bus_type, NULL, "watchdog"); > + dev = bus_find_device_by_name(&platform_bus_type, NULL, "davinci-wdt"); > if (!dev) { > pr_err("%s: failed to find watchdog device\n", __func__); > return; > diff --git a/arch/arm/mach-davinci/devices.c b/arch/arm/mach-davinci/devices.c > index 3996e98..5cf9a02 100644 > --- a/arch/arm/mach-davinci/devices.c > +++ b/arch/arm/mach-davinci/devices.c > @@ -302,7 +302,7 @@ static struct resource wdt_resources[] = { > }; > > struct platform_device davinci_wdt_device = { > - .name = "watchdog", > + .name = "davinci-wdt", > .id = -1, > .num_resources = ARRAY_SIZE(wdt_resources), > .resource = wdt_resources, > diff --git a/arch/arm/mach-davinci/dm355.c b/arch/arm/mach-davinci/dm355.c > index ef9ff1f..4bb8132 100644 > --- a/arch/arm/mach-davinci/dm355.c > +++ b/arch/arm/mach-davinci/dm355.c > @@ -375,7 +375,7 @@ static struct clk_lookup dm355_clks[] = { > CLK(NULL, "pwm3", &pwm3_clk), > CLK(NULL, "timer0", &timer0_clk), > CLK(NULL, "timer1", &timer1_clk), > - CLK("watchdog", NULL, &timer2_clk), > + CLK("davinci-wdt", NULL, &timer2_clk), > CLK(NULL, "timer3", &timer3_clk), > CLK(NULL, "rto", &rto_clk), > CLK(NULL, "usb", &usb_clk), > diff --git a/arch/arm/mach-davinci/dm365.c b/arch/arm/mach-davinci/dm365.c > index 1511a06..b0e8df3 100644 > --- a/arch/arm/mach-davinci/dm365.c > +++ b/arch/arm/mach-davinci/dm365.c > @@ -473,7 +473,7 @@ static struct clk_lookup dm365_clks[] = { > CLK(NULL, "pwm3", &pwm3_clk), > CLK(NULL, "timer0", &timer0_clk), > CLK(NULL, "timer1", &timer1_clk), > - CLK("watchdog", NULL, &timer2_clk), > + CLK("davinci-wdt", NULL, &timer2_clk), > CLK(NULL, "timer3", &timer3_clk), > CLK(NULL, "usb", &usb_clk), > CLK("davinci_emac.1", NULL, &emac_clk), > diff --git a/arch/arm/mach-davinci/dm644x.c b/arch/arm/mach-davinci/dm644x.c > index 143a321..aeaad95 100644 > --- a/arch/arm/mach-davinci/dm644x.c > +++ b/arch/arm/mach-davinci/dm644x.c > @@ -322,7 +322,7 @@ static struct clk_lookup dm644x_clks[] = { > CLK(NULL, "pwm2", &pwm2_clk), > CLK(NULL, "timer0", &timer0_clk), > CLK(NULL, "timer1", &timer1_clk), > - CLK("watchdog", NULL, &timer2_clk), > + CLK("davinci-wdt", NULL, &timer2_clk), > CLK(NULL, NULL, NULL), > }; > > diff --git a/arch/arm/mach-davinci/dm646x.c b/arch/arm/mach-davinci/dm646x.c > index 2a73f29..a56779b 100644 > --- a/arch/arm/mach-davinci/dm646x.c > +++ b/arch/arm/mach-davinci/dm646x.c > @@ -356,7 +356,7 @@ static struct clk_lookup dm646x_clks[] = { > CLK(NULL, "pwm1", &pwm1_clk), > CLK(NULL, "timer0", &timer0_clk), > CLK(NULL, "timer1", &timer1_clk), > - CLK("watchdog", NULL, &timer2_clk), > + CLK("davinci-wdt", NULL, &timer2_clk), > CLK("palm_bk3710", NULL, &ide_clk), > CLK(NULL, "vpif0", &vpif0_clk), > CLK(NULL, "vpif1", &vpif1_clk), > diff --git a/drivers/watchdog/davinci_wdt.c b/drivers/watchdog/davinci_wdt.c > index 884f696..9128c74 100644 > --- a/drivers/watchdog/davinci_wdt.c > +++ b/drivers/watchdog/davinci_wdt.c > @@ -218,7 +218,7 @@ MODULE_DEVICE_TABLE(of, davinci_wdt_of_match); > > static struct platform_driver platform_wdt_driver = { > .driver = { > - .name = "watchdog", > + .name = "davinci-wdt", > .owner = THIS_MODULE, > .of_match_table = davinci_wdt_of_match, > }, > @@ -238,4 +238,4 @@ MODULE_PARM_DESC(heartbeat, > __MODULE_STRING(DEFAULT_HEARTBEAT)); > > MODULE_LICENSE("GPL"); > -MODULE_ALIAS("platform:watchdog"); > +MODULE_ALIAS("platform:davinci-wdt"); > From mboxrd@z Thu Jan 1 00:00:00 1970 From: linux@roeck-us.net (Guenter Roeck) Date: Wed, 27 Nov 2013 07:56:43 -0800 Subject: [PATCH] watchdog: davinci: rename platform driver to davinci-wdt In-Reply-To: <1385559113-4354-1-git-send-email-ivan.khoronzhuk@ti.com> References: <1385559113-4354-1-git-send-email-ivan.khoronzhuk@ti.com> Message-ID: <5296163B.5040004@roeck-us.net> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 11/27/2013 05:31 AM, Ivan Khoronzhuk wrote: > As we switch to use the watchdog core which permits more than one > active watchdog in the system, rename platform driver to > "davinci-wdt" to be identifiable. > > Signed-off-by: Ivan Khoronzhuk Reviewed-by: Guenter Roeck > --- > based on: > https://lkml.org/lkml/2013/11/27/164 > > CC: > Sekhar Nori > Guenter Roeck > > arch/arm/mach-davinci/da830.c | 2 +- > arch/arm/mach-davinci/da850.c | 2 +- > arch/arm/mach-davinci/da8xx-dt.c | 2 +- > arch/arm/mach-davinci/devices-da8xx.c | 4 ++-- > arch/arm/mach-davinci/devices.c | 2 +- > arch/arm/mach-davinci/dm355.c | 2 +- > arch/arm/mach-davinci/dm365.c | 2 +- > arch/arm/mach-davinci/dm644x.c | 2 +- > arch/arm/mach-davinci/dm646x.c | 2 +- > drivers/watchdog/davinci_wdt.c | 4 ++-- > 10 files changed, 12 insertions(+), 12 deletions(-) > > diff --git a/arch/arm/mach-davinci/da830.c b/arch/arm/mach-davinci/da830.c > index 0813b51..82c6013 100644 > --- a/arch/arm/mach-davinci/da830.c > +++ b/arch/arm/mach-davinci/da830.c > @@ -385,7 +385,7 @@ static struct clk_lookup da830_clks[] = { > CLK(NULL, "pll0_sysclk7", &pll0_sysclk7), > CLK("i2c_davinci.1", NULL, &i2c0_clk), > CLK(NULL, "timer0", &timerp64_0_clk), > - CLK("watchdog", NULL, &timerp64_1_clk), > + CLK("davinci-wdt", NULL, &timerp64_1_clk), > CLK(NULL, "arm_rom", &arm_rom_clk), > CLK(NULL, "scr0_ss", &scr0_ss_clk), > CLK(NULL, "scr1_ss", &scr1_ss_clk), > diff --git a/arch/arm/mach-davinci/da850.c b/arch/arm/mach-davinci/da850.c > index 352984e..ccb2f58 100644 > --- a/arch/arm/mach-davinci/da850.c > +++ b/arch/arm/mach-davinci/da850.c > @@ -443,7 +443,7 @@ static struct clk_lookup da850_clks[] = { > CLK(NULL, "pll1_sysclk3", &pll1_sysclk3), > CLK("i2c_davinci.1", NULL, &i2c0_clk), > CLK(NULL, "timer0", &timerp64_0_clk), > - CLK("watchdog", NULL, &timerp64_1_clk), > + CLK("davinci-wdt", NULL, &timerp64_1_clk), > CLK(NULL, "arm_rom", &arm_rom_clk), > CLK(NULL, "tpcc0", &tpcc0_clk), > CLK(NULL, "tptc0", &tptc0_clk), > diff --git a/arch/arm/mach-davinci/da8xx-dt.c b/arch/arm/mach-davinci/da8xx-dt.c > index d2bc574..ed19287 100644 > --- a/arch/arm/mach-davinci/da8xx-dt.c > +++ b/arch/arm/mach-davinci/da8xx-dt.c > @@ -32,7 +32,7 @@ static void __init da8xx_init_irq(void) > > static struct of_dev_auxdata da850_auxdata_lookup[] __initdata = { > OF_DEV_AUXDATA("ti,davinci-i2c", 0x01c22000, "i2c_davinci.1", NULL), > - OF_DEV_AUXDATA("ti,davinci-wdt", 0x01c21000, "watchdog", NULL), > + OF_DEV_AUXDATA("ti,davinci-wdt", 0x01c21000, "davinci-wdt", NULL), > OF_DEV_AUXDATA("ti,da830-mmc", 0x01c40000, "da830-mmc.0", NULL), > OF_DEV_AUXDATA("ti,da850-ehrpwm", 0x01f00000, "ehrpwm", NULL), > OF_DEV_AUXDATA("ti,da850-ehrpwm", 0x01f02000, "ehrpwm", NULL), > diff --git a/arch/arm/mach-davinci/devices-da8xx.c b/arch/arm/mach-davinci/devices-da8xx.c > index c46eccb..f9ba74b 100644 > --- a/arch/arm/mach-davinci/devices-da8xx.c > +++ b/arch/arm/mach-davinci/devices-da8xx.c > @@ -389,7 +389,7 @@ static struct resource da8xx_watchdog_resources[] = { > }; > > static struct platform_device da8xx_wdt_device = { > - .name = "watchdog", > + .name = "davinci-wdt", > .id = -1, > .num_resources = ARRAY_SIZE(da8xx_watchdog_resources), > .resource = da8xx_watchdog_resources, > @@ -399,7 +399,7 @@ void da8xx_restart(enum reboot_mode mode, const char *cmd) > { > struct device *dev; > > - dev = bus_find_device_by_name(&platform_bus_type, NULL, "watchdog"); > + dev = bus_find_device_by_name(&platform_bus_type, NULL, "davinci-wdt"); > if (!dev) { > pr_err("%s: failed to find watchdog device\n", __func__); > return; > diff --git a/arch/arm/mach-davinci/devices.c b/arch/arm/mach-davinci/devices.c > index 3996e98..5cf9a02 100644 > --- a/arch/arm/mach-davinci/devices.c > +++ b/arch/arm/mach-davinci/devices.c > @@ -302,7 +302,7 @@ static struct resource wdt_resources[] = { > }; > > struct platform_device davinci_wdt_device = { > - .name = "watchdog", > + .name = "davinci-wdt", > .id = -1, > .num_resources = ARRAY_SIZE(wdt_resources), > .resource = wdt_resources, > diff --git a/arch/arm/mach-davinci/dm355.c b/arch/arm/mach-davinci/dm355.c > index ef9ff1f..4bb8132 100644 > --- a/arch/arm/mach-davinci/dm355.c > +++ b/arch/arm/mach-davinci/dm355.c > @@ -375,7 +375,7 @@ static struct clk_lookup dm355_clks[] = { > CLK(NULL, "pwm3", &pwm3_clk), > CLK(NULL, "timer0", &timer0_clk), > CLK(NULL, "timer1", &timer1_clk), > - CLK("watchdog", NULL, &timer2_clk), > + CLK("davinci-wdt", NULL, &timer2_clk), > CLK(NULL, "timer3", &timer3_clk), > CLK(NULL, "rto", &rto_clk), > CLK(NULL, "usb", &usb_clk), > diff --git a/arch/arm/mach-davinci/dm365.c b/arch/arm/mach-davinci/dm365.c > index 1511a06..b0e8df3 100644 > --- a/arch/arm/mach-davinci/dm365.c > +++ b/arch/arm/mach-davinci/dm365.c > @@ -473,7 +473,7 @@ static struct clk_lookup dm365_clks[] = { > CLK(NULL, "pwm3", &pwm3_clk), > CLK(NULL, "timer0", &timer0_clk), > CLK(NULL, "timer1", &timer1_clk), > - CLK("watchdog", NULL, &timer2_clk), > + CLK("davinci-wdt", NULL, &timer2_clk), > CLK(NULL, "timer3", &timer3_clk), > CLK(NULL, "usb", &usb_clk), > CLK("davinci_emac.1", NULL, &emac_clk), > diff --git a/arch/arm/mach-davinci/dm644x.c b/arch/arm/mach-davinci/dm644x.c > index 143a321..aeaad95 100644 > --- a/arch/arm/mach-davinci/dm644x.c > +++ b/arch/arm/mach-davinci/dm644x.c > @@ -322,7 +322,7 @@ static struct clk_lookup dm644x_clks[] = { > CLK(NULL, "pwm2", &pwm2_clk), > CLK(NULL, "timer0", &timer0_clk), > CLK(NULL, "timer1", &timer1_clk), > - CLK("watchdog", NULL, &timer2_clk), > + CLK("davinci-wdt", NULL, &timer2_clk), > CLK(NULL, NULL, NULL), > }; > > diff --git a/arch/arm/mach-davinci/dm646x.c b/arch/arm/mach-davinci/dm646x.c > index 2a73f29..a56779b 100644 > --- a/arch/arm/mach-davinci/dm646x.c > +++ b/arch/arm/mach-davinci/dm646x.c > @@ -356,7 +356,7 @@ static struct clk_lookup dm646x_clks[] = { > CLK(NULL, "pwm1", &pwm1_clk), > CLK(NULL, "timer0", &timer0_clk), > CLK(NULL, "timer1", &timer1_clk), > - CLK("watchdog", NULL, &timer2_clk), > + CLK("davinci-wdt", NULL, &timer2_clk), > CLK("palm_bk3710", NULL, &ide_clk), > CLK(NULL, "vpif0", &vpif0_clk), > CLK(NULL, "vpif1", &vpif1_clk), > diff --git a/drivers/watchdog/davinci_wdt.c b/drivers/watchdog/davinci_wdt.c > index 884f696..9128c74 100644 > --- a/drivers/watchdog/davinci_wdt.c > +++ b/drivers/watchdog/davinci_wdt.c > @@ -218,7 +218,7 @@ MODULE_DEVICE_TABLE(of, davinci_wdt_of_match); > > static struct platform_driver platform_wdt_driver = { > .driver = { > - .name = "watchdog", > + .name = "davinci-wdt", > .owner = THIS_MODULE, > .of_match_table = davinci_wdt_of_match, > }, > @@ -238,4 +238,4 @@ MODULE_PARM_DESC(heartbeat, > __MODULE_STRING(DEFAULT_HEARTBEAT)); > > MODULE_LICENSE("GPL"); > -MODULE_ALIAS("platform:watchdog"); > +MODULE_ALIAS("platform:davinci-wdt"); >