From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from nasmtp01.atmel.com ([192.199.1.246]:8638 "EHLO DVREDG02.corp.atmel.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751661AbbHEDbF (ORCPT ); Tue, 4 Aug 2015 23:31:05 -0400 Message-ID: <55C18374.4030707@atmel.com> Date: Wed, 5 Aug 2015 11:31:00 +0800 From: Josh Wu MIME-Version: 1.0 To: Laurent Pinchart CC: Subject: Re: [PATCH 0/4] atmel-isi: Remove platform data support References: <1438420976-7899-1-git-send-email-laurent.pinchart@ideasonboard.com> <55BED85D.4090905@atmel.com> <11161489.E6W8tYM4a4@avalon> In-Reply-To: <11161489.E6W8tYM4a4@avalon> Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-media-owner@vger.kernel.org List-ID: 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 >> Tested-by: Josh Wu > 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