* Re: [PATCH] sh: Rework sh7720 GPIO header usage
2008-09-29 10:46 [PATCH] sh: Rework sh7720 GPIO header usage Magnus Damm
@ 2008-09-29 11:17 ` Paul Mundt
2008-09-29 11:20 ` Paul Mundt
` (4 subsequent siblings)
5 siblings, 0 replies; 7+ messages in thread
From: Paul Mundt @ 2008-09-29 11:17 UTC (permalink / raw)
To: linux-sh
On Mon, Sep 29, 2008 at 07:46:26PM +0900, Magnus Damm wrote:
> From: Magnus Damm <damm@igel.co.jp>
>
> This patch reworks the sh7720 gpio includes. Makes space for the
> generic GPIO code.
>
> Signed-off-by: Magnus Damm <damm@igel.co.jp>
> ---
>
> Compile tested with the magicpanelr2_defconfig.
>
> arch/sh/include/asm/gpio.h | 8 --------
> arch/sh/include/asm/magicpanelr2.h | 2 +-
> drivers/serial/sh-sci.h | 4 +++-
> 3 files changed, 4 insertions(+), 10 deletions(-)
>
Churn in drivers/ for this kind of thing is pretty uninspiring. I'll just
change the asm/gpio.h ifdef to reference cpu-sh3 explicitly instead. It
will serve as a reminder that SH-3 still needs to be converted.
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] sh: Rework sh7720 GPIO header usage
2008-09-29 10:46 [PATCH] sh: Rework sh7720 GPIO header usage Magnus Damm
2008-09-29 11:17 ` Paul Mundt
@ 2008-09-29 11:20 ` Paul Mundt
2008-09-30 6:27 ` Mark Jonas
` (3 subsequent siblings)
5 siblings, 0 replies; 7+ messages in thread
From: Paul Mundt @ 2008-09-29 11:20 UTC (permalink / raw)
To: linux-sh
On Mon, Sep 29, 2008 at 08:17:39PM +0900, Paul Mundt wrote:
> On Mon, Sep 29, 2008 at 07:46:26PM +0900, Magnus Damm wrote:
> > From: Magnus Damm <damm@igel.co.jp>
> >
> > This patch reworks the sh7720 gpio includes. Makes space for the
> > generic GPIO code.
> >
> > Signed-off-by: Magnus Damm <damm@igel.co.jp>
> > ---
> >
> > Compile tested with the magicpanelr2_defconfig.
> >
> > arch/sh/include/asm/gpio.h | 8 --------
> > arch/sh/include/asm/magicpanelr2.h | 2 +-
> > drivers/serial/sh-sci.h | 4 +++-
> > 3 files changed, 4 insertions(+), 10 deletions(-)
> >
> Churn in drivers/ for this kind of thing is pretty uninspiring. I'll just
> change the asm/gpio.h ifdef to reference cpu-sh3 explicitly instead. It
> will serve as a reminder that SH-3 still needs to be converted.
Actually, there's not even any point in that. There's no harm in having
the ifdef as it is now remain in the file. Others were there first, so
you'll have to live with it.
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] sh: Rework sh7720 GPIO header usage
2008-09-29 10:46 [PATCH] sh: Rework sh7720 GPIO header usage Magnus Damm
2008-09-29 11:17 ` Paul Mundt
2008-09-29 11:20 ` Paul Mundt
@ 2008-09-30 6:27 ` Mark Jonas
2008-09-30 6:40 ` Paul Mundt
` (2 subsequent siblings)
5 siblings, 0 replies; 7+ messages in thread
From: Mark Jonas @ 2008-09-30 6:27 UTC (permalink / raw)
To: linux-sh
Magnus,
> --- 0001/arch/sh/include/asm/gpio.h
> +++ work/arch/sh/include/asm/gpio.h 2008-09-29 15:31:10.000000000 +0900
> @@ -1,10 +1,6 @@
> /*
> * include/asm-sh/gpio.h
> *
> - * Copyright (C) 2007 Markus Brunner, Mark Jonas
> - *
> - * Addresses for the Pin Function Controller
> - *
> * This file is subject to the terms and conditions of the GNU General Public
> * License. See the file "COPYING" in the main directory of this archive
> * for more details.
> @@ -12,8 +8,4 @@
> #ifndef __ASM_SH_GPIO_H
> #define __ASM_SH_GPIO_H
>
> -#if defined(CONFIG_CPU_SH3)
> -#include <cpu/gpio.h>
> -#endif
> -
> #endif /* __ASM_SH_GPIO_H */
This is most likely a stupid question but still:
Now that arch/sh/include/asm/gpio.h is completely empty from the
compiler's POV, why wasn't simply deleted?
Mark
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] sh: Rework sh7720 GPIO header usage
2008-09-29 10:46 [PATCH] sh: Rework sh7720 GPIO header usage Magnus Damm
` (2 preceding siblings ...)
2008-09-30 6:27 ` Mark Jonas
@ 2008-09-30 6:40 ` Paul Mundt
2008-09-30 9:31 ` Magnus Damm
2008-09-30 18:25 ` Mark Jonas
5 siblings, 0 replies; 7+ messages in thread
From: Paul Mundt @ 2008-09-30 6:40 UTC (permalink / raw)
To: linux-sh
On Tue, Sep 30, 2008 at 08:27:24AM +0200, Mark Jonas wrote:
> Magnus,
>
> > --- 0001/arch/sh/include/asm/gpio.h
> > +++ work/arch/sh/include/asm/gpio.h 2008-09-29 15:31:10.000000000 +0900
> > @@ -1,10 +1,6 @@
> > /*
> > * include/asm-sh/gpio.h
> > *
> > - * Copyright (C) 2007 Markus Brunner, Mark Jonas
> > - *
> > - * Addresses for the Pin Function Controller
> > - *
> > * This file is subject to the terms and conditions of the GNU General Public
> > * License. See the file "COPYING" in the main directory of this archive
> > * for more details.
> > @@ -12,8 +8,4 @@
> > #ifndef __ASM_SH_GPIO_H
> > #define __ASM_SH_GPIO_H
> >
> > -#if defined(CONFIG_CPU_SH3)
> > -#include <cpu/gpio.h>
> > -#endif
> > -
> > #endif /* __ASM_SH_GPIO_H */
>
> This is most likely a stupid question but still:
> Now that arch/sh/include/asm/gpio.h is completely empty from the
> compiler's POV, why wasn't simply deleted?
>
It was just in this patch, the subsequent patches added back to it.
Anyways, I've dropped that, and simply merged the subsequent patches
around the ifdef. There isn't any breakage for your platform, but you may
wish to look in to what it would take to get magicpanelr2 converted to
the generic framework.
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] sh: Rework sh7720 GPIO header usage
2008-09-29 10:46 [PATCH] sh: Rework sh7720 GPIO header usage Magnus Damm
` (3 preceding siblings ...)
2008-09-30 6:40 ` Paul Mundt
@ 2008-09-30 9:31 ` Magnus Damm
2008-09-30 18:25 ` Mark Jonas
5 siblings, 0 replies; 7+ messages in thread
From: Magnus Damm @ 2008-09-30 9:31 UTC (permalink / raw)
To: linux-sh
On Tue, Sep 30, 2008 at 3:40 PM, Paul Mundt <lethal@linux-sh.org> wrote:
> On Tue, Sep 30, 2008 at 08:27:24AM +0200, Mark Jonas wrote:
>> > --- 0001/arch/sh/include/asm/gpio.h
>> > +++ work/arch/sh/include/asm/gpio.h 2008-09-29 15:31:10.000000000 +0900
>> This is most likely a stupid question but still:
>> Now that arch/sh/include/asm/gpio.h is completely empty from the
>> compiler's POV, why wasn't simply deleted?
>>
> It was just in this patch, the subsequent patches added back to it.
> Anyways, I've dropped that, and simply merged the subsequent patches
> around the ifdef. There isn't any breakage for your platform, but you may
> wish to look in to what it would take to get magicpanelr2 converted to
> the generic framework.
I've got the sh7720 specific cpu bits mostly done. Mark, do you have
time to convert the board code?
/ magnus
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] sh: Rework sh7720 GPIO header usage
2008-09-29 10:46 [PATCH] sh: Rework sh7720 GPIO header usage Magnus Damm
` (4 preceding siblings ...)
2008-09-30 9:31 ` Magnus Damm
@ 2008-09-30 18:25 ` Mark Jonas
5 siblings, 0 replies; 7+ messages in thread
From: Mark Jonas @ 2008-09-30 18:25 UTC (permalink / raw)
To: linux-sh
Magnus,
> I've got the sh7720 specific cpu bits mostly done. Mark, do you have
> time to convert the board code?
To be honest, no. My employer is currently not interested in investing
in Linux and I'm overwhelmed with other projects. Sorry.
The good news is that I am still working on convincing my employer.
Mark
^ permalink raw reply [flat|nested] 7+ messages in thread