| 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
| ||
|
Date: Tue, 18 Dec 2012 17:37:44 -0200
From: Mauro Carvalho Chehab <mchehab@...hat.com>
To: Andrey Smirnov <andrey.smirnov@...vergeddevices.net>
Cc: Hans Verkuil <hverkuil@...all.nl>,
<andrey.smrinov@...vergeddevices.net>, <sameo@...ux.intel.com>,
<broonie@...nsource.wolfsonmicro.com>, <perex@...ex.cz>,
<tiwai@...e.de>, <linux-media@...r.kernel.org>,
<linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v2 1/6] Add header files and Kbuild plumbing for SI476x
MFD core
Em Mon, 8 Oct 2012 11:38:01 -0700
Andrey Smirnov <andrey.smirnov@...vergeddevices.net> escreveu:
> On 10/08/2012 01:43 AM, Hans Verkuil wrote:
> > On Sat October 6 2012 03:54:57 Andrey Smirnov wrote:
> >> This patch adds all necessary header files and Kbuild plumbing for the
> >> core driver for Silicon Laboratories Si476x series of AM/FM tuner
> >> chips.
> >>
> >> The driver as a whole is implemented as an MFD device and this patch
> >> adds a core portion of it that provides all the necessary
> >> functionality to the two other drivers that represent radio and audio
> >> codec subsystems of the chip.
> >>
> >> Signed-off-by: Andrey Smirnov <andrey.smirnov@...vergeddevices.net>
> >> ---
> >> drivers/mfd/Kconfig | 14 ++
> >> drivers/mfd/Makefile | 3 +
> >> include/linux/mfd/si476x-core.h | 529 +++++++++++++++++++++++++++++++++++++++
> >> include/media/si476x.h | 449 +++++++++++++++++++++++++++++++++
> >> 4 files changed, 995 insertions(+)
> >> create mode 100644 include/linux/mfd/si476x-core.h
> >> create mode 100644 include/media/si476x.h
> >>
> >> diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig
> >> index b1a1462..3fab06d 100644
> >> --- a/drivers/mfd/Kconfig
> >> +++ b/drivers/mfd/Kconfig
> >> @@ -895,6 +895,20 @@ config MFD_WL1273_CORE
> >> driver connects the radio-wl1273 V4L2 module and the wl1273
> >> audio codec.
> >>
> >> +config MFD_SI476X_CORE
> >> + tristate "Support for Silicon Laboratories 4761/64/68 AM/FM radio."
> >> + depends on I2C
> >> + select MFD_CORE
> >> + default n
> >> + help
> >> + This is the core driver for the SI476x series of AM/FM radio. This MFD
> >> + driver connects the radio-si476x V4L2 module and the si476x
> >> + audio codec.
> >> +
> >> + To compile this driver as a module, choose M here: the
> >> + module will be called si476x-core.
> >> +
> >> +
> >> config MFD_OMAP_USB_HOST
> >> bool "Support OMAP USBHS core driver"
> >> depends on USB_EHCI_HCD_OMAP || USB_OHCI_HCD_OMAP3
> >> diff --git a/drivers/mfd/Makefile b/drivers/mfd/Makefile
> >> index 79dd22d..942257b 100644
> >> --- a/drivers/mfd/Makefile
> >> +++ b/drivers/mfd/Makefile
> >> @@ -132,3 +132,6 @@ obj-$(CONFIG_MFD_RC5T583) += rc5t583.o rc5t583-irq.o
> >> obj-$(CONFIG_MFD_SEC_CORE) += sec-core.o sec-irq.o
> >> obj-$(CONFIG_MFD_ANATOP) += anatop-mfd.o
> >> obj-$(CONFIG_MFD_LM3533) += lm3533-core.o lm3533-ctrlbank.o
> >> +
> >> +si476x-core-objs := si476x-cmd.o si476x-prop.o si476x-i2c.o
> >> +obj-$(CONFIG_MFD_SI476X_CORE) += si476x-core.o
> >> diff --git a/include/linux/mfd/si476x-core.h b/include/linux/mfd/si476x-core.h
> >> new file mode 100644
> >> index 0000000..eb6f52a
> >> --- /dev/null
> >> +++ b/include/linux/mfd/si476x-core.h
> >> @@ -0,0 +1,529 @@
> >> +/*
> >> + * include/media/si476x-core.h -- Common definitions for si476x core
> >> + * device
> >> + *
> >> + * Copyright (C) 2012 Innovative Converged Devices(ICD)
> >> + *
> >> + * Author: Andrey Smirnov <andrey.smirnov@...vergeddevices.net>
> >> + *
> >> + * This program is free software; you can redistribute it and/or modify
> >> + * it under the terms of the GNU General Public License as published by
> >> + * the Free Software Foundation; version 2 of the License.
> >> + *
> >> + * This program is distributed in the hope that it will be useful, but
> >> + * WITHOUT ANY WARRANTY; without even the implied warranty of
> >> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
> >> + * General Public License for more details.
> >> + *
> >> + */
> >> +
> >> +#ifndef SI476X_CORE_H
> >> +#define SI476X_CORE_H
> >> +
> >> +#include <linux/kfifo.h>
> >> +#include <linux/atomic.h>
> >> +#include <linux/i2c.h>
> >> +#include <linux/mutex.h>
> >> +#include <linux/mfd/core.h>
> >> +#include <linux/videodev2.h>
> >> +
> >> +#include <media/si476x.h>
> >> +
> >> +#ifdef DEBUG
> >> +#define DBG_BUFFER(device, header, buffer, bcount) \
> >> + do { \
> >> + dev_info((device), header); \
> >> + print_hex_dump_bytes("", \
> >> + DUMP_PREFIX_OFFSET, \
> >> + buffer, bcount); \
> >> + } while (0)
> >> +#else
> >> +#define DBG_BUFFER(device, header, buffer, bcount) \
> >> + do {} while (0)
> >> +#endif
> >> +
> >> +enum si476x_freq_suppoted_chips {
> > typo: suppoted -> supported
> >
> >> + SI476X_CHIP_SI4761 = 1,
> >> + SI476X_CHIP_SI4762,
> >> + SI476X_CHIP_SI4763,
> >> + SI476X_CHIP_SI4764,
> >> + SI476X_CHIP_SI4768,
> >> + SI476X_CHIP_SI4769,
> >> +};
> >> +
> >> +enum si476x_mfd_cells {
> >> + SI476X_RADIO_CELL = 0,
> >> + SI476X_CODEC_CELL,
> >> + SI476X_MFD_CELLS,
> >> +};
> >> +
> >> +
> >> +/**
> >> + * enum si476x_power_state - possible power state of the si476x
> >> + * device.
> >> + *
> >> + * @SI476X_POWER_DOWN: In this state all regulators are turned off
> >> + * and the reset line is pulled low. The device is completely
> >> + * inactive.
> >> + * @SI476X_POWER_UP_FULL: In this state all the power regualtors are
> >> + * turned on, reset line pulled high, IRQ line is enabled(polling is
> >> + * active for polling use scenario) and device is turned on with
> >> + * POWER_UP command. The device is ready to be used.
> >> + * @SI476X_POWER_INCONSISTENT: This state indicates that previous
> >> + * power down was inconsisten meaning some of he regulators wer not
> >> + * turned down and thus the consequent use of the device, without
> >> + * power-cycling it is impossible.
> >> + */
> >> +enum si476x_power_state {
> >> + SI476X_POWER_DOWN = 0,
> >> + SI476X_POWER_UP_FULL = 1,
> >> + SI476X_POWER_INCONSISTENT = 2,
> >> +};
> >> +
> >> +/**
> >> + * struct si476x_core - internal data structure representing the
> >> + * underlying "core" device which all the MFD cell-devices use.
> >> + *
> >> + * @client: Actual I2C client used to transfer commands to the chip.
> >> + * @chip_id: Last digit of the chip model(E.g. "1" for SI4761)
> >> + * @cells: MFD cell devices created by this driver.
> >> + * @cmd_lock: Mutex used to serialize all the requests to the core
> >> + * device. This filed should not be used directly. Instead
> >> + * si476x_core_lock()/si476x_core_unlock() should be used to get
> >> + * exclusive access to the "core" device.
> >> + * @users: Active users counter(Used by the radio cell)
> >> + * @rds_read_queue: Wait queue used to wait for RDS data.
> >> + * @rds_fifo: FIFO in which all the RDS data received from the chip is
> >> + * placed.
> >> + * @rds_fifo_drainer: Worker that drains on-chip RDS FIFO.
> >> + * @rds_drainer_is_working: Flag used for launching only one instance
> >> + * of the @rds_fifo_drainer.
> >> + * @rds_drainer_status_lock: Lock used to guard access to the
> >> + * @rds_drainer_is_working variable.
> >> + * @command: Wait queue for wainting on the command comapletion.
> >> + * @cts: Clear To Send flag set upon receiving first status with CTS
> >> + * set.
> >> + * @tuning: Wait queue used for wainting for tune/seek comand
> >> + * completion.
> >> + * @stc: Similar to @cts, but for the STC bit of the status value.
> >> + * @power_up_parameters: Parameters used as argument for POWER_UP
> >> + * command when the device is started.
> >> + * @state: Current power state of the device.
> >> + * @supplues: Structure containing handles to all power supplies used
> >> + * by the device (NULL ones are ignored).
> >> + * @gpio_reset: GPIO pin connectet to the RSTB pin of the chip.
> >> + * @pinmux: Chip's configurable pins configuration.
> >> + * @diversity_mode: Chips role when functioning in diversity mode.
> >> + * @status_monitor: Polling worker used in polling use case scenarion
> >> + * (when IRQ is not avalible).
> >> + * @revision: Chip's running firmware revision number(Used for correct
> >> + * command set support).
> >> + */
> >> +
> >> +struct si476x_core {
> >> + struct i2c_client *client;
> >> + int chip_id;
> >> + struct mfd_cell cells[SI476X_MFD_CELLS];
> >> +
> >> + struct mutex cmd_lock; /* for serializing fm radio operations */
> >> + atomic_t users;
> >> +
> >> + wait_queue_head_t rds_read_queue;
> >> + struct kfifo rds_fifo;
> >> + struct work_struct rds_fifo_drainer;
> >> + bool rds_drainer_is_working;
> >> + struct mutex rds_drainer_status_lock;
> >> +
> >> +
> >> + wait_queue_head_t command;
> >> + atomic_t cts;
> >> +
> >> + wait_queue_head_t tuning;
> >> + atomic_t stc;
> >> +
> >> + struct si476x_power_up_args power_up_parameters;
> >> +
> >> + enum si476x_power_state power_state;
> >> +
> >> + struct {
> >> + struct regulator *vio1;
> >> + struct regulator *vd;
> >> + struct regulator *va;
> >> + struct regulator *vio2;
> >> + } supplies;
> >> +
> >> + int gpio_reset;
> >> +
> >> + struct si476x_pinmux pinmux;
> >> + enum si476x_phase_diversity_mode diversity_mode;
> >> +
> >> + atomic_t is_alive;
> >> +
> >> + struct delayed_work status_monitor;
> >> +#define SI476X_WORK_TO_CORE(w) container_of(to_delayed_work(w), \
> >> + struct si476x_core, \
> >> + status_monitor)
> >> +
> >> + int revision;
> >> +
> >> + int rds_fifo_depth;
> >> +
> >> + struct {
> >> + atomic_t tune;
> >> + atomic_t power_up;
> >> + atomic_t command;
> >> + } timeouts;
> >> +
> >> + atomic_t polling_interval;
> >> +};
> >> +
> >> +static inline struct si476x_core *i2c_mfd_cell_to_core(struct device *dev)
> >> +{
> >> + struct i2c_client *client = to_i2c_client(dev->parent);
> >> + return i2c_get_clientdata(client);
> >> +}
> >> +
> >> +
> >> +/**
> >> + * si476x_core_lock() - lock the core device to get an exclusive acces
> > acces -> access
> >
> >> + * to it.
> >> + */
> >> +static inline void si476x_core_lock(struct si476x_core *core)
> >> +{
> >> + mutex_lock(&core->cmd_lock);
> >> +}
> >> +
> >> +/**
> >> + * si476x_core_unlock() - unlock the core device to relinquish an
> >> + * exclusive acces to it.
> > Ditto
> >
> >> + */
> >> +static inline void si476x_core_unlock(struct si476x_core *core)
> >> +{
> >> + mutex_unlock(&core->cmd_lock);
> >> +}
> >> +
> >> +void si476x_core_get(struct si476x_core *core);
> >> +void si476x_core_put(struct si476x_core *core);
> >> +
> >> +
> >> +/* *_TUNE_FREQ family of commands accept frequency in multiples of
> >> + 10kHz */
> >> +static inline u16 hz_to_si476x(struct si476x_core *core, int freq)
> >> +{
> >> + u16 result;
> >> +
> >> + switch (core->power_up_parameters.func) {
> >> + default:
> >> + case SI476X_FUNC_FM_RECEIVER:
> >> + result = freq / 10000;
> >> + break;
> >> + case SI476X_FUNC_AM_RECEIVER:
> >> + result = freq / 1000;
> >> + break;
> >> + }
> >> +
> >> + return result;
> >> +}
> >> +
> >> +static inline int si476x_to_hz(struct si476x_core *core, u16 freq)
> >> +{
> >> + int result;
> >> +
> >> + switch (core->power_up_parameters.func) {
> >> + default:
> >> + case SI476X_FUNC_FM_RECEIVER:
> >> + result = freq * 10000;
> >> + break;
> >> + case SI476X_FUNC_AM_RECEIVER:
> >> + result = freq * 1000;
> >> + break;
> >> + }
> >> +
> >> + return result;
> >> +}
> >> +
> >> +/* Since the V4L2_TUNER_CAP_LOW flag is supplied, V4L2 subsystem
> >> + * mesures frequency in 62.5 Hz units */
> >> +
> >> +static inline int hz_to_v4l2(int freq)
> >> +{
> >> + return (freq * 10) / 625;
> >> +}
> >> +
> >> +static inline int v4l2_to_hz(int freq)
> >> +{
> >> + return (freq * 625) / 10;
> >> +}
> >> +
> >> +static inline u16 v4l2_to_si476x(struct si476x_core *core, int freq)
> >> +{
> >> + return hz_to_si476x(core, v4l2_to_hz(freq));
> >> +}
> >> +
> >> +static inline int si476x_to_v4l2(struct si476x_core *core, u16 freq)
> >> +{
> >> + return hz_to_v4l2(si476x_to_hz(core, freq));
> >> +}
> >> +
> >> +
> >> +
> >> +/**
> >> + * struct si476x_func_info - structure containing result of the
> >> + * FUNC_INFO command.
> >> + *
> >> + * @firmware.major: Firmare major number.
> > Firmare -> Firmware
> >
> >> + * @firmware.minor[...]: Firmare minor numbers.
> > ditto
> >
> >> + * @patch_id:
> >> + * @func: Mode tuner is working in.
> >> + */
> >> +struct si476x_func_info {
> >> + struct {
> >> + u8 major, minor[2];
> >> + } firmware;
> >> + u16 patch_id;
> >> + enum si476x_func func;
> >> +};
> >> +
> >> +/**
> >> + * struct si476x_power_down_args - structure used to pass parameters
> >> + * to POWER_DOWN command
> >> + *
> >> + * @xosc: true - Power down, but leav oscillator running.
> >> + * false - Full power down.
> >> + */
> >> +struct si476x_power_down_args {
> >> + bool xosc;
> >> +};
> >> +
> >> +/**
> >> + * enum si476x_tunemode - enum representing possible tune modes for
> >> + * the chip.
> >> + * @SI476X_TM_VALIDATED_NORMAL_TUNE: Unconditionally stay on the new
> >> + * channel after tune, tune status is valid.
> >> + * @SI476X_TM_INVALIDATED_FAST_TUNE: Unconditionally stay in the new
> >> + * channel after tune, tune status invalid.
> >> + * @SI476X_TM_VALIDATED_AF_TUNE: Jump back to previous channel if
> >> + * metric thresholds are not met.
> >> + * @SI476X_TM_VALIDATED_AF_CHECK: Unconditionally jump back to the
> >> + * previous channel.
> >> + */
> >> +enum si476x_tunemode {
> >> + SI476X_TM_VALIDATED_NORMAL_TUNE = 0,
> >> + SI476X_TM_INVALIDATED_FAST_TUNE = 1,
> >> + SI476X_TM_VALIDATED_AF_TUNE = 2,
> >> + SI476X_TM_VALIDATED_AF_CHECK = 3,
> >> +};
> >> +
> >> +/**
> >> + * enum si476x_smoothmetrics - enum containing the possible setting fo
> >> + * audio transitioning of the chip
> >> + * @SI476X_SM_INITIALIZE_AUDIO: Initialize audio state to match this
> >> + * new channel
> >> + * @SI476X_SM_TRANSITION_AUDIO: Transition audio state from previous
> >> + * channel values to the new values
> >> + */
> >> +enum si476x_smoothmetrics {
> >> + SI476X_SM_INITIALIZE_AUDIO = 0,
> >> + SI476X_SM_TRANSITION_AUDIO = 1,
> >> +};
> >> +
> >> +/**
> >> + * struct si476x_rds_status_report - the structure representing the
> >> + * response to 'FM_RD_STATUS' command
> >> + * @rdstpptyint: Traffic program flag(TP) and/or program type(PTY)
> >> + * code has changed.
> >> + * @rdspiint: Program indentifiaction(PI) code has changed.
> >> + * @rdssyncint: RDS synchronization has changed.
> >> + * @rdsfifoint: RDS was received and the RDS FIFO has at least
> >> + * 'FM_RDS_INTERRUPT_FIFO_COUNT' elements in it.
> >> + * @tpptyvalid: TP flag and PTY code are valid falg.
> >> + * @pivalid: PI code is valid flag.
> >> + * @rdssync: RDS is currently synchronized.
> >> + * @rdsfifolost: On or more RDS groups have been lost/discarded flag.
> >> + * @tp: Current channel's TP flag.
> >> + * @pty: Current channel's PTY code.
> >> + * @pi: Current channel's PI code.
> >> + * @rdsfifoused: Number of blocks remaining in the RDS FIFO (0 if
> >> + * empty).
> >> + */
> >> +struct si476x_rds_status_report {
> >> + bool rdstpptyint, rdspiint, rdssyncint, rdsfifoint;
> >> + bool tpptyvalid, pivalid, rdssync, rdsfifolost;
> >> + bool tp;
> >> +
> >> + u8 pty;
> >> + u16 pi;
> >> +
> >> + u8 rdsfifoused;
> >> + u8 ble[4];
> >> +
> >> + struct v4l2_rds_data rds[4];
> >> +};
> >> +
> >> +struct si476x_rsq_status_args {
> >> + bool primary;
> >> + bool rsqack;
> >> + bool attune;
> >> + bool cancel;
> >> + bool stcack;
> >> +};
> >> +
> >> +enum si476x_injside {
> >> + SI476X_INJSIDE_AUTO = 0,
> >> + SI476X_INJSIDE_LOW = 1,
> >> + SI476X_INJSIDE_HIGH = 2,
> >> +};
> >> +
> >> +struct si476x_tune_freq_args {
> >> + bool zifsr;
> >> + bool hd;
> >> + enum si476x_injside injside;
> >> + int freq;
> >> + enum si476x_tunemode tunemode;
> >> + enum si476x_smoothmetrics smoothmetrics;
> >> + int antcap;
> >> +};
> >> +
> >> +int si476x_core_stop(struct si476x_core *, bool);
> >> +int si476x_core_start(struct si476x_core *, bool);
> >> +int si476x_core_set_power_state(struct si476x_core *, enum si476x_power_state);
> >> +int si476x_core_cmd_func_info(struct si476x_core *, struct si476x_func_info *);
> >> +int si476x_core_cmd_set_property(struct si476x_core *, u16, u16);
> >> +int si476x_core_cmd_get_property(struct si476x_core *, u16);
> >> +int si476x_core_cmd_dig_audio_pin_cfg(struct si476x_core *,
> >> + enum si476x_dclk_config,
> >> + enum si476x_dfs_config,
> >> + enum si476x_dout_config,
> >> + enum si476x_xout_config);
> >> +int si476x_core_cmd_zif_pin_cfg(struct si476x_core *,
> >> + enum si476x_iqclk_config,
> >> + enum si476x_iqfs_config,
> >> + enum si476x_iout_config,
> >> + enum si476x_qout_config);
> >> +int si476x_core_cmd_ic_link_gpo_ctl_pin_cfg(struct si476x_core *,
> >> + enum si476x_icin_config,
> >> + enum si476x_icip_config,
> >> + enum si476x_icon_config,
> >> + enum si476x_icop_config);
> >> +int si476x_core_cmd_ana_audio_pin_cfg(struct si476x_core *,
> >> + enum si476x_lrout_config);
> >> +int si476x_core_cmd_intb_pin_cfg(struct si476x_core *, enum si476x_intb_config,
> >> + enum si476x_a1_config);
> >> +int si476x_core_cmd_fm_seek_start(struct si476x_core *, bool, bool);
> >> +int si476x_core_cmd_am_seek_start(struct si476x_core *, bool, bool);
> >> +int si476x_core_cmd_fm_rds_status(struct si476x_core *, bool, bool, bool,
> >> + struct si476x_rds_status_report *);
> >> +int si476x_core_cmd_fm_rds_blockcount(struct si476x_core *, bool,
> >> + struct si476x_rds_blockcount_report *);
> >> +int si476x_core_cmd_fm_tune_freq(struct si476x_core *,
> >> + struct si476x_tune_freq_args *);
> >> +int si476x_core_cmd_am_tune_freq(struct si476x_core *,
> >> + struct si476x_tune_freq_args *);
> >> +int si476x_core_cmd_am_rsq_status(struct si476x_core *,
> >> + struct si476x_rsq_status_args *,
> >> + struct si476x_rsq_status_report *);
> >> +int si476x_core_cmd_fm_rsq_status(struct si476x_core *,
> >> + struct si476x_rsq_status_args *,
> >> + struct si476x_rsq_status_report *);
> >> +int si476x_core_cmd_power_up(struct si476x_core *,
> >> + struct si476x_power_up_args *);
> >> +int si476x_core_cmd_power_down(struct si476x_core *,
> >> + struct si476x_power_down_args *);
> >> +int si476x_core_cmd_fm_phase_div_status(struct si476x_core *);
> >> +int si476x_core_cmd_fm_phase_diversity(struct si476x_core *,
> >> + enum si476x_phase_diversity_mode);
> >> +
> >> +int si476x_core_cmd_fm_acf_status(struct si476x_core *,
> >> + struct si476x_acf_status_report *);
> >> +int si476x_core_cmd_am_acf_status(struct si476x_core *,
> >> + struct si476x_acf_status_report *);
> >> +int si476x_core_cmd_agc_status(struct si476x_core *,
> >> + struct si476x_agc_status_report *);
> >> +
> >> +enum si476x_power_grid_type {
> >> + SI476X_POWER_GRID_50HZ = 0,
> >> + SI476X_POWER_GRID_60HZ,
> >> +};
> >> +
> >> +/* Properties */
> >> +
> >> +enum si476x_interrupt_flags {
> >> + SI476X_STCIEN = (1 << 0),
> >> + SI476X_ACFIEN = (1 << 1),
> >> + SI476X_RDSIEN = (1 << 2),
> >> + SI476X_RSQIEN = (1 << 3),
> >> +
> >> + SI476X_ERRIEN = (1 << 6),
> >> + SI476X_CTSIEN = (1 << 7),
> >> +
> >> + SI476X_STCREP = (1 << 8),
> >> + SI476X_ACFREP = (1 << 9),
> >> + SI476X_RDSREP = (1 << 10),
> >> + SI476X_RSQREP = (1 << 11),
> >> +};
> >> +
> >> +enum si476x_rdsint_sources {
> >> + SI476X_RDSTPPTY = (1 << 4),
> >> + SI476X_RDSPI = (1 << 3),
> >> + SI476X_RDSSYNC = (1 << 1),
> >> + SI476X_RDSRECV = (1 << 0),
> >> +};
> >> +
> >> +enum si476x_status_response_bits {
> >> + SI476X_CTS = (1 << 7),
> >> + SI476X_ERR = (1 << 6),
> >> + /* Status response for WB receiver */
> >> + SI476X_WB_ASQ_INT = (1 << 4),
> >> + SI476X_RSQ_INT = (1 << 3),
> >> + /* Status response for FM receiver */
> >> + SI476X_FM_RDS_INT = (1 << 2),
> >> + SI476X_ACF_INT = (1 << 1),
> >> + SI476X_STC_INT = (1 << 0),
> >> +};
> >> +
> >> +bool si476x_core_is_valid_property(struct si476x_core *, u16);
> >> +bool si476x_core_is_readonly_property(struct si476x_core *, u16);
> >> +int si476x_core_set_int_ctl_enable(struct si476x_core *,
> >> + enum si476x_interrupt_flags);
> >> +
> >> +int si476x_core_set_frequency_spacing(struct si476x_core *, int);
> >> +int si476x_core_set_seek_band_top(struct si476x_core *, int);
> >> +int si476x_core_set_seek_band_bottom(struct si476x_core *, int);
> >> +int si476x_core_set_audio_deemphasis(struct si476x_core *, int);
> >> +int si476x_core_set_rds_reception(struct si476x_core *, int);
> >> +int si476x_core_set_audio_pwr_line_filter(struct si476x_core *, bool,
> >> + enum si476x_power_grid_type, int);
> >> +
> >> +int si476x_core_set_valid_snr_threshold(struct si476x_core *, int);
> >> +int si476x_core_set_valid_rssi_threshold(struct si476x_core *, int);
> >> +int si476x_core_set_valid_max_tune_error(struct si476x_core *, int);
> >> +
> >> +int si476x_core_get_frequency_spacing(struct si476x_core *);
> >> +int si476x_core_get_seek_band_top(struct si476x_core *);
> >> +int si476x_core_get_seek_band_bottom(struct si476x_core *);
> >> +int si476x_core_get_audio_deemphasis(struct si476x_core *);
> >> +int si476x_core_get_rds_reception(struct si476x_core *);
> >> +int si476x_core_get_audio_pwr_line_filter(struct si476x_core *);
> >> +
> >> +int si476x_core_get_valid_snr_threshold(struct si476x_core *);
> >> +int si476x_core_get_valid_rssi_threshold(struct si476x_core *);
> >> +int si476x_core_get_valid_max_tune_error(struct si476x_core *);
> >> +
> >> +int si476x_core_set_fm_rds_interrupt_fifo_count(struct si476x_core *, int);
> >> +int si476x_core_set_rds_interrupt_source(struct si476x_core *,
> >> + enum si476x_rdsint_sources);
> >> +int si476x_core_set_digital_io_input_sample_rate(struct si476x_core *, u16);
> >> +int si476x_core_disable_digital_audio(struct si476x_core *);
> >> +
> >> +typedef int (*tune_freq_func_t) (struct si476x_core *,
> >> + struct si476x_tune_freq_args *);
> >> +
> >> +enum si476x_i2c_type {
> >> + SI476X_I2C_SEND,
> >> + SI476X_I2C_RECV
> >> +};
> >> +
> >> +int si476x_i2c_xfer(struct si476x_core *,
> >> + enum si476x_i2c_type,
> >> + char *, int);
> >> +#endif /* SI476X_CORE_H */
> >> diff --git a/include/media/si476x.h b/include/media/si476x.h
> >> new file mode 100644
> >> index 0000000..6f6c253
> >> --- /dev/null
> >> +++ b/include/media/si476x.h
> >> @@ -0,0 +1,449 @@
> >> +/*
> >> + * include/media/si476x.h -- Common definitions for si476x driver
> >> + *
> >> + * Copyright (C) 2012 Innovative Converged Devices(ICD)
> >> + *
> >> + * Author: Andrey Smirnov <andrey.smirnov@...vergeddevices.net>
> >> + *
> >> + * This program is free software; you can redistribute it and/or modify
> >> + * it under the terms of the GNU General Public License as published by
> >> + * the Free Software Foundation; version 2 of the License.
> >> + *
> >> + * This program is distributed in the hope that it will be useful, but
> >> + * WITHOUT ANY WARRANTY; without even the implied warranty of
> >> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
> >> + * General Public License for more details.
> >> + *
> >> + */
> >> +
> >> +#ifndef SI476X_H
> >> +#define SI476X_H
> >> +
> >> +#include <linux/types.h>
> >> +#include <linux/videodev2.h>
> >> +
> >> +struct si476x_device;
> >> +
> >> +/* It is possible to select one of the four adresses using pins A0
> >> + * and A1 on SI476x */
> >> +#define SI476X_I2C_ADDR_1 0x60
> >> +#define SI476X_I2C_ADDR_2 0x61
> >> +#define SI476X_I2C_ADDR_3 0x62
> >> +#define SI476X_I2C_ADDR_4 0x63
> >> +
> >> +enum si476x_iqclk_config {
> >> + SI476X_IQCLK_NOOP = 0,
> >> + SI476X_IQCLK_TRISTATE = 1,
> >> + SI476X_IQCLK_IQ = 21,
> >> +};
> >> +enum si476x_iqfs_config {
> >> + SI476X_IQFS_NOOP = 0,
> >> + SI476X_IQFS_TRISTATE = 1,
> >> + SI476X_IQFS_IQ = 21,
> >> +};
> >> +enum si476x_iout_config {
> >> + SI476X_IOUT_NOOP = 0,
> >> + SI476X_IOUT_TRISTATE = 1,
> >> + SI476X_IOUT_OUTPUT = 22,
> >> +};
> >> +enum si476x_qout_config {
> >> + SI476X_QOUT_NOOP = 0,
> >> + SI476X_QOUT_TRISTATE = 1,
> >> + SI476X_QOUT_OUTPUT = 22,
> >> +};
> >> +
> >> +enum si476x_dclk_config {
> >> + SI476X_DCLK_NOOP = 0,
> >> + SI476X_DCLK_TRISTATE = 1,
> >> + SI476X_DCLK_DAUDIO = 10,
> >> +};
> >> +
> >> +enum si476x_dfs_config {
> >> + SI476X_DFS_NOOP = 0,
> >> + SI476X_DFS_TRISTATE = 1,
> >> + SI476X_DFS_DAUDIO = 10,
> >> +};
> >> +
> >> +enum si476x_dout_config {
> >> + SI476X_DOUT_NOOP = 0,
> >> + SI476X_DOUT_TRISTATE = 1,
> >> + SI476X_DOUT_I2S_OUTPUT = 12,
> >> + SI476X_DOUT_I2S_INPUT = 13,
> >> +};
> >> +
> >> +enum si476x_xout_config {
> >> + SI476X_XOUT_NOOP = 0,
> >> + SI476X_XOUT_TRISTATE = 1,
> >> + SI476X_XOUT_I2S_INPUT = 13,
> >> + SI476X_XOUT_MODE_SELECT = 23,
> >> +};
> >> +
> >> +
> >> +enum si476x_icin_config {
> >> + SI476X_ICIN_NOOP = 0,
> >> + SI476X_ICIN_TRISTATE = 1,
> >> + SI476X_ICIN_GPO1_HIGH = 2,
> >> + SI476X_ICIN_GPO1_LOW = 3,
> >> + SI476X_ICIN_IC_LINK = 30,
> >> +};
> >> +
> >> +enum si476x_icip_config {
> >> + SI476X_ICIP_NOOP = 0,
> >> + SI476X_ICIP_TRISTATE = 1,
> >> + SI476X_ICIP_GPO2_HIGH = 2,
> >> + SI476X_ICIP_GPO2_LOW = 3,
> >> + SI476X_ICIP_IC_LINK = 30,
> >> +};
> >> +
> >> +enum si476x_icon_config {
> >> + SI476X_ICON_NOOP = 0,
> >> + SI476X_ICON_TRISTATE = 1,
> >> + SI476X_ICON_I2S = 10,
> >> + SI476X_ICON_IC_LINK = 30,
> >> +};
> >> +
> >> +enum si476x_icop_config {
> >> + SI476X_ICOP_NOOP = 0,
> >> + SI476X_ICOP_TRISTATE = 1,
> >> + SI476X_ICOP_I2S = 10,
> >> + SI476X_ICOP_IC_LINK = 30,
> >> +};
> >> +
> >> +
> >> +enum si476x_lrout_config {
> >> + SI476X_LROUT_NOOP = 0,
> >> + SI476X_LROUT_TRISTATE = 1,
> >> + SI476X_LROUT_AUDIO = 2,
> >> + SI476X_LROUT_MPX = 3,
> >> +};
> >> +
> >> +
> >> +enum si476x_intb_config {
> >> + SI476X_INTB_NOOP = 0,
> >> + SI476X_INTB_TRISTATE = 1,
> >> + SI476X_INTB_DAUDIO = 10,
> >> + SI476X_INTB_IRQ = 40,
> >> +};
> >> +
> >> +enum si476x_a1_config {
> >> + SI476X_A1_NOOP = 0,
> >> + SI476X_A1_TRISTATE = 1,
> >> + SI476X_A1_IRQ = 40,
> >> +};
> >> +
> >> +enum si476x_part_revisions {
> >> + SI476X_REVISION_A10 = 0,
> >> + SI476X_REVISION_A20 = 1,
> >> + SI476X_REVISION_A30 = 2,
> >> +};
> >> +
> >> +struct si476x_pinmux {
> >> + enum si476x_dclk_config dclk;
> >> + enum si476x_dfs_config dfs;
> >> + enum si476x_dout_config dout;
> >> + enum si476x_xout_config xout;
> >> +
> >> + enum si476x_iqclk_config iqclk;
> >> + enum si476x_iqfs_config iqfs;
> >> + enum si476x_iout_config iout;
> >> + enum si476x_qout_config qout;
> >> +
> >> + enum si476x_icin_config icin;
> >> + enum si476x_icip_config icip;
> >> + enum si476x_icon_config icon;
> >> + enum si476x_icop_config icop;
> >> +
> >> + enum si476x_lrout_config lrout;
> >> +
> >> + enum si476x_intb_config intb;
> >> + enum si476x_a1_config a1;
> >> +};
> >> +
> >> +/**
> >> + * enum si476x_phase_diversity_mode - possbile phase diversity modes
> >> + * for SI4764/5/6/7 chips.
> >> + *
> >> + * @SI476X_PHDIV_DISABLED: Phase diversity feature is
> >> + * disabled.
> >> + * @SI476X_PHDIV_PRIMARY_COMBINING: Tuner works as a primary tuner
> >> + * in combination with a
> >> + * secondary one.
> >> + * @SI476X_PHDIV_PRIMARY_ANTENNA: Tuner works as a primary tuner
> >> + * using only its own antenna.
> >> + * @SI476X_PHDIV_SECONDARY_ANTENNA: Tuner works as a primary tuner
> >> + * usning seconary tuner's antenna.
> >> + * @SI476X_PHDIV_SECONDARY_COMBINING: Tuner works as a secondary
> >> + * tuner in combination with the
> >> + * primary one.
> >> + */
> >> +enum si476x_phase_diversity_mode {
> >> + SI476X_PHDIV_DISABLED = 0,
> >> + SI476X_PHDIV_PRIMARY_COMBINING = 1,
> >> + SI476X_PHDIV_PRIMARY_ANTENNA = 2,
> >> + SI476X_PHDIV_SECONDARY_ANTENNA = 3,
> >> + SI476X_PHDIV_SECONDARY_COMBINING = 5,
> >> +};
> >> +
> >> +enum si476x_ibias6x {
> >> + SI476X_IBIAS6X_OTHER = 0,
> >> + SI476X_IBIAS6X_RCVR1_NON_4MHZ_CLK = 1,
> >> +};
> >> +
> >> +enum si476x_xstart {
> >> + SI476X_XSTART_MULTIPLE_TUNER = 0x11,
> >> + SI476X_XSTART_NORMAL = 0x77,
> >> +};
> >> +
> >> +enum si476x_freq {
> >> + SI476X_FREQ_4_MHZ = 0,
> >> + SI476X_FREQ_37P209375_MHZ = 1,
> >> + SI476X_FREQ_36P4_MHZ = 2,
> >> + SI476X_FREQ_37P8_MHZ = 3,
> >> +};
> >> +
> >> +enum si476x_xmode {
> >> + SI476X_XMODE_CRYSTAL_RCVR1 = 1,
> >> + SI476X_XMODE_EXT_CLOCK = 2,
> >> + SI476X_XMODE_CRYSTAL_RCVR2_3 = 3,
> >> +};
> >> +
> >> +enum si476x_xbiashc {
> >> + SI476X_XBIASHC_SINGLE_RECEIVER = 0,
> >> + SI476X_XBIASHC_MULTIPLE_RECEIVER = 1,
> >> +};
> >> +
> >> +enum si476x_xbias {
> >> + SI476X_XBIAS_RCVR2_3 = 0,
> >> + SI476X_XBIAS_4MHZ_RCVR1 = 3,
> >> + SI476X_XBIAS_RCVR1 = 7,
> >> +};
> >> +
> >> +enum si476x_func {
> >> + SI476X_FUNC_BOOTLOADER = 0,
> >> + SI476X_FUNC_FM_RECEIVER = 1,
> >> + SI476X_FUNC_AM_RECEIVER = 2,
> >> + SI476X_FUNC_WB_RECEIVER = 3,
> >> +};
> >> +
> >> +
> >> +/**
> >> + * @xcload: Selects the amount of additional on-chip capacitance to
> >> + * be connected between XTAL1 and gnd and between XTAL2 and
> >> + * GND. One half of the capacitance value shown here is the
> >> + * additional load capacitance presented to the xtal. The
> >> + * minimum step size is 0.277 pF. Recommended value is 0x28
> >> + * but it will be layout dependent. Range is 0–0x3F i.e.
> >> + * (0–16.33 pF)
> >> + * @ctsien: enable CTSINT(interrupt request when CTS condition
> >> + * arises) when set
> >> + * @intsel: when set A1 pin becomes the interrupt pin; otherwise,
> >> + * INTB is the interrupt pin
> >> + * @func: selects the boot function of the device. I.e.
> >> + * SI476X_BOOTLOADER - Boot loader
> >> + * SI476X_FM_RECEIVER - FM receiver
> >> + * SI476X_AM_RECEIVER - AM receiver
> >> + * SI476X_WB_RECEIVER - Weatherband receiver
> >> + * @freq: oscillator's crystal frequency:
> >> + * SI476X_XTAL_37P209375_MHZ - 37.209375 Mhz
> >> + * SI476X_XTAL_36P4_MHZ - 36.4 Mhz
> >> + * SI476X_XTAL_37P8_MHZ - 37.8 Mhz
> >> + */
> >> +struct si476x_power_up_args {
> >> + enum si476x_ibias6x ibias6x;
> >> + enum si476x_xstart xstart;
> >> + u8 xcload;
> >> + bool fastboot;
> >> + enum si476x_xbiashc xbiashc;
> >> + enum si476x_xbias xbias;
> >> + enum si476x_func func;
> >> + enum si476x_freq freq;
> >> + enum si476x_xmode xmode;
> >> +};
> >> +
> >> +
> >> +enum si476x_ctrl_id {
> >> + SI476X_CID_RSSI_THRESHOLD = (V4L2_CID_USER_BASE | 0x1001),
> >> + SI476X_CID_SNR_THRESHOLD = (V4L2_CID_USER_BASE | 0x1002),
> >> + SI476X_CID_MAX_TUNE_ERROR = (V4L2_CID_USER_BASE | 0x1003),
> >> + SI476X_CID_RDS_RECEPTION = (V4L2_CID_USER_BASE | 0x1004),
> >> + SI476X_CID_DEEMPHASIS = (V4L2_CID_USER_BASE | 0x1005),
> >> + SI476X_CID_HARMONICS_COUNT = (V4L2_CID_USER_BASE | 0x1006),
> >> +};
> > What do these controls do? Should they be standard controls instead?
> >
> >> +
> >> +/*
> >> + * Platform dependent definition
> >> + */
> >> +struct si476x_platform_data {
> >> + int gpio_reset; /* < 0 if not used */
> >> +
> >> + struct si476x_power_up_args power_up_parameters;
> >> + enum si476x_phase_diversity_mode diversity_mode;
> >> +
> >> + struct si476x_pinmux pinmux;
> >> +};
> >> +
> >> +/**
> >> + * struct si476x_rsq_status - structure containing received signal
> >> + * quality
> >> + * @multhint: Multipath Detect High.
> >> + * true - Indicatedes that the value is below
> >> + * FM_RSQ_MULTIPATH_HIGH_THRESHOLD
> >> + * false - Indicatedes that the value is above
> >> + * FM_RSQ_MULTIPATH_HIGH_THRESHOLD
> >> + * @multlint: Multipath Detect Low.
> >> + * true - Indicatedes that the value is below
> >> + * FM_RSQ_MULTIPATH_LOW_THRESHOLD
> >> + * false - Indicatedes that the value is above
> >> + * FM_RSQ_MULTIPATH_LOW_THRESHOLD
> >> + * @snrhint: SNR Detect High.
> >> + * true - Indicatedes that the value is below
> >> + * FM_RSQ_SNR_HIGH_THRESHOLD
> >> + * false - Indicatedes that the value is above
> >> + * FM_RSQ_SNR_HIGH_THRESHOLD
> >> + * @snrlint: SNR Detect Low.
> >> + * true - Indicatedes that the value is below
> >> + * FM_RSQ_SNR_LOW_THRESHOLD
> >> + * false - Indicatedes that the value is above
> >> + * FM_RSQ_SNR_LOW_THRESHOLD
> >> + * @rssihint: RSSI Detect High.
> >> + * true - Indicatedes that the value is below
> >> + * FM_RSQ_RSSI_HIGH_THRESHOLD
> >> + * false - Indicatedes that the value is above
> >> + * FM_RSQ_RSSI_HIGH_THRESHOLD
> >> + * @rssilint: RSSI Detect Low.
> >> + * true - Indicatedes that the value is below
> >> + * FM_RSQ_RSSI_LOW_THRESHOLD
> >> + * false - Indicatedes that the value is above
> >> + * FM_RSQ_RSSI_LOW_THRESHOLD
> >> + * @bltf: Band Limit.
> >> + * Set if seek command hits the band limit or wrapped to
> >> + * the original frequency.
> >> + * @snr_ready: SNR measurement in progress.
> >> + * @rssiready: RSSI measurement in progress.
> >> + * @afcrl: Set if FREQOFF >= MAX_TUNE_ERROR
> >> + * @valid: Set if the channel is valid
> >> + * rssi < FM_VALID_RSSI_THRESHOLD
> >> + * snr < FM_VALID_SNR_THRESHOLD
> >> + * tune_error < FM_VALID_MAX_TUNE_ERROR
> >> + * @readfreq: Current tuned frequency.
> >> + * @freqoff: Signed frequency offset.
> >> + * @rssi: Received Signal Strength Indicator(dBuV).
> >> + * @snr: RF SNR Indicator(dB).
> >> + * @lassi:
> >> + * @hassi: Low/High side Adjacent(100 kHz) Channel Strength Indicator
> >> + * @mult: Multipath indicator
> >> + * @dev: Who knows? But values may vary.
> >> + * @readantcap: Antenna tuning capacity value.
> >> + * @assi: Adjacent Channel(+/- 200kHz) Strength Indicator
> >> + * @usn: Ultrasonic Noise Inticator in -DBFS
> >> + */
> >> +struct si476x_rsq_status_report {
> >> + __u8 multhint, multlint;
> >> + __u8 snrhint, snrlint;
> >> + __u8 rssihint, rssilint;
> >> + __u8 bltf;
> >> + __u8 snr_ready;
> >> + __u8 rssiready;
> >> + __u8 injside;
> >> + __u8 afcrl;
> >> + __u8 valid;
> >> +
> >> + __u16 readfreq;
> >> + __s8 freqoff;
> >> + __s8 rssi;
> >> + __s8 snr;
> >> + __s8 issi;
> >> + __s8 lassi, hassi;
> >> + __s8 mult;
> >> + __u8 dev;
> >> + __u16 readantcap;
> >> + __s8 assi;
> >> + __s8 usn;
> >> +
> >> + __u8 pilotdev;
> >> + __u8 rdsdev;
> >> + __u8 assidev;
> >> + __u8 strongdev;
> >> + __u16 rdspi;
> >> +};
> >> +
> >> +/**
> >> + * si476x_acf_status_report - ACF report results
> >> + *
> >> + * @blend_int: If set, indicates that stereo separation has crossed
> >> + * below the blend threshold as set by FM_ACF_BLEND_THRESHOLD
> >> + * @hblend_int: If set, indicates that HiBlend cutoff frequency is
> >> + * lower than threshold as set by FM_ACF_HBLEND_THRESHOLD
> >> + * @hicut_int: If set, indicates that HiCut cutoff frequency is lower
> >> + * than the threshold set by ACF_
> >> +
> >> + */
> >> +struct si476x_acf_status_report {
> >> + __u8 blend_int;
> >> + __u8 hblend_int;
> >> + __u8 hicut_int;
> >> + __u8 chbw_int;
> >> + __u8 softmute_int;
> >> + __u8 smute;
> >> + __u8 smattn;
> >> + __u8 chbw;
> >> + __u8 hicut;
> >> + __u8 hiblend;
> >> + __u8 pilot;
> >> + __u8 stblend;
> >> +};
> >> +
> >> +enum si476x_fmagc {
> >> + SI476X_FMAGC_10K_OHM = 0,
> >> + SI476X_FMAGC_800_OHM = 1,
> >> + SI476X_FMAGC_400_OHM = 2,
> >> + SI476X_FMAGC_200_OHM = 4,
> >> + SI476X_FMAGC_100_OHM = 8,
> >> + SI476X_FMAGC_50_OHM = 16,
> >> + SI476X_FMAGC_25_OHM = 32,
> >> + SI476X_FMAGC_12P5_OHM = 64,
> >> + SI476X_FMAGC_6P25_OHM = 128,
> >> +};
> >> +
> >> +struct si476x_agc_status_report {
> >> + __u8 mxhi;
> >> + __u8 mxlo;
> >> + __u8 lnahi;
> >> + __u8 lnalo;
> >> + __u8 fmagc1;
> >> + __u8 fmagc2;
> >> + __u8 pgagain;
> >> + __u8 fmwblang;
> >> +};
> >> +
> >> +struct si476x_rds_blockcount_report {
> >> + __u16 expected;
> >> + __u16 received;
> >> + __u16 uncorrectable;
> >> +};
> >> +
> >> +#define SI476X_PHDIV_STATUS_LINK_LOCKED(status) (0b10000000 & (status))
> >> +#define SI476X_PHDIV_STATS_MODE(status) (0b111 & (status))
> >> +
> >> +#define SI476X_IOC_GET_RSQ _IOWR('V', BASE_VIDIOC_PRIVATE + 0, \
> >> + struct si476x_rsq_status_report)
> >> +
> >> +#define SI476X_IOC_SET_PHDIV_MODE _IOW('V', BASE_VIDIOC_PRIVATE + 1, \
> >> + enum si476x_phase_diversity_mode)
> >> +
> >> +#define SI476X_IOC_GET_PHDIV_STATUS _IOWR('V', BASE_VIDIOC_PRIVATE + 2, \
> >> + int)
> >> +
> >> +#define SI476X_IOC_GET_RSQ_PRIMARY _IOWR('V', BASE_VIDIOC_PRIVATE + 3, \
> >> + struct si476x_rsq_status_report)
> >> +
> >> +#define SI476X_IOC_GET_ACF _IOWR('V', BASE_VIDIOC_PRIVATE + 4, \
> >> + struct si476x_acf_status_report)
> >> +
> >> +#define SI476X_IOC_GET_AGC _IOWR('V', BASE_VIDIOC_PRIVATE + 5, \
> >> + struct si476x_agc_status_report)
> >> +
> >> +#define SI476X_IOC_GET_RDS_BLKCNT _IOWR('V', BASE_VIDIOC_PRIVATE + 6, \
> >> + struct si476x_rds_blockcount_report)
> > There is no documentation at all for these private ioctls. At the very least
> > these should be documentated (both the ioctl and the structs they receive).
> >
> > More importantly, are these ioctls really needed?
>
> I apologize for not writing the documentation for those ioctls. I
> thought I at least did so for all data structures returned by those
> calls, I guess I never got around to doing it. I'll document all the
> ioctls in the next version of the patch.
>
> SI476X_IOC_SET_PHDIV_MODE, SI476X_IOC_GET_PHDIV_STATUS are definitely
> needed since they are used to control antenna phase diversity modes of
> the tuners. In that mode two Si4764 chips connected to different
> antennas can act as a single tuner and use both signal to improve
> sensibility.
>
> The rest is useful to get different radio signal parameters from the
> chip. We used it for during RF performance evaluation of the
> boards(probably using it in EOL testing).
>
> > If the purpose is to return
> > status information for debugging, then you should consider implementing
> > VIDIOC_LOG_STATUS instead.
>
> For me, the problem with using VIDIOC_LOG_STATUS is that it dumps all
> the debugging information in kernel log buffer, meaning that if one
> wants to pass that information to some other application they would have
> to resort to screen-scraping of the output of dmesg. Unfortunately the
> people who were using this driver/as a part of a software suite during
> aforementioned RF performance testing are not Linux-savvy enough to be
> asked to use dmesg and they want a GUI solution. That small test utility
> was written and it uses those those ioctls to gather and display all
> signal related information. I guess the other solution for that problem
> would be to create corresponding files in sysfs, but I'm not sure if
> creating multitude of files in sysfs tree is a better option.
>
> Since this version of the driver has not been integrated into our
> software package the we install on laptops to do all the board testing I
> guess I can remove those 'ioctl', but I would love to find acceptable
> solution so that later I would be able to integrate upstream driver int
> our package and not use "special" version of the driver for our purposes.
One option would be to add a debugfs node for those, if they're used only
for debugging.
Regards,
Mauro
--
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