From mboxrd@z Thu Jan 1 00:00:00 1970 From: Domenico Andreoli Subject: Re: [PATCH 6/6] ARM: bcm476x: Instantiate console UART Date: Wed, 10 Oct 2012 01:37:43 +0200 Message-ID: <20121009233743.GC7758@glitch> References: <20121007015300.828366635@gmail.com> <20121007015407.840781689@gmail.com> <507394CB.5080300@wwwdotorg.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <507394CB.5080300-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: devicetree-discuss-bounces+gldd-devicetree-discuss=m.gmane.org-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org Sender: "devicetree-discuss" To: Stephen Warren Cc: Domenico Andreoli , devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org List-Id: devicetree@vger.kernel.org On Mon, Oct 08, 2012 at 09:06:51PM -0600, Stephen Warren wrote: > On 10/06/2012 07:53 PM, Domenico Andreoli wrote: > > From: Domenico Andreoli > > > > The BCM476x sports standard PL011 UARTs which are fully described and > > put to use with these few changes. > > Given that this SoC uses an interrupt controller for which there's > already a driver (so there's no need to wait for a later patch before > interrupts work), and that patch 1/6 adds debug/bcm476x.S anyway, I'd be > inclined to just squash this patch into the very first one myself. > > If not, this patch isn't really just instantiating the console UART, but > also allowing the DEBUG_LL implementation to be selected. I created a separate patch for the DEBUG_LL and the rest is squashed in the first infrastructure patch. Regards, Domenico