Re: [PATCH v3 05/14] [media] cxd2880: Add tuner part of the driver

From: Mauro Carvalho Chehab
Date: Sun Aug 27 2017 - 10:52:52 EST


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