From: horms@verge.net.au (Simon Horman)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/3] ARM: shmobile: Lager reference serial port support
Date: Fri, 1 Nov 2013 09:16:22 +0900 [thread overview]
Message-ID: <20131101001621.GC6818@verge.net.au> (raw)
In-Reply-To: <3965826.mx1T873Bg9@avalon>
[ CC Magnus ]
On Thu, Oct 31, 2013 at 01:21:23PM +0100, Laurent Pinchart wrote:
> Hi Simon,
>
> On Thursday 31 October 2013 14:31:44 Simon Horman wrote:
> > On Tue, Oct 29, 2013 at 04:23:10PM +0100, Laurent Pinchart wrote:
> > > Hello,
> > >
> > > This patch set implements serial port support for the Lager reference
> > > board, and adds early debugging support for the r8a7790 SoC.
> > >
> > > The code is available in my git tree at
> > >
> > > git://linuxtv.org/pinchartl/fbdev.git clocks/ccf/serial-lager
> > >
> > > The branch is based on a merge of renesas-devel-20131016, v3.12-rc7 and
> > > multiarch drivers, clk-prepare, r8a7790 CCF and sh-sci OF patches I've
> > > posted earlier today.
> > >
> > > Laurent Pinchart (3):
> > > ARM: shmobile: r8a7790: Add early debugging support
> >
> > The patch above could be applied now, right?
>
> That's correct. It will be useless though, as DEBUG_LL can only be used on
> multiarch.
I will wait, especially in the light of the point you raise below.
> Given that the SCIF/SCIFA/SCIFB port operate similarly, I wonder whether it
> wouldn't make more sense to add a generic DEBUG_SHMOBILE_SCIF implementation
> instead. We could then have sub-options to select the exact SoC type and port
> number, and compute the base address from that.
That makes a lot of sense to me.
Perhaps Magnus has an opinion?
> > If so, Magnus, could you review it?
> >
> > > ARM: shmobile: r8a7790: Add serial ports to the device tree
> > > ARM: shmobile: lager: Enable SCIF0 and SCIF1 serial ports in DT
> >
> > I would like to ask you to repost these patches once their
> > pre-requisites have been merged.
>
> Sure, no problem.
>
> --
> Regards,
>
> Laurent Pinchart
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-sh" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
prev parent reply other threads:[~2013-11-01 0:16 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-10-29 15:23 [PATCH 0/3] ARM: shmobile: Lager reference serial port support Laurent Pinchart
2013-10-29 15:23 ` [PATCH 1/3] ARM: shmobile: r8a7790: Add early debugging support Laurent Pinchart
2013-10-29 15:23 ` [PATCH 2/3] ARM: shmobile: r8a7790: Add serial ports to the device tree Laurent Pinchart
2013-10-29 15:23 ` [PATCH 3/3] ARM: shmobile: lager: Enable SCIF0 and SCIF1 serial ports in DT Laurent Pinchart
2013-10-29 18:09 ` Sergei Shtylyov
2013-10-29 18:30 ` Laurent Pinchart
2013-10-30 0:24 ` Simon Horman
2013-10-31 5:31 ` [PATCH 0/3] ARM: shmobile: Lager reference serial port support Simon Horman
2013-10-31 12:21 ` Laurent Pinchart
2013-11-01 0:16 ` Simon Horman [this message]
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=20131101001621.GC6818@verge.net.au \
--to=horms@verge.net.au \
--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).