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: Sat, 5 Feb 2022 17:20:54 +0000 From: Jonathan Cameron <jic23@...nel.org> To: Andy Shevchenko <andriy.shevchenko@...ux.intel.com> Cc: linux-iio@...r.kernel.org, linux-kernel@...r.kernel.org, Lars-Peter Clausen <lars@...afoo.de>, Michael Hennerich <Michael.Hennerich@...log.com> Subject: Re: [PATCH v2 1/1] iio: frequency: adf4350: Make use of device properties On Thu, 3 Feb 2022 13:00:06 +0200 Andy Shevchenko <andriy.shevchenko@...ux.intel.com> wrote: > Convert the module to be property provider agnostic and allow > it to be used on non-OF platforms. > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@...ux.intel.com> Whilst it's long each individual change is simple, so applied to the togreg branch of iio.git and pushed out as testing to see if the autobuilders can find anything we missed. Thanks, Jonathan > --- > v2: fixed typo (lkp) > drivers/iio/frequency/adf4350.c | 103 +++++++++++++------------------- > 1 file changed, 42 insertions(+), 61 deletions(-) > > diff --git a/drivers/iio/frequency/adf4350.c b/drivers/iio/frequency/adf4350.c > index 3d9eba716b69..9af20a51540d 100644 > --- a/drivers/iio/frequency/adf4350.c > +++ b/drivers/iio/frequency/adf4350.c > @@ -7,17 +7,18 @@ > > #include <linux/device.h> > #include <linux/kernel.h> > +#include <linux/mod_devicetable.h> > +#include <linux/module.h> > +#include <linux/property.h> > #include <linux/slab.h> > #include <linux/sysfs.h> > #include <linux/spi/spi.h> > #include <linux/regulator/consumer.h> > #include <linux/err.h> > -#include <linux/module.h> > #include <linux/gcd.h> > #include <linux/gpio/consumer.h> > #include <asm/div64.h> > #include <linux/clk.h> > -#include <linux/of.h> > > #include <linux/iio/iio.h> > #include <linux/iio/sysfs.h> > @@ -381,10 +382,8 @@ static const struct iio_info adf4350_info = { > .debugfs_reg_access = &adf4350_reg_access, > }; > > -#ifdef CONFIG_OF > static struct adf4350_platform_data *adf4350_parse_dt(struct device *dev) > { > - struct device_node *np = dev->of_node; > struct adf4350_platform_data *pdata; > unsigned int tmp; > > @@ -392,101 +391,83 @@ static struct adf4350_platform_data *adf4350_parse_dt(struct device *dev) > if (!pdata) > return NULL; > > - snprintf(&pdata->name[0], SPI_NAME_SIZE - 1, "%pOFn", np); > + snprintf(pdata->name, sizeof(pdata->name), "%pfw", dev_fwnode(dev)); > > tmp = 10000; > - of_property_read_u32(np, "adi,channel-spacing", &tmp); > + device_property_read_u32(dev, "adi,channel-spacing", &tmp); > pdata->channel_spacing = tmp; > > tmp = 0; > - of_property_read_u32(np, "adi,power-up-frequency", &tmp); > + device_property_read_u32(dev, "adi,power-up-frequency", &tmp); > pdata->power_up_frequency = tmp; > > tmp = 0; > - of_property_read_u32(np, "adi,reference-div-factor", &tmp); > + device_property_read_u32(dev, "adi,reference-div-factor", &tmp); > pdata->ref_div_factor = tmp; > > - pdata->ref_doubler_en = of_property_read_bool(np, > - "adi,reference-doubler-enable"); > - pdata->ref_div2_en = of_property_read_bool(np, > - "adi,reference-div2-enable"); > + pdata->ref_doubler_en = device_property_read_bool(dev, "adi,reference-doubler-enable"); > + pdata->ref_div2_en = device_property_read_bool(dev, "adi,reference-div2-enable"); > > /* r2_user_settings */ > - pdata->r2_user_settings = of_property_read_bool(np, > - "adi,phase-detector-polarity-positive-enable") ? > - ADF4350_REG2_PD_POLARITY_POS : 0; > - pdata->r2_user_settings |= of_property_read_bool(np, > - "adi,lock-detect-precision-6ns-enable") ? > - ADF4350_REG2_LDP_6ns : 0; > - pdata->r2_user_settings |= of_property_read_bool(np, > - "adi,lock-detect-function-integer-n-enable") ? > - ADF4350_REG2_LDF_INT_N : 0; > + pdata->r2_user_settings = 0; > + if (device_property_read_bool(dev, "adi,phase-detector-polarity-positive-enable")) > + pdata->r2_user_settings |= ADF4350_REG2_PD_POLARITY_POS; > + if (device_property_read_bool(dev, "adi,lock-detect-precision-6ns-enable")) > + pdata->r2_user_settings |= ADF4350_REG2_LDP_6ns; > + if (device_property_read_bool(dev, "adi,lock-detect-function-integer-n-enable")) > + pdata->r2_user_settings |= ADF4350_REG2_LDF_INT_N; > > tmp = 2500; > - of_property_read_u32(np, "adi,charge-pump-current", &tmp); > + device_property_read_u32(dev, "adi,charge-pump-current", &tmp); > pdata->r2_user_settings |= ADF4350_REG2_CHARGE_PUMP_CURR_uA(tmp); > > tmp = 0; > - of_property_read_u32(np, "adi,muxout-select", &tmp); > + device_property_read_u32(dev, "adi,muxout-select", &tmp); > pdata->r2_user_settings |= ADF4350_REG2_MUXOUT(tmp); > > - pdata->r2_user_settings |= of_property_read_bool(np, > - "adi,low-spur-mode-enable") ? > - ADF4350_REG2_NOISE_MODE(0x3) : 0; > + if (device_property_read_bool(dev, "adi,low-spur-mode-enable")) > + pdata->r2_user_settings |= ADF4350_REG2_NOISE_MODE(0x3); > > /* r3_user_settings */ > > - pdata->r3_user_settings = of_property_read_bool(np, > - "adi,cycle-slip-reduction-enable") ? > - ADF4350_REG3_12BIT_CSR_EN : 0; > - pdata->r3_user_settings |= of_property_read_bool(np, > - "adi,charge-cancellation-enable") ? > - ADF4351_REG3_CHARGE_CANCELLATION_EN : 0; > - > - pdata->r3_user_settings |= of_property_read_bool(np, > - "adi,anti-backlash-3ns-enable") ? > - ADF4351_REG3_ANTI_BACKLASH_3ns_EN : 0; > - pdata->r3_user_settings |= of_property_read_bool(np, > - "adi,band-select-clock-mode-high-enable") ? > - ADF4351_REG3_BAND_SEL_CLOCK_MODE_HIGH : 0; > + pdata->r3_user_settings = 0; > + if (device_property_read_bool(dev, "adi,cycle-slip-reduction-enable")) > + pdata->r3_user_settings |= ADF4350_REG3_12BIT_CSR_EN; > + if (device_property_read_bool(dev, "adi,charge-cancellation-enable")) > + pdata->r3_user_settings |= ADF4351_REG3_CHARGE_CANCELLATION_EN; > + if (device_property_read_bool(dev, "adi,anti-backlash-3ns-enable")) > + pdata->r3_user_settings |= ADF4351_REG3_ANTI_BACKLASH_3ns_EN; > + if (device_property_read_bool(dev, "adi,band-select-clock-mode-high-enable")) > + pdata->r3_user_settings |= ADF4351_REG3_BAND_SEL_CLOCK_MODE_HIGH; > > tmp = 0; > - of_property_read_u32(np, "adi,12bit-clk-divider", &tmp); > + device_property_read_u32(dev, "adi,12bit-clk-divider", &tmp); > pdata->r3_user_settings |= ADF4350_REG3_12BIT_CLKDIV(tmp); > > tmp = 0; > - of_property_read_u32(np, "adi,clk-divider-mode", &tmp); > + device_property_read_u32(dev, "adi,clk-divider-mode", &tmp); > pdata->r3_user_settings |= ADF4350_REG3_12BIT_CLKDIV_MODE(tmp); > > /* r4_user_settings */ > > - pdata->r4_user_settings = of_property_read_bool(np, > - "adi,aux-output-enable") ? > - ADF4350_REG4_AUX_OUTPUT_EN : 0; > - pdata->r4_user_settings |= of_property_read_bool(np, > - "adi,aux-output-fundamental-enable") ? > - ADF4350_REG4_AUX_OUTPUT_FUND : 0; > - pdata->r4_user_settings |= of_property_read_bool(np, > - "adi,mute-till-lock-enable") ? > - ADF4350_REG4_MUTE_TILL_LOCK_EN : 0; > + pdata->r4_user_settings = 0; > + if (device_property_read_bool(dev, "adi,aux-output-enable")) > + pdata->r4_user_settings |= ADF4350_REG4_AUX_OUTPUT_EN; > + if (device_property_read_bool(dev, "adi,aux-output-fundamental-enable")) > + pdata->r4_user_settings |= ADF4350_REG4_AUX_OUTPUT_FUND; > + if (device_property_read_bool(dev, "adi,mute-till-lock-enable")) > + pdata->r4_user_settings |= ADF4350_REG4_MUTE_TILL_LOCK_EN; > > tmp = 0; > - of_property_read_u32(np, "adi,output-power", &tmp); > + device_property_read_u32(dev, "adi,output-power", &tmp); > pdata->r4_user_settings |= ADF4350_REG4_OUTPUT_PWR(tmp); > > tmp = 0; > - of_property_read_u32(np, "adi,aux-output-power", &tmp); > + device_property_read_u32(dev, "adi,aux-output-power", &tmp); > pdata->r4_user_settings |= ADF4350_REG4_AUX_OUTPUT_PWR(tmp); > > return pdata; > } > -#else > -static > -struct adf4350_platform_data *adf4350_parse_dt(struct device *dev) > -{ > - return NULL; > -} > -#endif >
Powered by blists - more mailing lists