All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/1] ARM: Kconfig.debug: Update Samsung UART config options
@ 2014-06-24 10:19 ` Sachin Kamat
  0 siblings, 0 replies; 6+ messages in thread
From: Sachin Kamat @ 2014-06-24 10:19 UTC (permalink / raw)
  To: linux-arm-kernel; +Cc: linux-samsung-soc, linux, arnd, kgene.kim, spk.linux

In a multiplatform config, the low level debug option shows several
UART port entries. Improve the user visible string so that it becomes
clear to the user about Samsung UART ports.
While at it also remove some lines from the help text that are no
longer applicable across all Samsung platforms.

Signed-off-by: Sachin Kamat <sachin.kamat@samsung.com>
---
 arch/arm/Kconfig.debug |   20 ++++----------------
 1 file changed, 4 insertions(+), 16 deletions(-)

diff --git a/arch/arm/Kconfig.debug b/arch/arm/Kconfig.debug
index 3548612b0bfe..b92993e59a12 100644
--- a/arch/arm/Kconfig.debug
+++ b/arch/arm/Kconfig.debug
@@ -617,53 +617,41 @@ choice
 		depends on PLAT_SAMSUNG
 		select DEBUG_EXYNOS_UART if ARCH_EXYNOS
 		select DEBUG_S3C24XX_UART if ARCH_S3C24XX
-		bool "Use S3C UART 0 for low-level debug"
+		bool "Use Samsung S3C UART 0 for low-level debug"
 		help
 		  Say Y here if you want the debug print routines to direct
 		  their output to UART 0. The port must have been initialised
 		  by the boot-loader before use.
 
-		  The uncompressor code port configuration is now handled
-		  by CONFIG_S3C_LOWLEVEL_UART_PORT.
-
 	config DEBUG_S3C_UART1
 		depends on PLAT_SAMSUNG
 		select DEBUG_EXYNOS_UART if ARCH_EXYNOS
 		select DEBUG_S3C24XX_UART if ARCH_S3C24XX
-		bool "Use S3C UART 1 for low-level debug"
+		bool "Use Samsung S3C UART 1 for low-level debug"
 		help
 		  Say Y here if you want the debug print routines to direct
 		  their output to UART 1. The port must have been initialised
 		  by the boot-loader before use.
 
-		  The uncompressor code port configuration is now handled
-		  by CONFIG_S3C_LOWLEVEL_UART_PORT.
-
 	config DEBUG_S3C_UART2
 		depends on PLAT_SAMSUNG
 		select DEBUG_EXYNOS_UART if ARCH_EXYNOS
 		select DEBUG_S3C24XX_UART if ARCH_S3C24XX
-		bool "Use S3C UART 2 for low-level debug"
+		bool "Use Samsung S3C UART 2 for low-level debug"
 		help
 		  Say Y here if you want the debug print routines to direct
 		  their output to UART 2. The port must have been initialised
 		  by the boot-loader before use.
 
-		  The uncompressor code port configuration is now handled
-		  by CONFIG_S3C_LOWLEVEL_UART_PORT.
-
 	config DEBUG_S3C_UART3
 		depends on PLAT_SAMSUNG && ARCH_EXYNOS
 		select DEBUG_EXYNOS_UART
-		bool "Use S3C UART 3 for low-level debug"
+		bool "Use Samsung S3C UART 3 for low-level debug"
 		help
 		  Say Y here if you want the debug print routines to direct
 		  their output to UART 3. The port must have been initialised
 		  by the boot-loader before use.
 
-		  The uncompressor code port configuration is now handled
-		  by CONFIG_S3C_LOWLEVEL_UART_PORT.
-
 	config DEBUG_S3C2410_UART0
 		depends on ARCH_S3C24XX
 		select DEBUG_S3C2410_UART
-- 
1.7.9.5

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH 1/1] ARM: Kconfig.debug: Update Samsung UART config options
@ 2014-06-24 10:19 ` Sachin Kamat
  0 siblings, 0 replies; 6+ messages in thread
From: Sachin Kamat @ 2014-06-24 10:19 UTC (permalink / raw)
  To: linux-arm-kernel

In a multiplatform config, the low level debug option shows several
UART port entries. Improve the user visible string so that it becomes
clear to the user about Samsung UART ports.
While at it also remove some lines from the help text that are no
longer applicable across all Samsung platforms.

Signed-off-by: Sachin Kamat <sachin.kamat@samsung.com>
---
 arch/arm/Kconfig.debug |   20 ++++----------------
 1 file changed, 4 insertions(+), 16 deletions(-)

diff --git a/arch/arm/Kconfig.debug b/arch/arm/Kconfig.debug
index 3548612b0bfe..b92993e59a12 100644
--- a/arch/arm/Kconfig.debug
+++ b/arch/arm/Kconfig.debug
@@ -617,53 +617,41 @@ choice
 		depends on PLAT_SAMSUNG
 		select DEBUG_EXYNOS_UART if ARCH_EXYNOS
 		select DEBUG_S3C24XX_UART if ARCH_S3C24XX
-		bool "Use S3C UART 0 for low-level debug"
+		bool "Use Samsung S3C UART 0 for low-level debug"
 		help
 		  Say Y here if you want the debug print routines to direct
 		  their output to UART 0. The port must have been initialised
 		  by the boot-loader before use.
 
-		  The uncompressor code port configuration is now handled
-		  by CONFIG_S3C_LOWLEVEL_UART_PORT.
-
 	config DEBUG_S3C_UART1
 		depends on PLAT_SAMSUNG
 		select DEBUG_EXYNOS_UART if ARCH_EXYNOS
 		select DEBUG_S3C24XX_UART if ARCH_S3C24XX
-		bool "Use S3C UART 1 for low-level debug"
+		bool "Use Samsung S3C UART 1 for low-level debug"
 		help
 		  Say Y here if you want the debug print routines to direct
 		  their output to UART 1. The port must have been initialised
 		  by the boot-loader before use.
 
-		  The uncompressor code port configuration is now handled
-		  by CONFIG_S3C_LOWLEVEL_UART_PORT.
-
 	config DEBUG_S3C_UART2
 		depends on PLAT_SAMSUNG
 		select DEBUG_EXYNOS_UART if ARCH_EXYNOS
 		select DEBUG_S3C24XX_UART if ARCH_S3C24XX
-		bool "Use S3C UART 2 for low-level debug"
+		bool "Use Samsung S3C UART 2 for low-level debug"
 		help
 		  Say Y here if you want the debug print routines to direct
 		  their output to UART 2. The port must have been initialised
 		  by the boot-loader before use.
 
-		  The uncompressor code port configuration is now handled
-		  by CONFIG_S3C_LOWLEVEL_UART_PORT.
-
 	config DEBUG_S3C_UART3
 		depends on PLAT_SAMSUNG && ARCH_EXYNOS
 		select DEBUG_EXYNOS_UART
-		bool "Use S3C UART 3 for low-level debug"
+		bool "Use Samsung S3C UART 3 for low-level debug"
 		help
 		  Say Y here if you want the debug print routines to direct
 		  their output to UART 3. The port must have been initialised
 		  by the boot-loader before use.
 
-		  The uncompressor code port configuration is now handled
-		  by CONFIG_S3C_LOWLEVEL_UART_PORT.
-
 	config DEBUG_S3C2410_UART0
 		depends on ARCH_S3C24XX
 		select DEBUG_S3C2410_UART
-- 
1.7.9.5

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* RE: [PATCH 1/1] ARM: Kconfig.debug: Update Samsung UART config options
  2014-06-24 10:19 ` Sachin Kamat
@ 2014-06-25 11:41   ` Kukjin Kim
  -1 siblings, 0 replies; 6+ messages in thread
From: Kukjin Kim @ 2014-06-25 11:41 UTC (permalink / raw)
  To: 'Sachin Kamat', linux-arm-kernel, Russell King
  Cc: linux-samsung-soc, linux, arnd, spk.linux

Sachin Kamat wrote:
> 
+ Russell

> In a multiplatform config, the low level debug option shows several
> UART port entries. Improve the user visible string so that it becomes
> clear to the user about Samsung UART ports.
> While at it also remove some lines from the help text that are no
> longer applicable across all Samsung platforms.
> 
Looks good to me and will apply into cleanup.

Thanks,
Kukjin

> Signed-off-by: Sachin Kamat <sachin.kamat@samsung.com>
> ---
>  arch/arm/Kconfig.debug |   20 ++++----------------
>  1 file changed, 4 insertions(+), 16 deletions(-)
> 
> diff --git a/arch/arm/Kconfig.debug b/arch/arm/Kconfig.debug
> index 3548612b0bfe..b92993e59a12 100644
> --- a/arch/arm/Kconfig.debug
> +++ b/arch/arm/Kconfig.debug
> @@ -617,53 +617,41 @@ choice
>  		depends on PLAT_SAMSUNG
>  		select DEBUG_EXYNOS_UART if ARCH_EXYNOS
>  		select DEBUG_S3C24XX_UART if ARCH_S3C24XX
> -		bool "Use S3C UART 0 for low-level debug"
> +		bool "Use Samsung S3C UART 0 for low-level debug"
>  		help
>  		  Say Y here if you want the debug print routines to direct
>  		  their output to UART 0. The port must have been initialised
>  		  by the boot-loader before use.
> 
> -		  The uncompressor code port configuration is now handled
> -		  by CONFIG_S3C_LOWLEVEL_UART_PORT.
> -
>  	config DEBUG_S3C_UART1
>  		depends on PLAT_SAMSUNG
>  		select DEBUG_EXYNOS_UART if ARCH_EXYNOS
>  		select DEBUG_S3C24XX_UART if ARCH_S3C24XX
> -		bool "Use S3C UART 1 for low-level debug"
> +		bool "Use Samsung S3C UART 1 for low-level debug"
>  		help
>  		  Say Y here if you want the debug print routines to direct
>  		  their output to UART 1. The port must have been initialised
>  		  by the boot-loader before use.
> 
> -		  The uncompressor code port configuration is now handled
> -		  by CONFIG_S3C_LOWLEVEL_UART_PORT.
> -
>  	config DEBUG_S3C_UART2
>  		depends on PLAT_SAMSUNG
>  		select DEBUG_EXYNOS_UART if ARCH_EXYNOS
>  		select DEBUG_S3C24XX_UART if ARCH_S3C24XX
> -		bool "Use S3C UART 2 for low-level debug"
> +		bool "Use Samsung S3C UART 2 for low-level debug"
>  		help
>  		  Say Y here if you want the debug print routines to direct
>  		  their output to UART 2. The port must have been initialised
>  		  by the boot-loader before use.
> 
> -		  The uncompressor code port configuration is now handled
> -		  by CONFIG_S3C_LOWLEVEL_UART_PORT.
> -
>  	config DEBUG_S3C_UART3
>  		depends on PLAT_SAMSUNG && ARCH_EXYNOS
>  		select DEBUG_EXYNOS_UART
> -		bool "Use S3C UART 3 for low-level debug"
> +		bool "Use Samsung S3C UART 3 for low-level debug"
>  		help
>  		  Say Y here if you want the debug print routines to direct
>  		  their output to UART 3. The port must have been initialised
>  		  by the boot-loader before use.
> 
> -		  The uncompressor code port configuration is now handled
> -		  by CONFIG_S3C_LOWLEVEL_UART_PORT.
> -
>  	config DEBUG_S3C2410_UART0
>  		depends on ARCH_S3C24XX
>  		select DEBUG_S3C2410_UART
> --
> 1.7.9.5

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 1/1] ARM: Kconfig.debug: Update Samsung UART config options
@ 2014-06-25 11:41   ` Kukjin Kim
  0 siblings, 0 replies; 6+ messages in thread
From: Kukjin Kim @ 2014-06-25 11:41 UTC (permalink / raw)
  To: linux-arm-kernel

Sachin Kamat wrote:
> 
+ Russell

> In a multiplatform config, the low level debug option shows several
> UART port entries. Improve the user visible string so that it becomes
> clear to the user about Samsung UART ports.
> While at it also remove some lines from the help text that are no
> longer applicable across all Samsung platforms.
> 
Looks good to me and will apply into cleanup.

Thanks,
Kukjin

> Signed-off-by: Sachin Kamat <sachin.kamat@samsung.com>
> ---
>  arch/arm/Kconfig.debug |   20 ++++----------------
>  1 file changed, 4 insertions(+), 16 deletions(-)
> 
> diff --git a/arch/arm/Kconfig.debug b/arch/arm/Kconfig.debug
> index 3548612b0bfe..b92993e59a12 100644
> --- a/arch/arm/Kconfig.debug
> +++ b/arch/arm/Kconfig.debug
> @@ -617,53 +617,41 @@ choice
>  		depends on PLAT_SAMSUNG
>  		select DEBUG_EXYNOS_UART if ARCH_EXYNOS
>  		select DEBUG_S3C24XX_UART if ARCH_S3C24XX
> -		bool "Use S3C UART 0 for low-level debug"
> +		bool "Use Samsung S3C UART 0 for low-level debug"
>  		help
>  		  Say Y here if you want the debug print routines to direct
>  		  their output to UART 0. The port must have been initialised
>  		  by the boot-loader before use.
> 
> -		  The uncompressor code port configuration is now handled
> -		  by CONFIG_S3C_LOWLEVEL_UART_PORT.
> -
>  	config DEBUG_S3C_UART1
>  		depends on PLAT_SAMSUNG
>  		select DEBUG_EXYNOS_UART if ARCH_EXYNOS
>  		select DEBUG_S3C24XX_UART if ARCH_S3C24XX
> -		bool "Use S3C UART 1 for low-level debug"
> +		bool "Use Samsung S3C UART 1 for low-level debug"
>  		help
>  		  Say Y here if you want the debug print routines to direct
>  		  their output to UART 1. The port must have been initialised
>  		  by the boot-loader before use.
> 
> -		  The uncompressor code port configuration is now handled
> -		  by CONFIG_S3C_LOWLEVEL_UART_PORT.
> -
>  	config DEBUG_S3C_UART2
>  		depends on PLAT_SAMSUNG
>  		select DEBUG_EXYNOS_UART if ARCH_EXYNOS
>  		select DEBUG_S3C24XX_UART if ARCH_S3C24XX
> -		bool "Use S3C UART 2 for low-level debug"
> +		bool "Use Samsung S3C UART 2 for low-level debug"
>  		help
>  		  Say Y here if you want the debug print routines to direct
>  		  their output to UART 2. The port must have been initialised
>  		  by the boot-loader before use.
> 
> -		  The uncompressor code port configuration is now handled
> -		  by CONFIG_S3C_LOWLEVEL_UART_PORT.
> -
>  	config DEBUG_S3C_UART3
>  		depends on PLAT_SAMSUNG && ARCH_EXYNOS
>  		select DEBUG_EXYNOS_UART
> -		bool "Use S3C UART 3 for low-level debug"
> +		bool "Use Samsung S3C UART 3 for low-level debug"
>  		help
>  		  Say Y here if you want the debug print routines to direct
>  		  their output to UART 3. The port must have been initialised
>  		  by the boot-loader before use.
> 
> -		  The uncompressor code port configuration is now handled
> -		  by CONFIG_S3C_LOWLEVEL_UART_PORT.
> -
>  	config DEBUG_S3C2410_UART0
>  		depends on ARCH_S3C24XX
>  		select DEBUG_S3C2410_UART
> --
> 1.7.9.5

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/1] ARM: Kconfig.debug: Update Samsung UART config options
  2014-06-25 11:41   ` Kukjin Kim
@ 2014-06-25 12:51     ` Russell King - ARM Linux
  -1 siblings, 0 replies; 6+ messages in thread
From: Russell King - ARM Linux @ 2014-06-25 12:51 UTC (permalink / raw)
  To: Kukjin Kim
  Cc: 'Sachin Kamat',
	linux-arm-kernel, linux-samsung-soc, arnd, spk.linux

On Wed, Jun 25, 2014 at 08:41:13PM +0900, Kukjin Kim wrote:
> Sachin Kamat wrote:
> > 
> + Russell
> 
> > In a multiplatform config, the low level debug option shows several
> > UART port entries. Improve the user visible string so that it becomes
> > clear to the user about Samsung UART ports.
> > While at it also remove some lines from the help text that are no
> > longer applicable across all Samsung platforms.
> > 
> Looks good to me and will apply into cleanup.

That's fine.  I don't have anything which clashes with this.  Thanks for
asking.

-- 
FTTC broadband for 0.8mile line: now at 9.7Mbps down 460kbps up... slowly
improving, and getting towards what was expected from it.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 1/1] ARM: Kconfig.debug: Update Samsung UART config options
@ 2014-06-25 12:51     ` Russell King - ARM Linux
  0 siblings, 0 replies; 6+ messages in thread
From: Russell King - ARM Linux @ 2014-06-25 12:51 UTC (permalink / raw)
  To: linux-arm-kernel

On Wed, Jun 25, 2014 at 08:41:13PM +0900, Kukjin Kim wrote:
> Sachin Kamat wrote:
> > 
> + Russell
> 
> > In a multiplatform config, the low level debug option shows several
> > UART port entries. Improve the user visible string so that it becomes
> > clear to the user about Samsung UART ports.
> > While at it also remove some lines from the help text that are no
> > longer applicable across all Samsung platforms.
> > 
> Looks good to me and will apply into cleanup.

That's fine.  I don't have anything which clashes with this.  Thanks for
asking.

-- 
FTTC broadband for 0.8mile line: now at 9.7Mbps down 460kbps up... slowly
improving, and getting towards what was expected from it.

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2014-06-25 12:51 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-06-24 10:19 [PATCH 1/1] ARM: Kconfig.debug: Update Samsung UART config options Sachin Kamat
2014-06-24 10:19 ` Sachin Kamat
2014-06-25 11:41 ` Kukjin Kim
2014-06-25 11:41   ` Kukjin Kim
2014-06-25 12:51   ` Russell King - ARM Linux
2014-06-25 12:51     ` Russell King - ARM Linux

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.