tjh.dev / kernel
Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux
1
fork

Configure Feed

Select the types of activity you want to include in your feed.

iio: frequency: ad9523: fix multi-line dereferences

Platform data pointer dereferences for pll1_charge_pump_current_nA and
pll2_charge_pump_current_nA were split across multiple lines. Bring the
dereference chains onto a single line.

This resolves the following checkpatch.pl warnings:
WARNING: Avoid multiple line dereference - prefer 'pdata->pll1_charge_pump_current_nA'
WARNING: Avoid multiple line dereference - prefer 'pdata->pll2_charge_pump_current_nA'

Signed-off-by: Bhargav Joshi <rougueprince47@gmail.com>
Reviewed-by: Andy Shevchenko <andriy.shevchenko@intel.com>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>

authored by

Bhargav Joshi and committed by
Jonathan Cameron 787c9a9c e830a889

+2 -4
+2 -4
drivers/iio/frequency/ad9523.c
··· 797 797 return ret; 798 798 799 799 ret = ad9523_write(indio_dev, AD9523_PLL1_CHARGE_PUMP_CTRL, 800 - AD9523_PLL1_CHARGE_PUMP_CURRENT_nA(pdata-> 801 - pll1_charge_pump_current_nA) | 800 + AD9523_PLL1_CHARGE_PUMP_CURRENT_nA(pdata->pll1_charge_pump_current_nA) | 802 801 AD9523_PLL1_CHARGE_PUMP_MODE_NORMAL | 803 802 AD9523_PLL1_BACKLASH_PW_MIN); 804 803 if (ret < 0) ··· 841 842 */ 842 843 843 844 ret = ad9523_write(indio_dev, AD9523_PLL2_CHARGE_PUMP, 844 - AD9523_PLL2_CHARGE_PUMP_CURRENT_nA(pdata-> 845 - pll2_charge_pump_current_nA)); 845 + AD9523_PLL2_CHARGE_PUMP_CURRENT_nA(pdata->pll2_charge_pump_current_nA)); 846 846 if (ret < 0) 847 847 return ret; 848 848