* Re: [PATCH 02/20] serial: sh-sci: clean up white space
2011-11-11 2:43 [PATCH 02/20] serial: sh-sci: clean up white space Kuninori Morimoto
@ 2011-11-11 3:05 ` Paul Mundt
2011-11-11 3:25 ` Kuninori Morimoto
` (2 subsequent siblings)
3 siblings, 0 replies; 5+ messages in thread
From: Paul Mundt @ 2011-11-11 3:05 UTC (permalink / raw)
To: linux-sh
On Thu, Nov 10, 2011 at 06:43:31PM -0800, Kuninori Morimoto wrote:
> #define SCxSR_ERRORS(port) (to_sci_port(port)->cfg->error_mask)
>
> -#if defined(CONFIG_CPU_SUBTYPE_SH7705) || \
> - defined(CONFIG_CPU_SUBTYPE_SH7720) || \
> - defined(CONFIG_CPU_SUBTYPE_SH7721) || \
> - defined(CONFIG_ARCH_SH73A0) || \
> - defined(CONFIG_ARCH_SH7367) || \
> - defined(CONFIG_ARCH_SH7377) || \
> - defined(CONFIG_ARCH_SH7372)
> +#if defined(CONFIG_CPU_SUBTYPE_SH7705) ||\
> + defined(CONFIG_CPU_SUBTYPE_SH7720) ||\
> + defined(CONFIG_CPU_SUBTYPE_SH7721) ||\
> + defined(CONFIG_ARCH_SH73A0) ||\
> + defined(CONFIG_ARCH_SH7367) ||\
> + defined(CONFIG_ARCH_SH7377) ||\
> + defined(CONFIG_ARCH_SH7372)
> # define SCxSR_RDxF_CLEAR(port) (sci_in(port, SCxSR) & 0xfffc)
> # define SCxSR_ERROR_CLEAR(port) (sci_in(port, SCxSR) & 0xfd73)
> # define SCxSR_TDxE_CLEAR(port) (sci_in(port, SCxSR) & 0xffdf)
Err, no. I don't see this as an improvement, and I don't really see what
you think the issue with the first format is, either.
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH 02/20] serial: sh-sci: clean up white space
2011-11-11 2:43 [PATCH 02/20] serial: sh-sci: clean up white space Kuninori Morimoto
2011-11-11 3:05 ` Paul Mundt
@ 2011-11-11 3:25 ` Kuninori Morimoto
2011-11-11 3:47 ` Paul Mundt
2011-11-11 3:51 ` Kuninori Morimoto
3 siblings, 0 replies; 5+ messages in thread
From: Kuninori Morimoto @ 2011-11-11 3:25 UTC (permalink / raw)
To: linux-sh
Hi Paul
Thank you for checking.
> On Thu, Nov 10, 2011 at 06:43:31PM -0800, Kuninori Morimoto wrote:
> > #define SCxSR_ERRORS(port) (to_sci_port(port)->cfg->error_mask)
> >
> > -#if defined(CONFIG_CPU_SUBTYPE_SH7705) || \
> > - defined(CONFIG_CPU_SUBTYPE_SH7720) || \
> > - defined(CONFIG_CPU_SUBTYPE_SH7721) || \
> > - defined(CONFIG_ARCH_SH73A0) || \
> > - defined(CONFIG_ARCH_SH7367) || \
> > - defined(CONFIG_ARCH_SH7377) || \
> > - defined(CONFIG_ARCH_SH7372)
> > +#if defined(CONFIG_CPU_SUBTYPE_SH7705) ||\
> > + defined(CONFIG_CPU_SUBTYPE_SH7720) ||\
> > + defined(CONFIG_CPU_SUBTYPE_SH7721) ||\
> > + defined(CONFIG_ARCH_SH73A0) ||\
> > + defined(CONFIG_ARCH_SH7367) ||\
> > + defined(CONFIG_ARCH_SH7377) ||\
> > + defined(CONFIG_ARCH_SH7372)
> > # define SCxSR_RDxF_CLEAR(port) (sci_in(port, SCxSR) & 0xfffc)
> > # define SCxSR_ERROR_CLEAR(port) (sci_in(port, SCxSR) & 0xfd73)
> > # define SCxSR_TDxE_CLEAR(port) (sci_in(port, SCxSR) & 0xffdf)
>
> Err, no. I don't see this as an improvement, and I don't really see what
> you think the issue with the first format is, either.
OK. this is not important. please drop it.
but [PATCH 11/20] ARM: mach-shmobile: R-Mobile A1 support.
is depend on this patch.
I will re-post it.
Best regards
---
Kuninori Morimoto
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 02/20] serial: sh-sci: clean up white space
2011-11-11 2:43 [PATCH 02/20] serial: sh-sci: clean up white space Kuninori Morimoto
2011-11-11 3:05 ` Paul Mundt
2011-11-11 3:25 ` Kuninori Morimoto
@ 2011-11-11 3:47 ` Paul Mundt
2011-11-11 3:51 ` Kuninori Morimoto
3 siblings, 0 replies; 5+ messages in thread
From: Paul Mundt @ 2011-11-11 3:47 UTC (permalink / raw)
To: linux-sh
On Thu, Nov 10, 2011 at 07:25:33PM -0800, Kuninori Morimoto wrote:
> > On Thu, Nov 10, 2011 at 06:43:31PM -0800, Kuninori Morimoto wrote:
> > > #define SCxSR_ERRORS(port) (to_sci_port(port)->cfg->error_mask)
> > >
> > > -#if defined(CONFIG_CPU_SUBTYPE_SH7705) || \
> > > - defined(CONFIG_CPU_SUBTYPE_SH7720) || \
> > > - defined(CONFIG_CPU_SUBTYPE_SH7721) || \
> > > - defined(CONFIG_ARCH_SH73A0) || \
> > > - defined(CONFIG_ARCH_SH7367) || \
> > > - defined(CONFIG_ARCH_SH7377) || \
> > > - defined(CONFIG_ARCH_SH7372)
> > > +#if defined(CONFIG_CPU_SUBTYPE_SH7705) ||\
> > > + defined(CONFIG_CPU_SUBTYPE_SH7720) ||\
> > > + defined(CONFIG_CPU_SUBTYPE_SH7721) ||\
> > > + defined(CONFIG_ARCH_SH73A0) ||\
> > > + defined(CONFIG_ARCH_SH7367) ||\
> > > + defined(CONFIG_ARCH_SH7377) ||\
> > > + defined(CONFIG_ARCH_SH7372)
> > > # define SCxSR_RDxF_CLEAR(port) (sci_in(port, SCxSR) & 0xfffc)
> > > # define SCxSR_ERROR_CLEAR(port) (sci_in(port, SCxSR) & 0xfd73)
> > > # define SCxSR_TDxE_CLEAR(port) (sci_in(port, SCxSR) & 0xffdf)
> >
> > Err, no. I don't see this as an improvement, and I don't really see what
> > you think the issue with the first format is, either.
>
> OK. this is not important. please drop it.
> but [PATCH 11/20] ARM: mach-shmobile: R-Mobile A1 support.
> is depend on this patch.
> I will re-post it.
>
Not a big deal, I will just fix it up by hand.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 02/20] serial: sh-sci: clean up white space
2011-11-11 2:43 [PATCH 02/20] serial: sh-sci: clean up white space Kuninori Morimoto
` (2 preceding siblings ...)
2011-11-11 3:47 ` Paul Mundt
@ 2011-11-11 3:51 ` Kuninori Morimoto
3 siblings, 0 replies; 5+ messages in thread
From: Kuninori Morimoto @ 2011-11-11 3:51 UTC (permalink / raw)
To: linux-sh
Hi Paul
> > OK. this is not important. please drop it.
> > but [PATCH 11/20] ARM: mach-shmobile: R-Mobile A1 support.
> > is depend on this patch.
> > I will re-post it.
> >
> Not a big deal, I will just fix it up by hand.
Thank you
Best regards
---
Kuninori Morimoto
^ permalink raw reply [flat|nested] 5+ messages in thread