Re: [PATCH v4 36/48] mfd: ti_am335x_tscadc: Add ADC1/magnetic reader support

[Miquel Raynal <miquel.raynal@bootlin.com>]

Hello Lee,

- reduced a bit the Cc: list

lkp@intel.com wrote on Wed, 29 Sep 2021 06:32:16 +0800:

> Hi Miquel,
> 
> I love your patch! Perhaps something to improve:
> 
> [auto build test WARNING on lee-mfd/for-mfd-next]
> [also build test WARNING on jic23-iio/togreg robh/for-next v5.15-rc3 next-20210922]
> [If your patch is applied to the wrong git tree, kindly drop us a note.
> And when submitting patch, we suggest to use '--base' as documented in
> https://git-scm.com/docs/git-format-patch]
> 
> url:    https://github.com/0day-ci/linux/commits/Miquel-Raynal/TI-AM437X-ADC1/20210928-213524
> base:   https://git.kernel.org/pub/scm/linux/kernel/git/lee/mfd.git for-mfd-next
> config: microblaze-randconfig-r033-20210928 (attached as .config)
> compiler: microblaze-linux-gcc (GCC) 11.2.0
> reproduce (this is a W=1 build):
>         wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>         chmod +x ~/bin/make.cross
>         # https://github.com/0day-ci/linux/commit/43a01cf6413f2be038b0d466c7c3f6f16b40e2c3
>         git remote add linux-review https://github.com/0day-ci/linux
>         git fetch --no-tags linux-review Miquel-Raynal/TI-AM437X-ADC1/20210928-213524
>         git checkout 43a01cf6413f2be038b0d466c7c3f6f16b40e2c3
>         # save the attached .config to linux build tree
>         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross ARCH=microblaze 
> 
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp@intel.com>
> 
> All warnings (new ones prefixed by >>):
> 
>    drivers/mfd/ti_am335x_tscadc.c: In function 'ti_tscadc_probe':
> >> drivers/mfd/ti_am335x_tscadc.c:124:31: warning: variable 'use_mag' set but not used [-Wunused-but-set-variable]  
>      124 |         bool use_tsc = false, use_mag = false;
>          |                               ^~~~~~~

It's strange, on my side I didn't get any warnings (using an older GCC,
perhaps not with all the flags enabled either). Anyway, let's remove
the noise, I have a fixup patch which can be applied without disturbing
any of the other patches.

Lee, do you prefer that I send a fixup patch or a v5 for patch "mfd:
ti_am335x_tscadc: Add ADC1/magnetic reader support"? (provided of
course that the other patches are fine to you).

>    In file included from drivers/mfd/ti_am335x_tscadc.c:20:
>    drivers/mfd/ti_am335x_tscadc.c: At top level:
>    include/linux/mfd/ti_am335x_tscadc.h:126:38: error: 'HZ_PER_MHZ' undeclared here (not in a function)
>      126 | #define TSC_ADC_CLK             (3 * HZ_PER_MHZ)
>          |                                      ^~~~~~~~~~
>    drivers/mfd/ti_am335x_tscadc.c:354:28: note: in expansion of macro 'TSC_ADC_CLK'
>      354 |         .target_clk_rate = TSC_ADC_CLK,
>          |                            ^~~~~~~~~~~

For this one I don't believe this is a real warning, units.h is
introduced in a precedent patch, HZ_PER_MHZ is a new macro but it is
upstream now so I think it can safely be discarded.

Thanks,
Miquèl