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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux