All of lore.kernel.org
 help / color / mirror / Atom feed
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Josh Wu <josh.wu@atmel.com>
Cc: linux-media@vger.kernel.org
Subject: Re: [PATCH 0/4] atmel-isi: Remove platform data support
Date: Wed, 05 Aug 2015 01:08:19 +0300	[thread overview]
Message-ID: <11161489.E6W8tYM4a4@avalon> (raw)
In-Reply-To: <55BED85D.4090905@atmel.com>

Hi Josh,

On Monday 03 August 2015 10:56:29 Josh Wu wrote:
> On 8/1/2015 5:22 PM, Laurent Pinchart wrote:
> > Hello,
> > 
> > While reviewing patches for the atmel-isi I noticed a couple of small
> > issues with the driver. Here's a patch series to fix them, the main
> > change being the removal of platform data support now that all users have
> > migrated to DT.
>
> Thanks for the patches. It's perfectly make sense.
> 
> > The patches have been compile-tested only. Josh, would you be able to test
> > them on hardware ?
> 
> For the whole series, here is my:
> 
> Acked-by: Josh Wu <josh.wu@atmel.com>
> Tested-by: Josh Wu <josh.wu@atmel.com>

Thank you.

Do you plan to take those four patches in your tree and include them in your 
next pull request ?

> > Laurent Pinchart (4):
> >    v4l: atmel-isi: Simplify error handling during DT parsing
> >    v4l: atmel-isi: Remove unused variable
> >    v4l: atmel-isi: Remove support for platform data
> >    v4l: atmel-isi: Remove unused platform data fields
> >   
> >   drivers/media/platform/soc_camera/atmel-isi.c |  40 ++------
> >   drivers/media/platform/soc_camera/atmel-isi.h | 126 ++++++++++++++++++++
> >   include/media/atmel-isi.h                     | 131 --------------------
> >   3 files changed, 136 insertions(+), 161 deletions(-)
> >   create mode 100644 drivers/media/platform/soc_camera/atmel-isi.h
> >   delete mode 100644 include/media/atmel-isi.h

-- 
Regards,

Laurent Pinchart


  reply	other threads:[~2015-08-04 22:07 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-08-01  9:22 [PATCH 0/4] atmel-isi: Remove platform data support Laurent Pinchart
2015-08-01  9:22 ` [PATCH 1/4] v4l: atmel-isi: Simplify error handling during DT parsing Laurent Pinchart
2015-08-01  9:22 ` [PATCH 2/4] v4l: atmel-isi: Remove unused variable Laurent Pinchart
2015-08-06  8:19   ` Josh Wu
2015-08-01  9:22 ` [PATCH 3/4] v4l: atmel-isi: Remove support for platform data Laurent Pinchart
2015-08-01  9:22 ` [PATCH 4/4] v4l: atmel-isi: Remove unused platform data fields Laurent Pinchart
2015-08-03  2:56 ` [PATCH 0/4] atmel-isi: Remove platform data support Josh Wu
2015-08-04 22:08   ` Laurent Pinchart [this message]
2015-08-05  3:31     ` Josh Wu
2015-09-20 15:25 ` Guennadi Liakhovetski
2015-09-21  1:33   ` Josh Wu

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=11161489.E6W8tYM4a4@avalon \
    --to=laurent.pinchart@ideasonboard.com \
    --cc=josh.wu@atmel.com \
    --cc=linux-media@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 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.