* Re: [PATCH] ARM: shmobile: r8a7790: switch from scif to scifa
2014-09-16 16:10 [PATCH] ARM: shmobile: r8a7790: switch from scif to scifa Wolfram Sang
@ 2014-09-17 14:40 ` Laurent Pinchart
2014-09-17 23:55 ` Simon Horman
` (11 subsequent siblings)
12 siblings, 0 replies; 14+ messages in thread
From: Laurent Pinchart @ 2014-09-17 14:40 UTC (permalink / raw)
To: linux-sh
Hi Wolfram,
On Tuesday 16 September 2014 18:10:37 Wolfram Sang wrote:
> From: Wolfram Sang <wsa+renesas@sang-engineering.com>
>
> SCIF and SCIFA can be plexed onto the same wires on Lager board. The
> datasheet also describes the wires as SCIFA. So, to make use of the
> bigger FIFOs switch to SCIFA instead.
>
> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
> ---
>
> Currently, I use this patch to check if the DMA RX issue is the same on SCIF
> and SCIFA (yes, it is). However, I still think it makes sense to use the
> bigger FIFOs. I will check this next week with Bastian's FIFO patches.
> Until then, I send this as RFC to collect opinions speaking for/against
> this change.
I have nothing against this change, but I believe the aliases should now be
serial0 and serial1 to match the legacy code (OK, that's a lame excuse, we all
know that I just want to change the aliases ;-)).
> arch/arm/boot/dts/r8a7790-lager.dts | 24 ++++++++++++------------
> 1 file changed, 12 insertions(+), 12 deletions(-)
>
> diff --git a/arch/arm/boot/dts/r8a7790-lager.dts
> b/arch/arm/boot/dts/r8a7790-lager.dts index 84dcafa7179b..782f0aa938b3
> 100644
> --- a/arch/arm/boot/dts/r8a7790-lager.dts
> +++ b/arch/arm/boot/dts/r8a7790-lager.dts
> @@ -19,8 +19,8 @@
> compatible = "renesas,lager", "renesas,r8a7790";
>
> aliases {
> - serial6 = &scif0;
> - serial7 = &scif1;
> + serial6 = &scifa0;
> + serial7 = &scifa1;
> };
>
> chosen {
> @@ -159,9 +159,9 @@
> renesas,function = "du";
> };
>
> - scif0_pins: serial0 {
> - renesas,groups = "scif0_data";
> - renesas,function = "scif0";
> + scifa0_pins: serial0 {
> + renesas,groups = "scifa0_data";
> + renesas,function = "scifa0";
> };
>
> ether_pins: ether {
> @@ -174,9 +174,9 @@
> renesas,function = "intc";
> };
>
> - scif1_pins: serial1 {
> - renesas,groups = "scif1_data";
> - renesas,function = "scif1";
> + scifa1_pins: serial1 {
> + renesas,groups = "scifa1_data";
> + renesas,function = "scifa1";
> };
>
> sdhi0_pins: sd0 {
> @@ -308,15 +308,15 @@
> };
> };
>
> -&scif0 {
> - pinctrl-0 = <&scif0_pins>;
> +&scifa0 {
> + pinctrl-0 = <&scifa0_pins>;
> pinctrl-names = "default";
>
> status = "okay";
> };
>
> -&scif1 {
> - pinctrl-0 = <&scif1_pins>;
> +&scifa1 {
> + pinctrl-0 = <&scifa1_pins>;
> pinctrl-names = "default";
>
> status = "okay";
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCH] ARM: shmobile: r8a7790: switch from scif to scifa
2014-09-16 16:10 [PATCH] ARM: shmobile: r8a7790: switch from scif to scifa Wolfram Sang
2014-09-17 14:40 ` Laurent Pinchart
@ 2014-09-17 23:55 ` Simon Horman
2014-09-18 7:57 ` Laurent Pinchart
` (10 subsequent siblings)
12 siblings, 0 replies; 14+ messages in thread
From: Simon Horman @ 2014-09-17 23:55 UTC (permalink / raw)
To: linux-sh
On Wed, Sep 17, 2014 at 05:40:07PM +0300, Laurent Pinchart wrote:
> Hi Wolfram,
>
> On Tuesday 16 September 2014 18:10:37 Wolfram Sang wrote:
> > From: Wolfram Sang <wsa+renesas@sang-engineering.com>
> >
> > SCIF and SCIFA can be plexed onto the same wires on Lager board. The
> > datasheet also describes the wires as SCIFA. So, to make use of the
> > bigger FIFOs switch to SCIFA instead.
> >
> > Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
> > ---
> >
> > Currently, I use this patch to check if the DMA RX issue is the same on SCIF
> > and SCIFA (yes, it is). However, I still think it makes sense to use the
> > bigger FIFOs. I will check this next week with Bastian's FIFO patches.
> > Until then, I send this as RFC to collect opinions speaking for/against
> > this change.
>
> I have nothing against this change, but I believe the aliases should now be
> serial0 and serial1 to match the legacy code (OK, that's a lame excuse, we all
> know that I just want to change the aliases ;-)).
Thanks Laurent, I think its probably time we had that conversation again.
But I think that change can be done (or not ^) separately to this change.
As it is, I think this patch is just a little too late for v3.18.
I'll see about queuing it up for v3.19 in the not to distant future.
> > arch/arm/boot/dts/r8a7790-lager.dts | 24 ++++++++++++------------
> > 1 file changed, 12 insertions(+), 12 deletions(-)
> >
> > diff --git a/arch/arm/boot/dts/r8a7790-lager.dts
> > b/arch/arm/boot/dts/r8a7790-lager.dts index 84dcafa7179b..782f0aa938b3
> > 100644
> > --- a/arch/arm/boot/dts/r8a7790-lager.dts
> > +++ b/arch/arm/boot/dts/r8a7790-lager.dts
> > @@ -19,8 +19,8 @@
> > compatible = "renesas,lager", "renesas,r8a7790";
> >
> > aliases {
> > - serial6 = &scif0;
> > - serial7 = &scif1;
> > + serial6 = &scifa0;
> > + serial7 = &scifa1;
> > };
> >
> > chosen {
> > @@ -159,9 +159,9 @@
> > renesas,function = "du";
> > };
> >
> > - scif0_pins: serial0 {
> > - renesas,groups = "scif0_data";
> > - renesas,function = "scif0";
> > + scifa0_pins: serial0 {
> > + renesas,groups = "scifa0_data";
> > + renesas,function = "scifa0";
> > };
> >
> > ether_pins: ether {
> > @@ -174,9 +174,9 @@
> > renesas,function = "intc";
> > };
> >
> > - scif1_pins: serial1 {
> > - renesas,groups = "scif1_data";
> > - renesas,function = "scif1";
> > + scifa1_pins: serial1 {
> > + renesas,groups = "scifa1_data";
> > + renesas,function = "scifa1";
> > };
> >
> > sdhi0_pins: sd0 {
> > @@ -308,15 +308,15 @@
> > };
> > };
> >
> > -&scif0 {
> > - pinctrl-0 = <&scif0_pins>;
> > +&scifa0 {
> > + pinctrl-0 = <&scifa0_pins>;
> > pinctrl-names = "default";
> >
> > status = "okay";
> > };
> >
> > -&scif1 {
> > - pinctrl-0 = <&scif1_pins>;
> > +&scifa1 {
> > + pinctrl-0 = <&scifa1_pins>;
> > pinctrl-names = "default";
> >
> > status = "okay";
>
> --
> Regards,
>
> Laurent Pinchart
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-sh" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCH] ARM: shmobile: r8a7790: switch from scif to scifa
2014-09-16 16:10 [PATCH] ARM: shmobile: r8a7790: switch from scif to scifa Wolfram Sang
2014-09-17 14:40 ` Laurent Pinchart
2014-09-17 23:55 ` Simon Horman
@ 2014-09-18 7:57 ` Laurent Pinchart
2014-09-19 3:28 ` Simon Horman
` (9 subsequent siblings)
12 siblings, 0 replies; 14+ messages in thread
From: Laurent Pinchart @ 2014-09-18 7:57 UTC (permalink / raw)
To: linux-sh
Hi Simon,
On Thursday 18 September 2014 08:55:43 Simon Horman wrote:
> On Wed, Sep 17, 2014 at 05:40:07PM +0300, Laurent Pinchart wrote:
> > Hi Wolfram,
> >
> > On Tuesday 16 September 2014 18:10:37 Wolfram Sang wrote:
> > > From: Wolfram Sang <wsa+renesas@sang-engineering.com>
> > >
> > > SCIF and SCIFA can be plexed onto the same wires on Lager board. The
> > > datasheet also describes the wires as SCIFA. So, to make use of the
> > > bigger FIFOs switch to SCIFA instead.
> > >
> > > Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
> > > ---
> > >
> > > Currently, I use this patch to check if the DMA RX issue is the same on
> > > SCIF and SCIFA (yes, it is). However, I still think it makes sense to
> > > use the bigger FIFOs. I will check this next week with Bastian's FIFO
> > > patches. Until then, I send this as RFC to collect opinions speaking
> > > for/against this change.
> >
> > I have nothing against this change, but I believe the aliases should now
> > be serial0 and serial1 to match the legacy code (OK, that's a lame excuse,
> > we all know that I just want to change the aliases ;-)).
>
> Thanks Laurent, I think its probably time we had that conversation again.
> But I think that change can be done (or not ^) separately to this change.
It could, but I believe it would make sense to do both in one patch, otherwise
there will be a console serial port device mismatch between the legacy and
multiplatform cases.
> As it is, I think this patch is just a little too late for v3.18.
> I'll see about queuing it up for v3.19 in the not to distant future.
>
> > > arch/arm/boot/dts/r8a7790-lager.dts | 24 ++++++++++++------------
> > > 1 file changed, 12 insertions(+), 12 deletions(-)
> > >
> > > diff --git a/arch/arm/boot/dts/r8a7790-lager.dts
> > > b/arch/arm/boot/dts/r8a7790-lager.dts index 84dcafa7179b..782f0aa938b3
> > > 100644
> > > --- a/arch/arm/boot/dts/r8a7790-lager.dts
> > > +++ b/arch/arm/boot/dts/r8a7790-lager.dts
> > > @@ -19,8 +19,8 @@
> > > compatible = "renesas,lager", "renesas,r8a7790";
> > >
> > > aliases {
> > > - serial6 = &scif0;
> > > - serial7 = &scif1;
> > > + serial6 = &scifa0;
> > > + serial7 = &scifa1;
> > > };
> > >
> > > chosen {
> > > @@ -159,9 +159,9 @@
> > > renesas,function = "du";
> > > };
> > >
> > > - scif0_pins: serial0 {
> > > - renesas,groups = "scif0_data";
> > > - renesas,function = "scif0";
> > > + scifa0_pins: serial0 {
> > > + renesas,groups = "scifa0_data";
> > > + renesas,function = "scifa0";
> > > };
> > >
> > > ether_pins: ether {
> > > @@ -174,9 +174,9 @@
> > > renesas,function = "intc";
> > > };
> > >
> > > - scif1_pins: serial1 {
> > > - renesas,groups = "scif1_data";
> > > - renesas,function = "scif1";
> > > + scifa1_pins: serial1 {
> > > + renesas,groups = "scifa1_data";
> > > + renesas,function = "scifa1";
> > > };
> > >
> > > sdhi0_pins: sd0 {
> > > @@ -308,15 +308,15 @@
> > > };
> > > };
> > >
> > > -&scif0 {
> > > - pinctrl-0 = <&scif0_pins>;
> > > +&scifa0 {
> > > + pinctrl-0 = <&scifa0_pins>;
> > > pinctrl-names = "default";
> > >
> > > status = "okay";
> > > };
> > >
> > > -&scif1 {
> > > - pinctrl-0 = <&scif1_pins>;
> > > +&scifa1 {
> > > + pinctrl-0 = <&scifa1_pins>;
> > > pinctrl-names = "default";
> > >
> > > status = "okay";
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCH] ARM: shmobile: r8a7790: switch from scif to scifa
2014-09-16 16:10 [PATCH] ARM: shmobile: r8a7790: switch from scif to scifa Wolfram Sang
` (2 preceding siblings ...)
2014-09-18 7:57 ` Laurent Pinchart
@ 2014-09-19 3:28 ` Simon Horman
2014-09-30 4:30 ` Simon Horman
` (8 subsequent siblings)
12 siblings, 0 replies; 14+ messages in thread
From: Simon Horman @ 2014-09-19 3:28 UTC (permalink / raw)
To: linux-sh
On Thu, Sep 18, 2014 at 10:57:12AM +0300, Laurent Pinchart wrote:
> Hi Simon,
>
> On Thursday 18 September 2014 08:55:43 Simon Horman wrote:
> > On Wed, Sep 17, 2014 at 05:40:07PM +0300, Laurent Pinchart wrote:
> > > Hi Wolfram,
> > >
> > > On Tuesday 16 September 2014 18:10:37 Wolfram Sang wrote:
> > > > From: Wolfram Sang <wsa+renesas@sang-engineering.com>
> > > >
> > > > SCIF and SCIFA can be plexed onto the same wires on Lager board. The
> > > > datasheet also describes the wires as SCIFA. So, to make use of the
> > > > bigger FIFOs switch to SCIFA instead.
> > > >
> > > > Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
> > > > ---
> > > >
> > > > Currently, I use this patch to check if the DMA RX issue is the same on
> > > > SCIF and SCIFA (yes, it is). However, I still think it makes sense to
> > > > use the bigger FIFOs. I will check this next week with Bastian's FIFO
> > > > patches. Until then, I send this as RFC to collect opinions speaking
> > > > for/against this change.
> > >
> > > I have nothing against this change, but I believe the aliases should now
> > > be serial0 and serial1 to match the legacy code (OK, that's a lame excuse,
> > > we all know that I just want to change the aliases ;-)).
> >
> > Thanks Laurent, I think its probably time we had that conversation again.
> > But I think that change can be done (or not ^) separately to this change.
>
> It could, but I believe it would make sense to do both in one patch, otherwise
> there will be a console serial port device mismatch between the legacy and
> multiplatform cases.
I'm a little confused.
Does this patch introduce such a miss-match?
>
> > As it is, I think this patch is just a little too late for v3.18.
> > I'll see about queuing it up for v3.19 in the not to distant future.
> >
> > > > arch/arm/boot/dts/r8a7790-lager.dts | 24 ++++++++++++------------
> > > > 1 file changed, 12 insertions(+), 12 deletions(-)
> > > >
> > > > diff --git a/arch/arm/boot/dts/r8a7790-lager.dts
> > > > b/arch/arm/boot/dts/r8a7790-lager.dts index 84dcafa7179b..782f0aa938b3
> > > > 100644
> > > > --- a/arch/arm/boot/dts/r8a7790-lager.dts
> > > > +++ b/arch/arm/boot/dts/r8a7790-lager.dts
> > > > @@ -19,8 +19,8 @@
> > > > compatible = "renesas,lager", "renesas,r8a7790";
> > > >
> > > > aliases {
> > > > - serial6 = &scif0;
> > > > - serial7 = &scif1;
> > > > + serial6 = &scifa0;
> > > > + serial7 = &scifa1;
> > > > };
> > > >
> > > > chosen {
> > > > @@ -159,9 +159,9 @@
> > > > renesas,function = "du";
> > > > };
> > > >
> > > > - scif0_pins: serial0 {
> > > > - renesas,groups = "scif0_data";
> > > > - renesas,function = "scif0";
> > > > + scifa0_pins: serial0 {
> > > > + renesas,groups = "scifa0_data";
> > > > + renesas,function = "scifa0";
> > > > };
> > > >
> > > > ether_pins: ether {
> > > > @@ -174,9 +174,9 @@
> > > > renesas,function = "intc";
> > > > };
> > > >
> > > > - scif1_pins: serial1 {
> > > > - renesas,groups = "scif1_data";
> > > > - renesas,function = "scif1";
> > > > + scifa1_pins: serial1 {
> > > > + renesas,groups = "scifa1_data";
> > > > + renesas,function = "scifa1";
> > > > };
> > > >
> > > > sdhi0_pins: sd0 {
> > > > @@ -308,15 +308,15 @@
> > > > };
> > > > };
> > > >
> > > > -&scif0 {
> > > > - pinctrl-0 = <&scif0_pins>;
> > > > +&scifa0 {
> > > > + pinctrl-0 = <&scifa0_pins>;
> > > > pinctrl-names = "default";
> > > >
> > > > status = "okay";
> > > > };
> > > >
> > > > -&scif1 {
> > > > - pinctrl-0 = <&scif1_pins>;
> > > > +&scifa1 {
> > > > + pinctrl-0 = <&scifa1_pins>;
> > > > pinctrl-names = "default";
> > > >
> > > > status = "okay";
>
> --
> Regards,
>
> Laurent Pinchart
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-sh" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCH] ARM: shmobile: r8a7790: switch from scif to scifa
2014-09-16 16:10 [PATCH] ARM: shmobile: r8a7790: switch from scif to scifa Wolfram Sang
` (3 preceding siblings ...)
2014-09-19 3:28 ` Simon Horman
@ 2014-09-30 4:30 ` Simon Horman
2014-09-30 10:40 ` Laurent Pinchart
` (7 subsequent siblings)
12 siblings, 0 replies; 14+ messages in thread
From: Simon Horman @ 2014-09-30 4:30 UTC (permalink / raw)
To: linux-sh
On Fri, Sep 19, 2014 at 12:28:28PM +0900, Simon Horman wrote:
> On Thu, Sep 18, 2014 at 10:57:12AM +0300, Laurent Pinchart wrote:
> > Hi Simon,
> >
> > On Thursday 18 September 2014 08:55:43 Simon Horman wrote:
> > > On Wed, Sep 17, 2014 at 05:40:07PM +0300, Laurent Pinchart wrote:
> > > > Hi Wolfram,
> > > >
> > > > On Tuesday 16 September 2014 18:10:37 Wolfram Sang wrote:
> > > > > From: Wolfram Sang <wsa+renesas@sang-engineering.com>
> > > > >
> > > > > SCIF and SCIFA can be plexed onto the same wires on Lager board. The
> > > > > datasheet also describes the wires as SCIFA. So, to make use of the
> > > > > bigger FIFOs switch to SCIFA instead.
> > > > >
> > > > > Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
> > > > > ---
> > > > >
> > > > > Currently, I use this patch to check if the DMA RX issue is the same on
> > > > > SCIF and SCIFA (yes, it is). However, I still think it makes sense to
> > > > > use the bigger FIFOs. I will check this next week with Bastian's FIFO
> > > > > patches. Until then, I send this as RFC to collect opinions speaking
> > > > > for/against this change.
> > > >
> > > > I have nothing against this change, but I believe the aliases should now
> > > > be serial0 and serial1 to match the legacy code (OK, that's a lame excuse,
> > > > we all know that I just want to change the aliases ;-)).
> > >
> > > Thanks Laurent, I think its probably time we had that conversation again.
> > > But I think that change can be done (or not ^) separately to this change.
> >
> > It could, but I believe it would make sense to do both in one patch, otherwise
> > there will be a console serial port device mismatch between the legacy and
> > multiplatform cases.
>
> I'm a little confused.
> Does this patch introduce such a miss-match?
Hi Laurent,
I'd like to move this forwards somehow.
> > > As it is, I think this patch is just a little too late for v3.18.
> > > I'll see about queuing it up for v3.19 in the not to distant future.
> > >
> > > > > arch/arm/boot/dts/r8a7790-lager.dts | 24 ++++++++++++------------
> > > > > 1 file changed, 12 insertions(+), 12 deletions(-)
> > > > >
> > > > > diff --git a/arch/arm/boot/dts/r8a7790-lager.dts
> > > > > b/arch/arm/boot/dts/r8a7790-lager.dts index 84dcafa7179b..782f0aa938b3
> > > > > 100644
> > > > > --- a/arch/arm/boot/dts/r8a7790-lager.dts
> > > > > +++ b/arch/arm/boot/dts/r8a7790-lager.dts
> > > > > @@ -19,8 +19,8 @@
> > > > > compatible = "renesas,lager", "renesas,r8a7790";
> > > > >
> > > > > aliases {
> > > > > - serial6 = &scif0;
> > > > > - serial7 = &scif1;
> > > > > + serial6 = &scifa0;
> > > > > + serial7 = &scifa1;
> > > > > };
> > > > >
> > > > > chosen {
> > > > > @@ -159,9 +159,9 @@
> > > > > renesas,function = "du";
> > > > > };
> > > > >
> > > > > - scif0_pins: serial0 {
> > > > > - renesas,groups = "scif0_data";
> > > > > - renesas,function = "scif0";
> > > > > + scifa0_pins: serial0 {
> > > > > + renesas,groups = "scifa0_data";
> > > > > + renesas,function = "scifa0";
> > > > > };
> > > > >
> > > > > ether_pins: ether {
> > > > > @@ -174,9 +174,9 @@
> > > > > renesas,function = "intc";
> > > > > };
> > > > >
> > > > > - scif1_pins: serial1 {
> > > > > - renesas,groups = "scif1_data";
> > > > > - renesas,function = "scif1";
> > > > > + scifa1_pins: serial1 {
> > > > > + renesas,groups = "scifa1_data";
> > > > > + renesas,function = "scifa1";
> > > > > };
> > > > >
> > > > > sdhi0_pins: sd0 {
> > > > > @@ -308,15 +308,15 @@
> > > > > };
> > > > > };
> > > > >
> > > > > -&scif0 {
> > > > > - pinctrl-0 = <&scif0_pins>;
> > > > > +&scifa0 {
> > > > > + pinctrl-0 = <&scifa0_pins>;
> > > > > pinctrl-names = "default";
> > > > >
> > > > > status = "okay";
> > > > > };
> > > > >
> > > > > -&scif1 {
> > > > > - pinctrl-0 = <&scif1_pins>;
> > > > > +&scifa1 {
> > > > > + pinctrl-0 = <&scifa1_pins>;
> > > > > pinctrl-names = "default";
> > > > >
> > > > > status = "okay";
> >
> > --
> > Regards,
> >
> > Laurent Pinchart
> >
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-sh" in
> > the body of a message to majordomo@vger.kernel.org
> > More majordomo info at http://vger.kernel.org/majordomo-info.html
> >
> --
> To unsubscribe from this list: send the line "unsubscribe linux-sh" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCH] ARM: shmobile: r8a7790: switch from scif to scifa
2014-09-16 16:10 [PATCH] ARM: shmobile: r8a7790: switch from scif to scifa Wolfram Sang
` (4 preceding siblings ...)
2014-09-30 4:30 ` Simon Horman
@ 2014-09-30 10:40 ` Laurent Pinchart
2014-09-30 10:44 ` Geert Uytterhoeven
` (6 subsequent siblings)
12 siblings, 0 replies; 14+ messages in thread
From: Laurent Pinchart @ 2014-09-30 10:40 UTC (permalink / raw)
To: linux-sh
Hi Simon,
On Tuesday 30 September 2014 13:30:29 Simon Horman wrote:
> On Fri, Sep 19, 2014 at 12:28:28PM +0900, Simon Horman wrote:
> > On Thu, Sep 18, 2014 at 10:57:12AM +0300, Laurent Pinchart wrote:
> >> On Thursday 18 September 2014 08:55:43 Simon Horman wrote:
> >>> On Wed, Sep 17, 2014 at 05:40:07PM +0300, Laurent Pinchart wrote:
> >>>> On Tuesday 16 September 2014 18:10:37 Wolfram Sang wrote:
> >>>>> From: Wolfram Sang <wsa+renesas@sang-engineering.com>
> >>>>>
> >>>>> SCIF and SCIFA can be plexed onto the same wires on Lager board.
> >>>>> The datasheet also describes the wires as SCIFA. So, to make use
> >>>>> of the bigger FIFOs switch to SCIFA instead.
> >>>>>
> >>>>> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
> >>>>> ---
> >>>>>
> >>>>> Currently, I use this patch to check if the DMA RX issue is the
> >>>>> same on SCIF and SCIFA (yes, it is). However, I still think it
> >>>>> makes sense to use the bigger FIFOs. I will check this next week
> >>>>> with Bastian's FIFO patches. Until then, I send this as RFC to
> >>>>> collect opinions speaking for/against this change.
> >>>>
> >>>> I have nothing against this change, but I believe the aliases should
> >>>> now be serial0 and serial1 to match the legacy code (OK, that's a
> >>>> lame excuse, we all know that I just want to change the aliases
> >>>> ;-)).
> >>>
> >>> Thanks Laurent, I think its probably time we had that conversation
> >>> again. But I think that change can be done (or not ^) separately to
> >>> this change.
> >>
> >> It could, but I believe it would make sense to do both in one patch,
> >> otherwise there will be a console serial port device mismatch between
> >> the legacy and multiplatform cases.
> >
> > I'm a little confused.
> > Does this patch introduce such a miss-match?
>
> Hi Laurent,
>
> I'd like to move this forwards somehow.
This patch switches the Lager board to use SCIFA[01] instead of SCIF[01],
while keeping the Linux devices named /dev/ttySC[67]. With legacy boot,
SCIFA[01] are named /dev/ttySC[01]. There would thus be a mismatch in that
sense, even if this change doesn't prevent using the same Linux serial devices
on both legacy and DT boot.
I'll let you decide whether this is a problem that should be fixed.
> >>> As it is, I think this patch is just a little too late for v3.18.
> >>> I'll see about queuing it up for v3.19 in the not to distant future.
> >>>
> >>>>> arch/arm/boot/dts/r8a7790-lager.dts | 24 ++++++++++++------------
> >>>>> 1 file changed, 12 insertions(+), 12 deletions(-)
> >>>>>
> >>>>> diff --git a/arch/arm/boot/dts/r8a7790-lager.dts
> >>>>> b/arch/arm/boot/dts/r8a7790-lager.dts index
> >>>>> 84dcafa7179b..782f0aa938b3
> >>>>> 100644
> >>>>> --- a/arch/arm/boot/dts/r8a7790-lager.dts
> >>>>> +++ b/arch/arm/boot/dts/r8a7790-lager.dts
> >>>>> @@ -19,8 +19,8 @@
> >>>>> compatible = "renesas,lager", "renesas,r8a7790";
> >>>>>
> >>>>> aliases {
> >>>>> - serial6 = &scif0;
> >>>>> - serial7 = &scif1;
> >>>>> + serial6 = &scifa0;
> >>>>> + serial7 = &scifa1;
> >>>>> };
> >>>>>
> >>>>> chosen {
> >>>>> @@ -159,9 +159,9 @@
> >>>>> renesas,function = "du";
> >>>>> };
> >>>>>
> >>>>> - scif0_pins: serial0 {
> >>>>> - renesas,groups = "scif0_data";
> >>>>> - renesas,function = "scif0";
> >>>>> + scifa0_pins: serial0 {
> >>>>> + renesas,groups = "scifa0_data";
> >>>>> + renesas,function = "scifa0";
> >>>>> };
> >>>>>
> >>>>> ether_pins: ether {
> >>>>> @@ -174,9 +174,9 @@
> >>>>> renesas,function = "intc";
> >>>>> };
> >>>>>
> >>>>> - scif1_pins: serial1 {
> >>>>> - renesas,groups = "scif1_data";
> >>>>> - renesas,function = "scif1";
> >>>>> + scifa1_pins: serial1 {
> >>>>> + renesas,groups = "scifa1_data";
> >>>>> + renesas,function = "scifa1";
> >>>>> };
> >>>>>
> >>>>> sdhi0_pins: sd0 {
> >>>>> @@ -308,15 +308,15 @@
> >>>>> };
> >>>>> };
> >>>>>
> >>>>> -&scif0 {
> >>>>> - pinctrl-0 = <&scif0_pins>;
> >>>>> +&scifa0 {
> >>>>> + pinctrl-0 = <&scifa0_pins>;
> >>>>> pinctrl-names = "default";
> >>>>> status = "okay";
> >>>>> };
> >>>>>
> >>>>> -&scif1 {
> >>>>> - pinctrl-0 = <&scif1_pins>;
> >>>>> +&scifa1 {
> >>>>> + pinctrl-0 = <&scifa1_pins>;
> >>>>> pinctrl-names = "default";
> >>>>>
> >>>>> status = "okay";
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCH] ARM: shmobile: r8a7790: switch from scif to scifa
2014-09-16 16:10 [PATCH] ARM: shmobile: r8a7790: switch from scif to scifa Wolfram Sang
` (5 preceding siblings ...)
2014-09-30 10:40 ` Laurent Pinchart
@ 2014-09-30 10:44 ` Geert Uytterhoeven
2014-09-30 10:48 ` Laurent Pinchart
` (5 subsequent siblings)
12 siblings, 0 replies; 14+ messages in thread
From: Geert Uytterhoeven @ 2014-09-30 10:44 UTC (permalink / raw)
To: linux-sh
Hi Laurent,
On Tue, Sep 30, 2014 at 12:40 PM, Laurent Pinchart
<laurent.pinchart@ideasonboard.com> wrote:
> On Tuesday 30 September 2014 13:30:29 Simon Horman wrote:
>> On Fri, Sep 19, 2014 at 12:28:28PM +0900, Simon Horman wrote:
>> > On Thu, Sep 18, 2014 at 10:57:12AM +0300, Laurent Pinchart wrote:
>> >> On Thursday 18 September 2014 08:55:43 Simon Horman wrote:
>> >>> On Wed, Sep 17, 2014 at 05:40:07PM +0300, Laurent Pinchart wrote:
>> >>>> On Tuesday 16 September 2014 18:10:37 Wolfram Sang wrote:
>> >>>>> From: Wolfram Sang <wsa+renesas@sang-engineering.com>
>> >>>>>
>> >>>>> SCIF and SCIFA can be plexed onto the same wires on Lager board.
>> >>>>> The datasheet also describes the wires as SCIFA. So, to make use
>> >>>>> of the bigger FIFOs switch to SCIFA instead.
>> >>>>>
>> >>>>> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
>> >>>>> ---
>> >>>>>
>> >>>>> Currently, I use this patch to check if the DMA RX issue is the
>> >>>>> same on SCIF and SCIFA (yes, it is). However, I still think it
>> >>>>> makes sense to use the bigger FIFOs. I will check this next week
>> >>>>> with Bastian's FIFO patches. Until then, I send this as RFC to
>> >>>>> collect opinions speaking for/against this change.
>> >>>>
>> >>>> I have nothing against this change, but I believe the aliases should
>> >>>> now be serial0 and serial1 to match the legacy code (OK, that's a
>> >>>> lame excuse, we all know that I just want to change the aliases
>> >>>> ;-)).
>> >>>
>> >>> Thanks Laurent, I think its probably time we had that conversation
>> >>> again. But I think that change can be done (or not ^) separately to
>> >>> this change.
>> >>
>> >> It could, but I believe it would make sense to do both in one patch,
>> >> otherwise there will be a console serial port device mismatch between
>> >> the legacy and multiplatform cases.
>> >
>> > I'm a little confused.
>> > Does this patch introduce such a miss-match?
>>
>> Hi Laurent,
>>
>> I'd like to move this forwards somehow.
>
> This patch switches the Lager board to use SCIFA[01] instead of SCIF[01],
> while keeping the Linux devices named /dev/ttySC[67]. With legacy boot,
> SCIFA[01] are named /dev/ttySC[01]. There would thus be a mismatch in that
> sense, even if this change doesn't prevent using the same Linux serial devices
> on both legacy and DT boot.
So if I understand if correctly, booting with the current
"console=ttySC6,115200"
in /chosen/bootargs will give a working console for both legacy and
multi-platform,
but the underlying serial hardware block for the console differs (legacy uses
SCIF0, multi-platform uses SCIFA0)?
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCH] ARM: shmobile: r8a7790: switch from scif to scifa
2014-09-16 16:10 [PATCH] ARM: shmobile: r8a7790: switch from scif to scifa Wolfram Sang
` (6 preceding siblings ...)
2014-09-30 10:44 ` Geert Uytterhoeven
@ 2014-09-30 10:48 ` Laurent Pinchart
2014-09-30 23:58 ` Simon Horman
` (4 subsequent siblings)
12 siblings, 0 replies; 14+ messages in thread
From: Laurent Pinchart @ 2014-09-30 10:48 UTC (permalink / raw)
To: linux-sh
Hi Geert,
On Tuesday 30 September 2014 12:44:59 Geert Uytterhoeven wrote:
> On Tue, Sep 30, 2014 at 12:40 PM, Laurent Pinchart wrote:
> > On Tuesday 30 September 2014 13:30:29 Simon Horman wrote:
> >> On Fri, Sep 19, 2014 at 12:28:28PM +0900, Simon Horman wrote:
> >>> On Thu, Sep 18, 2014 at 10:57:12AM +0300, Laurent Pinchart wrote:
> >>>> On Thursday 18 September 2014 08:55:43 Simon Horman wrote:
> >>>>> On Wed, Sep 17, 2014 at 05:40:07PM +0300, Laurent Pinchart wrote:
> >>>>>> On Tuesday 16 September 2014 18:10:37 Wolfram Sang wrote:
> >>>>>>> From: Wolfram Sang <wsa+renesas@sang-engineering.com>
> >>>>>>>
> >>>>>>> SCIF and SCIFA can be plexed onto the same wires on Lager board.
> >>>>>>> The datasheet also describes the wires as SCIFA. So, to make use
> >>>>>>> of the bigger FIFOs switch to SCIFA instead.
> >>>>>>>
> >>>>>>> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
> >>>>>>> ---
> >>>>>>>
> >>>>>>> Currently, I use this patch to check if the DMA RX issue is the
> >> >>>>> same on SCIF and SCIFA (yes, it is). However, I still think it
> >>>>>>> makes sense to use the bigger FIFOs. I will check this next week
> >>>>>>> with Bastian's FIFO patches. Until then, I send this as RFC to
> >>>>>>> collect opinions speaking for/against this change.
> >>>>>>
> >>>>>> I have nothing against this change, but I believe the aliases should
> >>>>>> now be serial0 and serial1 to match the legacy code (OK, that's a
> >>>>>> lame excuse, we all know that I just want to change the aliases
> >>>>>> ;-)).
> >>>>>
> >>>>> Thanks Laurent, I think its probably time we had that conversation
> >>>>> again. But I think that change can be done (or not ^) separately to
> >>>>> this change.
> >>>>
> >>>> It could, but I believe it would make sense to do both in one patch,
> >>>> otherwise there will be a console serial port device mismatch between
> >>>> the legacy and multiplatform cases.
> >>>
> >>> I'm a little confused.
> >> > Does this patch introduce such a miss-match?
> >>
> >> Hi Laurent,
> >>
> >> I'd like to move this forwards somehow.
> >
> > This patch switches the Lager board to use SCIFA[01] instead of SCIF[01],
> > while keeping the Linux devices named /dev/ttySC[67]. With legacy boot,
> > SCIFA[01] are named /dev/ttySC[01]. There would thus be a mismatch in that
> > sense, even if this change doesn't prevent using the same Linux serial
> > devices on both legacy and DT boot.
>
> So if I understand if correctly, booting with the current
> "console=ttySC6,115200"
> in /chosen/bootargs will give a working console for both legacy and
> multi-platform,
> but the underlying serial hardware block for the console differs (legacy
> uses SCIF0, multi-platform uses SCIFA0)?
As far as I understand that's correct.
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCH] ARM: shmobile: r8a7790: switch from scif to scifa
2014-09-16 16:10 [PATCH] ARM: shmobile: r8a7790: switch from scif to scifa Wolfram Sang
` (7 preceding siblings ...)
2014-09-30 10:48 ` Laurent Pinchart
@ 2014-09-30 23:58 ` Simon Horman
2014-10-02 0:12 ` Simon Horman
` (3 subsequent siblings)
12 siblings, 0 replies; 14+ messages in thread
From: Simon Horman @ 2014-09-30 23:58 UTC (permalink / raw)
To: linux-sh
On Tue, Sep 30, 2014 at 01:48:27PM +0300, Laurent Pinchart wrote:
> Hi Geert,
>
> On Tuesday 30 September 2014 12:44:59 Geert Uytterhoeven wrote:
> > On Tue, Sep 30, 2014 at 12:40 PM, Laurent Pinchart wrote:
> > > On Tuesday 30 September 2014 13:30:29 Simon Horman wrote:
> > >> On Fri, Sep 19, 2014 at 12:28:28PM +0900, Simon Horman wrote:
> > >>> On Thu, Sep 18, 2014 at 10:57:12AM +0300, Laurent Pinchart wrote:
> > >>>> On Thursday 18 September 2014 08:55:43 Simon Horman wrote:
> > >>>>> On Wed, Sep 17, 2014 at 05:40:07PM +0300, Laurent Pinchart wrote:
> > >>>>>> On Tuesday 16 September 2014 18:10:37 Wolfram Sang wrote:
> > >>>>>>> From: Wolfram Sang <wsa+renesas@sang-engineering.com>
> > >>>>>>>
> > >>>>>>> SCIF and SCIFA can be plexed onto the same wires on Lager board.
> > >>>>>>> The datasheet also describes the wires as SCIFA. So, to make use
> > >>>>>>> of the bigger FIFOs switch to SCIFA instead.
> > >>>>>>>
> > >>>>>>> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
> > >>>>>>> ---
> > >>>>>>>
> > >>>>>>> Currently, I use this patch to check if the DMA RX issue is the
> > >> >>>>> same on SCIF and SCIFA (yes, it is). However, I still think it
> > >>>>>>> makes sense to use the bigger FIFOs. I will check this next week
> > >>>>>>> with Bastian's FIFO patches. Until then, I send this as RFC to
> > >>>>>>> collect opinions speaking for/against this change.
> > >>>>>>
> > >>>>>> I have nothing against this change, but I believe the aliases should
> > >>>>>> now be serial0 and serial1 to match the legacy code (OK, that's a
> > >>>>>> lame excuse, we all know that I just want to change the aliases
> > >>>>>> ;-)).
> > >>>>>
> > >>>>> Thanks Laurent, I think its probably time we had that conversation
> > >>>>> again. But I think that change can be done (or not ^) separately to
> > >>>>> this change.
> > >>>>
> > >>>> It could, but I believe it would make sense to do both in one patch,
> > >>>> otherwise there will be a console serial port device mismatch between
> > >>>> the legacy and multiplatform cases.
> > >>>
> > >>> I'm a little confused.
> > >> > Does this patch introduce such a miss-match?
> > >>
> > >> Hi Laurent,
> > >>
> > >> I'd like to move this forwards somehow.
> > >
> > > This patch switches the Lager board to use SCIFA[01] instead of SCIF[01],
> > > while keeping the Linux devices named /dev/ttySC[67]. With legacy boot,
> > > SCIFA[01] are named /dev/ttySC[01]. There would thus be a mismatch in that
> > > sense, even if this change doesn't prevent using the same Linux serial
> > > devices on both legacy and DT boot.
> >
> > So if I understand if correctly, booting with the current
> > "console=ttySC6,115200"
> > in /chosen/bootargs will give a working console for both legacy and
> > multi-platform,
> > but the underlying serial hardware block for the console differs (legacy
> > uses SCIF0, multi-platform uses SCIFA0)?
>
> As far as I understand that's correct.
I think that is a reasonable state of affairs so long as we have
Legacy C for Lager, which it seems we will have for a little while to come.
I will confer with Magnus as he has had strong opinions in this area
in the past.
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCH] ARM: shmobile: r8a7790: switch from scif to scifa
2014-09-16 16:10 [PATCH] ARM: shmobile: r8a7790: switch from scif to scifa Wolfram Sang
` (8 preceding siblings ...)
2014-09-30 23:58 ` Simon Horman
@ 2014-10-02 0:12 ` Simon Horman
2014-10-02 23:14 ` Khiem Nguyen
` (2 subsequent siblings)
12 siblings, 0 replies; 14+ messages in thread
From: Simon Horman @ 2014-10-02 0:12 UTC (permalink / raw)
To: linux-sh
On Wed, Oct 01, 2014 at 08:58:35AM +0900, Simon Horman wrote:
> On Tue, Sep 30, 2014 at 01:48:27PM +0300, Laurent Pinchart wrote:
> > Hi Geert,
> >
> > On Tuesday 30 September 2014 12:44:59 Geert Uytterhoeven wrote:
> > > On Tue, Sep 30, 2014 at 12:40 PM, Laurent Pinchart wrote:
> > > > On Tuesday 30 September 2014 13:30:29 Simon Horman wrote:
> > > >> On Fri, Sep 19, 2014 at 12:28:28PM +0900, Simon Horman wrote:
> > > >>> On Thu, Sep 18, 2014 at 10:57:12AM +0300, Laurent Pinchart wrote:
> > > >>>> On Thursday 18 September 2014 08:55:43 Simon Horman wrote:
> > > >>>>> On Wed, Sep 17, 2014 at 05:40:07PM +0300, Laurent Pinchart wrote:
> > > >>>>>> On Tuesday 16 September 2014 18:10:37 Wolfram Sang wrote:
> > > >>>>>>> From: Wolfram Sang <wsa+renesas@sang-engineering.com>
> > > >>>>>>>
> > > >>>>>>> SCIF and SCIFA can be plexed onto the same wires on Lager board.
> > > >>>>>>> The datasheet also describes the wires as SCIFA. So, to make use
> > > >>>>>>> of the bigger FIFOs switch to SCIFA instead.
> > > >>>>>>>
> > > >>>>>>> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
> > > >>>>>>> ---
> > > >>>>>>>
> > > >>>>>>> Currently, I use this patch to check if the DMA RX issue is the
> > > >> >>>>> same on SCIF and SCIFA (yes, it is). However, I still think it
> > > >>>>>>> makes sense to use the bigger FIFOs. I will check this next week
> > > >>>>>>> with Bastian's FIFO patches. Until then, I send this as RFC to
> > > >>>>>>> collect opinions speaking for/against this change.
> > > >>>>>>
> > > >>>>>> I have nothing against this change, but I believe the aliases should
> > > >>>>>> now be serial0 and serial1 to match the legacy code (OK, that's a
> > > >>>>>> lame excuse, we all know that I just want to change the aliases
> > > >>>>>> ;-)).
> > > >>>>>
> > > >>>>> Thanks Laurent, I think its probably time we had that conversation
> > > >>>>> again. But I think that change can be done (or not ^) separately to
> > > >>>>> this change.
> > > >>>>
> > > >>>> It could, but I believe it would make sense to do both in one patch,
> > > >>>> otherwise there will be a console serial port device mismatch between
> > > >>>> the legacy and multiplatform cases.
> > > >>>
> > > >>> I'm a little confused.
> > > >> > Does this patch introduce such a miss-match?
> > > >>
> > > >> Hi Laurent,
> > > >>
> > > >> I'd like to move this forwards somehow.
> > > >
> > > > This patch switches the Lager board to use SCIFA[01] instead of SCIF[01],
> > > > while keeping the Linux devices named /dev/ttySC[67]. With legacy boot,
> > > > SCIFA[01] are named /dev/ttySC[01]. There would thus be a mismatch in that
> > > > sense, even if this change doesn't prevent using the same Linux serial
> > > > devices on both legacy and DT boot.
> > >
> > > So if I understand if correctly, booting with the current
> > > "console=ttySC6,115200"
> > > in /chosen/bootargs will give a working console for both legacy and
> > > multi-platform,
> > > but the underlying serial hardware block for the console differs (legacy
> > > uses SCIF0, multi-platform uses SCIFA0)?
> >
> > As far as I understand that's correct.
>
> I think that is a reasonable state of affairs so long as we have
> Legacy C for Lager, which it seems we will have for a little while to come.
>
> I will confer with Magnus as he has had strong opinions in this area
> in the past.
Magnus also seems happy with this change so I have queued it up.
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCH] ARM: shmobile: r8a7790: switch from scif to scifa
2014-09-16 16:10 [PATCH] ARM: shmobile: r8a7790: switch from scif to scifa Wolfram Sang
` (9 preceding siblings ...)
2014-10-02 0:12 ` Simon Horman
@ 2014-10-02 23:14 ` Khiem Nguyen
2014-10-03 0:13 ` Wolfram Sang
2014-10-03 3:25 ` Khiem Nguyen
12 siblings, 0 replies; 14+ messages in thread
From: Khiem Nguyen @ 2014-10-02 23:14 UTC (permalink / raw)
To: linux-sh
Hi Wolfram, Simon,
Though the patch was already queued up by Simon,
I'd like to confirm one point in changelog.
On 10/2/2014 9:12 AM, Simon Horman wrote:
[snip]
>>>>>>>>>> On Tuesday 16 September 2014 18:10:37 Wolfram Sang wrote:
>>>>>>>>>>> From: Wolfram Sang <wsa+renesas@sang-engineering.com>
>>>>>>>>>>>
>>>>>>>>>>> SCIF and SCIFA can be plexed onto the same wires on Lager board.
>>>>>>>>>>> The datasheet also describes the wires as SCIFA. So, to make use
>>>>>>>>>>> of the bigger FIFOs switch to SCIFA instead.
Which datasheet you are mentioned about ?
I checked Lager board datasheet rev0.09 and it said debug interface is SCIF0 and SCIF1.
[snip]
>>>>>>>>>>> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
[snip]
>>>>>>>>>>> Currently, I use this patch to check if the DMA RX issue is the
>>>>>>>>>>> same on SCIF and SCIFA (yes, it is). However, I still think it
>>>>>>>>>>> makes sense to use the bigger FIFOs. I will check this next week
>>>>>>>>>>> with Bastian's FIFO patches. Until then, I send this as RFC to
>>>>>>>>>>> collect opinions speaking for/against this change.
[snip]
>>>>>>>>>> I have nothing against this change, but I believe the aliases should
>>>>>>>>>> now be serial0 and serial1 to match the legacy code (OK, that's a
>>>>>>>>>> lame excuse, we all know that I just want to change the aliases
>>>>>>>>>> ;-)).
[snip]
>>>>> This patch switches the Lager board to use SCIFA[01] instead of SCIF[01],
>>>>> while keeping the Linux devices named /dev/ttySC[67]. With legacy boot,
>>>>> SCIFA[01] are named /dev/ttySC[01]. There would thus be a mismatch in that
>>>>> sense, even if this change doesn't prevent using the same Linux serial
>>>>> devices on both legacy and DT boot.
[snip]
>>>> So if I understand if correctly, booting with the current
>>>> "console=ttySC6,115200"
>>>> in /chosen/bootargs will give a working console for both legacy and
>>>> multi-platform,
>>>> but the underlying serial hardware block for the console differs (legacy
>>>> uses SCIF0, multi-platform uses SCIFA0)?
[snip]
>> I will confer with Magnus as he has had strong opinions in this area
>> in the past.
[snip]
> Magnus also seems happy with this change so I have queued it up.
--
Best regards,
KHIEM Nguyen
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCH] ARM: shmobile: r8a7790: switch from scif to scifa
2014-09-16 16:10 [PATCH] ARM: shmobile: r8a7790: switch from scif to scifa Wolfram Sang
` (10 preceding siblings ...)
2014-10-02 23:14 ` Khiem Nguyen
@ 2014-10-03 0:13 ` Wolfram Sang
2014-10-03 3:25 ` Khiem Nguyen
12 siblings, 0 replies; 14+ messages in thread
From: Wolfram Sang @ 2014-10-03 0:13 UTC (permalink / raw)
To: linux-sh
[-- Attachment #1: Type: text/plain, Size: 372 bytes --]
Hello,
> Which datasheet you are mentioned about ?
> I checked Lager board datasheet rev0.09 and it said debug interface is SCIF0 and SCIF1.
V0.12D. On Page 22, the headline says "Debug Interface (SCIF0)", true.
However, the wires are named SCIFA0_TXD/RXD. Same for SCIF(A)1. On Page
2, as well. Check Y30 and AA29, they are named SCIFA0/A1.
Kind regards,
Wolfram
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCH] ARM: shmobile: r8a7790: switch from scif to scifa
2014-09-16 16:10 [PATCH] ARM: shmobile: r8a7790: switch from scif to scifa Wolfram Sang
` (11 preceding siblings ...)
2014-10-03 0:13 ` Wolfram Sang
@ 2014-10-03 3:25 ` Khiem Nguyen
12 siblings, 0 replies; 14+ messages in thread
From: Khiem Nguyen @ 2014-10-03 3:25 UTC (permalink / raw)
To: linux-sh
Hi Wolfram,
On 10/3/2014 9:13 AM, Wolfram Sang wrote:
> Hello,
>
>> Which datasheet you are mentioned about ?
>> I checked Lager board datasheet rev0.09 and it said debug interface is SCIF0 and SCIF1.
>
> V0.12D. On Page 22, the headline says "Debug Interface (SCIF0)", true.
> However, the wires are named SCIFA0_TXD/RXD. Same for SCIF(A)1. On Page
> 2, as well. Check Y30 and AA29, they are named SCIFA0/A1.
I think you got the information from circuit diagram of Lager board.
However, I got my information from Hardware manual of Lager board.
I also checked circuit diagram and see the mismatch
between circuit diagram and hardware manual.
I'm confirming with related persons.
Will let you know the update soon.
> Kind regards,
>
> Wolfram
>
--
Best regards,
KHIEM Nguyen
^ permalink raw reply [flat|nested] 14+ messages in thread