linux-tegra.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Nicolas Pitre <nico-vtqb6HGKxmzR7s880joybQ@public.gmane.org>
To: Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
Cc: Russell King - ARM Linux
	<linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org>,
	Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>,
	Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org>,
	Rob Herring <rob.herring-bsGFqQB8/DxBDgjK7y7TUQ@public.gmane.org>,
	Olof Johansson <olof-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org>,
	Tim Bird <tim.bird-mEdOJwZ7QcZBDgjK7y7TUQ@public.gmane.org>,
	"linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	"linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org"
	<linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org>
Subject: Re: [RFC PATCH 0/3] ARM: uncompress.h multi-platform enablement
Date: Thu, 27 Sep 2012 23:53:33 -0400 (EDT)	[thread overview]
Message-ID: <alpine.LFD.2.02.1209272343330.6667@xanadu.home> (raw)
In-Reply-To: <50632096.7080208-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>

On Wed, 26 Sep 2012, Stephen Warren wrote:

> In the Tegra case, by omitting uncompress.h, we lose more than just the
> "Decompressing Linux..." message; uncompress.h contains code to parse
> extract the platform's debug UART ID from data set up by the boot ROM
> (since we have 5 UARTs), and pass this on to debug-macro.S.

Let me tell you that this is totally evil.  OMAP and Davinci are doing 
the same abuse.

> Without uncompress.h running, debug-macro.S would either have to 
> hard-code a single UART ID (and different boards uses different UARTs, 
> which would make running a single kernel image across all Tegra 
> devices with earlyprintk enabled impossible) *or* we'd have to port 
> the UART ID extraction code to debug-macro.S too,

Please do the later.  That's the only sane solution.

> which I don't fancy very much since that's assembly whereas 
> uncompress.h is C.

Well, that's too bad.  Time to sharpen your assembly skills.  ;-)


Nicolas

  parent reply	other threads:[~2012-09-28  3:53 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-09-25 22:46 [RFC PATCH 0/3] ARM: uncompress.h multi-platform enablement Stephen Warren
     [not found] ` <1348613212-21897-1-git-send-email-swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2012-09-25 22:46   ` [RFC PATCH 1/3] ARM: move ICEDCC uncompress.h to common location Stephen Warren
     [not found]     ` <1348613212-21897-2-git-send-email-swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2012-09-26 21:56       ` Domenico Andreoli
2012-09-27  5:19         ` Stephen Warren
2012-09-25 22:46   ` [RFC PATCH 2/3] ARM: mvebu: restore uncompress.h Stephen Warren
2012-09-25 22:46   ` [RFC PATCH 3/3] ARM: tegra: move debug macros to include/debug Stephen Warren
2012-09-26 10:38   ` [RFC PATCH 0/3] ARM: uncompress.h multi-platform enablement Arnd Bergmann
     [not found]     ` <201209261038.36223.arnd-r2nGTMty4D4@public.gmane.org>
2012-09-26 10:44       ` Russell King - ARM Linux
     [not found]         ` <20120926104421.GA7040-l+eeeJia6m9vn6HldHNs0ANdhmdF6hFW@public.gmane.org>
2012-09-26 12:05           ` Arnd Bergmann
     [not found]             ` <201209261205.01351.arnd-r2nGTMty4D4@public.gmane.org>
2012-09-26 13:10               ` Domenico Andreoli
2012-09-26 13:48           ` Tim Bird
     [not found]             ` <506307A0.7010701-mEdOJwZ7QcZBDgjK7y7TUQ@public.gmane.org>
2012-09-26 14:17               ` Arnd Bergmann
     [not found]                 ` <201209261417.18883.arnd-r2nGTMty4D4@public.gmane.org>
2012-09-26 14:48                   ` Tim Bird
     [not found]                     ` <506315A6.3090300-mEdOJwZ7QcZBDgjK7y7TUQ@public.gmane.org>
2012-09-26 14:49                       ` Arnd Bergmann
     [not found]                         ` <201209261449.48468.arnd-r2nGTMty4D4@public.gmane.org>
2012-09-26 15:10                           ` Russell King - ARM Linux
     [not found]                             ` <20120926151023.GC30938-l+eeeJia6m9vn6HldHNs0ANdhmdF6hFW@public.gmane.org>
2012-09-26 15:34                               ` Stephen Warren
     [not found]                                 ` <50632096.7080208-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2012-09-28  3:53                                   ` Nicolas Pitre [this message]
2012-09-27  5:37 ` Jean-Christophe PLAGNIOL-VILLARD

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=alpine.LFD.2.02.1209272343330.6667@xanadu.home \
    --to=nico-vtqb6hgkxmzr7s880joybq@public.gmane.org \
    --cc=arnd-r2nGTMty4D4@public.gmane.org \
    --cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
    --cc=linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org \
    --cc=linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=olof-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org \
    --cc=rob.herring-bsGFqQB8/DxBDgjK7y7TUQ@public.gmane.org \
    --cc=swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org \
    --cc=swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org \
    --cc=tim.bird-mEdOJwZ7QcZBDgjK7y7TUQ@public.gmane.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).