[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