[U-Boot] [PATCH 3/9] dm: rtc: Correct rtc_read32() return value

Bin Meng bmeng.cn at gmail.com
Mon Oct 19 04:23:03 CEST 2015


On Mon, Oct 19, 2015 at 5:55 AM, Simon Glass <sjg at chromium.org> wrote:
> The current check is incorrect and will fail when any non-zero byte is read.
> Fix it.
>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
>
>  drivers/rtc/rtc-uclass.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/rtc/rtc-uclass.c b/drivers/rtc/rtc-uclass.c
> index fe74c69..300e9b3 100644
> --- a/drivers/rtc/rtc-uclass.c
> +++ b/drivers/rtc/rtc-uclass.c
> @@ -68,7 +68,7 @@ int rtc_read32(struct udevice *dev, unsigned int reg, u32 *valuep)
>
>         for (i = 0; i < sizeof(value); i++) {
>                 ret = rtc_read8(dev, reg + i);
> -               if (ret)
> +               if (ret < 0)
>                         return ret;
>                 value |= ret << (i << 3);
>         }
> --

Reviewed-by: Bin Meng <bmeng.cn at gmail.com>


More information about the U-Boot mailing list