Message ID | 5519AEB2.8050306@linaro.org |
---|---|
State | New |
Headers | show |
* Daniel Lezcano <daniel.lezcano@linaro.org> wrote: > From: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> > > The arch_timer_probed function returns whether the given time doesn't > need to be probed. This can be the case when the timer has been probed > already, but also when it has no corresponding enabled node in DT. > > Rename the function to arch_timer_need_probe and invert its return value > to better reflect the function's purpose and behaviour. > > Acked-by: Sudeep Holla <sudeep.holla@arm.com> > Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> > Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org> > --- > drivers/clocksource/arm_arch_timer.c | 13 +++++++------ > 1 file changed, 7 insertions(+), 6 deletions(-) > > diff --git a/drivers/clocksource/arm_arch_timer.c b/drivers/clocksource/arm_arch_timer.c > index a3025e7..50bb7f2 100644 > --- a/drivers/clocksource/arm_arch_timer.c > +++ b/drivers/clocksource/arm_arch_timer.c > @@ -661,17 +661,17 @@ static const struct of_device_id arch_timer_mem_of_match[] __initconst = { > }; > > static bool __init > -arch_timer_probed(int type, const struct of_device_id *matches) > +arch_timer_need_probe(int type, const struct of_device_id *matches) If we do a rename we might as well use valid English spelling such as 'arch_timer_needs_probing()'? Thanks, Ingo
* Daniel Lezcano <daniel.lezcano@linaro.org> wrote: > > Hi Thomas, Ingo, > > this pull request contains the following changes for 4.1: > > - Made IO endian agnostic for at91 and dw apb timers (Ben Dooks) > > - Maintained tegra endianess (Dmitry Osipenko) > > - Enabled the sun4i / sun5i timer for sched_clock, all others winner boards > have the arch arm timer which is more accurate (Hans de Goede) > > - Renamed the arch arm timer function name to reflect better its purpose > (Laurent Pinchart) > > - Cleaned up the sun5i timer code, replaced the setup_irq by the > request_irq function and handled the parent clock frequency change with the > frequency notifiers (Maxime Ripard) > > - Fixed features declaration for the efm32 timer (Viresh Kumar) > > Thanks ! So I had some comments about one of the patches, and fixes to changelogs, so I've applied all but the sun4/5i patches from email to move things forward. The sun4/5i patches conflicted - mind reposting those merged on top of the latest tip:timers:core tree? Thanks! Ingo
On 03/31/2015 09:17 AM, Ingo Molnar wrote: > > * Daniel Lezcano <daniel.lezcano@linaro.org> wrote: > >> >> Hi Thomas, Ingo, >> >> this pull request contains the following changes for 4.1: >> >> - Made IO endian agnostic for at91 and dw apb timers (Ben Dooks) >> >> - Maintained tegra endianess (Dmitry Osipenko) >> >> - Enabled the sun4i / sun5i timer for sched_clock, all others winner boards >> have the arch arm timer which is more accurate (Hans de Goede) >> >> - Renamed the arch arm timer function name to reflect better its purpose >> (Laurent Pinchart) >> >> - Cleaned up the sun5i timer code, replaced the setup_irq by the >> request_irq function and handled the parent clock frequency change with the >> frequency notifiers (Maxime Ripard) >> >> - Fixed features declaration for the efm32 timer (Viresh Kumar) >> >> Thanks ! > > So I had some comments about one of the patches, and fixes to > changelogs, so I've applied all but the sun4/5i patches from email to > move things forward. The subsystem prefix is now 'clocksource/drivers/<name>' ? > The sun4/5i patches conflicted - mind reposting those merged on top of > the latest tip:timers:core tree? Yes, sure. No problem. Thanks ! -- Daniel
* Daniel Lezcano <daniel.lezcano@linaro.org> wrote: > On 03/31/2015 09:17 AM, Ingo Molnar wrote: > > > >* Daniel Lezcano <daniel.lezcano@linaro.org> wrote: > > > >> > >>Hi Thomas, Ingo, > >> > >>this pull request contains the following changes for 4.1: > >> > >> - Made IO endian agnostic for at91 and dw apb timers (Ben Dooks) > >> > >> - Maintained tegra endianess (Dmitry Osipenko) > >> > >> - Enabled the sun4i / sun5i timer for sched_clock, all others winner boards > >>have the arch arm timer which is more accurate (Hans de Goede) > >> > >> - Renamed the arch arm timer function name to reflect better its purpose > >>(Laurent Pinchart) > >> > >> - Cleaned up the sun5i timer code, replaced the setup_irq by the > >>request_irq function and handled the parent clock frequency change with the > >>frequency notifiers (Maxime Ripard) > >> > >> - Fixed features declaration for the efm32 timer (Viresh Kumar) > >> > >>Thanks ! > > > >So I had some comments about one of the patches, and fixes to > >changelogs, so I've applied all but the sun4/5i patches from email to > >move things forward. > > The subsystem prefix is now 'clocksource/drivers/<name>' ? At least now that we have a good number of clocksource drivers, for such pure driver changes that's the logical prefix and makes it easier to tell apart clocksource core changes from driver changes. > > The sun4/5i patches conflicted - mind reposting those merged on > > top of the latest tip:timers:core tree? > > Yes, sure. No problem. Thanks! Ingo
Hi Ingo, On Tuesday 31 March 2015 09:14:52 Ingo Molnar wrote: > * Daniel Lezcano <daniel.lezcano@linaro.org> wrote: > > From: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> > > > > The arch_timer_probed function returns whether the given time doesn't > > need to be probed. This can be the case when the timer has been probed > > already, but also when it has no corresponding enabled node in DT. > > > > Rename the function to arch_timer_need_probe and invert its return value > > to better reflect the function's purpose and behaviour. > > > > Acked-by: Sudeep Holla <sudeep.holla@arm.com> > > Signed-off-by: Laurent Pinchart > > <laurent.pinchart+renesas@ideasonboard.com> > > Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org> > > --- > > > > drivers/clocksource/arm_arch_timer.c | 13 +++++++------ > > 1 file changed, 7 insertions(+), 6 deletions(-) > > > > diff --git a/drivers/clocksource/arm_arch_timer.c > > b/drivers/clocksource/arm_arch_timer.c index a3025e7..50bb7f2 100644 > > --- a/drivers/clocksource/arm_arch_timer.c > > +++ b/drivers/clocksource/arm_arch_timer.c > > @@ -661,17 +661,17 @@ static const struct of_device_id > > arch_timer_mem_of_match[] __initconst = {> > > }; > > > > static bool __init > > > > -arch_timer_probed(int type, const struct of_device_id *matches) > > +arch_timer_need_probe(int type, const struct of_device_id *matches) > > If we do a rename we might as well use valid English spelling such as > 'arch_timer_needs_probing()'? Of course. Should I resend the patch or can you fix that while applying it ?
* Laurent Pinchart <laurent.pinchart@ideasonboard.com> wrote: > Hi Ingo, > > On Tuesday 31 March 2015 09:14:52 Ingo Molnar wrote: > > * Daniel Lezcano <daniel.lezcano@linaro.org> wrote: > > > From: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> > > > > > > The arch_timer_probed function returns whether the given time doesn't > > > need to be probed. This can be the case when the timer has been probed > > > already, but also when it has no corresponding enabled node in DT. > > > > > > Rename the function to arch_timer_need_probe and invert its return value > > > to better reflect the function's purpose and behaviour. > > > > > > Acked-by: Sudeep Holla <sudeep.holla@arm.com> > > > Signed-off-by: Laurent Pinchart > > > <laurent.pinchart+renesas@ideasonboard.com> > > > Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org> > > > --- > > > > > > drivers/clocksource/arm_arch_timer.c | 13 +++++++------ > > > 1 file changed, 7 insertions(+), 6 deletions(-) > > > > > > diff --git a/drivers/clocksource/arm_arch_timer.c > > > b/drivers/clocksource/arm_arch_timer.c index a3025e7..50bb7f2 100644 > > > --- a/drivers/clocksource/arm_arch_timer.c > > > +++ b/drivers/clocksource/arm_arch_timer.c > > > @@ -661,17 +661,17 @@ static const struct of_device_id > > > arch_timer_mem_of_match[] __initconst = {> > > > }; > > > > > > static bool __init > > > > > > -arch_timer_probed(int type, const struct of_device_id *matches) > > > +arch_timer_need_probe(int type, const struct of_device_id *matches) > > > > If we do a rename we might as well use valid English spelling such as > > 'arch_timer_needs_probing()'? > > Of course. Should I resend the patch or can you fix that while applying it ? No need to resend, I've done the rename in the patch. Thanks, Ingo