On 14/03/22 09:04, Kever Yang wrote:
Hi Giulio,

     Thanks for your patch, it's a good catching.
     Could you please update the subject with "rockchip: adc: ", and
add a simple description in the commit message? Empy commit message is
not allowed according to U-Boot patch commit rules :(

Oh, I've forgotten about this in u-boot. Just sent V2:
https://patchwork.ozlabs.org/project/uboot/patch/20220314090943.821630-1-giulio.bene...@benettiengineering.com/

and tagged V1 as superseded in patchwork.

Best regards
--
Giulio Benetti
Benetti Engineering sas

Thanks,
- Kever

Giulio Benetti <giulio.bene...@benettiengineering.com> 于2022年2月4日周五 09:19写道:

Signed-off-by: Giulio Benetti <giulio.bene...@benettiengineering.com>
---
  drivers/adc/rockchip-saradc.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/adc/rockchip-saradc.c b/drivers/adc/rockchip-saradc.c
index e464d33f22..e0cbab6aa0 100644
--- a/drivers/adc/rockchip-saradc.c
+++ b/drivers/adc/rockchip-saradc.c
@@ -131,7 +131,7 @@ int rockchip_saradc_of_to_plat(struct udevice *dev)
         }

         priv->data = data;
-       uc_pdata->data_mask = (1 << priv->data->num_bits) - 1;;
+       uc_pdata->data_mask = (1 << priv->data->num_bits) - 1;
         uc_pdata->data_format = ADC_DATA_FORMAT_BIN;
         uc_pdata->data_timeout_us = SARADC_TIMEOUT / 5;
         uc_pdata->channel_mask = (1 << priv->data->num_channels) - 1;
--
2.25.1


Reply via email to