Hi Matti, Just a few comments:
On Monday 02 August 2010 16:06:42 Matti J. Aaltonen wrote: > This driver implements V4L2 controls for the Texas Instruments > WL1273 FM Radio. > > Signed-off-by: Matti J. Aaltonen <matti.j.aalto...@nokia.com> > --- > drivers/media/radio/Kconfig | 15 + > drivers/media/radio/Makefile | 1 + > drivers/media/radio/radio-wl1273.c | 1972 > ++++++++++++++++++++++++++++++++++++ > drivers/mfd/Kconfig | 6 + > drivers/mfd/Makefile | 2 + > 5 files changed, 1996 insertions(+), 0 deletions(-) > create mode 100644 drivers/media/radio/radio-wl1273.c > > diff --git a/drivers/media/radio/Kconfig b/drivers/media/radio/Kconfig > index 83567b8..209fd37 100644 > --- a/drivers/media/radio/Kconfig > +++ b/drivers/media/radio/Kconfig > @@ -452,4 +452,19 @@ config RADIO_TIMBERDALE > found behind the Timberdale FPGA on the Russellville board. > Enabling this driver will automatically select the DSP and tuner. > > +config RADIO_WL1273 > + tristate "Texas Instruments WL1273 I2C FM Radio" > + depends on I2C && VIDEO_V4L2 && SND > + select FW_LOADER > + ---help--- > + Choose Y here if you have this FM radio chip. > + > + In order to control your radio card, you will need to use programs > + that are compatible with the Video For Linux 2 API. Information on > + this API and pointers to "v4l2" programs may be found at > + <file:Documentation/video4linux/API.html>. > + > + To compile this driver as a module, choose M here: the > + module will be called radio-wl1273. > + > endif # RADIO_ADAPTERS > diff --git a/drivers/media/radio/Makefile b/drivers/media/radio/Makefile > index f615583..d297074 100644 > --- a/drivers/media/radio/Makefile > +++ b/drivers/media/radio/Makefile > @@ -26,5 +26,6 @@ obj-$(CONFIG_RADIO_TEA5764) += radio-tea5764.o > obj-$(CONFIG_RADIO_SAA7706H) += saa7706h.o > obj-$(CONFIG_RADIO_TEF6862) += tef6862.o > obj-$(CONFIG_RADIO_TIMBERDALE) += radio-timb.o > +obj-$(CONFIG_RADIO_WL1273) += radio-wl1273.o > > EXTRA_CFLAGS += -Isound > diff --git a/drivers/media/radio/radio-wl1273.c > b/drivers/media/radio/radio-wl1273.c > new file mode 100644 > index 0000000..9c7f01f > --- /dev/null > +++ b/drivers/media/radio/radio-wl1273.c > @@ -0,0 +1,1972 @@ > +/* > + * Driver for the Texas Instruments WL1273 FM radio. > + * > + * Copyright (C) Nokia Corporation > + * Author: Matti J. Aaltonen <matti.j.aalto...@nokia.com> > + * > + * This program is free software; you can redistribute it and/or > + * modify it under the terms of the GNU General Public License > + * version 2 as published by the Free Software Foundation. > + * > + * 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. > + * > + * You should have received a copy of the GNU General Public License > + * along with this program; if not, write to the Free Software > + * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA > + */ > + > +#include <linux/delay.h> > +#include <linux/firmware.h> > +#include <linux/mfd/wl1273-core.h> > +#include <linux/slab.h> > +#include <media/v4l2-device.h> > +#include <media/v4l2-ioctl.h> > + > +#define DRIVER_DESC "Wl1273 FM Radio - V4L2" > + > +#define WL1273_POWER_SET_OFF 0 > +#define WL1273_POWER_SET_FM (1 << 0) > +#define WL1273_POWER_SET_RDS (1 << 1) > +#define WL1273_POWER_SET_RETENTION (1 << 4) > + > +#define WL1273_PUPD_SET_OFF 0x00 > +#define WL1273_PUPD_SET_ON 0x01 > +#define WL1273_PUPD_SET_RETENTION 0x10 > + > +#define WL1273_FREQ(x) (x * 10000 / 625) > +#define WL1273_INV_FREQ(x) (x * 625 / 10000) > + > +/* > + * static int radio_nr - The number of the radio device > + * > + * The default is 0. > + */ > +static int radio_nr = -1; > +module_param(radio_nr, int, 0); > +MODULE_PARM_DESC(radio_nr, "Radio Nr"); > + > +struct wl1273_device { > + struct v4l2_device v4l2dev; > + struct video_device videodev; > + struct device *dev; > + struct wl1273_core *core; > + struct file *owner; > + char *write_buf; > + bool rds_on; > +}; > + > +static int wl1273_fm_set_tx_freq(struct wl1273_core *core, unsigned int freq) > +{ > + int r = 0; > + > + if (freq < 76000) { > + dev_err(&core->i2c_dev->dev, > + "Frequency out of range: %d < %d\n", > + freq, core->bands[core->band].bottom_frequency); > + return -EDOM; Use ERANGE instead of EDOM. EDOM is for math functions only. > + } > + > + if (freq > 108000) { > + dev_err(&core->i2c_dev->dev, > + "Frequency out of range: %d > %d\n", > + freq, core->bands[core->band].top_frequency); > + return -EDOM; > + } > + > + /* > + * The driver works better with this msleep, > + * the documentation doesn't mention it. > + */ > + msleep(5); > + dev_dbg(&core->i2c_dev->dev, "%s: freq: %d kHz\n", __func__, freq); > + > + INIT_COMPLETION(core->busy); > + /* Set the current tx channel */ > + r = wl1273_fm_write_cmd(core, WL1273_CHANL_SET, freq / 10); > + if (r) > + return r; > + > + /* wait for the FR IRQ */ > + r = wait_for_completion_timeout(&core->busy, msecs_to_jiffies(2000)); > + if (!r) > + return -ETIMEDOUT; > + > + dev_dbg(&core->i2c_dev->dev, "WL1273_CHANL_SET: %d\n", r); > + > + /* Enable the output power */ > + INIT_COMPLETION(core->busy); > + r = wl1273_fm_write_cmd(core, WL1273_POWER_ENB_SET, 1); > + if (r) > + return r; > + > + /* wait for the POWER_ENB IRQ */ > + r = wait_for_completion_timeout(&core->busy, msecs_to_jiffies(1000)); > + if (!r) > + return -ETIMEDOUT; > + > + core->tx_frequency = freq; > + dev_dbg(&core->i2c_dev->dev, "WL1273_POWER_ENB_SET: %d\n", r); > + > + return 0; > +} > + > +static int wl1273_fm_set_rx_freq(struct wl1273_core *core, unsigned int freq) > +{ > + int r; > + int f; > + > + if (freq < core->bands[core->band].bottom_frequency) { > + dev_err(&core->i2c_dev->dev, > + "Frequency out of range: %d < %d\n", > + freq, core->bands[core->band].bottom_frequency); > + r = -EDOM; > + goto err; > + } > + > + if (freq > core->bands[core->band].top_frequency) { > + dev_err(&core->i2c_dev->dev, > + "Frequency out of range: %d > %d\n", > + freq, core->bands[core->band].top_frequency); > + r = -EDOM; > + goto err; > + } > + > + dev_dbg(&core->i2c_dev->dev, "%s: %dkHz\n", __func__, freq); > + > + wl1273_fm_write_cmd(core, WL1273_INT_MASK_SET, core->irq_flags); > + > + f = (freq - core->bands[core->band].bottom_frequency) / 50; > + r = wl1273_fm_write_cmd(core, WL1273_FREQ_SET, f); > + if (r) { > + dev_err(&core->i2c_dev->dev, "FREQ_SET fails\n"); > + goto err; > + } > + > + INIT_COMPLETION(core->busy); > + r = wl1273_fm_write_cmd(core, WL1273_TUNER_MODE_SET, TUNER_MODE_PRESET); > + if (r) { > + dev_err(&core->i2c_dev->dev, "TUNER_MODE_SET fails\n"); > + complete(&core->busy); > + goto err; > + } > + > + r = wait_for_completion_timeout(&core->busy, msecs_to_jiffies(2000)); > + if (!r) { > + dev_err(&core->i2c_dev->dev, "TIMEOUT\n"); > + return -ETIMEDOUT; > + } > + > + core->rd_index = 0; > + core->wr_index = 0; > + core->rx_frequency = freq; > + return 0; > + > +err: > + return r; > +} > + > +static int wl1273_fm_get_freq(struct wl1273_core *core) > +{ > + unsigned int freq; > + u16 f; > + int r; > + > + if (core->mode == WL1273_MODE_RX) { > + r = wl1273_fm_read_reg(core, WL1273_FREQ_SET, &f); > + if (r) > + return r; > + > + dev_dbg(&core->i2c_dev->dev, "Freq get: 0x%04x\n", f); > + freq = core->bands[core->band].bottom_frequency + 50 * f; > + } else { > + r = wl1273_fm_read_reg(core, WL1273_CHANL_SET, &f); > + if (r) > + return r; > + > + freq = f * 10; > + } > + > + return freq; > +} > + > +/** > + * wl1273_fm_upload_firmware_patch() - Upload the firmware. > + * @core: A pointer to the device struct. > + * > + * The firmware file consists of arrays of bytes where the first byte > + * gives the array length. The first byte in the file gives the > + * number of these arrays. > + */ > +static int wl1273_fm_upload_firmware_patch(struct wl1273_core *core) > +{ > + unsigned int packet_num; > + const struct firmware *fw_p; > + const char *fw_name = "radio-wl1273-fw.bin"; > + struct device *dev = &core->i2c_dev->dev; > + __u8 *ptr; > + int i, n, len, r; > + struct i2c_msg *msgs; > + > + dev_dbg(dev, "%s:\n", __func__); > + > + if (request_firmware(&fw_p, fw_name, dev)) { > + dev_info(dev, "%s - %s not found\n", __func__, fw_name); > + > + return 0; > + } > + > + ptr = (__u8 *) fw_p->data; > + packet_num = ptr[0]; > + dev_dbg(dev, "%s: packets: %d\n", __func__, packet_num); > + > + msgs = kmalloc((packet_num + 1)*sizeof(struct i2c_msg), GFP_KERNEL); > + if (!msgs) { > + r = -ENOMEM; > + goto out; > + } > + > + i = 1; > + for (n = 0; n <= packet_num; n++) { > + len = ptr[i]; > + > + dev_dbg(dev, "%s: len[%d]: %d\n", __func__, n, len); > + > + if (i + len + 1 <= fw_p->size) { > + msgs[n].addr = core->i2c_dev->addr; > + msgs[n].flags = 0; > + msgs[n].len = len; > + msgs[n].buf = ptr + i + 1; > + } else { > + break; > + } > + > + i += len + 1; > + } > + > + r = i2c_transfer(core->i2c_dev->adapter, msgs, packet_num); > + kfree(msgs); > + > + if (r != packet_num) { > + dev_err(dev, "FW upload error: %d\n", r); > + dev_dbg(dev, "%d != %d\n", packet_num, r); > + > + r = -EREMOTEIO; > + goto out; > + } else { > + r = 0; > + } > + > + /* ignore possible error here */ > + wl1273_fm_write_cmd(core, WL1273_RESET, 0); > + dev_dbg(dev, "n: %d, i: %d\n", n, i); > + > + if (n - 1 != packet_num) > + dev_warn(dev, "%s - incorrect firmware size.\n", __func__); > + > + if (i != fw_p->size) > + dev_warn(dev, "%s - inconsistent firmware.\n", __func__); > + > + dev_dbg(dev, "%s - download OK, r: %d\n", __func__, r); > + > +out: > + release_firmware(fw_p); > + return r; > +} > + > +static int wl1273_fm_stop(struct wl1273_core *core) > +{ > + struct wl1273_fm_platform_data *pdata = > + core->i2c_dev->dev.platform_data; > + > + if (core->mode == WL1273_MODE_RX) { > + int r = wl1273_fm_write_cmd(core, WL1273_POWER_SET, > + WL1273_POWER_SET_OFF); > + if (r) > + dev_err(&core->i2c_dev->dev, > + "%s: POWER_SET fails: %d\n", __func__, r); > + } else if (core->mode == WL1273_MODE_TX) { > + int r = wl1273_fm_write_cmd(core, WL1273_PUPD_SET, > + WL1273_PUPD_SET_OFF); > + if (r) > + dev_err(&core->i2c_dev->dev, > + "%s: PUPD_SET fails: %d\n", __func__, r); > + } > + > + if (pdata->disable) { > + pdata->disable(); > + dev_dbg(&core->i2c_dev->dev, "Back to reset\n"); > + } > + > + return 0; > +} > + > +static int wl1273_fm_start(struct wl1273_core *core, int new_mode) > +{ > + struct wl1273_fm_platform_data *pdata = > + core->i2c_dev->dev.platform_data; > + struct device *dev = &core->i2c_dev->dev; > + int r = -EINVAL; > + > + if (pdata->enable && core->mode == WL1273_MODE_OFF) { > + dev_dbg(dev, "Out of reset\n"); > + > + pdata->enable(); > + msleep(250); > + } > + > + if (new_mode == WL1273_MODE_RX) { > + u16 val = WL1273_POWER_SET_FM; > + > + if (core->rds_on) > + val |= WL1273_POWER_SET_RDS; > + > + /* If this fails try again */ > + r = wl1273_fm_write_cmd(core, WL1273_POWER_SET, val); > + if (r) { > + msleep(100); > + > + r = wl1273_fm_write_cmd(core, WL1273_POWER_SET, val); > + if (r) { > + dev_err(dev, "%s: POWER_SET fails\n", __func__); > + goto fail; > + } > + } > + > + /* rds buffer configuration */ > + core->wr_index = 0; > + core->rd_index = 0; > + > + } else if (new_mode == WL1273_MODE_TX) { > + /* If this fails try again */ > + r = wl1273_fm_write_cmd(core, WL1273_PUPD_SET, > + WL1273_PUPD_SET_ON); > + if (r) { > + msleep(100); > + r = wl1273_fm_write_cmd(core, WL1273_PUPD_SET, > + WL1273_PUPD_SET_ON); > + if (r) { > + dev_err(dev, "%s: PUPD_SET fails\n", __func__); > + goto fail; > + } > + } > + > + if (core->rds_on) > + r = wl1273_fm_write_cmd(core, WL1273_RDS_DATA_ENB, 1); > + else > + r = wl1273_fm_write_cmd(core, WL1273_RDS_DATA_ENB, 0); > + } else { > + dev_warn(dev, "%s: Illegal mode.\n", __func__); > + } > + > + if (core->mode == WL1273_MODE_OFF) { > + r = wl1273_fm_upload_firmware_patch(core); > + if (r) > + dev_warn(dev, "Firmware upload failed.\n"); > + > + /* > + * Sometimes the chip is in a wrong power state at this point. > + * So we set the power once again. > + */ > + if (new_mode == WL1273_MODE_RX) { > + u16 val = WL1273_POWER_SET_FM; > + > + if (core->rds_on) > + val |= WL1273_POWER_SET_RDS; > + > + r = wl1273_fm_write_cmd(core, WL1273_POWER_SET, val); > + if (r) { > + dev_err(dev, "%s: POWER_SET fails\n", __func__); > + goto fail; > + } > + } else if (new_mode == WL1273_MODE_TX) { > + r = wl1273_fm_write_cmd(core, WL1273_PUPD_SET, > + WL1273_PUPD_SET_ON); > + if (r) { > + dev_err(dev, "%s: PUPD_SET fails\n", __func__); > + goto fail; > + } > + } > + } > + > + return 0; > +fail: > + if (pdata->disable) > + pdata->disable(); > + > + dev_dbg(dev, "%s: return: %d\n", __func__, r); > + return r; > +} > + > +static int wl1273_fm_suspend(struct wl1273_core *core) > +{ > + int r = 0; > + struct device *dev = &core->i2c_dev->dev; > + > + /* Cannot go from OFF to SUSPENDED */ > + if (core->mode == WL1273_MODE_RX) > + r = wl1273_fm_write_cmd(core, WL1273_POWER_SET, > + WL1273_POWER_SET_RETENTION); > + else if (core->mode == WL1273_MODE_TX) > + r = wl1273_fm_write_cmd(core, WL1273_PUPD_SET, > + WL1273_PUPD_SET_RETENTION); > + else > + r = -EINVAL; > + > + if (r) { > + dev_err(dev, "%s: POWER_SET fails: %d\n", __func__, r); > + goto out; > + } > + > +out: > + return r; > +} > + > +static int wl1273_fm_set_mode(struct wl1273_core *core, int mode) > +{ > + int r; > + int old_mode; > + struct device *dev = &core->i2c_dev->dev; > + > + dev_dbg(dev, "%s\n", __func__); > + dev_dbg(dev, "Forbidden modes: 0x%02x\n", core->forbidden); > + > + old_mode = core->mode; > + if (mode & core->forbidden) { > + r = -EPERM; > + goto out; > + } > + > + switch (mode) { > + case WL1273_MODE_RX: > + case WL1273_MODE_TX: > + r = wl1273_fm_start(core, mode); > + if (r) { > + dev_err(dev, "%s: Cannot start.\n", __func__); > + wl1273_fm_stop(core); > + goto out; > + } > + > + core->mode = mode; > + r = wl1273_fm_write_cmd(core, WL1273_INT_MASK_SET, > + core->irq_flags); > + if (r) { > + dev_err(dev, "INT_MASK_SET fails.\n"); > + goto out; > + } > + > + /* remember previous settings */ > + if (mode == WL1273_MODE_RX) { > + r = wl1273_fm_set_rx_freq(core, core->rx_frequency); > + if (r) { > + dev_err(dev, "set freq fails: %d.\n", r); > + goto out; > + } > + > + r = wl1273_fm_set_volume(core, core->volume); > + if (r) { > + dev_err(dev, "set volume fails: %d.\n", r); > + goto out; > + } > + > + dev_dbg(dev, "%s: Set vol: %d.\n", __func__, > + core->volume); > + } else { > + r = wl1273_fm_set_tx_freq(core, core->tx_frequency); > + if (r) { > + dev_err(dev, "set freq fails: %d.\n", r); > + goto out; > + } > + } > + > + dev_dbg(dev, "%s: Set audio mode.\n", __func__); > + > + r = wl1273_fm_set_audio(core, core->audio_mode); > + if (r) > + dev_err(dev, "Cannot set audio mode.\n"); > + break; > + > + case WL1273_MODE_OFF: > + r = wl1273_fm_stop(core); > + if (r) > + dev_err(dev, "%s: Off fails: %d\n", __func__, r); > + else > + core->mode = WL1273_MODE_OFF; > + > + break; > + > + case WL1273_MODE_SUSPENDED: > + r = wl1273_fm_suspend(core); > + if (r) > + dev_err(dev, "%s: Suspend fails: %d\n", __func__, r); > + else > + core->mode = WL1273_MODE_SUSPENDED; > + > + break; > + > + default: > + dev_err(dev, "%s: Unknown mode: %d\n", __func__, mode); > + r = -EINVAL; > + break; > + } > + > +out: > + if (r) > + core->mode = old_mode ; Remove space before ';'. > + > + return r; > +} > + > +static int wl1273_fm_set_seek(struct wl1273_core *core, > + unsigned int wrap_around, > + unsigned int seek_upward, > + int level) > +{ > + int r = 0; > + unsigned int dir = (seek_upward == 0) ? 0 : 1; > + unsigned int rx_frequency, top_frequency, bottom_frequency; > + > + rx_frequency = core->rx_frequency; > + top_frequency = core->bands[core->band].top_frequency; > + bottom_frequency = core->bands[core->band].bottom_frequency; > + dev_dbg(&core->i2c_dev->dev, "core->rx_frequency: %d\n", > + rx_frequency); > + > + if (dir && rx_frequency + core->spacing <= top_frequency) > + r = wl1273_fm_set_rx_freq(core, rx_frequency + core->spacing); > + else if (dir && wrap_around) > + r = wl1273_fm_set_rx_freq(core, bottom_frequency); > + else if (rx_frequency - core->spacing >= bottom_frequency) > + r = wl1273_fm_set_rx_freq(core, rx_frequency - core->spacing); > + else if (wrap_around) > + r = wl1273_fm_set_rx_freq(core, top_frequency); > + > + if (r) > + goto out; > + > + if (level < SCHAR_MIN || level > SCHAR_MAX) > + return -EINVAL; > + > + INIT_COMPLETION(core->busy); > + dev_dbg(&core->i2c_dev->dev, "%s: BUSY\n", __func__); > + > + r = wl1273_fm_write_cmd(core, WL1273_INT_MASK_SET, core->irq_flags); > + if (r) > + goto out; > + > + dev_dbg(&core->i2c_dev->dev, "%s\n", __func__); > + > + r = wl1273_fm_write_cmd(core, WL1273_SEARCH_LVL_SET, level); > + if (r) > + goto out; > + > + r = wl1273_fm_write_cmd(core, WL1273_SEARCH_DIR_SET, dir); > + if (r) > + goto out; > + > + r = wl1273_fm_write_cmd(core, WL1273_TUNER_MODE_SET, > + TUNER_MODE_AUTO_SEEK); > + if (r) > + goto out; > + > + wait_for_completion_timeout(&core->busy, msecs_to_jiffies(1000)); > + if (!(core->irq_received & WL1273_BL_EVENT)) > + goto out; > + > + core->irq_received &= ~WL1273_BL_EVENT; > + > + if (!wrap_around) > + goto out; > + > + /* Wrap around */ > + dev_dbg(&core->i2c_dev->dev, "Wrap around in HW seek.\n"); > + > + if (seek_upward) > + rx_frequency = bottom_frequency; > + else > + rx_frequency = top_frequency; > + > + r = wl1273_fm_set_rx_freq(core, rx_frequency); > + if (r) > + goto out; > + > + INIT_COMPLETION(core->busy); > + dev_dbg(&core->i2c_dev->dev, "%s: BUSY\n", __func__); > + > + r = wl1273_fm_write_cmd(core, WL1273_TUNER_MODE_SET, > + TUNER_MODE_AUTO_SEEK); > + if (r) > + goto out; > + > + wait_for_completion_timeout(&core->busy, msecs_to_jiffies(1000)); > +out: > + dev_dbg(&core->i2c_dev->dev, "%s: Err: %d\n", __func__, r); > + return r; > +} > + > +/** > + * wl1273_fm_set_band() - Change the current band. > + * @core: A pointer to the device structure. > + * @band: The ID of the new band. > + * > + * Wl1273 supports only two bands USA/Europe and Japan. > + */ > +static int wl1273_fm_set_band(struct wl1273_core *core, unsigned int band) > +{ > + int r = 0; > + unsigned int new_frequency = 0; > + > + dev_dbg(&core->i2c_dev->dev, "%s: Number of bands: %d\n", __func__, > + core->number_of_bands); > + > + if (band == core->band) > + return 0; > + > + if (core->mode == WL1273_MODE_OFF || > + core->mode == WL1273_MODE_SUSPENDED) > + return -EPERM; > + > + if (band >= core->number_of_bands) > + return -EINVAL; > + > + mutex_lock(&core->lock); > + > + core->band = band; > + > + if (core->rx_frequency < core->bands[core->band].bottom_frequency) > + new_frequency = core->bands[core->band].bottom_frequency; > + else if (core->rx_frequency > core->bands[core->band].top_frequency) > + new_frequency = core->bands[core->band].top_frequency; > + > + if (new_frequency) { > + core->rx_frequency = new_frequency; > + if (core->mode == WL1273_MODE_RX) { > + r = wl1273_fm_set_rx_freq(core, new_frequency); > + if (r) > + goto out; > + } > + } > + > + new_frequency = 0; > + if (core->tx_frequency < core->bands[core->band].bottom_frequency) > + new_frequency = core->bands[core->band].bottom_frequency; > + else if (core->tx_frequency > core->bands[core->band].top_frequency) > + new_frequency = core->bands[core->band].top_frequency; > + > + if (new_frequency) { > + core->tx_frequency = new_frequency; > + > + if (core->mode == WL1273_MODE_TX) { > + r = wl1273_fm_set_tx_freq(core, new_frequency); > + if (r) > + goto out; > + } > + } > + > +out: > + mutex_unlock(&core->lock); > + return r; > +} > + > +/** > + * wl1273_fm_get_tx_ctune() - Get the TX tuning capacitor value. > + * @core: A pointer to the device struct. > + */ > +static unsigned int wl1273_fm_get_tx_ctune(struct wl1273_core *core) > +{ > + struct device *dev = &core->i2c_dev->dev; > + u16 val; > + int r; > + > + if (core->mode == WL1273_MODE_OFF || > + core->mode == WL1273_MODE_SUSPENDED) > + return -EPERM; > + > + r = wl1273_fm_read_reg(core, WL1273_READ_FMANT_TUNE_VALUE, &val); > + if (r) { > + dev_err(dev, "%s: I2C error: %d\n", __func__, r); > + goto out; > + } > + > +out: > + return val; > +} > + > +/** > + * wl1273_fm_set_preemphasis() - Set the TX pre-emphasis value. > + * @core: A pointer to the device struct. > + * @preemphasis: The new pre-amphasis value. > + * > + * Possible pre-emphasis values are: V4L2_PREEMPHASIS_DISABLED, > + * V4L2_PREEMPHASIS_50_uS and V4L2_PREEMPHASIS_75_uS. > + */ > +static int wl1273_fm_set_preemphasis(struct wl1273_core *core, > + unsigned int preemphasis) > +{ > + int r; > + u16 em; > + > + if (core->mode == WL1273_MODE_OFF || > + core->mode == WL1273_MODE_SUSPENDED) > + return -EPERM; > + > + mutex_lock(&core->lock); > + > + switch (preemphasis) { > + case V4L2_PREEMPHASIS_DISABLED: > + em = 1; > + break; > + case V4L2_PREEMPHASIS_50_uS: > + em = 0; > + break; > + case V4L2_PREEMPHASIS_75_uS: > + em = 2; > + break; > + default: > + r = -EINVAL; > + goto out; > + } > + > + r = wl1273_fm_write_cmd(core, WL1273_PREMPH_SET, em); > + if (r) > + goto out; > + > + core->preemphasis = preemphasis; > + > +out: > + mutex_unlock(&core->lock); > + return r; > +} > + > +static int wl1273_fm_rds_on(struct wl1273_core *core) > +{ > + int r; > + > + dev_dbg(&core->i2c_dev->dev, "%s\n", __func__); > + if (core->rds_on) > + return 0; > + > + r = wl1273_fm_write_cmd(core, WL1273_POWER_SET, > + WL1273_POWER_SET_FM | WL1273_POWER_SET_RDS); > + if (r) > + goto out; > + > + r = wl1273_fm_set_rx_freq(core, core->rx_frequency); > + if (r) > + dev_err(&core->i2c_dev->dev, "set freq fails: %d.\n", r); > +out: > + return r; > +} > + > +static int wl1273_fm_rds_off(struct wl1273_core *core) > +{ > + struct device *dev = &core->i2c_dev->dev; > + int r; > + > + if (!core->rds_on) > + return 0; > + > + core->irq_flags &= ~WL1273_RDS_EVENT; > + > + r = wl1273_fm_write_cmd(core, WL1273_INT_MASK_SET, core->irq_flags); > + if (r) > + goto out; > + > + /* stop rds reception */ > + cancel_delayed_work(&core->work); > + > + /* Service pending read */ > + wake_up_interruptible(&core->read_queue); > + > + dev_dbg(dev, "%s\n", __func__); > + > + r = wl1273_fm_write_cmd(core, WL1273_POWER_SET, WL1273_POWER_SET_FM); > + if (r) > + goto out; > + > + r = wl1273_fm_set_rx_freq(core, core->rx_frequency); > + if (r) > + dev_err(&core->i2c_dev->dev, "set freq fails: %d.\n", r); > + > +out: > + dev_dbg(dev, "%s: exiting...\n", __func__); > + > + return r; > +} > + > +static int wl1273_fm_set_rds(struct wl1273_core *core, unsigned int new_mode) > +{ > + int r = 0; > + struct device *dev = &core->i2c_dev->dev; > + > + if (core->mode == WL1273_MODE_OFF || > + core->mode == WL1273_MODE_SUSPENDED) > + return -EPERM; > + > + if (new_mode == WL1273_RDS_RESET) { > + r = wl1273_fm_write_cmd(core, WL1273_RDS_CNTRL_SET, 1); > + return r; > + } > + > + if (core->mode == WL1273_MODE_TX && new_mode == WL1273_RDS_OFF) { > + r = wl1273_fm_write_cmd(core, WL1273_RDS_DATA_ENB, 0); > + } else if (core->mode == WL1273_MODE_TX && new_mode == WL1273_RDS_ON) { > + r = wl1273_fm_write_cmd(core, WL1273_RDS_DATA_ENB, 1); > + } else if (core->mode == WL1273_MODE_RX && new_mode == WL1273_RDS_OFF) { > + r = wl1273_fm_rds_off(core); > + } else if (core->mode == WL1273_MODE_RX && new_mode == WL1273_RDS_ON) { > + r = wl1273_fm_rds_on(core); > + } else { > + dev_err(dev, "%s: Unknown mode: %d\n", __func__, new_mode); > + r = -EINVAL; > + } > + > + if (!r) > + core->rds_on = (new_mode == WL1273_RDS_ON) ? true : false; > + > + return r; > +} > + > +static ssize_t wl1273_fm_fops_write(struct file *file, const char __user > *buf, > + size_t count, loff_t *ppos) > +{ > + struct wl1273_device *radio = video_get_drvdata(video_devdata(file)); > + u16 val; > + int r; > + > + dev_dbg(radio->dev, "%s\n", __func__); > + > + if (radio->core->mode != WL1273_MODE_TX) > + return count; > + > + if (!radio->rds_on) { > + dev_warn(radio->dev, "%s: RDS not on.\n", __func__); > + return 0; > + } > + > + if (mutex_lock_interruptible(&radio->core->lock)) > + return -EINTR; > + > + /* > + * Multiple processes can open the device, but only > + * one gets to write to it. > + */ > + if (radio->owner && radio->owner != file) { > + r = -EBUSY; > + goto out; > + } > + radio->owner = file; > + > + /* Manual Mode */ > + if (count > 255) > + val = 255; > + else > + val = count; > + > + wl1273_fm_write_cmd(radio->core, WL1273_RDS_CONFIG_DATA_SET, val); > + > + if (copy_from_user(radio->write_buf + 1, buf, val)) { > + r = -EFAULT; > + goto out; > + } > + > + dev_dbg(radio->dev, "Count: %d\n", val); > + dev_dbg(radio->dev, "From user: \"%s\"\n", radio->write_buf); > + > + radio->write_buf[0] = WL1273_RDS_DATA_SET; > + wl1273_fm_write_data(radio->core, radio->write_buf, val + 1); > + > + r = val; > +out: > + mutex_unlock(&radio->core->lock); > + > + return r; > +} > + > +static unsigned int wl1273_fm_fops_poll(struct file *file, > + struct poll_table_struct *pts) > +{ > + struct wl1273_device *radio = video_get_drvdata(video_devdata(file)); > + struct wl1273_core *core = radio->core; > + > + if (radio->owner && radio->owner != file) > + return -EBUSY; > + > + radio->owner = file; > + > + if (core->mode == WL1273_MODE_RX) { > + poll_wait(file, &core->read_queue, pts); > + > + if (core->rd_index != core->wr_index) > + return POLLIN | POLLRDNORM; > + > + } else if (core->mode == WL1273_MODE_TX) { > + return POLLOUT | POLLWRNORM; > + } > + > + return 0; > +} > + > +static int wl1273_fm_fops_open(struct file *file) > +{ > + struct wl1273_device *radio = video_get_drvdata(video_devdata(file)); > + struct wl1273_core *core = radio->core; > + int r = 0; > + > + dev_dbg(radio->dev, "%s\n", __func__); > + > + if (core->mode == WL1273_MODE_RX && core->rds_on && !radio->rds_on) { > + dev_dbg(radio->dev, "%s: Mode: %d\n", __func__, core->mode); > + > + if (mutex_lock_interruptible(&core->lock)) > + return -EINTR; > + > + core->irq_flags |= WL1273_RDS_EVENT; > + > + r = wl1273_fm_write_cmd(core, WL1273_INT_MASK_SET, > + core->irq_flags); > + if (r) { > + mutex_unlock(&core->lock); > + goto out; > + } > + > + radio->rds_on = true; > + mutex_unlock(&core->lock); > + } > +out: > + return r; > +} > + > +static int wl1273_fm_fops_release(struct file *file) > +{ > + struct wl1273_device *radio = video_get_drvdata(video_devdata(file)); > + struct wl1273_core *core = radio->core; > + int r = 0; > + > + dev_dbg(radio->dev, "%s\n", __func__); > + > + if (radio->rds_on) { > + if (mutex_lock_interruptible(&core->lock)) > + return -EINTR; > + > + core->irq_flags &= ~WL1273_RDS_EVENT; This is dangerous: you probably want to use a usecount instead. With this code opening the device one will turn on the RDS events, but opening and closing it via another application (e.g. v4l2-ctl) will disable it while the first still needs it. > + > + if (core->mode == WL1273_MODE_RX) { > + dev_dbg(radio->dev, "%s: A'\n", __func__); > + r = wl1273_fm_write_cmd(core, WL1273_INT_MASK_SET, > + core->irq_flags); > + if (r) { > + mutex_unlock(&core->lock); > + goto out; > + } > + } > + > + radio->rds_on = false; > + mutex_unlock(&core->lock); > + } > + > + if (file == radio->owner) > + radio->owner = NULL; > +out: > + return r; > +} > + > +static ssize_t wl1273_fm_fops_read(struct file *file, char __user *buf, > + size_t count, loff_t *ppos) > +{ > + int r = 0; > + struct wl1273_device *radio = video_get_drvdata(video_devdata(file)); > + struct wl1273_core *core = radio->core; > + unsigned int block_count = 0; > + u16 val; > + > + dev_dbg(radio->dev, "%s\n", __func__); > + > + if (radio->core->mode != WL1273_MODE_RX) > + return 0; > + > + if (!radio->rds_on) { > + dev_warn(radio->dev, "%s: RDS not on.\n", __func__); > + return 0; > + } > + > + if (mutex_lock_interruptible(&core->lock)) > + return -EINTR; > + > + /* > + * Multiple processes can open the device, but only > + * one gets to read from it. > + */ > + if (radio->owner && radio->owner != file) { > + r = -EBUSY; > + goto out; > + } > + radio->owner = file; > + > + r = wl1273_fm_read_reg(core, WL1273_RDS_SYNC_GET, &val); > + if (r) { > + dev_err(radio->dev, "%s: Get RDS_SYNC fails.\n", __func__); > + goto out; > + } else if (val == 0) { > + dev_info(radio->dev, "RDS_SYNC: Not synchronized\n"); > + r = -ENODATA; > + goto out; > + } > + > + /* block if no new data available */ > + while (core->wr_index == core->rd_index) { > + if (file->f_flags & O_NONBLOCK) { > + r = -EWOULDBLOCK; > + goto out; > + } > + > + dev_dbg(radio->dev, "%s: Wait for RDS data.\n", __func__); > + if (wait_event_interruptible(core->read_queue, > + core->wr_index != > + core->rd_index) < 0) { > + r = -EINTR; > + goto out; > + } > + } > + > + /* calculate block count from byte count */ > + count /= 3; > + > + /* copy RDS blocks from the internal buffer and to user buffer */ > + while (block_count < count) { > + if (core->rd_index == core->wr_index) > + break; > + > + /* always transfer complete RDS blocks */ > + if (copy_to_user(buf, &core->buffer[core->rd_index], 3)) > + break; > + > + /* increment and wrap the read pointer */ > + core->rd_index += 3; > + if (core->rd_index >= core->buf_size) > + core->rd_index = 0; > + > + /* increment counters */ > + block_count++; > + buf += 3; > + r += 3; > + } > + > +out: > + dev_dbg(radio->dev, "%s: exit\n", __func__); > + mutex_unlock(&core->lock); > + > + return r; > +} > + > +static const struct v4l2_file_operations wl1273_fops = { > + .owner = THIS_MODULE, > + .read = wl1273_fm_fops_read, > + .write = wl1273_fm_fops_write, > + .poll = wl1273_fm_fops_poll, > + .ioctl = video_ioctl2, > + .open = wl1273_fm_fops_open, > + .release = wl1273_fm_fops_release, > +}; > + > +static int wl1273_fm_vidioc_querycap(struct file *file, void *priv, > + struct v4l2_capability *capability) > +{ > + struct wl1273_device *radio = video_get_drvdata(video_devdata(file)); > + > + dev_dbg(radio->dev, "%s\n", __func__); > + > + strlcpy(capability->driver, WL1273_FM_DRIVER_NAME, > + sizeof(capability->driver)); > + strlcpy(capability->card, "Texas Instruments Wl1273 FM Radio", > + sizeof(capability->card)); > + strlcpy(capability->bus_info, "I2C", sizeof(capability->bus_info)); > + > + capability->capabilities = V4L2_CAP_HW_FREQ_SEEK | > + V4L2_CAP_TUNER | V4L2_CAP_RADIO | V4L2_CAP_AUDIO | > + V4L2_CAP_RDS_CAPTURE | V4L2_CAP_MODULATOR | > + V4L2_CAP_RDS_OUTPUT; > + > + return 0; > +} > + > +static int wl1273_fm_vidioc_g_input(struct file *file, void *priv, > + unsigned int *i) > +{ > + struct wl1273_device *radio = video_get_drvdata(video_devdata(file)); > + > + dev_dbg(radio->dev, "%s\n", __func__); > + > + *i = 0; > + > + return 0; > +} > + > +static int wl1273_fm_vidioc_s_input(struct file *file, void *priv, > + unsigned int i) > +{ > + struct wl1273_device *radio = video_get_drvdata(video_devdata(file)); > + > + dev_dbg(radio->dev, "%s\n", __func__); > + > + if (i != 0) > + return -EINVAL; > + > + return 0; > +} > + > +static int wl1273_fm_vidioc_queryctrl(struct file *file, void *priv, > + struct v4l2_queryctrl *qc) > +{ > + struct wl1273_device *radio = video_get_drvdata(video_devdata(file)); > + > + dev_dbg(radio->dev, "%s\n", __func__); > + > + switch (qc->id) { > + case V4L2_CID_AUDIO_VOLUME: > + return v4l2_ctrl_query_fill(qc, 0, WL1273_MAX_VOLUME, 1, > + WL1273_DEFAULT_VOLUME); > + case V4L2_CID_AUDIO_MUTE: > + return v4l2_ctrl_query_fill(qc, 0, 1, 1, 1); > + > + case V4L2_CID_TUNE_PREEMPHASIS: > + return v4l2_ctrl_query_fill(qc, V4L2_PREEMPHASIS_DISABLED, > + V4L2_PREEMPHASIS_75_uS, 1, > + V4L2_PREEMPHASIS_50_uS); > + case V4L2_CID_TUNE_POWER_LEVEL: > + return v4l2_ctrl_query_fill(qc, 0, 37, 1, 4); > + > + case V4L2_CID_FM_BAND: > + return v4l2_ctrl_query_fill(qc, V4L2_FM_BAND_OTHER, > + V4L2_FM_BAND_JAPAN, 1, > + V4L2_FM_BAND_OTHER); > + } > + return -EINVAL; > +} > + > +static int wl1273_fm_vidioc_querymenu(struct file *file, void *priv, > + struct v4l2_querymenu *qm) > +{ > + static u32 fm_rx_bands[] = { > + V4L2_FM_BAND_OTHER, > + V4L2_FM_BAND_JAPAN, > + V4L2_CTRL_MENU_IDS_END > + }; > + struct v4l2_queryctrl qctrl; > + int r; > + > + qctrl.id = qm->id; > + r = wl1273_fm_vidioc_queryctrl(file, priv, &qctrl); > + if (r) > + return r; > + > + switch (qm->id) { > + case V4L2_CID_FM_BAND: > + return v4l2_ctrl_query_menu_valid_items(qm, fm_rx_bands); > + } > + > + return v4l2_ctrl_query_menu(qm, &qctrl, NULL); > +} > + > +/** > + * wl1273_fm_set_tx_power() - Set the transmission power value. > + * @core: A pointer to the device struct. > + * @power: The new power value. > + */ > +static int wl1273_fm_set_tx_power(struct wl1273_core *core, u16 power) > +{ > + int r; > + > + if (core->mode == WL1273_MODE_OFF || > + core->mode == WL1273_MODE_SUSPENDED) > + return -EPERM; > + > + mutex_lock(&core->lock); > + > + r = wl1273_fm_write_cmd(core, WL1273_POWER_LEV_SET, power); > + if (r) > + goto out; > + > + core->tx_power = power; > + > +out: > + mutex_unlock(&core->lock); > + return r; > +} > + > +#define WL1273_SPACING_50kHz 1 > +#define WL1273_SPACING_100kHz 2 > +#define WL1273_SPACING_200kHz 4 > + > +static int wl1273_fm_tx_set_spacing(struct wl1273_core *core, > + unsigned int spacing) > +{ > + int r; > + > + if (spacing == 0) { > + r = wl1273_fm_write_cmd(core, WL1273_SCAN_SPACING_SET, > + WL1273_SPACING_100kHz); > + core->spacing = 100; > + } else if (spacing - 50000 < 25000) { > + r = wl1273_fm_write_cmd(core, WL1273_SCAN_SPACING_SET, > + WL1273_SPACING_50kHz); > + core->spacing = 50; > + } else if (spacing - 100000 < 50000) { > + r = wl1273_fm_write_cmd(core, WL1273_SCAN_SPACING_SET, > + WL1273_SPACING_100kHz); > + core->spacing = 100; > + } else { > + r = wl1273_fm_write_cmd(core, WL1273_SCAN_SPACING_SET, > + WL1273_SPACING_200kHz); > + core->spacing = 200; > + } > + > + return r; > +} > + > +static int wl1273_fm_vidioc_g_ctrl(struct file *file, void *priv, > + struct v4l2_control *ctrl) > +{ > + struct wl1273_device *radio = video_get_drvdata(video_devdata(file)); > + struct wl1273_core *core = radio->core; > + u16 val; > + int r = 0; > + > + dev_dbg(radio->dev, "%s\n", __func__); > + > + if (mutex_lock_interruptible(&core->lock)) > + return -EINTR; > + > + switch (ctrl->id) { > + case V4L2_CID_AUDIO_MUTE: > + if (core->mode == WL1273_MODE_RX) > + r = wl1273_fm_read_reg(core, WL1273_MUTE_STATUS_SET, > + &val); > + else > + r = wl1273_fm_read_reg(core, WL1273_MUTE, &val); > + > + if (r) > + goto out; > + > + dev_dbg(radio->dev, > + "MUTE STATUS GET: 0x%02x\n", val); > + > + if (val) > + ctrl->value = 1; > + else > + ctrl->value = 0; > + > + break; > + > + case V4L2_CID_AUDIO_VOLUME: > + ctrl->value = core->volume; > + break; > + > + case V4L2_CID_TUNE_ANTENNA_CAPACITOR: > + ctrl->value = wl1273_fm_get_tx_ctune(core); > + break; > + > + case V4L2_CID_TUNE_PREEMPHASIS: > + ctrl->value = core->preemphasis; > + break; > + > + case V4L2_CID_TUNE_POWER_LEVEL: > + ctrl->value = core->tx_power; > + break; > + > + case V4L2_CID_FM_BAND: > + ctrl->value = core->band; > + break; > + > + default: > + dev_warn(radio->dev, "%s: Unknown IOCTL: %d\n", > + __func__, ctrl->id); > + break; > + } > + > +out: > + mutex_unlock(&core->lock); > + > + return r; > +} > + > +#define WL1273_MUTE_SOFT_ENABLE (1 << 0) > +#define WL1273_MUTE_AC (1 << 1) > +#define WL1273_MUTE_HARD_LEFT (1 << 2) > +#define WL1273_MUTE_HARD_RIGHT (1 << 3) > +#define WL1273_MUTE_SOFT_FORCE (1 << 4) > + > +static int wl1273_fm_vidioc_s_ctrl(struct file *file, void *priv, > + struct v4l2_control *ctrl) > +{ > + struct wl1273_device *radio = video_get_drvdata(video_devdata(file)); > + struct wl1273_core *core = radio->core; > + int r = 0; > + > + dev_dbg(radio->dev, "%s\n", __func__); > + > + switch (ctrl->id) { > + case V4L2_CID_AUDIO_MUTE: > + if (mutex_lock_interruptible(&core->lock)) > + return -EINTR; > + > + if (core->mode == WL1273_MODE_RX && ctrl->value) > + r = wl1273_fm_write_cmd(core, > + WL1273_MUTE_STATUS_SET, > + WL1273_MUTE_HARD_LEFT | > + WL1273_MUTE_HARD_RIGHT); > + else if (core->mode == WL1273_MODE_RX) > + r = wl1273_fm_write_cmd(core, > + WL1273_MUTE_STATUS_SET, 0x0); > + else if (core->mode == WL1273_MODE_TX && ctrl->value) > + r = wl1273_fm_write_cmd(core, WL1273_MUTE, 1); > + else if (core->mode == WL1273_MODE_TX) > + r = wl1273_fm_write_cmd(core, WL1273_MUTE, 0); > + > + mutex_unlock(&core->lock); > + break; > + > + case V4L2_CID_AUDIO_VOLUME: > + if (ctrl->value == 0) > + r = wl1273_fm_set_mode(core, WL1273_MODE_OFF); > + else > + r = wl1273_fm_set_volume(core, core->volume); > + break; > + > + case V4L2_CID_TUNE_PREEMPHASIS: > + r = wl1273_fm_set_preemphasis(core, ctrl->value); > + break; > + > + case V4L2_CID_TUNE_POWER_LEVEL: > + r = wl1273_fm_set_tx_power(core, ctrl->value); > + break; > + > + case V4L2_CID_FM_BAND: > + r = wl1273_fm_set_band(core, ctrl->value); > + break; > + > + default: > + dev_warn(radio->dev, "%s: Unknown IOCTL: %d\n", > + __func__, ctrl->id); > + break; > + } > + > + dev_dbg(radio->dev, "%s\n", __func__); > + return r; > +} > + > +static int wl1273_fm_vidioc_g_audio(struct file *file, void *priv, > + struct v4l2_audio *audio) > +{ > + struct wl1273_device *radio = video_get_drvdata(video_devdata(file)); > + > + dev_dbg(radio->dev, "%s\n", __func__); > + > + if (audio->index > 1) > + return -EINVAL; > + > + strlcpy(audio->name, "Radio", sizeof(audio->name)); > + audio->capability = V4L2_AUDCAP_STEREO; > + > + return 0; > +} > + > +static int wl1273_fm_vidioc_s_audio(struct file *file, void *priv, > + struct v4l2_audio *audio) > +{ > + struct wl1273_device *radio = video_get_drvdata(video_devdata(file)); > + > + dev_dbg(radio->dev, "%s\n", __func__); > + > + if (audio->index != 0) > + return -EINVAL; > + > + return 0; > +} > + > +#define WL1273_RDS_NOT_SYNCHRONIZED 0 > +#define WL1273_RDS_SYNCHRONIZED 1 > + > +static int wl1273_fm_vidioc_g_tuner(struct file *file, void *priv, > + struct v4l2_tuner *tuner) > +{ > + struct wl1273_device *radio = video_get_drvdata(video_devdata(file)); > + struct wl1273_core *core = radio->core; > + u16 val; > + int r; > + > + dev_dbg(radio->dev, "%s\n", __func__); > + > + if (tuner->index > 0) > + return -EINVAL; > + > + strlcpy(tuner->name, WL1273_FM_DRIVER_NAME, sizeof(tuner->name)); > + tuner->type = V4L2_TUNER_RADIO; > + > + tuner->rangelow = > + WL1273_FREQ(core->bands[core->band].bottom_frequency); > + tuner->rangehigh = > + WL1273_FREQ(core->bands[core->band].top_frequency); > + > + tuner->capability = V4L2_TUNER_CAP_LOW | V4L2_TUNER_CAP_RDS | > + V4L2_TUNER_CAP_STEREO; > + > + if (core->stereo) > + tuner->audmode = V4L2_TUNER_MODE_STEREO; > + else > + tuner->audmode = V4L2_TUNER_MODE_MONO; > + > + if (core->mode != WL1273_MODE_RX) > + return 0; > + > + if (mutex_lock_interruptible(&core->lock)) > + return -EINTR; > + > + r = wl1273_fm_read_reg(core, WL1273_STEREO_GET, &val); > + if (r) > + goto out; > + > + if (val == 1) > + tuner->rxsubchans = V4L2_TUNER_SUB_STEREO; > + else > + tuner->rxsubchans = V4L2_TUNER_SUB_MONO; > + > + r = wl1273_fm_read_reg(core, WL1273_RSSI_LVL_GET, &val); > + if (r) > + goto out; > + > + tuner->signal = (s16) val; > + dev_dbg(radio->dev, "Signal: %d\n", tuner->signal); > + > + tuner->afc = 0; > + > + r = wl1273_fm_read_reg(core, WL1273_RDS_SYNC_GET, &val); > + if (r) > + goto out; > + > + if (val == WL1273_RDS_SYNCHRONIZED) > + tuner->rxsubchans |= V4L2_TUNER_SUB_RDS; > +out: > + mutex_unlock(&core->lock); > + > + return r; > +} > + > +static int wl1273_fm_vidioc_s_tuner(struct file *file, void *priv, > + struct v4l2_tuner *tuner) > +{ > + struct wl1273_device *radio = video_get_drvdata(video_devdata(file)); > + struct wl1273_core *core = radio->core; > + int r = 0; > + > + dev_dbg(radio->dev, "%s\n", __func__); > + dev_dbg(radio->dev, "tuner->index: %d\n", tuner->index); > + dev_dbg(radio->dev, "tuner->name: %s\n", tuner->name); > + dev_dbg(radio->dev, "tuner->capability: 0x%04x\n", tuner->capability); > + dev_dbg(radio->dev, "tuner->rxsubchans: 0x%04x\n", tuner->rxsubchans); > + dev_dbg(radio->dev, "tuner->rangelow: %d\n", tuner->rangelow); > + dev_dbg(radio->dev, "tuner->rangehigh: %d\n", tuner->rangehigh); > + > + if (tuner->index > 0) > + return -EINVAL; > + > + if (mutex_lock_interruptible(&core->lock)) > + return -EINTR; > + > + r = wl1273_fm_set_mode(core, WL1273_MODE_RX); > + if (r) > + goto out; > + > + if (tuner->rxsubchans & V4L2_TUNER_SUB_RDS) > + r = wl1273_fm_set_rds(core, WL1273_RDS_ON); > + else > + r = wl1273_fm_set_rds(core, WL1273_RDS_OFF); > + > + if (r) > + dev_warn(radio->dev, "%s: RDS fails: %d\n", __func__, r); > + > + if (tuner->audmode == V4L2_TUNER_MODE_MONO) { > + r = wl1273_fm_write_cmd(core, WL1273_MOST_MODE_SET, > + WL1273_RX_MONO); > + if (r < 0) { > + dev_warn(radio->dev, "%s: MOST_MODE fails: %d\n", > + __func__, r); > + goto out; > + } > + core->stereo = false; > + } else if (tuner->audmode == V4L2_TUNER_MODE_MONO) { > + r = wl1273_fm_write_cmd(core, WL1273_MOST_MODE_SET, > + WL1273_RX_STEREO); > + if (r < 0) { > + dev_warn(radio->dev, "%s: MOST_MODE fails: %d\n", > + __func__, r); > + goto out; > + } > + core->stereo = true; > + } else { > + dev_err(radio->dev, "%s: tuner->audmode: %d\n", > + __func__, tuner->audmode); > + r = -EINVAL; > + goto out; > + } > + > +out: > + mutex_unlock(&core->lock); > + > + return r; > +} > + > +static int wl1273_fm_vidioc_g_frequency(struct file *file, void *priv, > + struct v4l2_frequency *freq) > +{ > + struct wl1273_device *radio = video_get_drvdata(video_devdata(file)); > + struct wl1273_core *core = radio->core; > + > + dev_dbg(radio->dev, "%s\n", __func__); > + > + if (mutex_lock_interruptible(&core->lock)) > + return -EINTR; > + > + freq->type = V4L2_TUNER_RADIO; > + freq->frequency = WL1273_FREQ(wl1273_fm_get_freq(core)); > + > + mutex_unlock(&core->lock); > + > + return 0; > +} > + > +static int wl1273_fm_vidioc_s_frequency(struct file *file, void *priv, > + struct v4l2_frequency *freq) > +{ > + struct wl1273_device *radio = video_get_drvdata(video_devdata(file)); > + struct wl1273_core *core = radio->core; > + int r; > + > + dev_dbg(radio->dev, "%s: %d\n", __func__, freq->frequency); > + > + if (freq->type != V4L2_TUNER_RADIO) { > + dev_dbg(radio->dev, > + "freq->type != V4L2_TUNER_RADIO: %d\n", freq->type); > + return -EINVAL; > + } > + > + if (mutex_lock_interruptible(&core->lock)) > + return -EINTR; > + > + if (core->mode == WL1273_MODE_RX) { > + dev_dbg(radio->dev, "freq: %d\n", freq->frequency); > + > + r = wl1273_fm_set_rx_freq(core, > + WL1273_INV_FREQ(freq->frequency)); > + if (r) > + dev_warn(radio->dev, WL1273_FM_DRIVER_NAME > + ": set frequency failed with %d\n", r); > + } else { > + r = wl1273_fm_set_tx_freq(core, > + WL1273_INV_FREQ(freq->frequency)); > + if (r) > + dev_warn(radio->dev, WL1273_FM_DRIVER_NAME > + ": set frequency failed with %d\n", r); > + } > + > + mutex_unlock(&core->lock); > + > + dev_dbg(radio->dev, "wl1273_vidioc_s_frequency: DONE\n"); > + return r; > +} > + > +#define WL1273_DEFAULT_SEEK_LEVEL 7 > + > +static int wl1273_fm_vidioc_s_hw_freq_seek(struct file *file, void *priv, > + struct v4l2_hw_freq_seek *seek) > +{ > + struct wl1273_device *radio = video_get_drvdata(video_devdata(file)); > + struct wl1273_core *core = radio->core; > + int r; > + > + dev_dbg(radio->dev, "%s\n", __func__); > + > + if (seek->tuner != 0 || seek->type != V4L2_TUNER_RADIO) > + return -EINVAL; > + > + if (mutex_lock_interruptible(&core->lock)) > + return -EINTR; > + > + r = wl1273_fm_set_mode(core, WL1273_MODE_RX); > + if (r) > + goto out; > + > + r = wl1273_fm_tx_set_spacing(core, seek->spacing); > + if (r) > + dev_warn(radio->dev, "HW seek failed: %d\n", r); > + > + r = wl1273_fm_set_seek(core, seek->wrap_around, seek->seek_upward, > + WL1273_DEFAULT_SEEK_LEVEL); > + if (r) > + dev_warn(radio->dev, "HW seek failed: %d\n", r); > + > + out: > + mutex_unlock(&core->lock); > + return r; > +} > + > +static int wl1273_fm_vidioc_s_modulator(struct file *file, void *priv, > + struct v4l2_modulator *modulator) > +{ > + struct wl1273_device *radio = video_get_drvdata(video_devdata(file)); > + struct wl1273_core *core = radio->core; > + int r = 0; > + > + dev_dbg(radio->dev, "%s\n", __func__); > + > + if (modulator->index > 0) > + return -EINVAL; > + > + if (mutex_lock_interruptible(&core->lock)) > + return -EINTR; > + > + r = wl1273_fm_set_mode(core, WL1273_MODE_TX); > + if (r) > + goto out; > + > + if (modulator->txsubchans & V4L2_TUNER_SUB_RDS) > + r = wl1273_fm_set_rds(core, WL1273_RDS_ON); > + else > + r = wl1273_fm_set_rds(core, WL1273_RDS_OFF); > + > + if (modulator->txsubchans & V4L2_TUNER_SUB_MONO) > + r = wl1273_fm_write_cmd(core, WL1273_MONO_SET, WL1273_TX_MONO); > + else > + r = wl1273_fm_write_cmd(core, WL1273_MONO_SET, > + WL1273_RX_STEREO); > + if (r < 0) > + dev_warn(radio->dev, WL1273_FM_DRIVER_NAME > + "MONO_SET fails: %d\n", r); > +out: > + mutex_unlock(&core->lock); > + > + return r; > +} > + > +static int wl1273_fm_vidioc_g_modulator(struct file *file, void *priv, > + struct v4l2_modulator *modulator) > +{ > + struct wl1273_device *radio = video_get_drvdata(video_devdata(file)); > + struct wl1273_core *core = radio->core; > + u16 val; > + int r; > + > + dev_dbg(radio->dev, "%s\n", __func__); > + > + strlcpy(modulator->name, WL1273_FM_DRIVER_NAME, > + sizeof(modulator->name)); > + > + modulator->rangelow = > + WL1273_FREQ(core->bands[core->band].bottom_frequency); > + modulator->rangehigh = > + WL1273_FREQ(core->bands[core->band].top_frequency); > + > + modulator->capability = V4L2_TUNER_CAP_LOW | V4L2_TUNER_CAP_RDS | > + V4L2_TUNER_CAP_STEREO; > + > + if (core->mode != WL1273_MODE_TX) > + return 0; > + > + if (mutex_lock_interruptible(&core->lock)) > + return -EINTR; > + > + r = wl1273_fm_read_reg(core, WL1273_MONO_SET, &val); > + if (r) > + goto out; > + > + if (val == WL1273_TX_STEREO) > + modulator->txsubchans = V4L2_TUNER_SUB_STEREO; > + else > + modulator->txsubchans = V4L2_TUNER_SUB_MONO; > + > + if (core->rds_on) > + modulator->txsubchans |= V4L2_TUNER_SUB_RDS; > +out: > + mutex_unlock(&core->lock); > + > + return 0; > +} > + > +static int wl1273_fm_vidioc_log_status(struct file *file, void *priv) > +{ > + struct wl1273_device *radio = video_get_drvdata(video_devdata(file)); > + struct wl1273_core *core = radio->core; > + struct device *dev = radio->dev; > + u16 val; > + int r; > + > + dev_info(dev, DRIVER_DESC); > + > + if (core->mode == WL1273_MODE_OFF) { > + dev_info(dev, "Mode: Off\n"); > + return 0; > + } > + > + if (core->mode == WL1273_MODE_SUSPENDED) { > + dev_info(dev, "Mode: Suspended\n"); > + return 0; > + } > + > + r = wl1273_fm_read_reg(core, WL1273_ASIC_ID_GET, &val); > + if (r) > + dev_err(dev, "%s: Get ASIC_ID fails.\n", __func__); > + else > + dev_info(dev, "ASIC_ID: 0x%04x\n", val); > + > + r = wl1273_fm_read_reg(core, WL1273_ASIC_VER_GET, &val); > + if (r) > + dev_err(dev, "%s: Get ASIC_VER fails.\n", __func__); > + else > + dev_info(dev, "ASIC Version: 0x%04x\n", val); > + > + r = wl1273_fm_read_reg(core, WL1273_FIRM_VER_GET, &val); > + if (r) > + dev_err(dev, "%s: Get FIRM_VER fails.\n", __func__); > + else > + dev_info(dev, "FW version: %d(0x%04x)\n", val, val); > + > + r = wl1273_fm_read_reg(core, WL1273_BAND_SET, &val); > + if (r) > + dev_err(dev, "%s: Get BAND fails.\n", __func__); > + else > + dev_info(dev, "BAND: %d\n", val); > + > + if (core->mode == WL1273_MODE_TX) { > + r = wl1273_fm_read_reg(core, WL1273_PUPD_SET, &val); > + if (r) > + dev_err(dev, "%s: Get PUPD fails.\n", __func__); > + else > + dev_info(dev, "PUPD: 0x%04x\n", val); > + > + r = wl1273_fm_read_reg(core, WL1273_CHANL_SET, &val); > + if (r) > + dev_err(dev, "%s: Get CHANL fails.\n", __func__); > + else > + dev_info(dev, "Tx frequency: %dkHz\n", val*10); > + } else if (core->mode == WL1273_MODE_RX) { > + int bf = core->bands[core->band].bottom_frequency; > + > + r = wl1273_fm_read_reg(core, WL1273_FREQ_SET, &val); > + if (r) > + dev_err(dev, "%s: Get FREQ fails.\n", __func__); > + else > + dev_info(dev, "RX Frequency: %dkHz\n", bf + val*50); > + > + r = wl1273_fm_read_reg(core, WL1273_MOST_MODE_SET, &val); > + if (r) > + dev_err(dev, "%s: Get MOST_MODE fails.\n", > + __func__); > + else if (val == 0) > + dev_info(dev, "MOST_MODE: Stereo according to blend\n"); > + else if (val == 1) > + dev_info(dev, "MOST_MODE: Force mono output\n"); > + else > + dev_info(dev, "MOST_MODE: Unexpected value: %d\n", val); > + > + r = wl1273_fm_read_reg(core, WL1273_MOST_BLEND_SET, &val); > + if (r) > + dev_err(dev, "%s: Get MOST_BLEND fails.\n", __func__); > + else if (val == 0) > + dev_info(dev, > + "MOST_BLEND: Switched blend & hysteresis.\n"); > + else if (val == 1) > + dev_info(dev, "MOST_BLEND: Soft blend.\n"); > + else > + dev_info(dev, "MOST_BLEND: Unexpected val: %d\n", val); > + > + r = wl1273_fm_read_reg(core, WL1273_STEREO_GET, &val); > + if (r) > + dev_err(dev, "%s: Get STEREO fails.\n", __func__); > + else if (val == 0) > + dev_info(dev, "STEREO: Not detected\n"); > + else if (val == 1) > + dev_info(dev, "STEREO: Detected\n"); > + else > + dev_info(dev, "STEREO: Unexpected value: %d\n", val); > + > + r = wl1273_fm_read_reg(core, WL1273_RSSI_LVL_GET, &val); > + if (r) > + dev_err(dev, "%s: Get RSSI_LVL fails.\n", __func__); > + else > + dev_info(dev, "RX signal strength: %d\n", (s16) val); > + > + r = wl1273_fm_read_reg(core, WL1273_POWER_SET, &val); > + if (r) > + dev_err(dev, "%s: Get POWER fails.\n", __func__); > + else > + dev_info(dev, "POWER: 0x%04x\n", val); > + > + r = wl1273_fm_read_reg(core, WL1273_INT_MASK_SET, &val); > + if (r) > + dev_err(dev, "%s: Get INT_MASK fails.\n", __func__); > + else > + dev_info(dev, "INT_MASK: 0x%04x\n", val); > + > + r = wl1273_fm_read_reg(core, WL1273_RDS_SYNC_GET, &val); > + if (r) > + dev_err(dev, "%s: Get RDS_SYNC fails.\n", > + __func__); > + else if (val == 0) > + dev_info(dev, "RDS_SYNC: Not synchronized\n"); > + > + else if (val == 1) > + dev_info(dev, "RDS_SYNC: Synchronized\n"); > + else > + dev_info(dev, "RDS_SYNC: Unexpected value: %d\n", val); > + > + r = wl1273_fm_read_reg(core, WL1273_I2S_MODE_CONFIG_SET, &val); > + if (r) > + dev_err(dev, "%s: Get I2S_MODE_CONFIG fails.\n", > + __func__); > + else > + dev_info(dev, "I2S_MODE_CONFIG: 0x%04x\n", val); > + > + r = wl1273_fm_read_reg(core, WL1273_VOLUME_SET, &val); > + if (r) > + dev_err(dev, "%s: Get VOLUME fails.\n", __func__); > + else > + dev_info(dev, "VOLUME: 0x%04x\n", val); > + } > + > + return 0; > +} > + > +static void wl1273_vdev_release(struct video_device *dev) > +{ > +} > + > +static const struct v4l2_ioctl_ops wl1273_ioctl_ops = { > + .vidioc_querycap = wl1273_fm_vidioc_querycap, > + .vidioc_g_input = wl1273_fm_vidioc_g_input, > + .vidioc_s_input = wl1273_fm_vidioc_s_input, > + .vidioc_queryctrl = wl1273_fm_vidioc_queryctrl, > + .vidioc_querymenu = wl1273_fm_vidioc_querymenu, > + .vidioc_g_ctrl = wl1273_fm_vidioc_g_ctrl, > + .vidioc_s_ctrl = wl1273_fm_vidioc_s_ctrl, > + .vidioc_g_audio = wl1273_fm_vidioc_g_audio, > + .vidioc_s_audio = wl1273_fm_vidioc_s_audio, > + .vidioc_g_tuner = wl1273_fm_vidioc_g_tuner, > + .vidioc_s_tuner = wl1273_fm_vidioc_s_tuner, > + .vidioc_g_frequency = wl1273_fm_vidioc_g_frequency, > + .vidioc_s_frequency = wl1273_fm_vidioc_s_frequency, > + .vidioc_s_hw_freq_seek = wl1273_fm_vidioc_s_hw_freq_seek, > + .vidioc_g_modulator = wl1273_fm_vidioc_g_modulator, > + .vidioc_s_modulator = wl1273_fm_vidioc_s_modulator, > + .vidioc_log_status = wl1273_fm_vidioc_log_status, > +}; > + > +static struct video_device wl1273_viddev_template = { > + .fops = &wl1273_fops, > + .ioctl_ops = &wl1273_ioctl_ops, > + .name = WL1273_FM_DRIVER_NAME, > + .release = wl1273_vdev_release, > +}; > + > +static int wl1273_fm_radio_remove(struct platform_device *pdev) > +{ > + struct wl1273_device *radio = platform_get_drvdata(pdev);; > + > + dev_info(&pdev->dev, "%s.\n", __func__); > + > + video_unregister_device(&radio->videodev); > + v4l2_device_unregister(&radio->v4l2dev); > + kfree(radio->write_buf); > + kfree(radio); > + > + return 0; > +} > + > +static int __devinit wl1273_fm_radio_probe(struct platform_device *pdev) > +{ > + struct wl1273_core **pdata = pdev->dev.platform_data; > + struct wl1273_device *radio; > + int r = 0; > + > + dev_dbg(&pdev->dev, "%s\n", __func__); > + > + if (!pdata) { > + dev_err(&pdev->dev, "No platform data.\n"); > + return -EINVAL; > + } > + > + radio = kzalloc(sizeof(*radio), GFP_KERNEL); > + if (!radio) > + return -ENOMEM; > + > + radio->write_buf = kmalloc(256, GFP_KERNEL); > + if (!radio->write_buf) > + return -ENOMEM; > + > + radio->core = *pdata; > + radio->dev = &pdev->dev; > + radio->rds_on = false; > + > + r = v4l2_device_register(&pdev->dev, &radio->v4l2dev); > + if (r) { > + dev_err(&pdev->dev, "Cannot register v4l2_device.\n"); > + goto err_device_alloc; > + } > + > + /* V4L2 configuration */ > + memcpy(&radio->videodev, &wl1273_viddev_template, > + sizeof(wl1273_viddev_template)); > + > + radio->videodev.v4l2_dev = &radio->v4l2dev; > + > + /* register video device */ > + r = video_register_device(&radio->videodev, VFL_TYPE_RADIO, radio_nr); > + if (r) { > + dev_err(&pdev->dev, WL1273_FM_DRIVER_NAME > + ": Could not register video device\n"); > + goto err_video_register; > + } > + > + video_set_drvdata(&radio->videodev, radio); > + platform_set_drvdata(pdev, radio); > + > + return 0; > + > +err_video_register: > + v4l2_device_unregister(&radio->v4l2dev); > +err_device_alloc: > + kfree(radio); > + > + return r; > +} > + > +MODULE_ALIAS("platform:wl1273_fm_radio"); > + > +static struct platform_driver wl1273_fm_radio_driver = { > + .probe = wl1273_fm_radio_probe, > + .remove = __devexit_p(wl1273_fm_radio_remove), > + .driver = { > + .name = "wl1273_fm_radio", > + .owner = THIS_MODULE, > + }, > +}; > + > +static int __init wl1273_fm_module_init(void) > +{ > + pr_info("%s\n", __func__); > + return platform_driver_register(&wl1273_fm_radio_driver); > +} > +module_init(wl1273_fm_module_init); > + > +static void __exit wl1273_fm_module_exit(void) > +{ > + flush_scheduled_work(); > + platform_driver_unregister(&wl1273_fm_radio_driver); > + pr_info(DRIVER_DESC ", Exiting.\n"); > +} > +module_exit(wl1273_fm_module_exit); > + > +MODULE_AUTHOR("Matti Aaltonen <matti.j.aalto...@nokia.com>"); > +MODULE_DESCRIPTION(DRIVER_DESC); > +MODULE_LICENSE("GPL"); > diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig > index 413576a..c16d500 100644 > --- a/drivers/mfd/Kconfig > +++ b/drivers/mfd/Kconfig > @@ -135,6 +135,12 @@ config TWL4030_CODEC > select MFD_CORE > default n > > +config WL1273_CORE > + tristate > + depends on I2C > + select MFD_CORE > + default n > + > config MFD_TMIO > bool > default n > diff --git a/drivers/mfd/Makefile b/drivers/mfd/Makefile > index 78295d6..46e611d 100644 > --- a/drivers/mfd/Makefile > +++ b/drivers/mfd/Makefile > @@ -30,6 +30,8 @@ obj-$(CONFIG_TWL4030_CORE) += twl-core.o twl4030-irq.o > twl6030-irq.o > obj-$(CONFIG_TWL4030_POWER) += twl4030-power.o > obj-$(CONFIG_TWL4030_CODEC) += twl4030-codec.o > > +obj-$(CONFIG_WL1273_CORE) += wl1273-core.o > + > obj-$(CONFIG_MFD_MC13783) += mc13783-core.o > > obj-$(CONFIG_MFD_CORE) += mfd-core.o > Regards, Hans -- Hans Verkuil - video4linux developer - sponsored by TANDBERG, part of Cisco -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html