Em Sun, 27 Aug 2017 11:45:44 -0300 Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxxxxx> escreveu: > Em Wed, 16 Aug 2017 13:37:14 +0900 > <Yasunari.Takiguchi@xxxxxxxx> escreveu: > > > From: Yasunari Takiguchi <Yasunari.Takiguchi@xxxxxxxx> > > > > This part of the driver has the main routines to handle > > the tuner and demodulator functionality. The tnrdmd_mon.* files > > have monitor functions for the driver. > > This is part of the Sony CXD2880 DVB-T2/T tuner + demodulator driver. > > This series is on a much better state. Thanks! > > Patches 1-4 sound ok to me. > > Still, there are a few issues on this patch. See below. > > > > > [Change list] > > Changes in V3 > > drivers/media/dvb-frontends/cxd2880/cxd2880_dtv.h > > -removed code relevant to ISDB-T Forgot to mention... Just a small detail: the change list is important for us to understand the differences between submitted versions, but it makes no sense to track them when patches get merged upstream, as the previous versions of the patches won't be there at the git history. So, please put change lists after a blank line with just "---", e. g.: From: Yasunari Takiguchi <Yasunari.Takiguchi@xxxxxxxx> This part of the driver has the main routines to handle the tuner and demodulator functionality. The tnrdmd_mon.* files have monitor functions for the driver. This is part of the Sony CXD2880 DVB-T2/T tuner + demodulator driver. --- (change list and whatever other notes that are relevant to the patch reviewers but won't be merged upstream) The reason is simple: Kernel maintainers' scripts will just remove everything after the "---" line when applying the series ;-) Thanks, Mauro
