[U-Boot-Users] [U-boot] [PATCH 2/2] ds174x: Fix warning on return in rtc_get and rtc_set function

[Wolfgang Denk]

In message <1207116238-7253-3-git-send-email-plagnioj at jcrosoft.com> you wrote:
> ds174x.c: In function 'rtc_get':
> ds174x.c:117: warning: no return statement in function returning non-void
> ds174x.c: In function 'rtc_set':
> ds174x.c:146: warning: 'return' with a value, in function returning void
> 
> Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj at jcrosoft.com>
> 
> diff --git a/drivers/rtc/ds174x.c b/drivers/rtc/ds174x.c
> index 81a9cb3..eb3ca88 100644
> --- a/drivers/rtc/ds174x.c
> +++ b/drivers/rtc/ds174x.c
> @@ -114,6 +114,7 @@ int rtc_get( struct rtc_time *tmp )
>  		tmp->tm_year, tmp->tm_mon, tmp->tm_mday, tmp->tm_wday,
>  		tmp->tm_hour, tmp->tm_min, tmp->tm_sec );
>  #endif
> +	return 0;
>  }
>  
>  void rtc_set( struct rtc_time *tmp )
> @@ -142,8 +143,6 @@ void rtc_set( struct rtc_time *tmp )
>  
>  	/* unlock clock registers after read */
>  	rtc_write( RTC_CONTROLA, ( reg_a  & ~RTC_CA_WRITE ));
> -
> -	return 0;
>  }
>  
>  void rtc_reset (void)

I think this is actually not the right fix.

To make things right, both rtc_set() and rtc_get() should return
"int".

Yes, I'm aware that this is far beyond the scope of your fixes here,
but I wanted to at least note that.

Note that this is no NAK.



Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
The word "fit", as I understand it, means "appropriate to a purpose",
and I would say the body of the Dean is supremely appropriate to  the
purpose of sitting around all day and eating big heavy meals.
                                 - Terry Pratchett, _Moving Pictures_