From: nicolas.ferre@atmel.com (Nicolas Ferre)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 08/17] ARM: at91: stop using HAVE_AT91_DBGUx
Date: Tue, 20 Jan 2015 11:34:53 +0100 [thread overview]
Message-ID: <54BE2F4D.2080800@atmel.com> (raw)
In-Reply-To: <1421359100-8930-9-git-send-email-alexandre.belloni@free-electrons.com>
Le 15/01/2015 22:58, Alexandre Belloni a ?crit :
> In order to remove SOC_SAM9xxx options, stop using HAVE_AT91_DBGUx.
>
> Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
> ---
> arch/arm/Kconfig.debug | 6 +++---
> arch/arm/mach-at91/Kconfig | 19 -------------------
> 2 files changed, 3 insertions(+), 22 deletions(-)
>
> diff --git a/arch/arm/Kconfig.debug b/arch/arm/Kconfig.debug
> index e34d24949c6a..42a38731da99 100644
> --- a/arch/arm/Kconfig.debug
> +++ b/arch/arm/Kconfig.debug
> @@ -117,17 +117,17 @@ choice
> config AT91_DEBUG_LL_DBGU0
> bool "Kernel low-level debugging on rm9200, 9260/9g20, 9261/9g10, 9rl, 9x5, 9n12"
> select DEBUG_AT91_UART
> - depends on HAVE_AT91_DBGU0
> + depends on SOC_AT91RM9200 || SOC_AT91SAM9
>
> config AT91_DEBUG_LL_DBGU1
> bool "Kernel low-level debugging on 9263, 9g45 and sama5d3"
> select DEBUG_AT91_UART
> - depends on HAVE_AT91_DBGU1
> + depends on SOC_AT91SAM9 || SOC_SAMA5
>
> config AT91_DEBUG_LL_DBGU2
> bool "Kernel low-level debugging on sama5d4"
> select DEBUG_AT91_UART
> - depends on HAVE_AT91_DBGU2
These values are used below in the file: in:
"
default "debug/at91.S" if AT91_DEBUG_LL_DBGU0 || AT91_DEBUG_LL_DBGU1 || \
AT91_DEBUG_LL_DBGU2
"
So I guess there is a little patch missing: I answer to the
message "[PATCH 5/7] ARM: at91: move debug-macro.S into the common space" by fixing this.
Please tell me if it's okay.
Bye,
> + depends on SOC_SAMA5
>
> config DEBUG_BCM2835
> bool "Kernel low-level debugging on BCM2835 PL011 UART"
> diff --git a/arch/arm/mach-at91/Kconfig b/arch/arm/mach-at91/Kconfig
> index b7dcef50db23..c6956b863b9d 100644
> --- a/arch/arm/mach-at91/Kconfig
> +++ b/arch/arm/mach-at91/Kconfig
> @@ -6,15 +6,6 @@ config HAVE_AT91_UTMI
> config HAVE_AT91_USB_CLK
> bool
>
> -config HAVE_AT91_DBGU0
> - bool
> -
> -config HAVE_AT91_DBGU1
> - bool
> -
> -config HAVE_AT91_DBGU2
> - bool
> -
> config COMMON_CLK_AT91
> bool
> select COMMON_CLK
> @@ -70,7 +61,6 @@ config SOC_SAMA5D3
> bool "SAMA5D3 family"
> select SOC_SAMA5
> select HAVE_FB_ATMEL
> - select HAVE_AT91_DBGU1
> select HAVE_AT91_UTMI
> select HAVE_AT91_SMD
> select HAVE_AT91_USB_CLK
> @@ -81,7 +71,6 @@ config SOC_SAMA5D3
> config SOC_SAMA5D4
> bool "SAMA5D4 family"
> select SOC_SAMA5
> - select HAVE_AT91_DBGU2
> select CLKSRC_MMIO
> select CACHE_L2X0
> select CACHE_PL310
> @@ -101,12 +90,10 @@ config SOC_AT91RM9200
> select COMMON_CLK_AT91
> select CPU_ARM920T
> select GENERIC_CLOCKEVENTS
> - select HAVE_AT91_DBGU0
> select HAVE_AT91_USB_CLK
>
> config SOC_AT91SAM9260
> bool "AT91SAM9260, AT91SAM9XE or AT91SAM9G20"
> - select HAVE_AT91_DBGU0
> select SOC_AT91SAM9
> select HAVE_AT91_USB_CLK
> help
> @@ -115,7 +102,6 @@ config SOC_AT91SAM9260
>
> config SOC_AT91SAM9261
> bool "AT91SAM9261 or AT91SAM9G10"
> - select HAVE_AT91_DBGU0
> select HAVE_FB_ATMEL
> select SOC_AT91SAM9
> select HAVE_AT91_USB_CLK
> @@ -124,21 +110,18 @@ config SOC_AT91SAM9261
>
> config SOC_AT91SAM9263
> bool "AT91SAM9263"
> - select HAVE_AT91_DBGU1
> select HAVE_FB_ATMEL
> select SOC_AT91SAM9
> select HAVE_AT91_USB_CLK
>
> config SOC_AT91SAM9RL
> bool "AT91SAM9RL"
> - select HAVE_AT91_DBGU0
> select HAVE_FB_ATMEL
> select SOC_AT91SAM9
> select HAVE_AT91_UTMI
>
> config SOC_AT91SAM9G45
> bool "AT91SAM9G45 or AT91SAM9M10 families"
> - select HAVE_AT91_DBGU1
> select HAVE_FB_ATMEL
> select SOC_AT91SAM9
> select HAVE_AT91_UTMI
> @@ -149,7 +132,6 @@ config SOC_AT91SAM9G45
>
> config SOC_AT91SAM9X5
> bool "AT91SAM9x5 family"
> - select HAVE_AT91_DBGU0
> select HAVE_FB_ATMEL
> select SOC_AT91SAM9
> select HAVE_AT91_UTMI
> @@ -164,7 +146,6 @@ config SOC_AT91SAM9X5
>
> config SOC_AT91SAM9N12
> bool "AT91SAM9N12 family"
> - select HAVE_AT91_DBGU0
> select HAVE_FB_ATMEL
> select SOC_AT91SAM9
> select HAVE_AT91_USB_CLK
>
--
Nicolas Ferre
WARNING: multiple messages have this Message-ID (diff)
From: Nicolas Ferre <nicolas.ferre@atmel.com>
To: Alexandre Belloni <alexandre.belloni@free-electrons.com>
Cc: Jean-Christophe Plagniol-Villard <plagnioj@jcrosoft.com>,
Arnd Bergmann <arnd@arndb.de>,
Boris Brezillon <boris.brezillon@free-electrons.com>,
"Bo Shen" <voice.shen@atmel.com>,
Wenyou Yang <Wenyou.Yang@atmel.com>,
<linux-arm-kernel@lists.infradead.org>,
<linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 08/17] ARM: at91: stop using HAVE_AT91_DBGUx
Date: Tue, 20 Jan 2015 11:34:53 +0100 [thread overview]
Message-ID: <54BE2F4D.2080800@atmel.com> (raw)
In-Reply-To: <1421359100-8930-9-git-send-email-alexandre.belloni@free-electrons.com>
Le 15/01/2015 22:58, Alexandre Belloni a écrit :
> In order to remove SOC_SAM9xxx options, stop using HAVE_AT91_DBGUx.
>
> Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
> ---
> arch/arm/Kconfig.debug | 6 +++---
> arch/arm/mach-at91/Kconfig | 19 -------------------
> 2 files changed, 3 insertions(+), 22 deletions(-)
>
> diff --git a/arch/arm/Kconfig.debug b/arch/arm/Kconfig.debug
> index e34d24949c6a..42a38731da99 100644
> --- a/arch/arm/Kconfig.debug
> +++ b/arch/arm/Kconfig.debug
> @@ -117,17 +117,17 @@ choice
> config AT91_DEBUG_LL_DBGU0
> bool "Kernel low-level debugging on rm9200, 9260/9g20, 9261/9g10, 9rl, 9x5, 9n12"
> select DEBUG_AT91_UART
> - depends on HAVE_AT91_DBGU0
> + depends on SOC_AT91RM9200 || SOC_AT91SAM9
>
> config AT91_DEBUG_LL_DBGU1
> bool "Kernel low-level debugging on 9263, 9g45 and sama5d3"
> select DEBUG_AT91_UART
> - depends on HAVE_AT91_DBGU1
> + depends on SOC_AT91SAM9 || SOC_SAMA5
>
> config AT91_DEBUG_LL_DBGU2
> bool "Kernel low-level debugging on sama5d4"
> select DEBUG_AT91_UART
> - depends on HAVE_AT91_DBGU2
These values are used below in the file: in:
"
default "debug/at91.S" if AT91_DEBUG_LL_DBGU0 || AT91_DEBUG_LL_DBGU1 || \
AT91_DEBUG_LL_DBGU2
"
So I guess there is a little patch missing: I answer to the
message "[PATCH 5/7] ARM: at91: move debug-macro.S into the common space" by fixing this.
Please tell me if it's okay.
Bye,
> + depends on SOC_SAMA5
>
> config DEBUG_BCM2835
> bool "Kernel low-level debugging on BCM2835 PL011 UART"
> diff --git a/arch/arm/mach-at91/Kconfig b/arch/arm/mach-at91/Kconfig
> index b7dcef50db23..c6956b863b9d 100644
> --- a/arch/arm/mach-at91/Kconfig
> +++ b/arch/arm/mach-at91/Kconfig
> @@ -6,15 +6,6 @@ config HAVE_AT91_UTMI
> config HAVE_AT91_USB_CLK
> bool
>
> -config HAVE_AT91_DBGU0
> - bool
> -
> -config HAVE_AT91_DBGU1
> - bool
> -
> -config HAVE_AT91_DBGU2
> - bool
> -
> config COMMON_CLK_AT91
> bool
> select COMMON_CLK
> @@ -70,7 +61,6 @@ config SOC_SAMA5D3
> bool "SAMA5D3 family"
> select SOC_SAMA5
> select HAVE_FB_ATMEL
> - select HAVE_AT91_DBGU1
> select HAVE_AT91_UTMI
> select HAVE_AT91_SMD
> select HAVE_AT91_USB_CLK
> @@ -81,7 +71,6 @@ config SOC_SAMA5D3
> config SOC_SAMA5D4
> bool "SAMA5D4 family"
> select SOC_SAMA5
> - select HAVE_AT91_DBGU2
> select CLKSRC_MMIO
> select CACHE_L2X0
> select CACHE_PL310
> @@ -101,12 +90,10 @@ config SOC_AT91RM9200
> select COMMON_CLK_AT91
> select CPU_ARM920T
> select GENERIC_CLOCKEVENTS
> - select HAVE_AT91_DBGU0
> select HAVE_AT91_USB_CLK
>
> config SOC_AT91SAM9260
> bool "AT91SAM9260, AT91SAM9XE or AT91SAM9G20"
> - select HAVE_AT91_DBGU0
> select SOC_AT91SAM9
> select HAVE_AT91_USB_CLK
> help
> @@ -115,7 +102,6 @@ config SOC_AT91SAM9260
>
> config SOC_AT91SAM9261
> bool "AT91SAM9261 or AT91SAM9G10"
> - select HAVE_AT91_DBGU0
> select HAVE_FB_ATMEL
> select SOC_AT91SAM9
> select HAVE_AT91_USB_CLK
> @@ -124,21 +110,18 @@ config SOC_AT91SAM9261
>
> config SOC_AT91SAM9263
> bool "AT91SAM9263"
> - select HAVE_AT91_DBGU1
> select HAVE_FB_ATMEL
> select SOC_AT91SAM9
> select HAVE_AT91_USB_CLK
>
> config SOC_AT91SAM9RL
> bool "AT91SAM9RL"
> - select HAVE_AT91_DBGU0
> select HAVE_FB_ATMEL
> select SOC_AT91SAM9
> select HAVE_AT91_UTMI
>
> config SOC_AT91SAM9G45
> bool "AT91SAM9G45 or AT91SAM9M10 families"
> - select HAVE_AT91_DBGU1
> select HAVE_FB_ATMEL
> select SOC_AT91SAM9
> select HAVE_AT91_UTMI
> @@ -149,7 +132,6 @@ config SOC_AT91SAM9G45
>
> config SOC_AT91SAM9X5
> bool "AT91SAM9x5 family"
> - select HAVE_AT91_DBGU0
> select HAVE_FB_ATMEL
> select SOC_AT91SAM9
> select HAVE_AT91_UTMI
> @@ -164,7 +146,6 @@ config SOC_AT91SAM9X5
>
> config SOC_AT91SAM9N12
> bool "AT91SAM9N12 family"
> - select HAVE_AT91_DBGU0
> select HAVE_FB_ATMEL
> select SOC_AT91SAM9
> select HAVE_AT91_USB_CLK
>
--
Nicolas Ferre
next prev parent reply other threads:[~2015-01-20 10:34 UTC|newest]
Thread overview: 56+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-01-15 21:58 [PATCH 00/17] AT91 cleanup for 3.20 #3 Alexandre Belloni
2015-01-15 21:58 ` Alexandre Belloni
2015-01-15 21:58 ` [PATCH 01/17] ARM: at91: merge at91rm9200 files Alexandre Belloni
2015-01-15 21:58 ` Alexandre Belloni
2015-01-15 21:58 ` [PATCH 02/17] ARM: at91: move sama5 specific code into per-soc files Alexandre Belloni
2015-01-15 21:58 ` Alexandre Belloni
2015-01-15 21:58 ` [PATCH 03/17] ARM: at91:remove unused mach/system_rev.h Alexandre Belloni
2015-01-15 21:58 ` Alexandre Belloni
2015-01-15 21:58 ` [PATCH 04/17] ARM: at91: fix sam9n12 and sam9x5 arm_pm_idle Alexandre Belloni
2015-01-15 21:58 ` Alexandre Belloni
2015-01-15 21:58 ` [PATCH 05/17] ARM: at91: sam9: set arm_pm_idle from sam9_dt_device_init Alexandre Belloni
2015-01-15 21:58 ` Alexandre Belloni
2015-01-15 21:58 ` [PATCH 06/17] ARM: at91: remove unused at91_ioremap_matrix and header Alexandre Belloni
2015-01-15 21:58 ` Alexandre Belloni
2015-01-15 21:58 ` [PATCH 07/17] ARM: at91: remove unused _matrix.h headers Alexandre Belloni
2015-01-15 21:58 ` Alexandre Belloni
2015-01-15 21:58 ` [PATCH 08/17] ARM: at91: stop using HAVE_AT91_DBGUx Alexandre Belloni
2015-01-15 21:58 ` Alexandre Belloni
2015-01-20 10:34 ` Nicolas Ferre [this message]
2015-01-20 10:34 ` Nicolas Ferre
2015-01-20 15:12 ` Alexandre Belloni
2015-01-20 15:12 ` Alexandre Belloni
2015-01-15 21:58 ` [PATCH 09/17] ARM: at91: pm: remove warning to remove SOC_AT91SAM9263 usage Alexandre Belloni
2015-01-15 21:58 ` Alexandre Belloni
2015-01-15 21:58 ` [PATCH 10/17] ARM: at91: merge SOC_AT91SAM9xxx Alexandre Belloni
2015-01-15 21:58 ` Alexandre Belloni
2015-01-22 10:22 ` Nicolas Ferre
2015-01-22 10:22 ` Nicolas Ferre
2015-01-22 16:36 ` Alexandre Belloni
2015-01-22 16:36 ` Alexandre Belloni
2015-01-26 17:49 ` [PATCH v2] " Nicolas Ferre
2015-01-26 17:49 ` Nicolas Ferre
2015-01-15 21:58 ` [PATCH 11/17] ARM: at91: remove at91_boot_soc Alexandre Belloni
2015-01-15 21:58 ` Alexandre Belloni
2015-01-26 18:08 ` [PATCH v2] ARM: at91: at91rm9200: set idle and restart from rm9200_dt_device_init() Nicolas Ferre
2015-01-26 18:08 ` Nicolas Ferre
2015-01-26 19:01 ` Alexandre Belloni
2015-01-26 19:01 ` Alexandre Belloni
2015-01-15 21:58 ` [PATCH 12/17] ARM: at91: rework initialization for rm9200 based boards Alexandre Belloni
2015-01-15 21:58 ` Alexandre Belloni
2015-01-15 21:58 ` [PATCH 13/17] ARM: at91: rework initialization for sam9 " Alexandre Belloni
2015-01-15 21:58 ` Alexandre Belloni
2015-01-15 21:58 ` [PATCH 14/17] ARM: at91: rework initialization for sama5 " Alexandre Belloni
2015-01-15 21:58 ` Alexandre Belloni
2015-01-15 21:58 ` [PATCH 15/17] ARM: at91: remove old setup Alexandre Belloni
2015-01-15 21:58 ` Alexandre Belloni
2015-01-21 16:29 ` Nicolas Ferre
2015-01-21 16:29 ` Nicolas Ferre
2015-01-21 20:40 ` Alexandre Belloni
2015-01-21 20:40 ` Alexandre Belloni
2015-01-15 21:58 ` [PATCH 16/17] ARM: at91: mark const init data with __initconst instead of __initdata Alexandre Belloni
2015-01-15 21:58 ` Alexandre Belloni
2015-01-15 21:58 ` [PATCH 17/17] ARM: at91: remove at91_dt_initialize Alexandre Belloni
2015-01-15 21:58 ` Alexandre Belloni
2015-01-22 10:31 ` [PATCH 00/17] AT91 cleanup for 3.20 #3 Nicolas Ferre
2015-01-22 10:31 ` Nicolas Ferre
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=54BE2F4D.2080800@atmel.com \
--to=nicolas.ferre@atmel.com \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.