sh: Don't set sh-sci port_reg
authorLaurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Wed, 11 Jan 2017 14:43:29 +0000 (16:43 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 19 Jan 2017 13:17:05 +0000 (14:17 +0100)
The driver considers all negative or zero values of the port_reg field
as invalid. The four platforms that set the field to a register address
all use an address higher than 0x7fffffff, which is thus considered by
the driver as invalid. The feature is thus never used, remove it.

The feature could be implemented properly in the future using the
pinctrl and GPIO APIs if desired.

While at it, don't set the field to SCIx_NOT_SUPPORTED (-1) either,
leaving it unset leads to the same result. This will allow removing the
SCIx_NOT_SUPPORTED macro.

Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
arch/sh/kernel/cpu/sh3/setup-sh770x.c
arch/sh/kernel/cpu/sh4/setup-sh7750.c
arch/sh/kernel/cpu/sh4a/setup-sh7366.c
arch/sh/kernel/cpu/sh4a/setup-sh7723.c
arch/sh/kernel/cpu/sh4a/setup-sh7724.c

index e64fe24d860c9ccffbcee439138baa66717ff170..e1e54258b8220ccae87fdb040fdf646f3732f61d 100644 (file)
@@ -109,7 +109,6 @@ static struct platform_device rtc_device = {
 };
 
 static struct plat_sci_port scif0_platform_data = {
-       .port_reg       = 0xa4000136,
        .type           = PORT_SCI,
        .ops            = &sh770x_sci_port_ops,
        .regshift       = 1,
@@ -156,7 +155,6 @@ static struct platform_device scif1_device = {
 #if defined(CONFIG_CPU_SUBTYPE_SH7707) || \
     defined(CONFIG_CPU_SUBTYPE_SH7709)
 static struct plat_sci_port scif2_platform_data = {
-       .port_reg       = SCIx_NOT_SUPPORTED,
        .type           = PORT_IRDA,
        .ops            = &sh770x_sci_port_ops,
        .regshift       = 1,
index f693f122a2860389cf3f9650a32961f05d73f356..d98a55416306baef40ae6d0407f7b56f3d5d74bf 100644 (file)
@@ -38,7 +38,6 @@ static struct platform_device rtc_device = {
 };
 
 static struct plat_sci_port sci_platform_data = {
-       .port_reg       = 0xffe0001C,
        .type           = PORT_SCI,
        .regshift       = 2,
 };
index 3fc1a3ab634e414f46ea48592b87090a905be861..646918713d9afc3e35a2a0df6edfee2c95830253 100644 (file)
@@ -20,7 +20,6 @@
 #include <asm/clock.h>
 
 static struct plat_sci_port scif0_platform_data = {
-       .port_reg       = 0xa405013e,
        .scscr          = SCSCR_REIE,
        .type           = PORT_SCIF,
 };
index 9f47ec950af8c65735542d246ac6be7752fe72d4..6c5cb234aa33bbdddcd0e7309d77fc841704b4a4 100644 (file)
@@ -23,7 +23,6 @@
 
 /* Serial */
 static struct plat_sci_port scif0_platform_data = {
-       .port_reg       = 0xa4050160,
        .scscr          = SCSCR_REIE,
        .type           = PORT_SCIF,
        .regtype        = SCIx_SH4_SCIF_NO_SCSPTR_REGTYPE,
@@ -45,7 +44,6 @@ static struct platform_device scif0_device = {
 };
 
 static struct plat_sci_port scif1_platform_data = {
-       .port_reg       = SCIx_NOT_SUPPORTED,
        .scscr          = SCSCR_REIE,
        .type           = PORT_SCIF,
        .regtype        = SCIx_SH4_SCIF_NO_SCSPTR_REGTYPE,
@@ -67,7 +65,6 @@ static struct platform_device scif1_device = {
 };
 
 static struct plat_sci_port scif2_platform_data = {
-       .port_reg       = SCIx_NOT_SUPPORTED,
        .scscr          = SCSCR_REIE,
        .type           = PORT_SCIF,
        .regtype        = SCIx_SH4_SCIF_NO_SCSPTR_REGTYPE,
@@ -89,7 +86,6 @@ static struct platform_device scif2_device = {
 };
 
 static struct plat_sci_port scif3_platform_data = {
-       .port_reg       = SCIx_NOT_SUPPORTED,
        .scscr          = SCSCR_REIE,
        .sampling_rate  = 8,
        .type           = PORT_SCIFA,
@@ -111,7 +107,6 @@ static struct platform_device scif3_device = {
 };
 
 static struct plat_sci_port scif4_platform_data = {
-       .port_reg       = SCIx_NOT_SUPPORTED,
        .scscr          = SCSCR_REIE,
        .sampling_rate  = 8,
        .type           = PORT_SCIFA,
@@ -133,7 +128,6 @@ static struct platform_device scif4_device = {
 };
 
 static struct plat_sci_port scif5_platform_data = {
-       .port_reg       = SCIx_NOT_SUPPORTED,
        .scscr          = SCSCR_REIE,
        .sampling_rate  = 8,
        .type           = PORT_SCIFA,
index 3ce55884c7878d998abba0cef3bff454b168ff67..c20258b18775727f76637dc6de36bea44ddfebae 100644 (file)
@@ -290,7 +290,6 @@ static struct platform_device dma1_device = {
 
 /* Serial */
 static struct plat_sci_port scif0_platform_data = {
-       .port_reg       = SCIx_NOT_SUPPORTED,
        .scscr          = SCSCR_REIE,
        .type           = PORT_SCIF,
        .regtype        = SCIx_SH4_SCIF_NO_SCSPTR_REGTYPE,
@@ -312,7 +311,6 @@ static struct platform_device scif0_device = {
 };
 
 static struct plat_sci_port scif1_platform_data = {
-       .port_reg       = SCIx_NOT_SUPPORTED,
        .scscr          = SCSCR_REIE,
        .type           = PORT_SCIF,
        .regtype        = SCIx_SH4_SCIF_NO_SCSPTR_REGTYPE,
@@ -334,7 +332,6 @@ static struct platform_device scif1_device = {
 };
 
 static struct plat_sci_port scif2_platform_data = {
-       .port_reg       = SCIx_NOT_SUPPORTED,
        .scscr          = SCSCR_REIE,
        .type           = PORT_SCIF,
        .regtype        = SCIx_SH4_SCIF_NO_SCSPTR_REGTYPE,
@@ -356,7 +353,6 @@ static struct platform_device scif2_device = {
 };
 
 static struct plat_sci_port scif3_platform_data = {
-       .port_reg       = SCIx_NOT_SUPPORTED,
        .sampling_rate  = 8,
        .type           = PORT_SCIFA,
 };
@@ -377,7 +373,6 @@ static struct platform_device scif3_device = {
 };
 
 static struct plat_sci_port scif4_platform_data = {
-       .port_reg       = SCIx_NOT_SUPPORTED,
        .sampling_rate  = 8,
        .type           = PORT_SCIFA,
 };
@@ -398,7 +393,6 @@ static struct platform_device scif4_device = {
 };
 
 static struct plat_sci_port scif5_platform_data = {
-       .port_reg       = SCIx_NOT_SUPPORTED,
        .sampling_rate  = 8,
        .type           = PORT_SCIFA,
 };