* [PATCH RFC] clk: ux500: add range to usleep_range
@ 2019-04-07 3:13 Nicholas Mc Guire
2019-04-10 22:53 ` Stephen Boyd
2019-04-11 9:36 ` Ulf Hansson
0 siblings, 2 replies; 9+ messages in thread
From: Nicholas Mc Guire @ 2019-04-07 3:13 UTC (permalink / raw)
To: Ulf Hansson
Cc: Michael Turquette, Stephen Boyd, linux-arm-kernel, linux-clk,
linux-kernel, Nicholas Mc Guire
Providing a range for usleep_range() allows the hrtimer subsystem to
coalesce timers - the delay is runtime configurable so a factor 2
is taken to provide the range.
Signed-off-by: Nicholas Mc Guire <hofrat@opentech.at>
---
Problem located with an experimental coccinelle script
Q: Basically usleep_range() with min == max never makes much sense notably
in non-atomic context. If the factor of 2 is tolerable or a fixed
offset of e.g. 1000 would be more suitable is not clear to me - maybe
someone familiar with that driver can clarify this.
Patch was compile tested with: u8500_defconfig (implies COMMON_CLK=y)
(with some sparse warnings about not implemented system calls)
Patch is against 5.1-rc3 (localversion-next is next=20190405)
drivers/clk/ux500/clk-sysctrl.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/clk/ux500/clk-sysctrl.c b/drivers/clk/ux500/clk-sysctrl.c
index 7c0403b..a1fa3fb 100644
--- a/drivers/clk/ux500/clk-sysctrl.c
+++ b/drivers/clk/ux500/clk-sysctrl.c
@@ -42,7 +42,7 @@ static int clk_sysctrl_prepare(struct clk_hw *hw)
clk->reg_bits[0]);
if (!ret && clk->enable_delay_us)
- usleep_range(clk->enable_delay_us, clk->enable_delay_us);
+ usleep_range(clk->enable_delay_us, clk->enable_delay_us*2);
return ret;
}
--
2.1.4
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH RFC] clk: ux500: add range to usleep_range
2019-04-07 3:13 [PATCH RFC] clk: ux500: add range to usleep_range Nicholas Mc Guire
@ 2019-04-10 22:53 ` Stephen Boyd
2019-04-11 2:56 ` Nicholas Mc Guire
2019-04-11 9:36 ` Ulf Hansson
1 sibling, 1 reply; 9+ messages in thread
From: Stephen Boyd @ 2019-04-10 22:53 UTC (permalink / raw)
To: Nicholas Mc Guire, Ulf Hansson
Cc: Michael Turquette, linux-arm-kernel, linux-clk, linux-kernel,
Nicholas Mc Guire
Quoting Nicholas Mc Guire (2019-04-06 20:13:24)
> Providing a range for usleep_range() allows the hrtimer subsystem to
> coalesce timers - the delay is runtime configurable so a factor 2
> is taken to provide the range.
>
> Signed-off-by: Nicholas Mc Guire <hofrat@opentech.at>
> ---
I think this driver is in maintenance mode. I'll wait for Ulf to ack or
review this change before applying.
> diff --git a/drivers/clk/ux500/clk-sysctrl.c b/drivers/clk/ux500/clk-sysctrl.c
> index 7c0403b..a1fa3fb 100644
> --- a/drivers/clk/ux500/clk-sysctrl.c
> +++ b/drivers/clk/ux500/clk-sysctrl.c
> @@ -42,7 +42,7 @@ static int clk_sysctrl_prepare(struct clk_hw *hw)
> clk->reg_bits[0]);
>
> if (!ret && clk->enable_delay_us)
> - usleep_range(clk->enable_delay_us, clk->enable_delay_us);
> + usleep_range(clk->enable_delay_us, clk->enable_delay_us*2);
Please add space around that multiply.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH RFC] clk: ux500: add range to usleep_range
2019-04-10 22:53 ` Stephen Boyd
@ 2019-04-11 2:56 ` Nicholas Mc Guire
2019-04-11 11:51 ` Joe Perches
0 siblings, 1 reply; 9+ messages in thread
From: Nicholas Mc Guire @ 2019-04-11 2:56 UTC (permalink / raw)
To: Stephen Boyd
Cc: Nicholas Mc Guire, Ulf Hansson, Michael Turquette,
linux-arm-kernel, linux-clk, linux-kernel
On Wed, Apr 10, 2019 at 03:53:51PM -0700, Stephen Boyd wrote:
> Quoting Nicholas Mc Guire (2019-04-06 20:13:24)
> > Providing a range for usleep_range() allows the hrtimer subsystem to
> > coalesce timers - the delay is runtime configurable so a factor 2
> > is taken to provide the range.
> >
> > Signed-off-by: Nicholas Mc Guire <hofrat@opentech.at>
> > ---
>
> I think this driver is in maintenance mode. I'll wait for Ulf to ack or
> review this change before applying.
>
> > diff --git a/drivers/clk/ux500/clk-sysctrl.c b/drivers/clk/ux500/clk-sysctrl.c
> > index 7c0403b..a1fa3fb 100644
> > --- a/drivers/clk/ux500/clk-sysctrl.c
> > +++ b/drivers/clk/ux500/clk-sysctrl.c
> > @@ -42,7 +42,7 @@ static int clk_sysctrl_prepare(struct clk_hw *hw)
> > clk->reg_bits[0]);
> >
> > if (!ret && clk->enable_delay_us)
> > - usleep_range(clk->enable_delay_us, clk->enable_delay_us);
> > + usleep_range(clk->enable_delay_us, clk->enable_delay_us*2);
>
> Please add space around that multiply.
>
I can do that but it does not seem common and also checkpatch
did not complain about this - now a simple grep -re "\*10" on the
kernel shows that it seems more common not to use spaces around *
that to use them. Greping specifically for cases using usleep_range()
(not that many) it seems more or less evenly devided between space
and no space - so the concern is overlooking that factor 2 ?
thx!
hofrat
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH RFC] clk: ux500: add range to usleep_range
2019-04-07 3:13 [PATCH RFC] clk: ux500: add range to usleep_range Nicholas Mc Guire
2019-04-10 22:53 ` Stephen Boyd
@ 2019-04-11 9:36 ` Ulf Hansson
2019-04-11 10:38 ` Nicholas Mc Guire
1 sibling, 1 reply; 9+ messages in thread
From: Ulf Hansson @ 2019-04-11 9:36 UTC (permalink / raw)
To: Nicholas Mc Guire
Cc: Michael Turquette, Stephen Boyd, Linux ARM, linux-clk,
Linux Kernel Mailing List
On Sun, 7 Apr 2019 at 05:13, Nicholas Mc Guire <hofrat@opentech.at> wrote:
>
> Providing a range for usleep_range() allows the hrtimer subsystem to
> coalesce timers - the delay is runtime configurable so a factor 2
> is taken to provide the range.
>
> Signed-off-by: Nicholas Mc Guire <hofrat@opentech.at>
> ---
>
> Problem located with an experimental coccinelle script
>
> Q: Basically usleep_range() with min == max never makes much sense notably
> in non-atomic context. If the factor of 2 is tolerable or a fixed
> offset of e.g. 1000 would be more suitable is not clear to me - maybe
> someone familiar with that driver can clarify this.
>
> Patch was compile tested with: u8500_defconfig (implies COMMON_CLK=y)
> (with some sparse warnings about not implemented system calls)
>
> Patch is against 5.1-rc3 (localversion-next is next=20190405)
>
> drivers/clk/ux500/clk-sysctrl.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/clk/ux500/clk-sysctrl.c b/drivers/clk/ux500/clk-sysctrl.c
> index 7c0403b..a1fa3fb 100644
> --- a/drivers/clk/ux500/clk-sysctrl.c
> +++ b/drivers/clk/ux500/clk-sysctrl.c
> @@ -42,7 +42,7 @@ static int clk_sysctrl_prepare(struct clk_hw *hw)
> clk->reg_bits[0]);
>
> if (!ret && clk->enable_delay_us)
> - usleep_range(clk->enable_delay_us, clk->enable_delay_us);
> + usleep_range(clk->enable_delay_us, clk->enable_delay_us*2);
The range being used is actually in ms, so not sure we actually need
to double it for the range.
How about adding ~25% instead, along the lines of below:
usleep_range(clk->enable_delay_us, clk->enable_delay_us +
(clk->enable_delay_us >> 2));
Kind regards
Uffe
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH RFC] clk: ux500: add range to usleep_range
2019-04-11 9:36 ` Ulf Hansson
@ 2019-04-11 10:38 ` Nicholas Mc Guire
0 siblings, 0 replies; 9+ messages in thread
From: Nicholas Mc Guire @ 2019-04-11 10:38 UTC (permalink / raw)
To: Ulf Hansson
Cc: Nicholas Mc Guire, Michael Turquette, Stephen Boyd, Linux ARM,
linux-clk, Linux Kernel Mailing List
On Thu, Apr 11, 2019 at 11:36:45AM +0200, Ulf Hansson wrote:
> On Sun, 7 Apr 2019 at 05:13, Nicholas Mc Guire <hofrat@opentech.at> wrote:
> >
> > Providing a range for usleep_range() allows the hrtimer subsystem to
> > coalesce timers - the delay is runtime configurable so a factor 2
> > is taken to provide the range.
> >
> > Signed-off-by: Nicholas Mc Guire <hofrat@opentech.at>
> > ---
> >
> > Problem located with an experimental coccinelle script
> >
> > Q: Basically usleep_range() with min == max never makes much sense notably
> > in non-atomic context. If the factor of 2 is tolerable or a fixed
> > offset of e.g. 1000 would be more suitable is not clear to me - maybe
> > someone familiar with that driver can clarify this.
> >
> > Patch was compile tested with: u8500_defconfig (implies COMMON_CLK=y)
> > (with some sparse warnings about not implemented system calls)
> >
> > Patch is against 5.1-rc3 (localversion-next is next=20190405)
> >
> > drivers/clk/ux500/clk-sysctrl.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/clk/ux500/clk-sysctrl.c b/drivers/clk/ux500/clk-sysctrl.c
> > index 7c0403b..a1fa3fb 100644
> > --- a/drivers/clk/ux500/clk-sysctrl.c
> > +++ b/drivers/clk/ux500/clk-sysctrl.c
> > @@ -42,7 +42,7 @@ static int clk_sysctrl_prepare(struct clk_hw *hw)
> > clk->reg_bits[0]);
> >
> > if (!ret && clk->enable_delay_us)
> > - usleep_range(clk->enable_delay_us, clk->enable_delay_us);
> > + usleep_range(clk->enable_delay_us, clk->enable_delay_us*2);
>
> The range being used is actually in ms, so not sure we actually need
> to double it for the range.
>
> How about adding ~25% instead, along the lines of below:
> usleep_range(clk->enable_delay_us, clk->enable_delay_us +
> (clk->enable_delay_us >> 2));
>
that would be perfectly sufficient for hrtimers - if the range is
in ms - so I´ll send out a V2 shortly.
thx!
hofrat
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH RFC] clk: ux500: add range to usleep_range
2019-04-11 2:56 ` Nicholas Mc Guire
@ 2019-04-11 11:51 ` Joe Perches
2019-04-11 12:59 ` Nicholas Mc Guire
0 siblings, 1 reply; 9+ messages in thread
From: Joe Perches @ 2019-04-11 11:51 UTC (permalink / raw)
To: Nicholas Mc Guire, Stephen Boyd
Cc: Nicholas Mc Guire, Ulf Hansson, Michael Turquette,
linux-arm-kernel, linux-clk, linux-kernel
On Thu, 2019-04-11 at 04:56 +0200, Nicholas Mc Guire wrote:
> On Wed, Apr 10, 2019 at 03:53:51PM -0700, Stephen Boyd wrote:
> > Quoting Nicholas Mc Guire (2019-04-06 20:13:24)
> > > Providing a range for usleep_range() allows the hrtimer subsystem to
> > > coalesce timers - the delay is runtime configurable so a factor 2
> > > is taken to provide the range.
> > >
> > > Signed-off-by: Nicholas Mc Guire <hofrat@opentech.at>
> > > ---
> >
> > I think this driver is in maintenance mode. I'll wait for Ulf to ack or
> > review this change before applying.
> >
> > > diff --git a/drivers/clk/ux500/clk-sysctrl.c b/drivers/clk/ux500/clk-sysctrl.c
> > > index 7c0403b..a1fa3fb 100644
> > > --- a/drivers/clk/ux500/clk-sysctrl.c
> > > +++ b/drivers/clk/ux500/clk-sysctrl.c
> > > @@ -42,7 +42,7 @@ static int clk_sysctrl_prepare(struct clk_hw *hw)
> > > clk->reg_bits[0]);
> > >
> > > if (!ret && clk->enable_delay_us)
> > > - usleep_range(clk->enable_delay_us, clk->enable_delay_us);
> > > + usleep_range(clk->enable_delay_us, clk->enable_delay_us*2);
> >
> > Please add space around that multiply.
> >
> I can do that but it does not seem common and also checkpatch
> did not complain about this - now a simple grep -re "\*10" on the
> kernel shows that it seems more common not to use spaces around *
> that to use them.
Not really
$ git grep -P '\*\s*10' | grep -oh -P '\*\s*10' | \
sort | uniq -c | sort -rn | head
11800 * 10
1705 *10
179 * 10
74 * 10
67 * 10
20 * 10
20 * 10
14 * 10
14 * 10
12 * 10
> Greping specifically for cases using usleep_range()
> (not that many) it seems more or less evenly devided between space
> and no space - so the concern is overlooking that factor 2 ?
>
> thx!
> hofrat
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH RFC] clk: ux500: add range to usleep_range
2019-04-11 11:51 ` Joe Perches
@ 2019-04-11 12:59 ` Nicholas Mc Guire
2019-04-11 13:12 ` Joe Perches
0 siblings, 1 reply; 9+ messages in thread
From: Nicholas Mc Guire @ 2019-04-11 12:59 UTC (permalink / raw)
To: Joe Perches
Cc: Stephen Boyd, Nicholas Mc Guire, Ulf Hansson, Michael Turquette,
linux-arm-kernel, linux-clk, linux-kernel
On Thu, Apr 11, 2019 at 04:51:25AM -0700, Joe Perches wrote:
> On Thu, 2019-04-11 at 04:56 +0200, Nicholas Mc Guire wrote:
> > On Wed, Apr 10, 2019 at 03:53:51PM -0700, Stephen Boyd wrote:
> > > Quoting Nicholas Mc Guire (2019-04-06 20:13:24)
> > > > Providing a range for usleep_range() allows the hrtimer subsystem to
> > > > coalesce timers - the delay is runtime configurable so a factor 2
> > > > is taken to provide the range.
> > > >
> > > > Signed-off-by: Nicholas Mc Guire <hofrat@opentech.at>
> > > > ---
> > >
> > > I think this driver is in maintenance mode. I'll wait for Ulf to ack or
> > > review this change before applying.
> > >
> > > > diff --git a/drivers/clk/ux500/clk-sysctrl.c b/drivers/clk/ux500/clk-sysctrl.c
> > > > index 7c0403b..a1fa3fb 100644
> > > > --- a/drivers/clk/ux500/clk-sysctrl.c
> > > > +++ b/drivers/clk/ux500/clk-sysctrl.c
> > > > @@ -42,7 +42,7 @@ static int clk_sysctrl_prepare(struct clk_hw *hw)
> > > > clk->reg_bits[0]);
> > > >
> > > > if (!ret && clk->enable_delay_us)
> > > > - usleep_range(clk->enable_delay_us, clk->enable_delay_us);
> > > > + usleep_range(clk->enable_delay_us, clk->enable_delay_us*2);
> > >
> > > Please add space around that multiply.
> > >
> > I can do that but it does not seem common and also checkpatch
> > did not complain about this - now a simple grep -re "\*10" on the
> > kernel shows that it seems more common not to use spaces around *
> > that to use them.
>
> Not really
>
> $ git grep -P '\*\s*10' | grep -oh -P '\*\s*10' | \
> sort | uniq -c | sort -rn | head
> 11800 * 10
> 1705 *10
> 179 * 10
> 74 * 10
> 67 * 10
> 20 * 10
> 20 * 10
> 14 * 10
> 14 * 10
> 12 * 10
yup - my bad - If you restrict it to code lines - its 1:10
not quite sure how I got the first numbers - sloppy check.
hofrat@debian:~/git/linux-next$ grep -re '.*\*\s*10.*;$' * | grep -oh '\*\s*10' | sort | uniq -c | sort -nr | more
8568 * 10
860 *10
Anyway - is there a reason checkpatch will not flag this ?
thx!
hofrat
>
> > Greping specifically for cases using usleep_range()
> > (not that many) it seems more or less evenly devided between space
> > and no space - so the concern is overlooking that factor 2 ?
> >
> > thx!
> > hofrat
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH RFC] clk: ux500: add range to usleep_range
2019-04-11 12:59 ` Nicholas Mc Guire
@ 2019-04-11 13:12 ` Joe Perches
2019-04-11 13:53 ` Nicholas Mc Guire
0 siblings, 1 reply; 9+ messages in thread
From: Joe Perches @ 2019-04-11 13:12 UTC (permalink / raw)
To: Nicholas Mc Guire
Cc: Stephen Boyd, Nicholas Mc Guire, Ulf Hansson, Michael Turquette,
linux-arm-kernel, linux-clk, linux-kernel
On Thu, 2019-04-11 at 14:59 +0200, Nicholas Mc Guire wrote:
> On Thu, Apr 11, 2019 at 04:51:25AM -0700, Joe Perches wrote:
> > On Thu, 2019-04-11 at 04:56 +0200, Nicholas Mc Guire wrote:
> > > On Wed, Apr 10, 2019 at 03:53:51PM -0700, Stephen Boyd wrote:
> > > > Quoting Nicholas Mc Guire (2019-04-06 20:13:24)
> > > > > Providing a range for usleep_range() allows the hrtimer subsystem to
> > > > > coalesce timers - the delay is runtime configurable so a factor 2
> > > > > is taken to provide the range.
> > > > >
> > > > > Signed-off-by: Nicholas Mc Guire <hofrat@opentech.at>
> > > > > ---
> > > >
> > > > I think this driver is in maintenance mode. I'll wait for Ulf to ack or
> > > > review this change before applying.
> > > >
> > > > > diff --git a/drivers/clk/ux500/clk-sysctrl.c b/drivers/clk/ux500/clk-sysctrl.c
> > > > > index 7c0403b..a1fa3fb 100644
> > > > > --- a/drivers/clk/ux500/clk-sysctrl.c
> > > > > +++ b/drivers/clk/ux500/clk-sysctrl.c
> > > > > @@ -42,7 +42,7 @@ static int clk_sysctrl_prepare(struct clk_hw *hw)
> > > > > clk->reg_bits[0]);
> > > > >
> > > > > if (!ret && clk->enable_delay_us)
> > > > > - usleep_range(clk->enable_delay_us, clk->enable_delay_us);
> > > > > + usleep_range(clk->enable_delay_us, clk->enable_delay_us*2);
> > > >
> > > > Please add space around that multiply.
> > > >
> > > I can do that but it does not seem common and also checkpatch
> > > did not complain about this - now a simple grep -re "\*10" on the
> > > kernel shows that it seems more common not to use spaces around *
> > > that to use them.
> >
> > Not really
> >
> > $ git grep -P '\*\s*10' | grep -oh -P '\*\s*10' | \
> > sort | uniq -c | sort -rn | head
> > 11800 * 10
> > 1705 *10
> > 179 * 10
> > 74 * 10
> > 67 * 10
> > 20 * 10
> > 20 * 10
> > 14 * 10
> > 14 * 10
> > 12 * 10
>
> yup - my bad - If you restrict it to code lines - its 1:10
> not quite sure how I got the first numbers - sloppy check.
>
> hofrat@debian:~/git/linux-next$ grep -re '.*\*\s*10.*;$' * | grep -oh '\*\s*10' | sort | uniq -c | sort -nr | more
> 8568 * 10
> 860 *10
The ratio is about the same in any case.
> Anyway - is there a reason checkpatch will not flag this ?
Because the style is not mentioned in coding style.
checkpatch flags it only when using the --strict option
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH RFC] clk: ux500: add range to usleep_range
2019-04-11 13:12 ` Joe Perches
@ 2019-04-11 13:53 ` Nicholas Mc Guire
0 siblings, 0 replies; 9+ messages in thread
From: Nicholas Mc Guire @ 2019-04-11 13:53 UTC (permalink / raw)
To: Joe Perches
Cc: Stephen Boyd, Nicholas Mc Guire, Ulf Hansson, Michael Turquette,
linux-arm-kernel, linux-clk, linux-kernel
On Thu, Apr 11, 2019 at 06:12:49AM -0700, Joe Perches wrote:
> On Thu, 2019-04-11 at 14:59 +0200, Nicholas Mc Guire wrote:
> > On Thu, Apr 11, 2019 at 04:51:25AM -0700, Joe Perches wrote:
> > > On Thu, 2019-04-11 at 04:56 +0200, Nicholas Mc Guire wrote:
> > > > On Wed, Apr 10, 2019 at 03:53:51PM -0700, Stephen Boyd wrote:
> > > > > Quoting Nicholas Mc Guire (2019-04-06 20:13:24)
> > > > > > Providing a range for usleep_range() allows the hrtimer subsystem to
> > > > > > coalesce timers - the delay is runtime configurable so a factor 2
> > > > > > is taken to provide the range.
> > > > > >
> > > > > > Signed-off-by: Nicholas Mc Guire <hofrat@opentech.at>
> > > > > > ---
> > > > >
> > > > > I think this driver is in maintenance mode. I'll wait for Ulf to ack or
> > > > > review this change before applying.
> > > > >
> > > > > > diff --git a/drivers/clk/ux500/clk-sysctrl.c b/drivers/clk/ux500/clk-sysctrl.c
> > > > > > index 7c0403b..a1fa3fb 100644
> > > > > > --- a/drivers/clk/ux500/clk-sysctrl.c
> > > > > > +++ b/drivers/clk/ux500/clk-sysctrl.c
> > > > > > @@ -42,7 +42,7 @@ static int clk_sysctrl_prepare(struct clk_hw *hw)
> > > > > > clk->reg_bits[0]);
> > > > > >
> > > > > > if (!ret && clk->enable_delay_us)
> > > > > > - usleep_range(clk->enable_delay_us, clk->enable_delay_us);
> > > > > > + usleep_range(clk->enable_delay_us, clk->enable_delay_us*2);
> > > > >
> > > > > Please add space around that multiply.
> > > > >
> > > > I can do that but it does not seem common and also checkpatch
> > > > did not complain about this - now a simple grep -re "\*10" on the
> > > > kernel shows that it seems more common not to use spaces around *
> > > > that to use them.
> > >
> > > Not really
> > >
> > > $ git grep -P '\*\s*10' | grep -oh -P '\*\s*10' | \
> > > sort | uniq -c | sort -rn | head
> > > 11800 * 10
> > > 1705 *10
> > > 179 * 10
> > > 74 * 10
> > > 67 * 10
> > > 20 * 10
> > > 20 * 10
> > > 14 * 10
> > > 14 * 10
> > > 12 * 10
> >
> > yup - my bad - If you restrict it to code lines - its 1:10
> > not quite sure how I got the first numbers - sloppy check.
> >
> > hofrat@debian:~/git/linux-next$ grep -re '.*\*\s*10.*;$' * | grep -oh '\*\s*10' | sort | uniq -c | sort -nr | more
> > 8568 * 10
> > 860 *10
>
> The ratio is about the same in any case.
>
> > Anyway - is there a reason checkpatch will not flag this ?
>
> Because the style is not mentioned in coding style.
> checkpatch flags it only when using the --strict option
>
hmm...It sees it is:
3.1) Spaces
***********
...
Use one space around (on each side of) most binary and ternary operators,
such as any of these::
= + - < > * / % | & ^ <= >= == != ? :
thx!
hofrat
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2019-04-11 13:54 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-04-07 3:13 [PATCH RFC] clk: ux500: add range to usleep_range Nicholas Mc Guire
2019-04-10 22:53 ` Stephen Boyd
2019-04-11 2:56 ` Nicholas Mc Guire
2019-04-11 11:51 ` Joe Perches
2019-04-11 12:59 ` Nicholas Mc Guire
2019-04-11 13:12 ` Joe Perches
2019-04-11 13:53 ` Nicholas Mc Guire
2019-04-11 9:36 ` Ulf Hansson
2019-04-11 10:38 ` Nicholas Mc Guire
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).