All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ezequiel Garcia <ezequiel@vanguardiasur.com.ar>
To: Dave Airlie <airlied@gmail.com>
Cc: dri-devel <dri-devel@lists.freedesktop.org>
Subject: Re: [PATCH/RESEND 0/8] tilcdc-panel: Backlight and GPIO devicetree support
Date: Mon, 15 Sep 2014 19:51:33 +0100	[thread overview]
Message-ID: <20140915185133.GB1358@arch> (raw)
In-Reply-To: <CAPM=9tzF6FG=RBjiP-8RdjaufEoUDHHWbs2bivCJhKpB+KCtEQ@mail.gmail.com>

On 15 Sep 05:59 PM, Dave Airlie wrote:
> On 15 September 2014 17:50, Ezequiel Garcia
> <ezequiel@vanguardiasur.com.ar> wrote:
> > Dave,
> >
> > On 03 Sep 08:08 AM, Johannes Pointner wrote:
> >> 2014-09-02 14:51 GMT+02:00 Ezequiel Garcia <ezequiel@vanguardiasur.com.ar>:
> >> > Dave,
> >> >
> >> > I'm resending this, hoping it can be pushed for v3.18. The patchset was
> >> > ready for v3.17, but it got no maintainer feedback or review. Maybe it fell
> >> > through some crack?
> >> >
> >> > Just for reference, here goes the details about this series and why it's
> >> > needed:
> >> >
> >> > This patchset adds the required changes to support an optional backlight
> >> > and GPIO for the tilcdc panel driver.
> >> >
> >> > There was some code to support a backlight, but it was broken and undocumented.
> >> > I've followed the nice implementation in panel-simple and added a similar
> >> > one here.
> >> >
> >> > The enable GPIO is required to turn on and off devices with such capability.
> >> > Also here, I've followed panel-simple which looks correct.
> >> >
> >> > In addition to this there are very minor cosmetic cleanups and a larger
> >> > fix for the error path in tilcdc's DRM driver .load error path.
> >> >
> >>
> >> I tested the series with 3.16.1 (with additonal patches from Guido and
> >> Sachin) and with 3.17-rc3 with a custom AM335x board and it worked for
> >> me without an issue. I tried it with and without the backlight
> >> addition in the dts file.
> >>
> >> For the series:
> >> Tested-by: Johannes Pointner <johannes.pointner@br-automation.com>
> >>
> >
> > Any feedback for this series?
> >
> > If at all possible, it'd be great to not miss the merge this time.
> 
> Could you stick it in a git tree somewhere? and send a pull request for it?
> 

Hm.. I really don't have a git tree ready :/

Do you think you can pick the patches this time? I'll setup a tree as soon
as possible.
-- 
Ezequiel Garcia, VanguardiaSur
www.vanguardiasur.com.ar

  reply	other threads:[~2014-09-15 18:52 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-09-02 12:51 [PATCH/RESEND 0/8] tilcdc-panel: Backlight and GPIO devicetree support Ezequiel Garcia
2014-09-02 12:51 ` [PATCH/RESEND 1/8] drm/tilcdc: Fix the error path in tilcdc_load() Ezequiel Garcia
2014-09-02 12:51 ` [PATCH/RESEND 2/8] drm/tilcdc: panel: Add missing of_node_put Ezequiel Garcia
2014-09-02 12:51 ` [PATCH/RESEND 3/8] drm/tilcdc: panel: Remove unused variable Ezequiel Garcia
2014-09-02 12:51 ` [PATCH/RESEND 4/8] drm/tilcdc: panel: Spurious whitespace removal Ezequiel Garcia
2014-09-02 12:51 ` [PATCH/RESEND 5/8] drm/tilcdc: panel: Use devm_kzalloc to simplify the error path Ezequiel Garcia
2014-09-02 12:51 ` [PATCH/RESEND 6/8] drm/tilcdc: panel: Fix backlight devicetree support Ezequiel Garcia
2014-09-02 12:51 ` [PATCH/RESEND 7/8] drm/tilcdc: panel: Set return value explicitly Ezequiel Garcia
2014-09-02 12:51 ` [PATCH/RESEND 8/8] drm/tilcdc: panel: Add support for enable GPIO Ezequiel Garcia
2014-09-03  6:08 ` [PATCH/RESEND 0/8] tilcdc-panel: Backlight and GPIO devicetree support Johannes Pointner
2014-09-03 11:08   ` Ezequiel Garcia
2014-09-15  7:50   ` Ezequiel Garcia
2014-09-15  7:59     ` Dave Airlie
2014-09-15 18:51       ` Ezequiel Garcia [this message]
2014-09-17  0:56         ` Dave Airlie

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=20140915185133.GB1358@arch \
    --to=ezequiel@vanguardiasur.com.ar \
    --cc=airlied@gmail.com \
    --cc=dri-devel@lists.freedesktop.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.