| 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 19:07:28 -0800
From: Andrey Smirnov <andrey.smirnov@...vergeddevices.net>
To: Mauro Carvalho Chehab <mchehab@...hat.com>
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
On 12-12-18 11:37 AM, Mauro Carvalho Chehab wrote:
> 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.
Yeah. Can't find a link for it right now, but that was the way I went
about it in my third version of the patches
>
> 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