Re: [PATCH 6/9] serial: samsung: Add support for EXYNOS5250
On Wednesday 01 February 2012, Kyungmin Park wrote: > > > > diff --git a/drivers/tty/serial/samsung.c b/drivers/tty/serial/samsung.c > > index 3b07fb9..c55e5fb 100644 > > --- a/drivers/tty/serial/samsung.c > > +++ b/drivers/tty/serial/samsung.c > > @@ -1594,7 +1594,7 @@ static struct s3c24xx_serial_drv_data > > s5pv210_serial_drv_data = { > > #endif > > > > #if defined(CONFIG_CPU_EXYNOS4210) || defined(CONFIG_SOC_EXYNOS4212) || \ > > - defined(CONFIG_SOC_EXYNOS4412) > > + defined(CONFIG_SOC_EXYNOS4412) || defined(CONFIG_SOC_EXYNOS5250) > I think you can make it simple just use "#ifdef CONFIG_ARCH_EXYNOS " Or you could just remove all the #ifdef. Most drivers just register for all devices that they support, even if you know which one to expect. Arnd -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
RE: [PATCH 6/9] serial: samsung: Add support for EXYNOS5250
Kyungmin Park wrote: > > On 2/1/12, Kukjin Kim wrote: > > Cc: Thomas Abraham > > Cc: Greg Kroah-Hartman > > Signed-off-by: Kukjin Kim > > --- > > drivers/tty/serial/samsung.c |2 +- > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > diff --git a/drivers/tty/serial/samsung.c b/drivers/tty/serial/samsung.c > > index 3b07fb9..c55e5fb 100644 > > --- a/drivers/tty/serial/samsung.c > > +++ b/drivers/tty/serial/samsung.c > > @@ -1594,7 +1594,7 @@ static struct s3c24xx_serial_drv_data > > s5pv210_serial_drv_data = { > > #endif > > > > #if defined(CONFIG_CPU_EXYNOS4210) || defined(CONFIG_SOC_EXYNOS4212) || > \ > > - defined(CONFIG_SOC_EXYNOS4412) > > + defined(CONFIG_SOC_EXYNOS4412) || defined(CONFIG_SOC_EXYNOS5250) > I think you can make it simple just use "#ifdef CONFIG_ARCH_EXYNOS " I think, should be 'depends on SoC' not 'ARCH' Thanks. Best regards, Kgene. -- Kukjin Kim , Senior Engineer, SW Solution Development Team, Samsung Electronics Co., Ltd. -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 6/9] serial: samsung: Add support for EXYNOS5250
On 2/1/12, Kukjin Kim wrote: > Cc: Thomas Abraham > Cc: Greg Kroah-Hartman > Signed-off-by: Kukjin Kim > --- > drivers/tty/serial/samsung.c |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/drivers/tty/serial/samsung.c b/drivers/tty/serial/samsung.c > index 3b07fb9..c55e5fb 100644 > --- a/drivers/tty/serial/samsung.c > +++ b/drivers/tty/serial/samsung.c > @@ -1594,7 +1594,7 @@ static struct s3c24xx_serial_drv_data > s5pv210_serial_drv_data = { > #endif > > #if defined(CONFIG_CPU_EXYNOS4210) || defined(CONFIG_SOC_EXYNOS4212) || \ > - defined(CONFIG_SOC_EXYNOS4412) > + defined(CONFIG_SOC_EXYNOS4412) || defined(CONFIG_SOC_EXYNOS5250) I think you can make it simple just use "#ifdef CONFIG_ARCH_EXYNOS " > static struct s3c24xx_serial_drv_data exynos4210_serial_drv_data = { > .info = &(struct s3c24xx_uart_info) { > .name = "Samsung Exynos4 UART", > -- > 1.7.4.4 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" > in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH 6/9] serial: samsung: Add support for EXYNOS5250
Cc: Thomas Abraham Cc: Greg Kroah-Hartman Signed-off-by: Kukjin Kim --- drivers/tty/serial/samsung.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/tty/serial/samsung.c b/drivers/tty/serial/samsung.c index 3b07fb9..c55e5fb 100644 --- a/drivers/tty/serial/samsung.c +++ b/drivers/tty/serial/samsung.c @@ -1594,7 +1594,7 @@ static struct s3c24xx_serial_drv_data s5pv210_serial_drv_data = { #endif #if defined(CONFIG_CPU_EXYNOS4210) || defined(CONFIG_SOC_EXYNOS4212) || \ - defined(CONFIG_SOC_EXYNOS4412) + defined(CONFIG_SOC_EXYNOS4412) || defined(CONFIG_SOC_EXYNOS5250) static struct s3c24xx_serial_drv_data exynos4210_serial_drv_data = { .info = &(struct s3c24xx_uart_info) { .name = "Samsung Exynos4 UART", -- 1.7.4.4 -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html