* [PATCHv2 3/3] mpc52xx/wdt: remove obsolete old WDT implementation @ 2009-11-12 18:45 ` Albrecht Dreß 0 siblings, 0 replies; 12+ messages in thread From: Albrecht Dreß @ 2009-11-12 18:45 UTC (permalink / raw) To: Likely, Grant, Linux PPC Development, Devicetree Discussions, Wim Van Sebroeck Remove the old WDT implementation. Signed-off-by: Albrecht Dre=DF <albrecht.dress@arcor.de> --- Change against v1: WDT stuff now fully merged into the file arch/powerpc/platforms/52xx/mpc52xx_gpt.c. Note: The file drivers/watchdog/mpc5200_wdt.c is now obsolete and should be= removed. drivers/watchdog/Kconfig | 4 +++- drivers/watchdog/Makefile | 1 - 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig index 3711b88..d958b76 100644 --- a/drivers/watchdog/Kconfig +++ b/drivers/watchdog/Kconfig @@ -861,8 +861,10 @@ config GEF_WDT Watchdog timer found in a number of GE Fanuc single board computers. =20 config MPC5200_WDT - tristate "MPC5200 Watchdog Timer" + bool "MPC52xx Watchdog Timer" depends on PPC_MPC52xx + help + Use General Purpose Timer (GPT) 0 on the MPC5200 as Watchdog. =20 config 8xxx_WDT tristate "MPC8xxx Platform Watchdog Timer" diff --git a/drivers/watchdog/Makefile b/drivers/watchdog/Makefile index 699199b..89c045d 100644 --- a/drivers/watchdog/Makefile +++ b/drivers/watchdog/Makefile @@ -118,7 +118,6 @@ obj-$(CONFIG_TXX9_WDT) +=3D txx9wdt.o =20 # POWERPC Architecture obj-$(CONFIG_GEF_WDT) +=3D gef_wdt.o -obj-$(CONFIG_MPC5200_WDT) +=3D mpc5200_wdt.o obj-$(CONFIG_8xxx_WDT) +=3D mpc8xxx_wdt.o obj-$(CONFIG_MV64X60_WDT) +=3D mv64x60_wdt.o obj-$(CONFIG_PIKA_WDT) +=3D pika_wdt.o ^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCHv2 3/3] mpc52xx/wdt: remove obsolete old WDT implementation @ 2009-11-12 18:45 ` Albrecht Dreß 0 siblings, 0 replies; 12+ messages in thread From: Albrecht Dreß @ 2009-11-12 18:45 UTC (permalink / raw) To: Likely, Grant, Linux PPC Development, Devicetree Discussions, Wim Van Sebroeck Remove the old WDT implementation. Signed-off-by: Albrecht Dreß <albrecht.dress@arcor.de> --- Change against v1: WDT stuff now fully merged into the file arch/powerpc/platforms/52xx/mpc52xx_gpt.c. Note: The file drivers/watchdog/mpc5200_wdt.c is now obsolete and should be removed. drivers/watchdog/Kconfig | 4 +++- drivers/watchdog/Makefile | 1 - 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig index 3711b88..d958b76 100644 --- a/drivers/watchdog/Kconfig +++ b/drivers/watchdog/Kconfig @@ -861,8 +861,10 @@ config GEF_WDT Watchdog timer found in a number of GE Fanuc single board computers. config MPC5200_WDT - tristate "MPC5200 Watchdog Timer" + bool "MPC52xx Watchdog Timer" depends on PPC_MPC52xx + help + Use General Purpose Timer (GPT) 0 on the MPC5200 as Watchdog. config 8xxx_WDT tristate "MPC8xxx Platform Watchdog Timer" diff --git a/drivers/watchdog/Makefile b/drivers/watchdog/Makefile index 699199b..89c045d 100644 --- a/drivers/watchdog/Makefile +++ b/drivers/watchdog/Makefile @@ -118,7 +118,6 @@ obj-$(CONFIG_TXX9_WDT) += txx9wdt.o # POWERPC Architecture obj-$(CONFIG_GEF_WDT) += gef_wdt.o -obj-$(CONFIG_MPC5200_WDT) += mpc5200_wdt.o obj-$(CONFIG_8xxx_WDT) += mpc8xxx_wdt.o obj-$(CONFIG_MV64X60_WDT) += mv64x60_wdt.o obj-$(CONFIG_PIKA_WDT) += pika_wdt.o ^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCHv2 3/3] mpc52xx/wdt: remove obsolete old WDT implementation 2009-11-12 18:45 ` Albrecht Dreß @ 2009-11-12 19:14 ` Grant Likely -1 siblings, 0 replies; 12+ messages in thread From: Grant Likely @ 2009-11-12 19:14 UTC (permalink / raw) To: Albrecht Dreß Cc: Linux PPC Development, Devicetree Discussions, Wim Van Sebroeck On Thu, Nov 12, 2009 at 11:45 AM, Albrecht Dre=DF <albrecht.dress@arcor.de>= wrote: > Remove the old WDT implementation. > > Signed-off-by: Albrecht Dre=DF <albrecht.dress@arcor.de> Acked-by: Grant Likely <grant.likely@secretlab.ca> Wim, I'm picking up the other 2 patches into my tree. Do you want me to push this patch through my tree, or do you want to pick it up? g. > --- > > Change against v1: WDT stuff now fully merged into the file > arch/powerpc/platforms/52xx/mpc52xx_gpt.c. > > Note: The file drivers/watchdog/mpc5200_wdt.c is now obsolete and should = be removed. > > =A0drivers/watchdog/Kconfig =A0| =A0 =A04 +++- > =A0drivers/watchdog/Makefile | =A0 =A01 - > =A02 files changed, 3 insertions(+), 2 deletions(-) > > diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig > index 3711b88..d958b76 100644 > --- a/drivers/watchdog/Kconfig > +++ b/drivers/watchdog/Kconfig > @@ -861,8 +861,10 @@ config GEF_WDT > =A0 =A0 =A0 =A0 =A0Watchdog timer found in a number of GE Fanuc single bo= ard computers. > > =A0config MPC5200_WDT > - =A0 =A0 =A0 tristate "MPC5200 Watchdog Timer" > + =A0 =A0 =A0 bool "MPC52xx Watchdog Timer" > =A0 =A0 =A0 =A0depends on PPC_MPC52xx > + =A0 =A0 =A0 help > + =A0 =A0 =A0 =A0 Use General Purpose Timer (GPT) 0 on the MPC5200 as Wat= chdog. > > =A0config 8xxx_WDT > =A0 =A0 =A0 =A0tristate "MPC8xxx Platform Watchdog Timer" > diff --git a/drivers/watchdog/Makefile b/drivers/watchdog/Makefile > index 699199b..89c045d 100644 > --- a/drivers/watchdog/Makefile > +++ b/drivers/watchdog/Makefile > @@ -118,7 +118,6 @@ obj-$(CONFIG_TXX9_WDT) +=3D txx9wdt.o > > =A0# POWERPC Architecture > =A0obj-$(CONFIG_GEF_WDT) +=3D gef_wdt.o > -obj-$(CONFIG_MPC5200_WDT) +=3D mpc5200_wdt.o > =A0obj-$(CONFIG_8xxx_WDT) +=3D mpc8xxx_wdt.o > =A0obj-$(CONFIG_MV64X60_WDT) +=3D mv64x60_wdt.o > =A0obj-$(CONFIG_PIKA_WDT) +=3D pika_wdt.o > > --=20 Grant Likely, B.Sc., P.Eng. Secret Lab Technologies Ltd. ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCHv2 3/3] mpc52xx/wdt: remove obsolete old WDT implementation @ 2009-11-12 19:14 ` Grant Likely 0 siblings, 0 replies; 12+ messages in thread From: Grant Likely @ 2009-11-12 19:14 UTC (permalink / raw) To: Albrecht Dreß Cc: Linux PPC Development, Devicetree Discussions, Wim Van Sebroeck On Thu, Nov 12, 2009 at 11:45 AM, Albrecht Dreß <albrecht.dress@arcor.de> wrote: > Remove the old WDT implementation. > > Signed-off-by: Albrecht Dreß <albrecht.dress@arcor.de> Acked-by: Grant Likely <grant.likely@secretlab.ca> Wim, I'm picking up the other 2 patches into my tree. Do you want me to push this patch through my tree, or do you want to pick it up? g. > --- > > Change against v1: WDT stuff now fully merged into the file > arch/powerpc/platforms/52xx/mpc52xx_gpt.c. > > Note: The file drivers/watchdog/mpc5200_wdt.c is now obsolete and should be removed. > > drivers/watchdog/Kconfig | 4 +++- > drivers/watchdog/Makefile | 1 - > 2 files changed, 3 insertions(+), 2 deletions(-) > > diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig > index 3711b88..d958b76 100644 > --- a/drivers/watchdog/Kconfig > +++ b/drivers/watchdog/Kconfig > @@ -861,8 +861,10 @@ config GEF_WDT > Watchdog timer found in a number of GE Fanuc single board computers. > > config MPC5200_WDT > - tristate "MPC5200 Watchdog Timer" > + bool "MPC52xx Watchdog Timer" > depends on PPC_MPC52xx > + help > + Use General Purpose Timer (GPT) 0 on the MPC5200 as Watchdog. > > config 8xxx_WDT > tristate "MPC8xxx Platform Watchdog Timer" > diff --git a/drivers/watchdog/Makefile b/drivers/watchdog/Makefile > index 699199b..89c045d 100644 > --- a/drivers/watchdog/Makefile > +++ b/drivers/watchdog/Makefile > @@ -118,7 +118,6 @@ obj-$(CONFIG_TXX9_WDT) += txx9wdt.o > > # POWERPC Architecture > obj-$(CONFIG_GEF_WDT) += gef_wdt.o > -obj-$(CONFIG_MPC5200_WDT) += mpc5200_wdt.o > obj-$(CONFIG_8xxx_WDT) += mpc8xxx_wdt.o > obj-$(CONFIG_MV64X60_WDT) += mv64x60_wdt.o > obj-$(CONFIG_PIKA_WDT) += pika_wdt.o > > -- Grant Likely, B.Sc., P.Eng. Secret Lab Technologies Ltd. ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCHv2 3/3] mpc52xx/wdt: remove obsolete old WDT implementation 2009-11-12 18:45 ` Albrecht Dreß @ 2009-11-12 19:56 ` Wolfram Sang -1 siblings, 0 replies; 12+ messages in thread From: Wolfram Sang @ 2009-11-12 19:56 UTC (permalink / raw) To: Albrecht Dreß Cc: Linux PPC Development, Devicetree Discussions, Wim Van Sebroeck [-- Attachment #1: Type: text/plain, Size: 1966 bytes --] On Thu, Nov 12, 2009 at 07:45:20PM +0100, Albrecht Dreß wrote: > Remove the old WDT implementation. > > Signed-off-by: Albrecht Dreß <albrecht.dress@arcor.de> > --- > > Change against v1: WDT stuff now fully merged into the file > arch/powerpc/platforms/52xx/mpc52xx_gpt.c. > > Note: The file drivers/watchdog/mpc5200_wdt.c is now obsolete and should be removed. Ehrm, this is possible within a patch :) > > drivers/watchdog/Kconfig | 4 +++- > drivers/watchdog/Makefile | 1 - > 2 files changed, 3 insertions(+), 2 deletions(-) > > diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig > index 3711b88..d958b76 100644 > --- a/drivers/watchdog/Kconfig > +++ b/drivers/watchdog/Kconfig > @@ -861,8 +861,10 @@ config GEF_WDT > Watchdog timer found in a number of GE Fanuc single board computers. > > config MPC5200_WDT > - tristate "MPC5200 Watchdog Timer" > + bool "MPC52xx Watchdog Timer" > depends on PPC_MPC52xx > + help > + Use General Purpose Timer (GPT) 0 on the MPC5200 as Watchdog. s/5200/52xx/ here too. > > config 8xxx_WDT > tristate "MPC8xxx Platform Watchdog Timer" > diff --git a/drivers/watchdog/Makefile b/drivers/watchdog/Makefile > index 699199b..89c045d 100644 > --- a/drivers/watchdog/Makefile > +++ b/drivers/watchdog/Makefile > @@ -118,7 +118,6 @@ obj-$(CONFIG_TXX9_WDT) += txx9wdt.o > > # POWERPC Architecture > obj-$(CONFIG_GEF_WDT) += gef_wdt.o > -obj-$(CONFIG_MPC5200_WDT) += mpc5200_wdt.o > obj-$(CONFIG_8xxx_WDT) += mpc8xxx_wdt.o > obj-$(CONFIG_MV64X60_WDT) += mv64x60_wdt.o > obj-$(CONFIG_PIKA_WDT) += pika_wdt.o > > _______________________________________________ > Linuxppc-dev mailing list > Linuxppc-dev@lists.ozlabs.org > https://lists.ozlabs.org/listinfo/linuxppc-dev -- Pengutronix e.K. | Wolfram Sang | Industrial Linux Solutions | http://www.pengutronix.de/ | [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 197 bytes --] ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCHv2 3/3] mpc52xx/wdt: remove obsolete old WDT implementation @ 2009-11-12 19:56 ` Wolfram Sang 0 siblings, 0 replies; 12+ messages in thread From: Wolfram Sang @ 2009-11-12 19:56 UTC (permalink / raw) To: Albrecht Dreß Cc: Linux PPC Development, Devicetree Discussions, Wim Van Sebroeck [-- Attachment #1.1: Type: text/plain, Size: 2009 bytes --] On Thu, Nov 12, 2009 at 07:45:20PM +0100, Albrecht Dreß wrote: > Remove the old WDT implementation. > > Signed-off-by: Albrecht Dreß <albrecht.dress-KvP5wT2u2U0@public.gmane.org> > --- > > Change against v1: WDT stuff now fully merged into the file > arch/powerpc/platforms/52xx/mpc52xx_gpt.c. > > Note: The file drivers/watchdog/mpc5200_wdt.c is now obsolete and should be removed. Ehrm, this is possible within a patch :) > > drivers/watchdog/Kconfig | 4 +++- > drivers/watchdog/Makefile | 1 - > 2 files changed, 3 insertions(+), 2 deletions(-) > > diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig > index 3711b88..d958b76 100644 > --- a/drivers/watchdog/Kconfig > +++ b/drivers/watchdog/Kconfig > @@ -861,8 +861,10 @@ config GEF_WDT > Watchdog timer found in a number of GE Fanuc single board computers. > > config MPC5200_WDT > - tristate "MPC5200 Watchdog Timer" > + bool "MPC52xx Watchdog Timer" > depends on PPC_MPC52xx > + help > + Use General Purpose Timer (GPT) 0 on the MPC5200 as Watchdog. s/5200/52xx/ here too. > > config 8xxx_WDT > tristate "MPC8xxx Platform Watchdog Timer" > diff --git a/drivers/watchdog/Makefile b/drivers/watchdog/Makefile > index 699199b..89c045d 100644 > --- a/drivers/watchdog/Makefile > +++ b/drivers/watchdog/Makefile > @@ -118,7 +118,6 @@ obj-$(CONFIG_TXX9_WDT) += txx9wdt.o > > # POWERPC Architecture > obj-$(CONFIG_GEF_WDT) += gef_wdt.o > -obj-$(CONFIG_MPC5200_WDT) += mpc5200_wdt.o > obj-$(CONFIG_8xxx_WDT) += mpc8xxx_wdt.o > obj-$(CONFIG_MV64X60_WDT) += mv64x60_wdt.o > obj-$(CONFIG_PIKA_WDT) += pika_wdt.o > > _______________________________________________ > Linuxppc-dev mailing list > Linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org > https://lists.ozlabs.org/listinfo/linuxppc-dev -- Pengutronix e.K. | Wolfram Sang | Industrial Linux Solutions | http://www.pengutronix.de/ | [-- Attachment #1.2: Digital signature --] [-- Type: application/pgp-signature, Size: 197 bytes --] [-- Attachment #2: Type: text/plain, Size: 192 bytes --] _______________________________________________ devicetree-discuss mailing list devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org https://lists.ozlabs.org/listinfo/devicetree-discuss ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCHv2 3/3] mpc52xx/wdt: remove obsolete old WDT implementation 2009-11-12 19:56 ` Wolfram Sang @ 2009-11-12 20:36 ` Albrecht Dreß -1 siblings, 0 replies; 12+ messages in thread From: Albrecht Dreß @ 2009-11-12 20:36 UTC (permalink / raw) To: Wolfram Sang Cc: Linux PPC Development, Devicetree Discussions, Wim Van Sebroeck [-- Attachment #1: Type: text/plain, Size: 880 bytes --] Hi Wolfram: Am 12.11.09 20:56 schrieb(en) Wolfram Sang: > > Note: The file drivers/watchdog/mpc5200_wdt.c is now obsolete and should be removed. > > Ehrm, this is possible within a patch :) Ummm, I tried that (removed the file from git locally), but apparently I'm too dumb to find the proper options to get this into the diff (to my defence, I'm still using cvs/svn for my projects). Can you give me a hint? > > config MPC5200_WDT > > - tristate "MPC5200 Watchdog Timer" > > + bool "MPC52xx Watchdog Timer" > > depends on PPC_MPC52xx > > + help > > + Use General Purpose Timer (GPT) 0 on the MPC5200 as Watchdog. > > s/5200/52xx/ here too. Yes, you're right. The config option should still be 'MPC5200_WDT', though, as otherwise all the default configs had to be touched - IMO, on the low level, this does no harm. Thoughts? Cheers, Albrecht. [-- Attachment #2: Type: application/pgp-signature, Size: 190 bytes --] ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCHv2 3/3] mpc52xx/wdt: remove obsolete old WDT implementation @ 2009-11-12 20:36 ` Albrecht Dreß 0 siblings, 0 replies; 12+ messages in thread From: Albrecht Dreß @ 2009-11-12 20:36 UTC (permalink / raw) To: Wolfram Sang Cc: Linux PPC Development, Devicetree Discussions, Wim Van Sebroeck [-- Attachment #1.1: Type: text/plain, Size: 880 bytes --] Hi Wolfram: Am 12.11.09 20:56 schrieb(en) Wolfram Sang: > > Note: The file drivers/watchdog/mpc5200_wdt.c is now obsolete and should be removed. > > Ehrm, this is possible within a patch :) Ummm, I tried that (removed the file from git locally), but apparently I'm too dumb to find the proper options to get this into the diff (to my defence, I'm still using cvs/svn for my projects). Can you give me a hint? > > config MPC5200_WDT > > - tristate "MPC5200 Watchdog Timer" > > + bool "MPC52xx Watchdog Timer" > > depends on PPC_MPC52xx > > + help > > + Use General Purpose Timer (GPT) 0 on the MPC5200 as Watchdog. > > s/5200/52xx/ here too. Yes, you're right. The config option should still be 'MPC5200_WDT', though, as otherwise all the default configs had to be touched - IMO, on the low level, this does no harm. Thoughts? Cheers, Albrecht. [-- Attachment #1.2: Type: application/pgp-signature, Size: 190 bytes --] [-- Attachment #2: Type: text/plain, Size: 150 bytes --] _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCHv2 3/3] mpc52xx/wdt: remove obsolete old WDT implementation 2009-11-12 20:36 ` Albrecht Dreß @ 2009-11-12 20:43 ` Grant Likely -1 siblings, 0 replies; 12+ messages in thread From: Grant Likely @ 2009-11-12 20:43 UTC (permalink / raw) To: Albrecht Dreß Cc: Linux PPC Development, Devicetree Discussions, Wim Van Sebroeck On Thu, Nov 12, 2009 at 1:36 PM, Albrecht Dre=DF <albrecht.dress@arcor.de> = wrote: > Hi Wolfram: > > Am 12.11.09 20:56 schrieb(en) Wolfram Sang: >> >> > Note: The file drivers/watchdog/mpc5200_wdt.c is now obsolete and shou= ld >> > be removed. >> >> Ehrm, this is possible within a patch :) > > Ummm, I tried that (removed the file from git locally), but apparently I'= m > too dumb to find the proper options to get this into the diff (to my > defence, I'm still using cvs/svn for my projects). =A0Can you give me a h= int? > >> > =A0config MPC5200_WDT >> > - =A0 =A0 tristate "MPC5200 Watchdog Timer" >> > + =A0 =A0 bool "MPC52xx Watchdog Timer" >> > =A0 =A0 =A0 depends on PPC_MPC52xx >> > + =A0 =A0 help >> > + =A0 =A0 =A0 Use General Purpose Timer (GPT) 0 on the MPC5200 as Watc= hdog. >> >> s/5200/52xx/ here too. > > Yes, you're right. =A0The config option should still be 'MPC5200_WDT', th= ough, > as otherwise all the default configs had to be touched - IMO, on the low > level, this does no harm. =A0Thoughts? I don't care. Don't bother respinning just for this. g. --=20 Grant Likely, B.Sc., P.Eng. Secret Lab Technologies Ltd. ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCHv2 3/3] mpc52xx/wdt: remove obsolete old WDT implementation @ 2009-11-12 20:43 ` Grant Likely 0 siblings, 0 replies; 12+ messages in thread From: Grant Likely @ 2009-11-12 20:43 UTC (permalink / raw) To: Albrecht Dreß Cc: Linux PPC Development, Devicetree Discussions, Wim Van Sebroeck On Thu, Nov 12, 2009 at 1:36 PM, Albrecht Dreß <albrecht.dress-KvP5wT2u2U0@public.gmane.org> wrote: > Hi Wolfram: > > Am 12.11.09 20:56 schrieb(en) Wolfram Sang: >> >> > Note: The file drivers/watchdog/mpc5200_wdt.c is now obsolete and should >> > be removed. >> >> Ehrm, this is possible within a patch :) > > Ummm, I tried that (removed the file from git locally), but apparently I'm > too dumb to find the proper options to get this into the diff (to my > defence, I'm still using cvs/svn for my projects). Can you give me a hint? > >> > config MPC5200_WDT >> > - tristate "MPC5200 Watchdog Timer" >> > + bool "MPC52xx Watchdog Timer" >> > depends on PPC_MPC52xx >> > + help >> > + Use General Purpose Timer (GPT) 0 on the MPC5200 as Watchdog. >> >> s/5200/52xx/ here too. > > Yes, you're right. The config option should still be 'MPC5200_WDT', though, > as otherwise all the default configs had to be touched - IMO, on the low > level, this does no harm. Thoughts? I don't care. Don't bother respinning just for this. g. -- Grant Likely, B.Sc., P.Eng. Secret Lab Technologies Ltd. ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCHv2 3/3] mpc52xx/wdt: remove obsolete old WDT implementation 2009-11-12 20:36 ` Albrecht Dreß @ 2009-11-12 20:47 ` Wolfram Sang -1 siblings, 0 replies; 12+ messages in thread From: Wolfram Sang @ 2009-11-12 20:47 UTC (permalink / raw) To: Albrecht Dreß Cc: Linux PPC Development, Devicetree Discussions, Wim Van Sebroeck [-- Attachment #1: Type: text/plain, Size: 1152 bytes --] On Thu, Nov 12, 2009 at 09:36:42PM +0100, Albrecht Dreß wrote: > Hi Wolfram: > > Am 12.11.09 20:56 schrieb(en) Wolfram Sang: >> > Note: The file drivers/watchdog/mpc5200_wdt.c is now obsolete and should >> > be removed. >> >> Ehrm, this is possible within a patch :) > > Ummm, I tried that (removed the file from git locally), but apparently I'm > too dumb to find the proper options to get this into the diff (to my defence, > I'm still using cvs/svn for my projects). Can you give me a hint? git rm file You can even do this after you removed the file using rm already. 'git status' gives you an overview about the current changes and how to add them to the commit. > Yes, you're right. The config option should still be 'MPC5200_WDT', though, > as otherwise all the default configs had to be touched - IMO, on the low > level, this does no harm. Thoughts? I'd also leave the config name. But the help text should match the prompt :) Regards, Wolfram -- Pengutronix e.K. | Wolfram Sang | Industrial Linux Solutions | http://www.pengutronix.de/ | [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 197 bytes --] ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCHv2 3/3] mpc52xx/wdt: remove obsolete old WDT implementation @ 2009-11-12 20:47 ` Wolfram Sang 0 siblings, 0 replies; 12+ messages in thread From: Wolfram Sang @ 2009-11-12 20:47 UTC (permalink / raw) To: Albrecht Dreß Cc: Linux PPC Development, Devicetree Discussions, Wim Van Sebroeck [-- Attachment #1.1: Type: text/plain, Size: 1152 bytes --] On Thu, Nov 12, 2009 at 09:36:42PM +0100, Albrecht Dreß wrote: > Hi Wolfram: > > Am 12.11.09 20:56 schrieb(en) Wolfram Sang: >> > Note: The file drivers/watchdog/mpc5200_wdt.c is now obsolete and should >> > be removed. >> >> Ehrm, this is possible within a patch :) > > Ummm, I tried that (removed the file from git locally), but apparently I'm > too dumb to find the proper options to get this into the diff (to my defence, > I'm still using cvs/svn for my projects). Can you give me a hint? git rm file You can even do this after you removed the file using rm already. 'git status' gives you an overview about the current changes and how to add them to the commit. > Yes, you're right. The config option should still be 'MPC5200_WDT', though, > as otherwise all the default configs had to be touched - IMO, on the low > level, this does no harm. Thoughts? I'd also leave the config name. But the help text should match the prompt :) Regards, Wolfram -- Pengutronix e.K. | Wolfram Sang | Industrial Linux Solutions | http://www.pengutronix.de/ | [-- Attachment #1.2: Digital signature --] [-- Type: application/pgp-signature, Size: 197 bytes --] [-- Attachment #2: Type: text/plain, Size: 150 bytes --] _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev ^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2009-11-12 20:47 UTC | newest] Thread overview: 12+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2009-11-12 18:45 [PATCHv2 3/3] mpc52xx/wdt: remove obsolete old WDT implementation Albrecht Dreß 2009-11-12 18:45 ` Albrecht Dreß 2009-11-12 19:14 ` Grant Likely 2009-11-12 19:14 ` Grant Likely 2009-11-12 19:56 ` Wolfram Sang 2009-11-12 19:56 ` Wolfram Sang 2009-11-12 20:36 ` Albrecht Dreß 2009-11-12 20:36 ` Albrecht Dreß 2009-11-12 20:43 ` Grant Likely 2009-11-12 20:43 ` Grant Likely 2009-11-12 20:47 ` Wolfram Sang 2009-11-12 20:47 ` Wolfram Sang
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.