From: Tony Lindgren <tony@atomide.com>
To: Daniel Thompson <daniel.thompson@linaro.org>
Cc: Russell King <linux@arm.linux.org.uk>,
Arnd Bergmann <arnd@arndb.de>,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, spear-devel@list.st.com,
Paul Bolle <pebolle@tiscali.nl>,
Srinivas Kandagatla <srinivas.kandagatla@linaro.org>,
John Stultz <john.stultz@linaro.org>,
Sumit Semwal <sumit.semwal@linaro.org>,
patches@linaro.org, linaro-kernel@lists.linaro.org,
Arnd Bergmann <arnd.bergmann@linaro.org>,
linux-omap@vger.kernel.org
Subject: Re: [PATCH v13 5/9] arm: omap1: Migrate debug_ll macros to use 8250.S
Date: Fri, 21 Nov 2014 15:39:59 -0800 [thread overview]
Message-ID: <20141121233958.GB7046@atomide.com> (raw)
In-Reply-To: <1416235750-4766-6-git-send-email-daniel.thompson@linaro.org>
* Daniel Thompson <daniel.thompson@linaro.org> [141117 06:53]:
> The omap1's debug-macro.S is similar to the generic 8250 code. Compared to
> the 8520 code the omap1 macro automatically determines what UART to use
> based on breadcrumbs left by the bootloader and automatically copes with
> the eccentric register layout on OMAP7XX.
>
> This patch drops both these features and relies instead on the generic
> 8250 macros:
>
> 1. Dropping support for the bootloader breadcrumbs is identical to the
> way the migration was handled for OMAP2 (see 808b7e07464d...).
>
> 2. Support for OMAP7XX still exists but it must be configured by hand
> (DEBUG_OMAP7XXUART1/2/3) rather than handled at runtime.
>
> Signed-off-by: Daniel Thompson <daniel.thompson@linaro.org>
> Cc: Russell King <linux@arm.linux.org.uk>
> Cc: linux-arm-kernel@lists.infradead.org
> Cc: Tony Lindgren <tony@atomide.com>
> Cc: Arnd Bergmann <arnd.bergmann@linaro.org>
> Cc: linux-omap@vger.kernel.org
> Tested-by: Aaro Koskinen <aaro.koskinen@iki.fi>
Looks OK to me, probably best that these are queued all together so:
Acked-by: Tony Lindgren <tony@atomide.com>
WARNING: multiple messages have this Message-ID (diff)
From: tony@atomide.com (Tony Lindgren)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v13 5/9] arm: omap1: Migrate debug_ll macros to use 8250.S
Date: Fri, 21 Nov 2014 15:39:59 -0800 [thread overview]
Message-ID: <20141121233958.GB7046@atomide.com> (raw)
In-Reply-To: <1416235750-4766-6-git-send-email-daniel.thompson@linaro.org>
* Daniel Thompson <daniel.thompson@linaro.org> [141117 06:53]:
> The omap1's debug-macro.S is similar to the generic 8250 code. Compared to
> the 8520 code the omap1 macro automatically determines what UART to use
> based on breadcrumbs left by the bootloader and automatically copes with
> the eccentric register layout on OMAP7XX.
>
> This patch drops both these features and relies instead on the generic
> 8250 macros:
>
> 1. Dropping support for the bootloader breadcrumbs is identical to the
> way the migration was handled for OMAP2 (see 808b7e07464d...).
>
> 2. Support for OMAP7XX still exists but it must be configured by hand
> (DEBUG_OMAP7XXUART1/2/3) rather than handled at runtime.
>
> Signed-off-by: Daniel Thompson <daniel.thompson@linaro.org>
> Cc: Russell King <linux@arm.linux.org.uk>
> Cc: linux-arm-kernel at lists.infradead.org
> Cc: Tony Lindgren <tony@atomide.com>
> Cc: Arnd Bergmann <arnd.bergmann@linaro.org>
> Cc: linux-omap at vger.kernel.org
> Tested-by: Aaro Koskinen <aaro.koskinen@iki.fi>
Looks OK to me, probably best that these are queued all together so:
Acked-by: Tony Lindgren <tony@atomide.com>
next prev parent reply other threads:[~2014-11-21 23:39 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-11-17 14:49 [PATCH v13 0/9] arm: Fix DEBUG_LL for multi-platform kernels (without PL01X) Daniel Thompson
2014-11-17 14:49 ` Daniel Thompson
2014-11-17 14:49 ` [PATCH v13 1/9] arm: cnx3xxxx: Remove spurious default for DEBUG_CNS3XXX Daniel Thompson
2014-11-17 14:53 ` Fwd: [PATCH v13 1/9] arm: cnx3xxx: Remove spurious default for DEBUG_CNS3xxx Daniel Thompson
2014-11-17 14:49 ` [PATCH v13 2/9] arm: configs: Enable DEBUG_LL_UART_8250 where needed Daniel Thompson
2014-11-17 14:49 ` Daniel Thompson
2014-11-17 14:49 ` [PATCH v13 3/9] arm: Remove DEBUG_LL_UART_NONE Daniel Thompson
2014-11-17 14:49 ` Daniel Thompson
2014-11-17 14:49 ` [PATCH v13 4/9] arm: ks8695: Migrate debug_ll macros to shared directory Daniel Thompson
2014-11-17 14:49 ` Daniel Thompson
2014-11-17 14:49 ` [PATCH v13 5/9] arm: omap1: Migrate debug_ll macros to use 8250.S Daniel Thompson
2014-11-17 14:49 ` Daniel Thompson
2014-11-21 23:39 ` Tony Lindgren [this message]
2014-11-21 23:39 ` Tony Lindgren
2014-11-17 14:49 ` [PATCH v13 6/9] arm: netx: Migrate DEBUG_LL macros to shared directory Daniel Thompson
2014-11-17 14:49 ` Daniel Thompson
2014-11-17 14:49 ` [PATCH v13 7/9] arm: sa1100: " Daniel Thompson
2014-11-17 14:49 ` Daniel Thompson
2014-11-17 14:49 ` [PATCH v13 8/9] arm: Seperate DEBUG_UART_PHYS from DEBUG_LL on EP93XX Daniel Thompson
2014-11-17 14:49 ` Daniel Thompson
2014-11-17 14:49 ` [PATCH v13 9/9] arm: Fix DEBUG_LL for multi-platform kernels (without PL01X) Daniel Thompson
2014-11-17 14:49 ` Daniel Thompson
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=20141121233958.GB7046@atomide.com \
--to=tony@atomide.com \
--cc=arnd.bergmann@linaro.org \
--cc=arnd@arndb.de \
--cc=daniel.thompson@linaro.org \
--cc=john.stultz@linaro.org \
--cc=linaro-kernel@lists.linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-omap@vger.kernel.org \
--cc=linux@arm.linux.org.uk \
--cc=patches@linaro.org \
--cc=pebolle@tiscali.nl \
--cc=spear-devel@list.st.com \
--cc=srinivas.kandagatla@linaro.org \
--cc=sumit.semwal@linaro.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.