Hi Takiguchi-san, Em Thu, 25 May 2017 15:15:39 +0900 "Takiguchi, Yasunari" <Yasunari.Takiguchi@xxxxxxxx> 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 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
