| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-Id: <201302260930.07222.hverkuil@xs4all.nl> Date: Tue, 26 Feb 2013 09:30:07 +0100 From: Hans Verkuil <hverkuil@...all.nl> To: Andrey Smirnov <andrew.smirnov@...il.com> Cc: mchehab@...hat.com, sameo@...ux.intel.com, perex@...ex.cz, tiwai@...e.de, linux-media@...r.kernel.org, linux-kernel@...r.kernel.org Subject: Re: [PATCH v5 0/8] Driver for Si476x series of chips On Tue February 26 2013 07:38:46 Andrey Smirnov wrote: > This is a fourth version of the patchset originaly posted here: > https://lkml.org/lkml/2012/9/13/590 > > Second version of the patch was posted here: > https://lkml.org/lkml/2012/10/5/598 > > Third version of the patch was posted here: > https://lkml.org/lkml/2012/10/23/510 > > Fourth version of the patch was posted here: > https://lkml.org/lkml/2013/2/18/572 > > To save everyone's time I'll repost the original description of it: > > This patchset contains a driver for a Silicon Laboratories 476x series > of radio tuners. The driver itself is implemented as an MFD devices > comprised of three parts: > 1. Core device that provides all the other devices with basic > functionality and locking scheme. > 2. Radio device that translates between V4L2 subsystem requests into > Core device commands. > 3. Codec device that does similar to the earlier described task, but > for ALSA SoC subsystem. > > v5 of this driver has following changes: > - Generic controls are converted to standard ones > - Custom controls use a differend offest as base > - Added documentation with controls description Reviewed this. Found some small stuff, although I need a bit more info regarding the diversity mode (see my question in the review of patch 8/8). Depending on that you can likely make a v6 which I can ack. Regards, Hans > > > Andrey Smirnov (8): > mfd: Add header files and Kbuild plumbing for SI476x MFD core > mfd: Add commands abstraction layer for SI476X MFD > mfd: Add the main bulk of core driver for SI476x code > mfd: Add chip properties handling code for SI476X MFD > v4l2: Add standard controls for FM receivers > v4l2: Add documentation for the FM RX controls > v4l2: Add private controls base for SI476X > v4l2: Add a V4L2 driver for SI476X MFD > > Documentation/DocBook/media/v4l/compat.xml | 3 + > Documentation/DocBook/media/v4l/controls.xml | 72 + > .../DocBook/media/v4l/vidioc-g-ext-ctrls.xml | 11 +- > Documentation/video4linux/si476x.txt | 187 +++ > drivers/media/radio/Kconfig | 17 + > drivers/media/radio/Makefile | 1 + > drivers/media/radio/radio-si476x.c | 1581 ++++++++++++++++++++ > drivers/media/v4l2-core/v4l2-ctrls.c | 14 +- > drivers/mfd/Kconfig | 13 + > drivers/mfd/Makefile | 4 + > drivers/mfd/si476x-cmd.c | 1553 +++++++++++++++++++ > drivers/mfd/si476x-i2c.c | 878 +++++++++++ > drivers/mfd/si476x-prop.c | 234 +++ > include/linux/mfd/si476x-core.h | 525 +++++++ > include/media/si476x.h | 426 ++++++ > include/uapi/linux/v4l2-controls.h | 17 +- > 16 files changed, 5531 insertions(+), 5 deletions(-) > create mode 100644 Documentation/video4linux/si476x.txt > create mode 100644 drivers/media/radio/radio-si476x.c > create mode 100644 drivers/mfd/si476x-cmd.c > create mode 100644 drivers/mfd/si476x-i2c.c > create mode 100644 drivers/mfd/si476x-prop.c > create mode 100644 include/linux/mfd/si476x-core.h > create mode 100644 include/media/si476x.h > > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists