From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from bombadil.infradead.org ([198.137.202.133]:43640 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752609AbeGDQIh (ORCPT ); Wed, 4 Jul 2018 12:08:37 -0400 Date: Wed, 4 Jul 2018 13:08:31 -0300 From: Mauro Carvalho Chehab To: Daniel Scheller Cc: linux-media@vger.kernel.org, mchehab@kernel.org, mchehab@s-opensource.com, Jonathan Corbet , Andrew Morton Subject: Re: [PATCH v3 0/3] IOCTLs in ddbridge. Message-ID: <20180704130831.1073f094@coco.lan> In-Reply-To: <20180512112432.30887-1-d.scheller.oss@gmail.com> References: <20180512112432.30887-1-d.scheller.oss@gmail.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: Hi Daniel, Em Sat, 12 May 2018 13:24:29 +0200 Daniel Scheller escreveu: > From: Daniel Scheller > > Third iteration of the IOCTL patches for ddbridge, split into multiple > patches: > > Patch 1 just adds the reservation/information of the used IOCTLs into > ioctl-numbers.txt in the Docs dir. Doc, s390 and LKML are Cc'ed on > this patch. Patch looks ok, although it would be great to get some acks there. I don't know who currently maintains Documentation/ioctl/ioctl-number.txt. Just in case, I would explicitly c/c LKML, Andrew Morton and Jonathan Corbet. Please c/c them on a next respin. > Patch 2 adds the header which defines the IOCTLs in include/uapi/ so > userspace applications can directly reuse the IOCTL definitions by > including this file. > > Patch 3 (re)implements the IOCTL handling into ddbridge. This is > basically code that was there since literally forever, but had to be > removed along with the initial ddbridge-0.9.x bump. Also looked ok. What I miss here is a forth patch to Documentation/media/dvb-drivers/, adding a documentation for ddbridge, in special explaining those new ioctls. > The whole functionality gets more important these days since ie. the > new MaxSX8 cards may require updating from time to time since these > cards implement the demod/tuner communication in their FPGA (which > normally I2C drivers exist for). Also, the CineS2v7 and derivatives > received some important updates and the possibility to receive higher > bitrate transponders these days, so users should be able to update > their cards. > > Changes since the last versions: > - Docs, headers and code split apart and sent out separately to > the subsystems. > - Only the two absolutely necessary IOCTLs (DDB_FLASHIO and DDB_ID) > are implemented for now. > > Daniel Scheller (3): > Documentation: ioctl-number: add ddbridge IOCTLs > [media] ddbridge: uAPI header for IOCTL definitions and related data > structs > [media] ddbridge: implement IOCTL handling > > Documentation/ioctl/ioctl-number.txt | 1 + > MAINTAINERS | 1 + > drivers/media/pci/ddbridge/Makefile | 3 +- > drivers/media/pci/ddbridge/ddbridge-core.c | 111 +---------------- > drivers/media/pci/ddbridge/ddbridge-ioctl.c | 179 ++++++++++++++++++++++++++++ > drivers/media/pci/ddbridge/ddbridge-ioctl.h | 32 +++++ > include/uapi/linux/ddbridge-ioctl.h | 61 ++++++++++ > 7 files changed, 278 insertions(+), 110 deletions(-) > create mode 100644 drivers/media/pci/ddbridge/ddbridge-ioctl.c > create mode 100644 drivers/media/pci/ddbridge/ddbridge-ioctl.h > create mode 100644 include/uapi/linux/ddbridge-ioctl.h Thanks, Mauro