From: tony@atomide.com (Tony Lindgren)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 3/5] omap: Add support for CONFIG_AUTO_ZRELADDR for DEBUG_LL
Date: Fri, 4 Feb 2011 12:24:11 -0800 [thread overview]
Message-ID: <20110204202411.GA20795@atomide.com> (raw)
In-Reply-To: <alpine.LFD.2.00.1102041445360.12104@xanadu.home>
* Nicolas Pitre <nico@fluxnic.net> [110204 12:14]:
>
> Just create a get_config_ptr macro or similar and the trickery will be
> nicely encapsulated. You'd have:
>
> .macro get_config_ptr ptr, tmp
> b 9002f
> .align
> 9001: .long .
> .long uart_param_storage
> 9002: adr \ptr, 9001b
> ldr \tmp, [\ptr]
> sub \tmp, \tmp, \ptr
> ldr \ptr, [\ptr, #4]
> sub \ptr, \ptr, \tmp
> .endm
>
> .macro addruart rp, rv
> get_config_ptr \rv
> ldr \rp, [\rv]
> ldr \rv, [\rv, #4]
> .endm
>
> And for the other macros, you get the hardware address to use already,
> but you can still use get_config_ptr for extra config values you
> need if necessary.
Thanks, will try that out next week. That might run into problems with
busyuart as it only has rd register to play with.
> > In that case the mapping of IO space for the serial port in head.S
> > would have to be changed as that relies on rx containing the port
> > IO address..
>
> Yes, and I'm not sure if it is worth changing that at this point when it
> is possible to do it without introducing such a change which would
> affect all machines again.
Agreed, let's see how far we can get with the existing setup.
What might need to be done is to save and restore rx in
busyuart to be able to use your macro above.
Tony
next prev parent reply other threads:[~2011-02-04 20:24 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-02-04 1:26 [PATCH 0/5] Change omap DEBUG_LL code to use inituart for AUTO_ZRELADDR and ARM_PATCH_PHYS_VIRT Tony Lindgren
2011-02-04 1:27 ` [PATCH 1/5] omap: Remove code configuring the DEBUG_LL serial port using uncompress code Tony Lindgren
2011-02-04 1:27 ` [PATCH 2/5] omap: Use inituart to configure the debug serial port based on machine ID Tony Lindgren
2011-02-04 12:04 ` Grazvydas Ignotas
2011-02-04 16:49 ` Tony Lindgren
2011-02-04 1:27 ` [PATCH 3/5] omap: Add support for CONFIG_AUTO_ZRELADDR for DEBUG_LL Tony Lindgren
2011-02-04 3:33 ` Nicolas Pitre
2011-02-04 17:02 ` Tony Lindgren
2011-02-04 20:16 ` Nicolas Pitre
2011-02-04 20:24 ` Tony Lindgren [this message]
2011-02-04 20:33 ` Nicolas Pitre
2011-02-04 17:15 ` Russell King - ARM Linux
2011-02-04 18:36 ` Tony Lindgren
2011-02-04 1:27 ` [PATCH 4/5] omap: Combine debug-macro.S for omap1 and omap2+ Tony Lindgren
2011-02-04 1:27 ` [PATCH 5/5] omap: Add CONFIG_AUTO_ZRELADDR and CONFIG_ARM_PATCH_PHYS_VIRT to defconfigs Tony Lindgren
2011-02-09 6:08 ` [PATCH 0/5] Change omap DEBUG_LL code to use inituart for AUTO_ZRELADDR and ARM_PATCH_PHYS_VIRT Poddar, Sourav
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=20110204202411.GA20795@atomide.com \
--to=tony@atomide.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 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).