Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753469AbdFMOi0 (ORCPT ); Tue, 13 Jun 2017 10:38:26 -0400 Received: from ec2-52-27-115-49.us-west-2.compute.amazonaws.com ([52.27.115.49]:48857 "EHLO osg.samsung.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752781AbdFMOiY (ORCPT ); Tue, 13 Jun 2017 10:38:24 -0400 Date: Tue, 13 Jun 2017 11:38:14 -0300 From: Mauro Carvalho Chehab To: "Takiguchi, Yasunari" Cc: "akpm@linux-foundation.org" , "linux-kernel@vger.kernel.org" , "devicetree@vger.kernel.org" , "linux-media@vger.kernel.org" , "tbird20d@gmail.com" , "frowand.list@gmail.com" , "Yamamoto, Masayuki" , "Nozawa, Hideki (STWN)" , "Yonezawa, Kota" , "Matsumoto, Toshihiko" , "Watanabe, Satoshi (SSS)" Subject: Re: [PATCH v2 0/15] [dt-bindings] [media] Add document file and driver for Sony CXD2880 DVB-T2/T tuner + demodulator Message-ID: <20170613113814.0094536f@vento.lan> In-Reply-To: References: <20170414015043.16731-1-Yasunari.Takiguchi@sony.com> <5188b958-9a34-4519-5845-a318273592e0@sony.com> Organization: Samsung X-Mailer: Claws Mail 3.14.1 (GTK+ 2.24.31; x86_64-redhat-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1367 Lines: 34 Hi Takiguchi-san, Em Thu, 25 May 2017 15:15:39 +0900 "Takiguchi, Yasunari" escreveu: > Hi, all > > I sent the patch series of Sony CXD2880 DVB-T2/T tuner + demodulator driver on Apr/14. > Are there any comments, advices and review results for them? Usually, reviewing drivers takes more time, as one needs to reserve a long period of time for reviewing. Sorry for the delay. > I'd like to get better understanding of current review status for our codes. Just sent today a review. There are some things that need to be changed in order to get it into a better shape and make it easier to review. In particular, it should be using some Kernel internal APIs, instead of coming with re-implementation of existing core code. That's fine. It is very unusual that the first contributions from a new Kernel developer to gets everything the way as it is expected mainstream ;-) One thing that come into my mind, besides what was already commented, is that it seems you added an abstraction layer. We don't like such layers very much, as it makes harder to understand the driver, usually for very little benefit. On this first review, I didn't actually try to understand what's going on there. As the driver doesn't contain any comments inside, it makes harder to understand why some things were coded using such approach. Thanks, Mauro