On Wed, Jul 18, 2018 at 3:49 PM, Finn Thain fthain@telegraphics.com.au wrote:
On Wed, 18 Jul 2018, Arnd Bergmann wrote:
-static long via_read_time(void) +static time64_t via_read_time(void) { union { __u8 cdata[4];
long idata;
__u32 idata; } result, last_result;time64_t ret;ret isn't used.
int count = 1; via_pram_command(0x81, &last_result.cdata[3]);@@ -279,12 +280,8 @@ static long via_read_time(void) via_pram_command(0x89, &result.cdata[1]); via_pram_command(0x8D, &result.cdata[0]);
if (result.idata == last_result.idata) {if (result.idata < RTC_OFFSET)result.idata += 0x100000000ull;return result.idata - RTC_OFFSET;}
if (result.idata == last_result.idata)return (time64_t(result.idata) - RTC_OFFSET);Did you mean to write,
return (time64_t)result.idata - RTC_OFFSET;?
Right, I should have at least tried to build it again.
Arnd