From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mailout4.w2.samsung.com ([211.189.100.14]:65006 "EHLO usmailout4.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750975Ab3KDMTX (ORCPT ); Mon, 4 Nov 2013 07:19:23 -0500 Received: from uscpsbgm2.samsung.com (u115.gpu85.samsung.co.kr [203.254.195.115]) by usmailout4.samsung.com (Oracle Communications Messaging Server 7u4-24.01(7.0.4.24.0) 64bit (built Nov 17 2011)) with ESMTP id <0MVQ004HVNK74W40@usmailout4.samsung.com> for linux-media@vger.kernel.org; Mon, 04 Nov 2013 07:19:22 -0500 (EST) Date: Mon, 04 Nov 2013 10:19:18 -0200 From: Mauro Carvalho Chehab To: Maik Broemme Cc: Linux Media Mailing List Subject: Re: [PATCH 00/12] DDBridge 0.9.10 driver updates Message-id: <20131104101918.453355b9@samsung.com> In-reply-to: <20131103124601.GS7956@parallels.com> References: <20131103002235.GD7956@parallels.com> <20131103085822.08e8406e@samsung.com> <20131103124601.GS7956@parallels.com> MIME-version: 1.0 Content-type: text/plain; charset=US-ASCII Content-transfer-encoding: 7bit Sender: linux-media-owner@vger.kernel.org List-ID: Em Sun, 03 Nov 2013 13:46:01 +0100 Maik Broemme escreveu: > Hi Mauro, > > Mauro Carvalho Chehab wrote: > > Em Sun, 3 Nov 2013 01:22:35 +0100 > > Maik Broemme escreveu: > > > > > I've updated the current DDBridge to latest version 0.9.10 from Ralph > > > Metzler available at: > > > > > > http://www.metzlerbros.de/dddvb/dddvb-0.9.10.tar.bz2 > > > > > > I've merged the driver to work with current v4l/dvb tree and I will > > > maintain the driver for v4l/dvb in future. > > > > Works for me. > > > > > The coming patch series is > > > the first version and I explicitly want to get feedback and hints if > > > some parts are merged at wrong places, etc... The following changes > > > were made: > > > > > > - MSI enabled by default (some issues left with i2c timeouts) > > > - no support for Digital Devices Octonet > > > - no support for DVB Netstream > > > - removed unused module parameters 'tt' and 'vlan' (used by Octonet) > > > - removed unused registers to cleanup code (might be added later again > > > if needed) > > > > Be sure to not remove any feature that are currently needed for the > > already supported devices to work. > > Of course I won't do. The Octonet and DVB Netstream weren't supported in > current driver. MSI is already supported but was not enabled by default > because the old 0.5 version currently in kernel had some problems with > it. However new one works fine with MSI - at least for me I'm using the > patchset myself already - but needs some further testing. Ok. From the above, it sounded that some features got disabled. Please make it clearer on the next patchset (and move/copy this kind of description to the ddbridge patches). With regards to MSI, we may use a quirk to disable (or enable) support for it, on devices where this is know to have troubles. > > > > > > > The following devices are supported by the driver update: > > > > > > - Octopus DVB adapter > > > - Octopus V3 DVB adapter > > > - Octopus LE DVB adapter > > > - Octopus OEM > > > - Octopus Mini > > > - Cine S2 V6 DVB adapter > > > - Cine S2 V6.5 DVB adapter > > > - Octopus CI > > > - Octopus CI single > > > - DVBCT V6.1 DVB adapter > > > - DVB-C modulator > > > - SaTiX-S2 V3 DVB adapter > > > > > > I might merge the Octonet and DVB Netstream drivers from Ralphs source > > > later once the current committed DDBridge driver updates are merged in > > > mainline. > > > > > > Signed-off-by: Maik Broemme > > > > > > Maik Broemme (12): > > > dvb-frontends: Support for DVB-C2 to DVB frontends > > > tda18271c2dd: Fix description of NXP TDA18271C2 silicon tuner > > > stv0367dd: Support for STV 0367 DVB-C/T (DD) demodulator > > > tda18212dd: Support for NXP TDA18212 (DD) silicon tuner > > > cxd2843: Support for CXD2843ER demodulator for DVB-T/T2/C/C2 > > > dvb-core: export dvb_usercopy and new DVB device constants > > > ddbridge: Updated ddbridge registers > > > ddbridge: Moved i2c interfaces into separate file > > > ddbridge: Support for the Digital Devices Resi DVB-C Modulator card > > > ddbridge: Update ddbridge driver to version 0.9.10 > > > ddbridge: Update ddbridge header for 0.9.10 changes > > > ddbridge: Kconfig and Makefile fixes to build latest ddbridge > > > > > > drivers/media/dvb-core/dvbdev.c | 1 > > > drivers/media/dvb-core/dvbdev.h | 2 > > > drivers/media/dvb-frontends/Kconfig | 31 > > > drivers/media/dvb-frontends/Makefile | 3 > > > drivers/media/dvb-frontends/cxd2843.c | 1647 ++++++++++++ > > > drivers/media/dvb-frontends/cxd2843.h | 47 > > > drivers/media/dvb-frontends/stv0367dd.c | 2329 ++++++++++++++++++ > > > drivers/media/dvb-frontends/stv0367dd.h | 48 > > > drivers/media/dvb-frontends/stv0367dd_regs.h | 3442 +++++++++++++++++++++++++++ > > > drivers/media/dvb-frontends/tda18212dd.c | 934 +++++++ > > > drivers/media/dvb-frontends/tda18212dd.h | 37 > > > drivers/media/pci/ddbridge/Kconfig | 21 > > > drivers/media/pci/ddbridge/Makefile | 2 > > > drivers/media/pci/ddbridge/ddbridge-core.c | 3085 +++++++++++++++++------- > > > drivers/media/pci/ddbridge/ddbridge-i2c.c | 239 + > > > drivers/media/pci/ddbridge/ddbridge-mod.c | 1033 ++++++++ > > > drivers/media/pci/ddbridge/ddbridge-regs.h | 273 +- > > > drivers/media/pci/ddbridge/ddbridge.h | 408 ++- > > > include/uapi/linux/dvb/frontend.h | 1 > > > 19 files changed, 12555 insertions(+), 1028 deletions(-) > > > > > > --Maik > > > -- > > > To unsubscribe from this list: send the line "unsubscribe linux-media" in > > > the body of a message to majordomo@vger.kernel.org > > > More majordomo info at http://vger.kernel.org/majordomo-info.html > > > > Thanks for your submission. > > > > I'm seeing this entire patch series as an RFC. There are simply too much > > changes required for us to be able to do a more detailed review on it. > > > > Many thanks for the feedback. > > > Please do the pointed changes, in special: > > > > - Don't let any patch to break compilation; > > > > - Please verify the Documentation/CodingStyle and check it with > > ./scripts/checkpatch.pl; > > > > - Please discuss on a separate thread the API changes for CI, > > modulator and DVB-C2; > > > > - Please try to break the ddbridge changes into one change per > > patch. If not possible, please try to at least break them more, > > to help us to review the changes; > > > > - Please don't duplicate existing drivers without a very very good > > reason. > > > > I will address the concerns with re-submission of the patches. Most > probably it is worth to split the patchset a bit. The CXD2843 > demodulator can also be used by other drivers so I will address your > feedback on this driver first and re-submit. Once it is fine I will > re-send changes for ddbridge. Hope it is a good approach. :) Works for me. > > > Thanks! > > Mauro > > --Maik -- Cheers, Mauro