[U-Boot] [PATCH 06/20] adc: stm32-adc: Fix warnings when compiling with W=1
Fabrice Gasnier
fabrice.gasnier at st.com
Fri Jun 21 13:40:45 UTC 2019
On 6/21/19 3:26 PM, Patrick Delaunay wrote:
> This patch solves the following warnings:
>
> drivers/adc/stm32-adc.c: In function 'stm32_adc_chan_of_init':
> warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
> if (num_channels > adc->cfg->max_channels) {
> ^
> Signed-off-by: Patrice Chotard <patrice.chotard at st.com>
>
> Signed-off-by: Patrick Delaunay <patrick.delaunay at st.com>
Hi Patrick,
Acked-by: Fabrice Gasnier <fabrice.gasnier at st.com>
Thanks,
Fabrice
> ---
>
> drivers/adc/stm32-adc.c | 13 +++++++------
> 1 file changed, 7 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/adc/stm32-adc.c b/drivers/adc/stm32-adc.c
> index e108062..029338e 100644
> --- a/drivers/adc/stm32-adc.c
> +++ b/drivers/adc/stm32-adc.c
> @@ -163,15 +163,16 @@ static int stm32_adc_chan_of_init(struct udevice *dev)
> struct adc_uclass_platdata *uc_pdata = dev_get_uclass_platdata(dev);
> struct stm32_adc *adc = dev_get_priv(dev);
> u32 chans[STM32_ADC_CH_MAX];
> - int i, num_channels, ret;
> + unsigned int i, num_channels;
> + int ret;
>
> /* Retrieve single ended channels listed in device tree */
> - num_channels = dev_read_size(dev, "st,adc-channels");
> - if (num_channels < 0) {
> - dev_err(dev, "can't get st,adc-channels: %d\n", num_channels);
> - return num_channels;
> + ret = dev_read_size(dev, "st,adc-channels");
> + if (ret < 0) {
> + dev_err(dev, "can't get st,adc-channels: %d\n", ret);
> + return ret;
> }
> - num_channels /= sizeof(u32);
> + num_channels = ret / sizeof(u32);
>
> if (num_channels > adc->cfg->max_channels) {
> dev_err(dev, "too many st,adc-channels: %d\n", num_channels);
>
More information about the U-Boot
mailing list