SUPERH platform development
 help / color / mirror / Atom feed
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: linux-sh@vger.kernel.org
Subject: Re: [PATCH v3 0/8] Enable DU support for Marzen and Lager boards
Date: Fri, 09 Aug 2013 10:25:11 +0000	[thread overview]
Message-ID: <2721876.CaeM0Iyynj@avalon> (raw)
In-Reply-To: <1375963414-31096-1-git-send-email-laurent.pinchart+renesas@ideasonboard.com>

Hi Morimoto-san,

On Thursday 08 August 2013 18:44:49 Kuninori Morimoto wrote:
> > This is the third version of a patch set that registers DU devices for the
> > Marzen and Lager boards. Only platform data registration is supported as
> > DU DT bindings are work in progress.
> > 
> > Compared to v2, this version annotates const platform data and resources
> > as __initconst, splits the r8a7779 DU clock rename to a patch of its own,
> > and moves DU device registration from setup-*.c files to board files.
> > 
> > The patches depend on both the DU r8a7790 DRM and PFC series available
> > from
> > 
> >         git://linuxtv.org/pinchartl/fbdev.git drm/next/du
> >         git://linuxtv.org/pinchartl/fbdev.git pinmux/next/r8a7790
> > 
> > They are based respectively on the latest drm-next branch from
> > git://people.freedesktop.org/~airlied/linux and the latest for-next branch
> > from
> > git://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-pinctrl.git.
> > Both series are scheduled for upstream merge in v3.12.
> > 
> > The dependency on pinmux/next/r8a7790 is runtime only, compilation will
> > not break and the boards won't fail to boot. The only regression will be a
> > pinmux error message at boot time. The drm/next/du series, however, is a
> > compile-time dependency. I've asked Dave Airlie to provide a stable
> > branch with those patches.
> > 
> > As a side note, patches 1/8 to 3/8 and 6/8 can be merged first
> > independently of this series if desired.
> > 
> > Laurent Pinchart (8):
> >   ARM: shmobile: r8a7779: Rename DU device in clock lookups list
> >   ARM: shmobile: r8a7790: Constify platform data and resources
> >   ARM: shmobile: r8a7790: Add DU and LVDS clocks
> >   ARM: shmobile: marzen: Add Display Unit support
> >   ARM: shmobile: marzen: Select DRM_RCAR_DU in defconfig
> >   ARM: shmobile: lager: Constify platform data and resources
> >   ARM: shmobile: lager: Add Display Unit support
> >   ARM: shmobile: lager: Select DRM_RCAR_DU in defconfig
> >  
> >  arch/arm/configs/lager_defconfig       |  2 +
> >  arch/arm/configs/marzen_defconfig      |  2 +
> >  arch/arm/mach-shmobile/board-lager.c   | 81 ++++++++++++++++++++++++++---
> >  arch/arm/mach-shmobile/board-marzen.c  | 72 +++++++++++++++++++++++++++++
> >  arch/arm/mach-shmobile/clock-r8a7779.c |  2 +-
> >  arch/arm/mach-shmobile/clock-r8a7790.c | 12 ++++-
> >  arch/arm/mach-shmobile/setup-r8a7790.c | 21 ++++-----
> >  7 files changed, 173 insertions(+), 19 deletions(-)
> 
> [7/8] has extra blank line.
> I don't care so much about it,
> but please remove it if you have chance to send v4 patch ;P

Oops, my bad.

Simon, I've pushed all the patches with the blank line removed and the 
collected Acked-by to

	git://linuxtv.org/pinchartl/fbdev.git drm/du/r8a7790

You will need to cherry-pick them as they're based on a merge of local 
branches.

> For all patches, from driver registration point of view
> 
> Acked-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>

Thank you.

-- 
Regards,

Laurent Pinchart


      parent reply	other threads:[~2013-08-09 10:25 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-08-08 12:03 [PATCH v3 0/8] Enable DU support for Marzen and Lager boards Laurent Pinchart
2013-08-09  1:44 ` Kuninori Morimoto
2013-08-09 10:25 ` Laurent Pinchart [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=2721876.CaeM0Iyynj@avalon \
    --to=laurent.pinchart@ideasonboard.com \
    --cc=linux-sh@vger.kernel.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