From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from smtp-vbr12.xs4all.nl ([194.109.24.32]:1400 "EHLO smtp-vbr12.xs4all.nl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751055AbaEIJFE (ORCPT ); Fri, 9 May 2014 05:05:04 -0400 Message-ID: <536C9A32.10703@xs4all.nl> Date: Fri, 09 May 2014 11:04:50 +0200 From: Hans Verkuil MIME-Version: 1.0 To: =?UTF-8?B?RnJhbmsgU2Now6RmZXI=?= , m.chehab@samsung.com CC: linux-media@vger.kernel.org Subject: Re: [PATCH 00/19] em28xx: clean up the main device struct and move sub-module specific data to its own data structs References: <1395689605-2705-1-git-send-email-fschaefer.oss@googlemail.com> In-Reply-To: <1395689605-2705-1-git-send-email-fschaefer.oss@googlemail.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sender: linux-media-owner@vger.kernel.org List-ID: Hi Frank, This looks good to me. I do have some comments for future cleanups and I'll reply to the relevant patches for that. However, before I can apply this patch series you need to take a look at my comments for this pre-requisite patch: https://patchwork.linuxtv.org/patch/23179/ That needs to be sorted before I can apply this series. Regards, Hans On 03/24/2014 08:33 PM, Frank Schäfer wrote: > This patch series cleans up the main device struct of the em28xx driver. > > Most of the patches (patches 3-16) are about moving the em28xx-v4l specific data > to it's own dynamically allocated data structure. > Patch 19 moves two em28xx-alsa specific fields to the em28xx_audio struct. > Patches 17 and 18 remove two fields which aren't needed. > > > Frank Schäfer (19): > em28xx: move sub-module data structs to a common place in the main > struct > em28xx-video: simplify usage of the pointer to struct > v4l2_ctrl_handler in em28xx_v4l2_init() > em28xx: start moving em28xx-v4l specific data to its own struct > em28xx: move struct v4l2_ctrl_handler ctrl_handler from struct em28xx > to struct v4l2 > em28xx: move struct v4l2_clk *clk from struct em28xx to struct v4l2 > em28xx: move video_device structs from struct em28xx to struct v4l2 > em28xx: move videobuf2 related data from struct em28xx to struct v4l2 > em28xx: move v4l2 frame resolutions and scale data from struct em28xx > to struct v4l2 > em28xx: move vinmode and vinctrl data from struct em28xx to struct > v4l2 > em28xx: move TV norm from struct em28xx to struct v4l2 > em28xx: move struct em28xx_fmt *format from struct em28xx to struct > v4l2 > em28xx: move progressive/interlaced fields from struct em28xx to > struct v4l2 > em28xx: move sensor parameter fields from struct em28xx to struct v4l2 > em28xx: move capture state tracking fields from struct em28xx to > struct v4l2 > em28xx: move v4l2 user counting fields from struct em28xx to struct > v4l2 > em28xx: move tuner frequency field from struct em28xx to struct v4l2 > em28xx: remove field tda9887_conf from struct em28xx > em28xx: remove field tuner_addr from struct em28xx > em28xx: move fields wq_trigger and streaming_started from struct > em28xx to struct em28xx_audio > > drivers/media/usb/em28xx/em28xx-audio.c | 39 +- > drivers/media/usb/em28xx/em28xx-camera.c | 51 +-- > drivers/media/usb/em28xx/em28xx-cards.c | 9 - > drivers/media/usb/em28xx/em28xx-vbi.c | 10 +- > drivers/media/usb/em28xx/em28xx-video.c | 592 +++++++++++++++++-------------- > drivers/media/usb/em28xx/em28xx.h | 120 ++++--- > 6 files changed, 452 insertions(+), 369 deletions(-) >