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

Hi, Laurent

On 8/5/2015 6:08 AM, Laurent Pinchart wrote:
> 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 ?

yes, I plan to take them with my other patches for atmel-isi (about the 
configure_geometry(), already sent for review).
And I will sent a pull request to Gueannadi.

Best Regards,
Josh Wu

>>> 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


  reply	other threads:[~2015-08-05  3:31 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
2015-08-05  3:31     ` Josh Wu [this message]
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=55C18374.4030707@atmel.com \
    --to=josh.wu@atmel.com \
    --cc=laurent.pinchart@ideasonboard.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.