* [PATCH] tee: replace getnstimeofday64() with ktime_get_real_ts64()
@ 2018-06-18 14:24 Arnd Bergmann
2018-06-19 13:37 ` Jens Wiklander
0 siblings, 1 reply; 4+ messages in thread
From: Arnd Bergmann @ 2018-06-18 14:24 UTC (permalink / raw)
To: Jens Wiklander
Cc: y2038, Arnd Bergmann, Jerome Forissier, Etienne Carriere,
Joakim Bech, Volodymyr Babchuk, tiger-yu99, David Wang,
linux-kernel
The two do the same thing, but we want to have a consistent
naming in the kernel.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
drivers/tee/optee/rpc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/tee/optee/rpc.c b/drivers/tee/optee/rpc.c
index 41aea12e2bcc..b45c73dd37a5 100644
--- a/drivers/tee/optee/rpc.c
+++ b/drivers/tee/optee/rpc.c
@@ -48,7 +48,7 @@ static void handle_rpc_func_cmd_get_time(struct optee_msg_arg *arg)
OPTEE_MSG_ATTR_TYPE_VALUE_OUTPUT)
goto bad;
- getnstimeofday64(&ts);
+ ktime_get_real_ts64(&ts);
arg->params[0].u.value.a = ts.tv_sec;
arg->params[0].u.value.b = ts.tv_nsec;
--
2.9.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] tee: replace getnstimeofday64() with ktime_get_real_ts64()
2018-06-18 14:24 [PATCH] tee: replace getnstimeofday64() with ktime_get_real_ts64() Arnd Bergmann
@ 2018-06-19 13:37 ` Jens Wiklander
2018-06-19 14:05 ` Arnd Bergmann
0 siblings, 1 reply; 4+ messages in thread
From: Jens Wiklander @ 2018-06-19 13:37 UTC (permalink / raw)
To: Arnd Bergmann
Cc: y2038, Jerome Forissier, Etienne Carriere, Joakim Bech,
Volodymyr Babchuk, tiger-yu99, David Wang, linux-kernel
On Mon, Jun 18, 2018 at 04:24:56PM +0200, Arnd Bergmann wrote:
> The two do the same thing, but we want to have a consistent
> naming in the kernel.
>
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> ---
> drivers/tee/optee/rpc.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/tee/optee/rpc.c b/drivers/tee/optee/rpc.c
> index 41aea12e2bcc..b45c73dd37a5 100644
> --- a/drivers/tee/optee/rpc.c
> +++ b/drivers/tee/optee/rpc.c
> @@ -48,7 +48,7 @@ static void handle_rpc_func_cmd_get_time(struct optee_msg_arg *arg)
> OPTEE_MSG_ATTR_TYPE_VALUE_OUTPUT)
> goto bad;
>
> - getnstimeofday64(&ts);
> + ktime_get_real_ts64(&ts);
> arg->params[0].u.value.a = ts.tv_sec;
> arg->params[0].u.value.b = ts.tv_nsec;
>
> --
> 2.9.0
>
Looks good.
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
Thanks,
Jens
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] tee: replace getnstimeofday64() with ktime_get_real_ts64()
2018-06-19 13:37 ` Jens Wiklander
@ 2018-06-19 14:05 ` Arnd Bergmann
2018-06-19 14:23 ` Jens Wiklander
0 siblings, 1 reply; 4+ messages in thread
From: Arnd Bergmann @ 2018-06-19 14:05 UTC (permalink / raw)
To: Jens Wiklander
Cc: y2038 Mailman List, Jerome Forissier, Etienne Carriere,
Joakim Bech, Volodymyr Babchuk, tiger-yu99, David Wang,
Linux Kernel Mailing List
On Tue, Jun 19, 2018 at 3:37 PM, Jens Wiklander
<jens.wiklander@linaro.org> wrote:
> On Mon, Jun 18, 2018 at 04:24:56PM +0200, Arnd Bergmann wrote:
>> The two do the same thing, but we want to have a consistent
>> naming in the kernel.
>>
>> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
>> ---
>> drivers/tee/optee/rpc.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/tee/optee/rpc.c b/drivers/tee/optee/rpc.c
>> index 41aea12e2bcc..b45c73dd37a5 100644
>> --- a/drivers/tee/optee/rpc.c
>> +++ b/drivers/tee/optee/rpc.c
>> @@ -48,7 +48,7 @@ static void handle_rpc_func_cmd_get_time(struct optee_msg_arg *arg)
>> OPTEE_MSG_ATTR_TYPE_VALUE_OUTPUT)
>> goto bad;
>>
>> - getnstimeofday64(&ts);
>> + ktime_get_real_ts64(&ts);
>> arg->params[0].u.value.a = ts.tv_sec;
>> arg->params[0].u.value.b = ts.tv_nsec;
>>
>> --
>> 2.9.0
>>
>
> Looks good.
>
> Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
Thanks for the review. Can you pick it up in your tree?
Arnd
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] tee: replace getnstimeofday64() with ktime_get_real_ts64()
2018-06-19 14:05 ` Arnd Bergmann
@ 2018-06-19 14:23 ` Jens Wiklander
0 siblings, 0 replies; 4+ messages in thread
From: Jens Wiklander @ 2018-06-19 14:23 UTC (permalink / raw)
To: Arnd Bergmann
Cc: y2038 Mailman List, Jerome Forissier, Etienne Carriere,
Joakim Bech, Volodymyr Babchuk, tiger-yu99, David Wang,
Linux Kernel Mailing List
On Tue, Jun 19, 2018 at 04:05:38PM +0200, Arnd Bergmann wrote:
> On Tue, Jun 19, 2018 at 3:37 PM, Jens Wiklander
> <jens.wiklander@linaro.org> wrote:
> > On Mon, Jun 18, 2018 at 04:24:56PM +0200, Arnd Bergmann wrote:
> >> The two do the same thing, but we want to have a consistent
> >> naming in the kernel.
> >>
> >> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> >> ---
> >> drivers/tee/optee/rpc.c | 2 +-
> >> 1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> diff --git a/drivers/tee/optee/rpc.c b/drivers/tee/optee/rpc.c
> >> index 41aea12e2bcc..b45c73dd37a5 100644
> >> --- a/drivers/tee/optee/rpc.c
> >> +++ b/drivers/tee/optee/rpc.c
> >> @@ -48,7 +48,7 @@ static void handle_rpc_func_cmd_get_time(struct optee_msg_arg *arg)
> >> OPTEE_MSG_ATTR_TYPE_VALUE_OUTPUT)
> >> goto bad;
> >>
> >> - getnstimeofday64(&ts);
> >> + ktime_get_real_ts64(&ts);
> >> arg->params[0].u.value.a = ts.tv_sec;
> >> arg->params[0].u.value.b = ts.tv_nsec;
> >>
> >> --
> >> 2.9.0
> >>
> >
> > Looks good.
> >
> > Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
>
> Thanks for the review. Can you pick it up in your tree?
OK, I'll pick it up.
Thanks,
Jens
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-06-19 14:23 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-06-18 14:24 [PATCH] tee: replace getnstimeofday64() with ktime_get_real_ts64() Arnd Bergmann
2018-06-19 13:37 ` Jens Wiklander
2018-06-19 14:05 ` Arnd Bergmann
2018-06-19 14:23 ` Jens Wiklander
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).