diff mbox

[v3,04/10] serial: samsung: select Exynos specific driver data if ARCH_EXYNOS is defined

Message ID 1371466836-4111-5-git-send-email-chander.kashyap@linaro.org (mailing list archive)
State New, archived
Headers show

Commit Message

Chander Kashyap June 17, 2013, 11 a.m. UTC
All Exynos4/5 SoCs share a common driver data in the samsung serial
driver. Hence, let the driver data inclusion be based on ARCH_EXYNOS
instead of SOC specific definition.

Signed-off-by: Chander Kashyap <chander.kashyap@linaro.org>
---
 drivers/tty/serial/samsung.c |    4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

Comments

Kim Kukjin June 17, 2013, 11:45 a.m. UTC | #1
Chander Kashyap wrote:
> 
> All Exynos4/5 SoCs share a common driver data in the samsung serial
> driver. Hence, let the driver data inclusion be based on ARCH_EXYNOS
> instead of SOC specific definition.
> 
> Signed-off-by: Chander Kashyap <chander.kashyap@linaro.org>

(+ Greg, Jiri)

Acked-by: Kukjin Kim <kgene.kim@samsung.com>

Greg, Jiri, please take this in your tree with my ack if no problem on this.

Thanks,
- Kukjin

> ---
>  drivers/tty/serial/samsung.c |    4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
> 
> diff --git a/drivers/tty/serial/samsung.c b/drivers/tty/serial/samsung.c
> index 2552c7f..e70d19a 100644
> --- a/drivers/tty/serial/samsung.c
> +++ b/drivers/tty/serial/samsung.c
> @@ -1703,9 +1703,7 @@ static struct s3c24xx_serial_drv_data
> s5pv210_serial_drv_data = {
>  #define S5PV210_SERIAL_DRV_DATA	(kernel_ulong_t)NULL
>  #endif
> 
> -#if defined(CONFIG_CPU_EXYNOS4210) || defined(CONFIG_SOC_EXYNOS4212) || \
> -	defined(CONFIG_SOC_EXYNOS4412) || defined(CONFIG_SOC_EXYNOS5250) ||
> \
> -	defined(CONFIG_SOC_EXYNOS5440)
> +#if defined(CONFIG_ARCH_EXYNOS)
>  static struct s3c24xx_serial_drv_data exynos4210_serial_drv_data = {
>  	.info = &(struct s3c24xx_uart_info) {
>  		.name		= "Samsung Exynos4 UART",
> --
> 1.7.9.5
Kim Kukjin June 18, 2013, 10:31 a.m. UTC | #2
Kukjin Kim wrote:
> 
> Chander Kashyap wrote:
> >
> > All Exynos4/5 SoCs share a common driver data in the samsung serial
> > driver. Hence, let the driver data inclusion be based on ARCH_EXYNOS
> > instead of SOC specific definition.
> >
> > Signed-off-by: Chander Kashyap <chander.kashyap@linaro.org>
> 
> (+ Greg, Jiri)
> 
> Acked-by: Kukjin Kim <kgene.kim@samsung.com>
> 
> Greg, Jiri, please take this in your tree with my ack if no problem on
> this.
> 
Greg, Jiri, since there are other changes for this in samsung tree so would
be nice if this could be sent to upstream with others via samsung tree. If
you have any concerns, please kindly let me know.

Thanks,
- Kukjin
> 
> > ---
> >  drivers/tty/serial/samsung.c |    4 +---
> >  1 file changed, 1 insertion(+), 3 deletions(-)
> >
> > diff --git a/drivers/tty/serial/samsung.c b/drivers/tty/serial/samsung.c
> > index 2552c7f..e70d19a 100644
> > --- a/drivers/tty/serial/samsung.c
> > +++ b/drivers/tty/serial/samsung.c
> > @@ -1703,9 +1703,7 @@ static struct s3c24xx_serial_drv_data
> > s5pv210_serial_drv_data = {
> >  #define S5PV210_SERIAL_DRV_DATA	(kernel_ulong_t)NULL
> >  #endif
> >
> > -#if defined(CONFIG_CPU_EXYNOS4210) || defined(CONFIG_SOC_EXYNOS4212) ||
> \
> > -	defined(CONFIG_SOC_EXYNOS4412) || defined(CONFIG_SOC_EXYNOS5250) ||
> > \
> > -	defined(CONFIG_SOC_EXYNOS5440)
> > +#if defined(CONFIG_ARCH_EXYNOS)
> >  static struct s3c24xx_serial_drv_data exynos4210_serial_drv_data = {
> >  	.info = &(struct s3c24xx_uart_info) {
> >  		.name		= "Samsung Exynos4 UART",
> > --
> > 1.7.9.5
Greg KH June 18, 2013, 3:19 p.m. UTC | #3
On Tue, Jun 18, 2013 at 07:31:06PM +0900, Kukjin Kim wrote:
> Kukjin Kim wrote:
> > 
> > Chander Kashyap wrote:
> > >
> > > All Exynos4/5 SoCs share a common driver data in the samsung serial
> > > driver. Hence, let the driver data inclusion be based on ARCH_EXYNOS
> > > instead of SOC specific definition.
> > >
> > > Signed-off-by: Chander Kashyap <chander.kashyap@linaro.org>
> > 
> > (+ Greg, Jiri)
> > 
> > Acked-by: Kukjin Kim <kgene.kim@samsung.com>
> > 
> > Greg, Jiri, please take this in your tree with my ack if no problem on
> > this.
> > 
> Greg, Jiri, since there are other changes for this in samsung tree so would
> be nice if this could be sent to upstream with others via samsung tree. If
> you have any concerns, please kindly let me know.
> 
> Thanks,
> - Kukjin
> > 
> > > ---
> > >  drivers/tty/serial/samsung.c |    4 +---
> > >  1 file changed, 1 insertion(+), 3 deletions(-)
> > >
> > > diff --git a/drivers/tty/serial/samsung.c b/drivers/tty/serial/samsung.c
> > > index 2552c7f..e70d19a 100644
> > > --- a/drivers/tty/serial/samsung.c
> > > +++ b/drivers/tty/serial/samsung.c
> > > @@ -1703,9 +1703,7 @@ static struct s3c24xx_serial_drv_data
> > > s5pv210_serial_drv_data = {
> > >  #define S5PV210_SERIAL_DRV_DATA	(kernel_ulong_t)NULL
> > >  #endif
> > >
> > > -#if defined(CONFIG_CPU_EXYNOS4210) || defined(CONFIG_SOC_EXYNOS4212) ||
> > \
> > > -	defined(CONFIG_SOC_EXYNOS4412) || defined(CONFIG_SOC_EXYNOS5250) ||
> > > \
> > > -	defined(CONFIG_SOC_EXYNOS5440)
> > > +#if defined(CONFIG_ARCH_EXYNOS)
> > >  static struct s3c24xx_serial_drv_data exynos4210_serial_drv_data = {
> > >  	.info = &(struct s3c24xx_uart_info) {
> > >  		.name		= "Samsung Exynos4 UART",

Acked-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
diff mbox

Patch

diff --git a/drivers/tty/serial/samsung.c b/drivers/tty/serial/samsung.c
index 2552c7f..e70d19a 100644
--- a/drivers/tty/serial/samsung.c
+++ b/drivers/tty/serial/samsung.c
@@ -1703,9 +1703,7 @@  static struct s3c24xx_serial_drv_data s5pv210_serial_drv_data = {
 #define S5PV210_SERIAL_DRV_DATA	(kernel_ulong_t)NULL
 #endif
 
-#if defined(CONFIG_CPU_EXYNOS4210) || defined(CONFIG_SOC_EXYNOS4212) || \
-	defined(CONFIG_SOC_EXYNOS4412) || defined(CONFIG_SOC_EXYNOS5250) || \
-	defined(CONFIG_SOC_EXYNOS5440)
+#if defined(CONFIG_ARCH_EXYNOS)
 static struct s3c24xx_serial_drv_data exynos4210_serial_drv_data = {
 	.info = &(struct s3c24xx_uart_info) {
 		.name		= "Samsung Exynos4 UART",