From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S1751871AbbATJAZ (ORCPT <rfc822;w@1wt.eu>);
	Tue, 20 Jan 2015 04:00:25 -0500
Received: from mail-we0-f173.google.com ([74.125.82.173]:42335 "EHLO
	mail-we0-f173.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S1751241AbbATJAY (ORCPT
	<rfc822;linux-kernel@vger.kernel.org>);
	Tue, 20 Jan 2015 04:00:24 -0500
Message-ID: <54BE1921.5050902@linaro.org>
Date: Tue, 20 Jan 2015 09:00:17 +0000
From: Daniel Thompson <daniel.thompson@linaro.org>
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.3.0
MIME-Version: 1.0
To: Tony Lindgren <tony@atomide.com>
CC: Russell King <linux@arm.linux.org.uk>, Arnd Bergmann <arnd@arndb.de>,
        Paul Bolle <pebolle@tiscali.nl>, linaro-kernel@lists.linaro.org,
        patches@linaro.org, spear-devel@list.st.com,
        linux-kernel@vger.kernel.org,
        Srinivas Kandagatla <srinivas.kandagatla@linaro.org>,
        John Stultz <john.stultz@linaro.org>,
        Arnd Bergmann <arnd.bergmann@linaro.org>, linux-omap@vger.kernel.org,
        Sumit Semwal <sumit.semwal@linaro.org>,
        linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 3.19-rc2 v15 5/8] arm: omap1: Migrate debug_ll macros
 to use 8250.S
References: <1420461624-4410-1-git-send-email-daniel.thompson@linaro.org> <1420461624-4410-6-git-send-email-daniel.thompson@linaro.org> <20150119213845.GL18552@atomide.com>
In-Reply-To: <20150119213845.GL18552@atomide.com>
Content-Type: text/plain; charset=windows-1252
Content-Transfer-Encoding: 8bit
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 19/01/15 21:38, Tony Lindgren wrote:
> * Daniel Thompson <daniel.thompson@linaro.org> [150105 04:49]:
>> 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: Arnd Bergmann <arnd.bergmann@linaro.org>
>> Cc: linux-omap@vger.kernel.org
>> Tested-by: Aaro Koskinen <aaro.koskinen@iki.fi>
>> Acked-by: Tony Lindgren <tony@atomide.com>
> 
> Daniel, I suggest you upload this patch into Russell's patch tracking
> system to get it merged. That at least shrinks down your patch series
> if the other patches need more work.

This has been in the patch tracker for a week or so:
http://www.arm.linux.org.uk/developer/patches/viewpatch.php?id=8271/1

I'm very aware that this patch (and indeed the whole patch set) has been
knocking round for longer than it should have. The changelog for this
patchset is certainly not one to be especially proud off ;-) .


Daniel.