All of lore.kernel.org
 help / color / mirror / Atom feed
From: tony@atomide.com (Tony Lindgren)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCHv2] ARM: debug: uncompress debug support for omap2plus
Date: Wed, 3 Sep 2014 15:04:39 -0700	[thread overview]
Message-ID: <20140903220433.GQ11766@atomide.com> (raw)
In-Reply-To: <20140826075831.GU30401@n2100.arm.linux.org.uk>

* Russell King - ARM Linux <linux@arm.linux.org.uk> [140826 00:58]:
> On Tue, Aug 26, 2014 at 04:28:59PM +0900, kpark3469 at gmail.com wrote:
> > From: Sahara <keun-o.park@windriver.com>
> > 
> > Since OMAP low-level debug code places data in the .data section,
> > The symbol DEBUG_UNCOMPRESS was defined with !DEBUG_OMAP2PLUS_UART.
> > This patch removes the part using data section in debug/omap2plus.S,
> > so DEBUG_UNCOMPRESS is now available on OMAP system.
> > 
> > Signed-off-by: Sahara <keun-o.park@windriver.com>
> > Tested-by: Afzal Mohammed <afzal.mohd.ma@gmail.com> (on am335x beagle
> > bone white)
> 
> You need to consider whom you are sending this to.  It would be well
> worth including the OMAP maintainer, Tony Lindgren.

Thanks saw it and replied yesterday that we should just rip out
the SoC detection and use standard 8250 debug_ll code.

Regards,

Tony

WARNING: multiple messages have this Message-ID (diff)
From: Tony Lindgren <tony@atomide.com>
To: Russell King - ARM Linux <linux@arm.linux.org.uk>
Cc: kpark3469@gmail.com, keun-o.park@windriver.com,
	linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org, afzal.mohd.ma@gmail.com
Subject: Re: [PATCHv2] ARM: debug: uncompress debug support for omap2plus
Date: Wed, 3 Sep 2014 15:04:39 -0700	[thread overview]
Message-ID: <20140903220433.GQ11766@atomide.com> (raw)
In-Reply-To: <20140826075831.GU30401@n2100.arm.linux.org.uk>

* Russell King - ARM Linux <linux@arm.linux.org.uk> [140826 00:58]:
> On Tue, Aug 26, 2014 at 04:28:59PM +0900, kpark3469@gmail.com wrote:
> > From: Sahara <keun-o.park@windriver.com>
> > 
> > Since OMAP low-level debug code places data in the .data section,
> > The symbol DEBUG_UNCOMPRESS was defined with !DEBUG_OMAP2PLUS_UART.
> > This patch removes the part using data section in debug/omap2plus.S,
> > so DEBUG_UNCOMPRESS is now available on OMAP system.
> > 
> > Signed-off-by: Sahara <keun-o.park@windriver.com>
> > Tested-by: Afzal Mohammed <afzal.mohd.ma@gmail.com> (on am335x beagle
> > bone white)
> 
> You need to consider whom you are sending this to.  It would be well
> worth including the OMAP maintainer, Tony Lindgren.

Thanks saw it and replied yesterday that we should just rip out
the SoC detection and use standard 8250 debug_ll code.

Regards,

Tony

  reply	other threads:[~2014-09-03 22:04 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-08-26  7:28 [PATCHv2] ARM: debug: uncompress debug support for omap2plus kpark3469 at gmail.com
2014-08-26  7:28 ` kpark3469
2014-08-26  7:58 ` Russell King - ARM Linux
2014-08-26  7:58   ` Russell King - ARM Linux
2014-09-03 22:04   ` Tony Lindgren [this message]
2014-09-03 22:04     ` Tony Lindgren
  -- strict thread matches above, loose matches on Subject: below --
2014-08-26  8:28 kpark3469 at gmail.com
2014-08-26  8:28 ` kpark3469
2014-09-02 16:37 ` Tony Lindgren
2014-09-02 16:37   ` Tony Lindgren

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=20140903220433.GQ11766@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 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.