On Fri, 2022-11-04 at 11:28 +0200, Antoniu Miclaus wrote: > The ADF4377 is a high performance, ultralow jitter, dual output > integer-N > phased locked loop (PLL) with integrated voltage controlled > oscillator > (VCO) ideally suited for data converter and mixed signal front end > (MxFE) > clock applications. > > Datasheet: > https://www.analog.com/media/en/technical-documentation/data-sheets/adf4377.pdf > Signed-off-by: Antoniu Miclaus <antoniu.miclaus@xxxxxxxxxx> > --- > drivers/iio/frequency/Kconfig | 10 + > drivers/iio/frequency/Makefile | 1 + > drivers/iio/frequency/adf4377.c | 1154 > +++++++++++++++++++++++++++++++ > 3 files changed, 1165 insertions(+) > create mode 100644 drivers/iio/frequency/adf4377.c > > diff --git a/drivers/iio/frequency/Kconfig > b/drivers/iio/frequency/Kconfig > index f3702f36436c..9e85dfa58508 100644 > --- a/drivers/iio/frequency/Kconfig > +++ b/drivers/iio/frequency/Kconfig > @@ -50,6 +50,16 @@ config ADF4371 > To compile this driver as a module, choose M here: the > module will be called adf4371. > > +config ADF4377 > + tristate "Analog Devices ADF4377 Microwave Wideband > Synthesizer" > + depends on SPI && COMMON_CLK > + help > + Say yes here to build support for Analog Devices ADF4377 > Microwave > + Wideband Synthesizer. > + > + To compile this driver as a module, choose M here: the > + module will be called adf4377. > + > config ADMV1013 > tristate "Analog Devices ADMV1013 Microwave Upconverter" > depends on SPI && COMMON_CLK > diff --git a/drivers/iio/frequency/Makefile > b/drivers/iio/frequency/Makefile > index 48add732f1d3..b616c29b4a08 100644 > --- a/drivers/iio/frequency/Makefile > +++ b/drivers/iio/frequency/Makefile > @@ -7,6 +7,7 @@ > obj-$(CONFIG_AD9523) += ad9523.o > obj-$(CONFIG_ADF4350) += adf4350.o > obj-$(CONFIG_ADF4371) += adf4371.o > +obj-$(CONFIG_ADF4377) += adf4377.o > obj-$(CONFIG_ADMV1013) += admv1013.o > obj-$(CONFIG_ADMV1014) += admv1014.o > obj-$(CONFIG_ADMV4420) += admv4420.o > diff --git a/drivers/iio/frequency/adf4377.c > b/drivers/iio/frequency/adf4377.c > new file mode 100644 > index 000000000000..1901dde1003e > --- /dev/null > +++ b/drivers/iio/frequency/adf4377.c > @@ -0,0 +1,1154 @@ > +// SPDX-License-Identifier: GPL-2.0-only > +/* > + * ADF4377 driver > + * > + * Copyright 2022 Analog Devices Inc. > + */ > + > +#include <linux/bitfield.h> > +#include <linux/bits.h> > +#include <linux/clk.h> > +#include <linux/clkdev.h> > +#include <linux/delay.h> > +#include <linux/device.h> > +#include <linux/module.h> > +#include <linux/notifier.h> > +#include <linux/property.h> > +#include <linux/spi/spi.h> > +#include <linux/iio/iio.h> > +#include <linux/regmap.h> > +#include <linux/units.h> > + > +#include <linux/gpio/consumer.h> > + > +/* ADF4377 REG0000 Map */ > +#define ADF4377_SOFT_RESET_R_MSK BIT(7) > +#define ADF4377_LSB_FIRST_R_MSK BIT(6) > +#define ADF4377_ADDRESS_ASC_R_MSK BIT(5) > +#define ADF4377_SDO_ACTIVE_R_MSK BIT(4) > +#define ADF4377_SDO_ACTIVE_MSK BIT(3) > +#define ADF4377_ADDRESS_ASC_MSK BIT(2) > +#define ADF4377_LSB_FIRST_MSK BIT(1) > +#define ADF4377_SOFT_RESET_MSK BIT(0) > + > +/* ADF4377 REG0000 Bit Definition */ > +#define ADF4377_SDO_ACTIVE_SPI_3W 0x0 > +#define ADF4377_SDO_ACTIVE_SPI_4W 0x1 > + > +#define ADF4377_ADDR_ASC_AUTO_DECR 0x0 > +#define ADF4377_ADDR_ASC_AUTO_INCR 0x1 > + > +#define ADF4377_LSB_FIRST_MSB 0x0 > +#define ADF4377_LSB_FIRST_LSB 0x1 > + > +#define ADF4377_SOFT_RESET_N_OP 0x0 > +#define ADF4377_SOFT_RESET_EN 0x1 > + > +/* ADF4377 REG0001 Map */ > +#define ADF4377_SINGLE_INSTR_MSK BIT(7) > +#define ADF4377_MASTER_RB_CTRL_MSK BIT(5) > + > +/* ADF4377 REG0001 Bit Definition */ > +#define ADF4377_SPI_STREAM_EN 0x0 > +#define ADF4377_SPI_STREAM_DIS 0x1 > + > +#define ADF4377_RB_SLAVE_REG 0x0 > +#define ADF4377_RB_MASTER_REG 0x1 > + > +/* ADF4377 REG0003 Bit Definition */ > +#define ADF4377_CHIP_TYPE 0x06 > + > +/* ADF4377 REG0004 Bit Definition */ > +#define ADF4377_PRODUCT_ID_LSB 0x0005 > + > +/* ADF4377 REG0005 Bit Definition */ > +#define ADF4377_PRODUCT_ID_MSB 0x0005 > + > +/* ADF4377 REG000A Map */ > +#define ADF4377_SCRATCHPAD_MSK GENMASK(7, 0) > + > +/* ADF4377 REG000C Bit Definition */ > +#define ADF4377_VENDOR_ID_LSB 0x56 > + > +/* ADF4377 REG000D Bit Definition */ > +#define ADF4377_VENDOR_ID_MSB 0x04 > + > +/* ADF4377 REG000F Bit Definition */ > +#define ADF4377_R00F_RSV1_MSK GENMASK(7, 0) > + > +/* ADF4377 REG0010 Map*/ > +#define ADF4377_N_INT_LSB_MSK GENMASK(7, 0) > + > +/* ADF4377 REG0011 Map*/ > +#define ADF4377_EN_AUTOCAL_MSK BIT(7) > +#define ADF4377_EN_RDBLR_MSK BIT(6) > +#define ADF4377_DCLK_DIV2_MSK GENMASK(5, 4) > +#define ADF4377_N_INT_MSB_MSK GENMASK(3, 0) > + > +/* ADF4377 REG0011 Bit Definition */ > +#define ADF4377_VCO_CALIB_DIS 0x0 > +#define ADF4377_VCO_CALIB_EN 0x1 > + > +#define ADF4377_REF_DBLR_DIS 0x0 > +#define ADF4377_REF_DBLR_EN 0x1 > + > +#define ADF4377_DCLK_DIV2_1 0x0 > +#define ADF4377_DCLK_DIV2_2 0x1 > +#define ADF4377_DCLK_DIV2_4 0x2 > +#define ADF4377_DCLK_DIV2_8 0x3 > + > +/* ADF4377 REG0012 Map*/ > +#define ADF4377_CLKOUT_DIV_MSK GENMASK(7, 6) > +#define ADF4377_R_DIV_MSK GENMASK(5, 0) > + > +/* ADF4377 REG0012 Bit Definition */ > +#define ADF4377_CLKOUT_DIV_1 0x0 > +#define ADF4377_CLKOUT_DIV_2 0x1 > +#define ADF4377_CLKOUT_DIV_4 0x2 > +#define ADF4377_CLKOUT_DIV_8 0x3 > + > +/* ADF4377 REG0013 Map */ > +#define ADF4377_M_VCO_CORE_MSK GENMASK(5, 4) > +#define ADF4377_M_VCO_BIAS_MSK GENMASK(3, 0) > + > +/* ADF4377 REG0013 Bit Definition */ > +#define ADF4377_M_VCO_0 0x0 > +#define ADF4377_M_VCO_1 0x1 > +#define ADF4377_M_VCO_2 0x2 > +#define ADF4377_M_VCO_3 0x3 > + > +/* ADF4377 REG0014 Map */ > +#define ADF4377_M_VCO_BAND_MSK GENMASK(7, 0) > + > +/* ADF4377 REG0015 Map */ > +#define ADF4377_BLEED_I_LSB_MSK GENMASK(7, 6) > +#define ADF4377_BLEED_POL_MSK BIT(5) > +#define ADF4377_EN_BLEED_MSK BIT(4) > +#define ADF4377_CP_I_MSK GENMASK(3, 0) > + > +/* ADF4377 REG0015 Bit Definition */ > +#define ADF4377_CURRENT_SINK 0x0 > +#define ADF4377_CURRENT_SOURCE 0x1 > + > +#define ADF4377_BLEED_CURR_DIS 0x0 > +#define ADF4377_BLEED_CURR_EN 0x1 > + > +#define ADF4377_CP_0MA7 0x0 > +#define ADF4377_CP_0MA9 0x1 > +#define ADF4377_CP_1MA1 0x2 > +#define ADF4377_CP_1MA3 0x3 > +#define ADF4377_CP_1MA4 0x4 > +#define ADF4377_CP_1MA8 0x5 > +#define ADF4377_CP_2MA2 0x6 > +#define ADF4377_CP_2MA5 0x7 > +#define ADF4377_CP_2MA9 0x8 > +#define ADF4377_CP_3MA6 0x9 > +#define ADF4377_CP_4MA3 0xA > +#define ADF4377_CP_5MA0 0xB > +#define ADF4377_CP_5MA7 0xC > +#define ADF4377_CP_7MA2 0xD > +#define ADF4377_CP_8MA6 0xE > +#define ADF4377_CP_10MA1 0xF > + > +/* ADF4377 REG0016 Map */ > +#define ADF4377_BLEED_I_MSB_MSK GENMASK(7, 0) > + > +/* ADF4377 REG0017 Map */ > +#define ADF4377_INV_CLKOUT_MSK BIT(7) > +#define ADF4377_N_DEL_MSK GENMASK(6, 0) > + > +/* ADF4377 REG0017 Bit Definition */ > +#define ADF4377_CLKOUT_INV_DIS 0x0 > +#define ADF4377_CLKOUT_INV_EN 0x1 > + > +/* ADF4377 REG0018 Map */ > +#define ADF4377_CMOS_OV_MSK BIT(7) > +#define ADF4377_R_DEL_MSK GENMASK(6, 0) > + > +/* ADF4377 REG0018 Bit Definition */ > +#define ADF4377_1V8_LOGIC 0x0 > +#define ADF4377_3V3_LOGIC 0x1 > + > +/* ADF4377 REG0019 Map */ > +#define ADF4377_CLKOUT2_OP_MSK GENMASK(7, 6) > +#define ADF4377_CLKOUT1_OP_MSK GENMASK(5, 4) > +#define ADF4377_PD_CLK_MSK BIT(3) > +#define ADF4377_PD_RDET_MSK BIT(2) > +#define ADF4377_PD_ADC_MSK BIT(1) > +#define ADF4377_PD_CALADC_MSK BIT(0) > + > +/* ADF4377 REG0019 Bit Definition */ > +#define ADF4377_CLKOUT_320MV 0x0 > +#define ADF4377_CLKOUT_420MV 0x1 > +#define ADF4377_CLKOUT_530MV 0x2 > +#define ADF4377_CLKOUT_640MV 0x3 > + > +#define ADF4377_PD_CLK_N_OP 0x0 > +#define ADF4377_PD_CLK_PD 0x1 > + > +#define ADF4377_PD_RDET_N_OP 0x0 > +#define ADF4377_PD_RDET_PD 0x1 > + > +#define ADF4377_PD_ADC_N_OP 0x0 > +#define ADF4377_PD_ADC_PD 0x1 > + > +#define ADF4377_PD_CALADC_N_OP 0x0 > +#define ADF4377_PD_CALADC_PD 0x1 > + > +/* ADF4377 REG001A Map */ > +#define ADF4377_PD_ALL_MSK BIT(7) > +#define ADF4377_PD_RDIV_MSK BIT(6) > +#define ADF4377_PD_NDIV_MSK BIT(5) > +#define ADF4377_PD_VCO_MSK BIT(4) > +#define ADF4377_PD_LD_MSK BIT(3) > +#define ADF4377_PD_PFDCP_MSK BIT(2) > +#define ADF4377_PD_CLKOUT1_MSK BIT(1) > +#define ADF4377_PD_CLKOUT2_MSK BIT(0) > + > +/* ADF4377 REG001A Bit Definition */ > +#define ADF4377_PD_ALL_N_OP 0x0 > +#define ADF4377_PD_ALL_PD 0x1 > + > +#define ADF4377_PD_RDIV_N_OP 0x0 > +#define ADF4377_PD_RDIV_PD 0x1 > + > +#define ADF4377_PD_NDIV_N_OP 0x0 > +#define ADF4377_PD_NDIV_PD 0x1 > + > +#define ADF4377_PD_VCO_N_OP 0x0 > +#define ADF4377_PD_VCO_PD 0x1 > + > +#define ADF4377_PD_LD_N_OP 0x0 > +#define ADF4377_PD_LD_PD 0x1 > + > +#define ADF4377_PD_PFDCP_N_OP 0x0 > +#define ADF4377_PD_PFDCP_PD 0x1 > + > +#define ADF4377_PD_CLKOUT1_N_OP 0x0 > +#define ADF4377_PD_CLKOUT1_PD 0x1 > + > +#define ADF4377_PD_CLKOUT2_N_OP 0x0 > +#define ADF4377_PD_CLKOUT2_PD 0x1 > + > +/* ADF4377 REG001B Map */ > +#define ADF4377_EN_LOL_MSK BIT(7) > +#define ADF4377_LDWIN_PW_MSK BIT(6) > +#define ADF4377_EN_LDWIN_MSK BIT(5) > +#define ADF4377_LD_COUNT_MSK GENMASK(4, 0) > + > +/* ADF4377 REG001B Bit Definition */ > +#define ADF4377_EN_LOL_DIS 0x0 > +#define ADF4377_EN_LOL_EN 0x1 > + > +#define ADF4377_LDWIN_PW_NARROW 0x0 > +#define ADF4377_LDWIN_PW_WIDE 0x1 > + > +#define ADF4377_EN_LDWIN_DIS 0x0 > +#define ADF4377_EN_LDWIN_EN 0x1 > + > +/* ADF4377 REG001C Map */ > +#define ADF4377_EN_DNCLK_MSK BIT(7) > +#define ADF4377_EN_DRCLK_MSK BIT(6) > +#define ADF4377_RST_LD_MSK BIT(2) > +#define ADF4377_R01C_RSV1_MSK BIT(0) > + > +/* ADF4377 REG001C Bit Definition */ > +#define ADF4377_EN_DNCLK_OFF 0x0 > +#define ADF4377_EN_DNCLK_ON 0x1 > + > +#define ADF4377_EN_DRCLK_OFF 0x0 > +#define ADF4377_EN_DRCLK_ON 0x1 > + > +#define ADF4377_RST_LD_INACTIVE 0x0 > +#define ADF4377_RST_LD_ACTIVE 0x1 > + > +/* ADF4377 REG001D Map */ > +#define ADF4377_MUXOUT_MSK GENMASK(7, 4) > +#define ADF4377_EN_CPTEST_MSK BIT(2) > +#define ADF4377_CP_DOWN_MSK BIT(1) > +#define ADF4377_CP_UP_MSK BIT(0) > + > +#define ADF4377_EN_CPTEST_OFF 0x0 > +#define ADF4377_EN_CPTEST_ON 0x1 > + > +#define ADF4377_CP_DOWN_OFF 0x0 > +#define ADF4377_CP_DOWN_ON 0x1 > + > +#define ADF4377_CP_UP_OFF 0x0 > +#define ADF4377_CP_UP_ON 0x1 > + > +/* ADF4377 REG001F Map */ > +#define ADF4377_BST_REF_MSK BIT(7) > +#define ADF4377_FILT_REF_MSK BIT(6) > +#define ADF4377_REF_SEL_MSK BIT(5) > +#define ADF4377_R01F_RSV1_MSK GENMASK(4, 0) > + > +/* ADF4377 REG001F Bit Definition */ > +#define ADF4377_BST_LARGE_REF_IN 0x0 > +#define ADF4377_BST_SMALL_REF_IN 0x1 > + > +#define ADF4377_FILT_REF_OFF 0x0 > +#define ADF4377_FILT_REF_ON 0x1 > + > +#define ADF4377_REF_SEL_DMA 0x0 > +#define ADF4377_REF_SEL_LNA 0x1 > + > +/* ADF4377 REG0020 Map */ > +#define ADF4377_RST_SYS_MSK BIT(4) > +#define ADF4377_EN_ADC_CLK_MSK BIT(3) > +#define ADF4377_R020_RSV1_MSK BIT(0) > + > +/* ADF4377 REG0020 Bit Definition */ > +#define ADF4377_RST_SYS_INACTIVE 0x0 > +#define ADF4377_RST_SYS_ACTIVE 0x1 > + > +#define ADF4377_EN_ADC_CLK_DIS 0x0 > +#define ADF4377_EN_ADC_CLK_EN 0x1 > + > +/* ADF4377 REG0021 Bit Definition */ > +#define ADF4377_R021_RSV1 0xD3 > + > +/* ADF4377 REG0022 Bit Definition */ > +#define ADF4377_R022_RSV1 0x32 > + > +/* ADF4377 REG0023 Map */ > +#define ADF4377_CAT_CT_SEL BIT(7) > +#define ADF4377_R023_RSV1_MSK GENMASK(6, 0) > + > +/* ADF4377 REG0024 Map */ > +#define ADF4377_DCLK_MODE_MSK BIT(2) > + > +/* ADF4377 REG0024 Bit Definition */ > +#define ADF4377_DCLK_MODE_DIS 0x0 > +#define ADF4377_DCLK_MODE_EN 0x1 > + > +/* ADF4377 REG0025 Map */ > +#define ADF4377_CLKODIV_DB_MSK BIT(7) > +#define ADF4377_DCLK_DB_MSK BIT(6) > +#define ADF4377_R025_RSV1_MSK GENMASK(5, 0) > + > +/* ADF4377 REG0025 Bit Definition */ > +#define ADF4377_CLKODIV_DB_DIS 0x0 > +#define ADF4377_CLKODIV_DB_EN 0x1 > + > +#define ADF4377_DCLK_DIV_DB_DIS 0x0 > +#define ADF4377_DCLK_DIV_DB_EN 0x1 > + > +/* ADF4377 REG0026 Map */ > +#define ADF4377_VCO_BAND_DIV_MSK GENMASK(7, 0) > + > +/* ADF4377 REG0027 Map */ > +#define ADF4377_SYNTH_LOCK_TO_LSB_MSK GENMASK(7, 0) > + > +/* ADF4377 REG0028 Map */ > +#define ADF4377_O_VCO_DB_MSK BIT(7) > +#define ADF4377_SYNTH_LOCK_TO_MSB_MSK GENMASK(6, 0) > + > +/* ADF4377 REG0028 Bit Definition */ > +#define ADF4377_O_VCO_DB_DIS 0x0 > +#define ADF4377_O_VCO_DB_EN 0x1 > + > +/* ADF4377 REG0029 Map */ > +#define ADF4377_VCO_ALC_TO_LSB_MSK GENMASK(7, 0) > + > +/* ADF4377 REG002A Map */ > +#define ADF4377_DEL_CTRL_DB_MSK BIT(7) > +#define ADF4377_VCO_ALC_TO_MSB_MSK GENMASK(6, 0) > + > +/* ADF4377 REG002A Bit Definition */ > +#define ADF4377_DEL_CTRL_DB_DIS 0x0 > +#define ADF4377_DEL_CTRL_DB_EN 0x1 > + > +/* ADF4377 REG002C Map */ > +#define ADF4377_R02C_RSV1 0xC0 > + > +/* ADF4377 REG002D Map */ > +#define ADF4377_ADC_CLK_DIV_MSK GENMASK(7, 0) > + > +/* ADF4377 REG002E Map */ > +#define ADF4377_EN_ADC_CNV_MSK BIT(7) > +#define ADF4377_EN_ADC_MSK BIT(1) > +#define ADF4377_ADC_A_CONV_MSK BIT(0) > + > +/* ADF4377 REG002E Bit Definition */ > +#define ADF4377_EN_ADC_CNV_DIS 0x0 > +#define ADF4377_EN_ADC_CNV_EN 0x1 > + > +#define ADF4377_EN_ADC_DIS 0x0 > +#define ADF4377_EN_ADC_EN 0x1 > + > +#define ADF4377_ADC_A_CONV_ADC_ST_CNV 0x0 > +#define ADF4377_ADC_A_CONV_VCO_CALIB 0x1 > + > +/* ADF4377 REG002F Map */ > +#define ADF4377_DCLK_DIV1_MSK GENMASK(1, 0) > + > +/* ADF4377 REG002F Bit Definition */ > +#define ADF4377_DCLK_DIV1_1 0x0 > +#define ADF4377_DCLK_DIV1_2 0x1 > +#define ADF4377_DCLK_DIV1_8 0x2 > +#define ADF4377_DCLK_DIV1_32 0x3 > + > +/* ADF4377 REG0031 Bit Definition */ > +#define ADF4377_R031_RSV1 0x09 > + > +/* ADF4377 REG0032 Map */ > +#define ADF4377_ADC_CLK_SEL_MSK BIT(6) > +#define ADF4377_R032_RSV1_MSK GENMASK(5, 0) > + > +/* ADF4377 REG0032 Bit Definition */ > +#define ADF4377_R032_RSV1 0x9 > + > +/* ADF4377 REG0032 Bit Definition */ > +#define ADF4377_ADC_CLK_SEL_N_OP 0x0 > +#define ADF4377_ADC_CLK_SEL_SPI_CLK 0x1 > + > +/* ADF4377 REG0033 Bit Definition */ > +#define ADF4377_R033_RSV1 0x18 > + > +/* ADF4377 REG0034 Bit Definition */ > +#define ADF4377_R034_RSV1 0x08 > + > +/* ADF4377 REG003A Bit Definition */ > +#define ADF4377_R03A_RSV1 0x5D > + > +/* ADF4377 REG003B Bit Definition */ > +#define ADF4377_R03B_RSV1 0x2B > + > +/* ADF4377 REG003D Map */ > +#define ADF4377_O_VCO_BAND_MSK BIT(3) > +#define ADF4377_O_VCO_CORE_MSK BIT(2) > +#define ADF4377_O_VCO_BIAS_MSK BIT(1) > + > +/* ADF4377 REG003D Bit Definition */ > +#define ADF4377_O_VCO_BAND_VCO_CALIB 0x0 > +#define ADF4377_O_VCO_BAND_M_VCO 0x1 > + > +#define ADF4377_O_VCO_CORE_VCO_CALIB 0x0 > +#define ADF4377_O_VCO_CORE_M_VCO 0x1 > + > +#define ADF4377_O_VCO_BIAS_VCO_CALIB 0x0 > +#define ADF4377_O_VCO_BIAS_M_VCO 0x1 > + > +/* ADF4377 REG0042 Map */ > +#define ADF4377_R042_RSV1 0x05 > + > +/* ADF4377 REG0045 Map */ > +#define ADF4377_ADC_ST_CNV_MSK BIT(0) > + > +/* ADF4377 REG0045 Bit Definition */ > +#define ADF4377_ADC_ST_ADC_DIS 0x0 > +#define ADF4377_ADC_ST_ADC_EN 0x1 > + > +/* ADF4377 REG0049 Map */ > +#define ADF4377_EN_CLK2_MSK BIT(7) > +#define ADF4377_EN_CLK1_MSK BIT(6) > +#define ADF4377_REF_OK_MSK BIT(3) > +#define ADF4377_ADC_BUSY_MSK BIT(2) > +#define ADF4377_FSM_BUSY_MSK BIT(1) > +#define ADF4377_LOCKED_MSK BIT(0) > + > +/* ADF4377 REG004B Map */ > +#define ADF4377_VCO_CORE_MSK GENMASK(1, 0) > + > +/* ADF4377 REG004C Map */ > +#define ADF4377_CHIP_TEMP_LSB_MSK GENMASK(7, 0) > + > +/* ADF4377 REG004D Map */ > +#define ADF4377_CHIP_TEMP_MSB_MSK BIT(0) > + > +/* ADF4377 REG004F Map */ > +#define ADF4377_VCO_BAND_MSK GENMASK(7, 0) > + > +/* ADF4377 REG0051 Map */ > +#define ADF4377_VCO_BIAS_MSK GENMASK(3, 0) > + > +/* ADF4377 REG0054 Map */ > +#define ADF4377_CHIP_VERSION_MSK GENMASK(7, 0) > + > +/* Specifications */ > +#define ADF4377_SPI_READ_CMD BIT(7) > +#define ADF4377_MAX_VCO_FREQ (12800ULL * HZ_PER_MHZ) > +#define ADF4377_MIN_VCO_FREQ (6400ULL * HZ_PER_MHZ) > +#define ADF4377_MAX_REFIN_FREQ (1000 * HZ_PER_MHZ) > +#define ADF4377_MIN_REFIN_FREQ (10 * HZ_PER_MHZ) > +#define ADF4377_MAX_FREQ_PFD (500 * HZ_PER_MHZ) > +#define ADF4377_MIN_FREQ_PFD (3 * HZ_PER_MHZ) > +#define ADF4377_MAX_CLKPN_FREQ ADF4377_MAX_VCO_FREQ > +#define ADF4377_MIN_CLKPN_FREQ (ADF4377_MIN_VCO_FREQ / 8) > +#define ADF4377_FREQ_PFD_80MHZ (80 * HZ_PER_MHZ) > +#define ADF4377_FREQ_PFD_125MHZ (125 * HZ_PER_MHZ) > +#define ADF4377_FREQ_PFD_160MHZ (160 * HZ_PER_MHZ) > +#define ADF4377_FREQ_PFD_250MHZ (250 * HZ_PER_MHZ) > +#define ADF4377_FREQ_PFD_320MHZ (320 * HZ_PER_MHZ) > + > +enum adf4377_dev_type { > + ADF4377, > + ADF4378, > +}; > + > +enum { > + ADF4377_FREQ, > +}; > + > +enum muxout_select_modes { > + ADF4377_MUXOUT_HIGH_Z = 0x0, > + ADF4377_MUXOUT_LKDET = 0x1, > + ADF4377_MUXOUT_LOW = 0x2, > + ADF4377_MUXOUT_DIV_RCLK_2 = 0x4, > + ADF4377_MUXOUT_DIV_NCLK_2 = 0x5, > + ADF4377_MUXOUT_HIGH = 0x8, > +}; > + > +struct adf4377_state { > + struct spi_device *spi; > + enum adf4377_dev_type type; > + struct regmap *regmap; > + struct clk *clkin; > + /* Protect against concurrent accesses to the device and data > content */ > + struct mutex lock; > + struct notifier_block nb; > + /* Reference Divider */ > + unsigned int ref_div_factor; > + /* PFD Frequency */ > + unsigned int f_pfd; > + /* Input Reference Clock */ > + unsigned int clkin_freq; > + /* CLKOUT Divider */ > + u8 clkout_div_sel; > + /* Feedback Divider (N) */ > + u16 n_int; > + u16 synth_lock_timeout; > + u16 vco_alc_timeout; > + u16 adc_clk_div; > + u16 vco_band_div; > + u8 dclk_div1; > + u8 dclk_div2; > + u8 dclk_mode; > + unsigned int f_div_rclk; > + struct gpio_desc *gpio_ce; > + struct gpio_desc *gpio_enclk1; > + struct gpio_desc *gpio_enclk2; > + u8 buf[2] ____cacheline_aligned; > +}; > + > +static const char * const adf4377_muxout_modes[] = { > + [ADF4377_MUXOUT_HIGH_Z] = "high_z", > + [ADF4377_MUXOUT_LKDET] = "lock_detect", > + [ADF4377_MUXOUT_LOW] = "muxout_low", > + [ADF4377_MUXOUT_DIV_RCLK_2] = "f_div_rclk_2", > + [ADF4377_MUXOUT_DIV_NCLK_2] = "f_div_nclk_2", > + [ADF4377_MUXOUT_HIGH] = "muxout_high", > +}; > + > +static const struct regmap_config adf4377_regmap_config = { > + .reg_bits = 16, > + .val_bits = 8, > + .read_flag_mask = BIT(7), > + .max_register = 0x54, > +}; > + > +static int adf4377_reg_access(struct iio_dev *indio_dev, > + unsigned int reg, > + unsigned int write_val, > + unsigned int *read_val) > +{ > + struct adf4377_state *st = iio_priv(indio_dev); > + > + if (read_val) > + return regmap_read(st->regmap, reg, read_val); > + > + return regmap_write(st->regmap, reg, write_val); > +} > + > +static const struct iio_info adf4377_info = { > + .debugfs_reg_access = &adf4377_reg_access, > +}; > + > +static int adf4377_soft_reset(struct adf4377_state *st) > +{ > + unsigned int read_val; > + int ret; > + > + ret = regmap_update_bits(st->regmap, 0x0, > ADF4377_SOFT_RESET_MSK | > + ADF4377_SOFT_RESET_R_MSK, > + FIELD_PREP(ADF4377_SOFT_RESET_MSK, > 1) | > + FIELD_PREP(ADF4377_SOFT_RESET_R_MSK, > 1)); > + if (ret) > + return ret; > + > + return regmap_read_poll_timeout(st->regmap, 0x0, read_val, > + !(read_val & > (ADF4377_SOFT_RESET_R_MSK | > + ADF4377_SOFT_RESET_R_MSK)), > 200, 200 * 100); > +} > + > +static int adf4377_set_default(struct adf4377_state *st) > +{ > + int ret; > + > + /* Set default registers bits */ > + ret = regmap_write(st->regmap, 0x42, ADF4377_R042_RSV1); > + if (ret) > + return ret; > + > + ret = regmap_write(st->regmap, 0x3B, ADF4377_R03B_RSV1); > + if (ret) > + return ret; > + > + ret = regmap_write(st->regmap, 0x3A, ADF4377_R03A_RSV1); > + if (ret) > + return ret; > + > + ret = regmap_write(st->regmap, 0x34, ADF4377_R034_RSV1); > + if (ret) > + return ret; > + > + ret = regmap_write(st->regmap, 0x33, ADF4377_R033_RSV1); > + if (ret) > + return ret; > + > + ret = regmap_update_bits(st->regmap, 0x32, > ADF4377_R032_RSV1_MSK, > + FIELD_PREP(ADF4377_R032_RSV1_MSK, > 0x9)); > + if (ret) > + return ret; > + > + ret = regmap_write(st->regmap, 0x31, ADF4377_R031_RSV1); > + if (ret) > + return ret; > + > + ret = regmap_write(st->regmap, 0x2C, ADF4377_R02C_RSV1); > + if (ret) > + return ret; > + > + ret = regmap_update_bits(st->regmap, 0x25, > ADF4377_R025_RSV1_MSK, > + FIELD_PREP(ADF4377_R025_RSV1_MSK, > 0x16)); > + if (ret) > + return ret; > + > + ret = regmap_update_bits(st->regmap, 0x23, > ADF4377_R023_RSV1_MSK, > + FIELD_PREP(ADF4377_R023_RSV1_MSK, > 0x18)); > + if (ret) > + return ret; > + > + ret = regmap_write(st->regmap, 0x22, ADF4377_R022_RSV1); > + if (ret) > + return ret; > + > + ret = regmap_write(st->regmap, 0x21, ADF4377_R021_RSV1); > + if (ret) > + return ret; > + > + ret = regmap_update_bits(st->regmap, 0x1f, > ADF4377_R01F_RSV1_MSK, > + FIELD_PREP(ADF4377_R01F_RSV1_MSK, > 0x7)); > + if (ret) > + return ret; > + > + return regmap_update_bits(st->regmap, 0x1c, > ADF4377_R01C_RSV1_MSK, > + FIELD_PREP(ADF4377_R01C_RSV1_MSK, > 0x1)); > +} > + > +int adf4377_get_freq(struct adf4377_state *st, u64 *freq) > +{ > + unsigned int ref_div_factor, n_int; > + u64 clkin_freq; > + int ret; > + > + mutex_lock(&st->lock); > + ret = regmap_read(st->regmap, 0x12, &ref_div_factor); > + if (ret) > + goto exit; > + > + ret = regmap_bulk_read(st->regmap, 0x10, st->buf, sizeof(st- > >buf)); > + if (ret) > + goto exit; > + > + clkin_freq = clk_get_rate(st->clkin); > + ref_div_factor = FIELD_GET(ADF4377_R_DIV_MSK, > ref_div_factor); > + n_int = FIELD_GET(ADF4377_N_INT_LSB_MSK, st->buf[0]) | > + (FIELD_GET(ADF4377_N_INT_MSB_MSK, st->buf[1]) << 8); > + Maybe we could refactor this to use the proper get_unaligned() call? I think we would also need one FIELD_GET() in that case... > + *freq = div_u64(clkin_freq, ref_div_factor) * n_int; > +exit: > + mutex_unlock(&st->lock); > + > + return ret; > +} > + > +int adf4377_set_freq(struct adf4377_state *st, u64 freq) > +{ > + unsigned int read_val; > + u64 f_vco; > + int ret; > + > + ret = regmap_update_bits(st->regmap, 0x1C, > ADF4377_EN_DNCLK_MSK | ADF4377_EN_DRCLK_MSK, > + FIELD_PREP(ADF4377_EN_DNCLK_MSK, > ADF4377_EN_DNCLK_ON) | > + FIELD_PREP(ADF4377_EN_DRCLK_MSK, > ADF4377_EN_DRCLK_ON)); > + if (ret) > + return ret; > + > + ret = regmap_update_bits(st->regmap, 0x11, > ADF4377_EN_AUTOCAL_MSK | ADF4377_DCLK_DIV2_MSK, > + FIELD_PREP(ADF4377_EN_AUTOCAL_MSK, > ADF4377_VCO_CALIB_EN) | > + FIELD_PREP(ADF4377_DCLK_DIV2_MSK, > st->dclk_div2)); > + if (ret) > + return ret; > + > + ret = regmap_update_bits(st->regmap, 0x2E, > ADF4377_EN_ADC_CNV_MSK | ADF4377_EN_ADC_MSK | > + ADF4377_ADC_A_CONV_MSK, > + FIELD_PREP(ADF4377_EN_ADC_CNV_MSK, > ADF4377_EN_ADC_CNV_EN) | > + FIELD_PREP(ADF4377_EN_ADC_MSK, > ADF4377_EN_ADC_EN) | > + FIELD_PREP(ADF4377_ADC_A_CONV_MSK, > ADF4377_ADC_A_CONV_VCO_CALIB)); > + if (ret) > + return ret; > + > + ret = regmap_update_bits(st->regmap, 0x20, > ADF4377_EN_ADC_CLK_MSK, > + FIELD_PREP(ADF4377_EN_ADC_CLK_MSK, > ADF4377_EN_ADC_CLK_EN)); > + if (ret) > + return ret; > + > + ret = regmap_update_bits(st->regmap, 0x2F, > ADF4377_DCLK_DIV1_MSK, > + FIELD_PREP(ADF4377_DCLK_DIV1_MSK, > st->dclk_div1)); > + if (ret) > + return ret; > + > + ret = regmap_update_bits(st->regmap, 0x24, > ADF4377_DCLK_MODE_MSK, > + FIELD_PREP(ADF4377_DCLK_MODE_MSK, > st->dclk_mode)); > + if (ret) > + return ret; > + > + ret = regmap_write(st->regmap, 0x27, > + FIELD_PREP(ADF4377_SYNTH_LOCK_TO_LSB_MSK, > st->synth_lock_timeout)); > + if (ret) > + return ret; > + > + ret = regmap_update_bits(st->regmap, 0x28, > ADF4377_SYNTH_LOCK_TO_MSB_MSK, > + > FIELD_PREP(ADF4377_SYNTH_LOCK_TO_MSB_MSK, > + st->synth_lock_timeout >> > 8)); > + if (ret) > + return ret; > + > + ret = regmap_write(st->regmap, 0x29, > + FIELD_PREP(ADF4377_VCO_ALC_TO_LSB_MSK, st- > >vco_alc_timeout)); > + if (ret) > + return ret; > + > + ret = regmap_update_bits(st->regmap, 0x2A, > ADF4377_VCO_ALC_TO_MSB_MSK, > + > FIELD_PREP(ADF4377_VCO_ALC_TO_MSB_MSK, st->vco_alc_timeout >> 8)); > + if (ret) > + return ret; > + > + ret = regmap_write(st->regmap, 0x26, > + FIELD_PREP(ADF4377_VCO_BAND_DIV_MSK, st- > >vco_band_div)); > + if (ret) > + return ret; > + > + ret = regmap_write(st->regmap, 0x2D, > + FIELD_PREP(ADF4377_ADC_CLK_DIV_MSK, st- > >adc_clk_div)); > + if (ret) > + return ret; > + > + st->clkout_div_sel = 0; > + > + if (freq > ADF4377_MAX_CLKPN_FREQ || freq < > ADF4377_MIN_CLKPN_FREQ) { > + ret = -EINVAL; > + return ret; > + } > + > + f_vco = freq; > + > + while (f_vco < ADF4377_MIN_VCO_FREQ) { > + f_vco <<= 1; > + st->clkout_div_sel++; > + } > + > + st->n_int = div_u64(freq, st->f_pfd); > + > + ret = regmap_update_bits(st->regmap, 0x11, > ADF4377_EN_RDBLR_MSK | ADF4377_N_INT_MSB_MSK, > + FIELD_PREP(ADF4377_EN_RDBLR_MSK, > ADF4377_REF_DBLR_DIS) | > + FIELD_PREP(ADF4377_N_INT_MSB_MSK, > st->n_int >> 8)); > + if (ret) > + return ret; > + > + ret = regmap_update_bits(st->regmap, 0x12, ADF4377_R_DIV_MSK > | ADF4377_CLKOUT_DIV_MSK, > + FIELD_PREP(ADF4377_CLKOUT_DIV_MSK, > st->clkout_div_sel) | > + FIELD_PREP(ADF4377_R_DIV_MSK, st- > >ref_div_factor)); > + if (ret) > + return ret; > + > + ret = regmap_write(st->regmap, 0x10, > + FIELD_PREP(ADF4377_N_INT_LSB_MSK, st- > >n_int)); > + if (ret) > + return ret; > + > + ret = regmap_read_poll_timeout(st->regmap, 0x49, read_val, > + !(read_val & > (ADF4377_FSM_BUSY_MSK)), 200, 200 * 100); > + if (ret) > + return ret; > + > + /* Disable EN_DNCLK, EN_DRCLK */ > + ret = regmap_update_bits(st->regmap, 0x1C, > ADF4377_EN_DNCLK_MSK | ADF4377_EN_DRCLK_MSK, > + FIELD_PREP(ADF4377_EN_DNCLK_MSK, > ADF4377_EN_DNCLK_OFF) | > + FIELD_PREP(ADF4377_EN_DRCLK_MSK, > ADF4377_EN_DRCLK_OFF)); > + if (ret) > + return ret; > + > + /* Disable EN_ADC_CLK */ > + ret = regmap_update_bits(st->regmap, 0x20, > ADF4377_EN_ADC_CLK_MSK, > + FIELD_PREP(ADF4377_EN_ADC_CLK_MSK, > ADF4377_EN_ADC_CLK_DIS)); > + if (ret) > + return ret; > + > + /* Set output Amplitude */ > + return regmap_update_bits(st->regmap, 0x19, > ADF4377_CLKOUT2_OP_MSK | ADF4377_CLKOUT1_OP_MSK, > + FIELD_PREP(ADF4377_CLKOUT1_OP_MSK, > ADF4377_CLKOUT_420MV) | > + FIELD_PREP(ADF4377_CLKOUT2_OP_MSK, > ADF4377_CLKOUT_420MV)); > +} > + > +static void adf4377_gpio_init(struct adf4377_state *st) > +{ > + if (st->gpio_ce) { > + gpiod_set_value(st->gpio_ce, 1); > + > + /* Delay for SPI register bits to settle to their > power-on reset state */ > + usleep_range(200, 250); fsleep()? > + } > + > + if (st->gpio_enclk1) > + gpiod_set_value(st->gpio_enclk1, 1); > + > + if (st->gpio_enclk2 && st->type == ADF4377) > + gpiod_set_value(st->gpio_enclk2, 1); > +} > + > +static int adf4377_init(struct adf4377_state *st) > +{ > + int ret; > + > + /* GPIO Inititalization */ > + adf4377_gpio_init(st); > + > + /* Software reset */ > + ret = adf4377_soft_reset(st); > + if (ret) > + return ret; > + > + /* Set Default Registers */ > + ret = adf4377_set_default(st); > + if (ret) > + return ret; > + > + ret = regmap_update_bits(st->regmap, 0x15, ADF4377_CP_I_MSK, > + FIELD_PREP(ADF4377_CP_I_MSK, > ADF4377_CP_10MA1)); > + if (ret) > + return ret; > + > + ret = regmap_update_bits(st->regmap, 0x00, we should have some meaningful names for the registers... > + ADF4377_SDO_ACTIVE_MSK | > ADF4377_SDO_ACTIVE_R_MSK, > + FIELD_PREP(ADF4377_SDO_ACTIVE_MSK, > ADF4377_SDO_ACTIVE_SPI_4W) | > + FIELD_PREP(ADF4377_SDO_ACTIVE_R_MSK, > ADF4377_SDO_ACTIVE_SPI_4W)); > + if (ret) > + return ret; > + > + st->clkin_freq = clk_get_rate(st->clkin); > + > + /* Power Up */ > + ret = regmap_write(st->regmap, 0x1a, > + FIELD_PREP(ADF4377_PD_ALL_MSK, > ADF4377_PD_ALL_N_OP) | > + FIELD_PREP(ADF4377_PD_RDIV_MSK, > ADF4377_PD_RDIV_N_OP) | > + FIELD_PREP(ADF4377_PD_NDIV_MSK, > ADF4377_PD_NDIV_N_OP) | > + FIELD_PREP(ADF4377_PD_VCO_MSK, > ADF4377_PD_VCO_N_OP) | > + FIELD_PREP(ADF4377_PD_LD_MSK, > ADF4377_PD_LD_N_OP) | > + FIELD_PREP(ADF4377_PD_PFDCP_MSK, > ADF4377_PD_PFDCP_N_OP) | > + FIELD_PREP(ADF4377_PD_CLKOUT1_MSK, > ADF4377_PD_CLKOUT1_N_OP) | > + FIELD_PREP(ADF4377_PD_CLKOUT2_MSK, > ADF4377_PD_CLKOUT2_N_OP)); > + if (ret) > + return ret; > + > + /* Compute PFD */ > + st->ref_div_factor = 0; > + do { > + st->ref_div_factor++; > + st->f_pfd = st->clkin_freq / st->ref_div_factor; > + } while (st->f_pfd > ADF4377_MAX_FREQ_PFD); > + > + if (st->f_pfd > ADF4377_MAX_FREQ_PFD || st->f_pfd < > ADF4377_MIN_FREQ_PFD) > + return -EINVAL; > + > + st->f_div_rclk = st->f_pfd; > + > + if (st->f_pfd <= ADF4377_FREQ_PFD_80MHZ) { > + st->dclk_div1 = ADF4377_DCLK_DIV1_1; > + st->dclk_div2 = ADF4377_DCLK_DIV2_1; > + st->dclk_mode = ADF4377_DCLK_MODE_DIS; > + } else if (st->f_pfd <= ADF4377_FREQ_PFD_125MHZ) { > + st->dclk_div1 = ADF4377_DCLK_DIV1_1; > + st->dclk_div2 = ADF4377_DCLK_DIV2_1; > + st->dclk_mode = ADF4377_DCLK_MODE_EN; > + } else if (st->f_pfd <= ADF4377_FREQ_PFD_160MHZ) { > + st->dclk_div1 = ADF4377_DCLK_DIV1_2; > + st->dclk_div2 = ADF4377_DCLK_DIV2_1; > + st->dclk_mode = ADF4377_DCLK_MODE_DIS; > + st->f_div_rclk /= 2; > + } else if (st->f_pfd <= ADF4377_FREQ_PFD_250MHZ) { > + st->dclk_div1 = ADF4377_DCLK_DIV1_2; > + st->dclk_div2 = ADF4377_DCLK_DIV2_1; > + st->dclk_mode = ADF4377_DCLK_MODE_EN; > + st->f_div_rclk /= 2; > + } else if (st->f_pfd <= ADF4377_FREQ_PFD_320MHZ) { > + st->dclk_div1 = ADF4377_DCLK_DIV1_2; > + st->dclk_div2 = ADF4377_DCLK_DIV2_2; > + st->dclk_mode = ADF4377_DCLK_MODE_DIS; > + st->f_div_rclk /= 4; > + } else { > + st->dclk_div1 = ADF4377_DCLK_DIV1_2; > + st->dclk_div2 = ADF4377_DCLK_DIV2_2; > + st->dclk_mode = ADF4377_DCLK_MODE_EN; > + st->f_div_rclk /= 4; > + } > + > + st->synth_lock_timeout = DIV_ROUND_UP(st->f_div_rclk, 50000); > + st->vco_alc_timeout = DIV_ROUND_UP(st->f_div_rclk, 20000); > + st->vco_band_div = DIV_ROUND_UP(st->f_div_rclk, 150000 * 16 * > (1 << st->dclk_mode)); > + st->adc_clk_div = DIV_ROUND_UP((st->f_div_rclk / 400000 - 2), > 4); > + > + return 0; > +} > + > +static ssize_t adf4377_read(struct iio_dev *indio_dev, uintptr_t > private, > + const struct iio_chan_spec *chan, char > *buf) > +{ > + struct adf4377_state *st = iio_priv(indio_dev); > + u64 val = 0; > + int ret; > + > + switch ((u32)private) { > + case ADF4377_FREQ: > + ret = adf4377_get_freq(st, &val); > + break; > + default: > + ret = -EINVAL; > + val = 0; > + break; > + } > + > + return ret ?: sysfs_emit(buf, "%llu\n", val); I would also return in place. I've come to prefer it but that's me :) > +} > + > +static ssize_t adf4377_write(struct iio_dev *indio_dev, uintptr_t > private, > + const struct iio_chan_spec *chan, const > char *buf, > + size_t len) > +{ > + struct adf4377_state *st = iio_priv(indio_dev); > + unsigned long long freq; > + int ret; > + > + mutex_lock(&st->lock); > + switch ((u32)private) { > + case ADF4377_FREQ: > + ret = kstrtoull(buf, 10, &freq); > + if (ret) > + break; > + > + ret = adf4377_set_freq(st, freq); > + break; > + default: > + ret = -EINVAL; > + break; > + } > + I think we can just return in place on the each switch()... We can just move the lock inside adf4377_set_freq()? > + mutex_unlock(&st->lock); > + > + return ret ? ret : len; > +} > + > +static int adf4377_set_muxout_mode(struct iio_dev *indio_dev, > + const struct iio_chan_spec *chan, > + unsigned int mode) > +{ > + struct adf4377_state *st = iio_priv(indio_dev); > + > + return regmap_update_bits(st->regmap, 0x1D, > + ADF4377_MUXOUT_MSK, > + FIELD_PREP(ADF4377_MUXOUT_MSK, > mode)); > +} > + > +static int adf4377_get_muxout_mode(struct iio_dev *indio_dev, > + const struct iio_chan_spec *chan) > +{ > + struct adf4377_state *st = iio_priv(indio_dev); > + unsigned int mode; > + int ret; > + > + ret = regmap_read(st->regmap, 0x1D, &mode); > + Maybe something more descriptive than 0x1D? > + return ret ? : FIELD_GET(ADF4377_MUXOUT_MSK, mode); > +} > + > +static const struct iio_enum adf4377_muxout_enum = { > + .items = adf4377_muxout_modes, > + .num_items = ARRAY_SIZE(adf4377_muxout_modes), > + .get = adf4377_get_muxout_mode, > + .set = adf4377_set_muxout_mode, > +}; > + > +#define _ADF4377_EXT_INFO(_name, _shared, _ident) { \ > + .name = _name, \ > + .read = adf4377_read, \ > + .write = adf4377_write, \ > + .private = _ident, \ > + .shared = _shared, \ > + } > + > +static const struct iio_chan_spec_ext_info adf4377_ext_info[] = { > + /* > + * Usually we use IIO_CHAN_INFO_FREQUENCY, but there are > + * values > 2^32 in order to support the entire frequency > range > + * in Hz. > + */ > + _ADF4377_EXT_INFO("frequency", IIO_SHARED_BY_ALL, > ADF4377_FREQ), Can't we have u64 already in IIO_CHAN_INFO_FREQUENCY? I know the write side is a bit awkward but I think we can make it better. > + IIO_ENUM("muxout_select", IIO_SHARED_BY_ALL, > &adf4377_muxout_enum), > + IIO_ENUM_AVAILABLE("muxout_select", IIO_SHARED_BY_ALL, > &adf4377_muxout_enum), > + { }, > +}; > + > +static const struct iio_chan_spec adf4377_channels[] = { > + { > + .type = IIO_ALTVOLTAGE, > + .indexed = 1, > + .output = 1, > + .channel = 0, > + .ext_info = adf4377_ext_info, > + }, > +}; > + > +static int adf4377_properties_parse(struct adf4377_state *st) > +{ > + struct spi_device *spi = st->spi; > + > + st->clkin = devm_clk_get(&spi->dev, "ref_in"); > + if (IS_ERR(st->clkin)) > + return dev_err_probe(&spi->dev, PTR_ERR(st->clkin), > + "failed to get the reference > input clock\n"); > + > + st->gpio_ce = devm_gpiod_get_optional(&st->spi->dev, "ce-en", > + GPIOD_OUT_HIGH); > + if (IS_ERR(st->gpio_ce)) > + return dev_err_probe(&spi->dev, PTR_ERR(st->gpio_ce), > + "failed to get the CE GPIO\n"); > + > + st->gpio_enclk1 = devm_gpiod_get_optional(&st->spi->dev, > "enclk1", > + GPIOD_OUT_HIGH); > + if (IS_ERR(st->gpio_enclk1)) > + return dev_err_probe(&spi->dev, PTR_ERR(st- > >gpio_enclk1), > + "failed to get the CE GPIO\n"); > + > + if (st->type == ADF4377) { > + st->gpio_enclk2 = devm_gpiod_get_optional(&st->spi- > >dev, "enclk2", > + > GPIOD_OUT_HIGH); > + if (IS_ERR(st->gpio_enclk2)) > + return dev_err_probe(&spi->dev, PTR_ERR(st- > >gpio_enclk2), > + "failed to get the CE > GPIO\n"); > + } > + > + return 0; > +} > + > +static int adf4377_freq_change(struct notifier_block *nb, unsigned > long action, void *data) > +{ > + struct adf4377_state *st = container_of(nb, struct > adf4377_state, nb); > + int ret; > + > + if (action == POST_RATE_CHANGE) { > + mutex_lock(&st->lock); > + ret = notifier_from_errno(adf4377_init(st)); > + mutex_unlock(&st->lock); > + return ret; > + } > + > + return NOTIFY_OK; > +} > + > +static void adf4377_clk_disable(void *data) > +{ > + clk_disable_unprepare(data); > +} > + > +static int adf4377_probe(struct spi_device *spi) > +{ > + struct iio_dev *indio_dev; > + struct regmap *regmap; > + struct adf4377_state *st; > + int ret; > + > + indio_dev = devm_iio_device_alloc(&spi->dev, sizeof(*st)); > + if (!indio_dev) > + return -ENOMEM; > + > + regmap = devm_regmap_init_spi(spi, &adf4377_regmap_config); > + if (IS_ERR(regmap)) > + return PTR_ERR(regmap); > + > + st = iio_priv(indio_dev); > + > + indio_dev->info = &adf4377_info; > + indio_dev->name = "adf4377"; > + indio_dev->channels = adf4377_channels; > + indio_dev->num_channels = ARRAY_SIZE(adf4377_channels); > + > + st->regmap = regmap; > + st->spi = spi; > + st->type = spi_get_device_id(spi)->driver_data; Hmm this is something that came up internally the other day. Are we guaranteed that this will always work? For OF I think it is but I'm not sure about ACPI? At first glance, it seems that it might be ok but I did not went too deep in the ACPI code. > + mutex_init(&st->lock); > + > + ret = adf4377_properties_parse(st); > + if (ret) > + return ret; > + > + ret = clk_prepare_enable(st->clkin); > + if (ret) > + return ret; > + > + ret = devm_add_action_or_reset(&spi->dev, > adf4377_clk_disable, st->clkin); > + if (ret) > + return ret; > + > + st->nb.notifier_call = adf4377_freq_change; > + ret = devm_clk_notifier_register(&spi->dev, st->clkin, &st- > >nb); > + if (ret) > + return ret; > + > + ret = adf4377_init(st); > + if (ret) { > + dev_err(&spi->dev, "adf4377 init failed\n"); I would make sure to print all the error paths (that matter) in adf4377_init() and just return in here... - Nuno Sá