Re: [PATCH v2 05/12] ARM: davinci: da850: switch to using the clocksource driver

From: Bartosz Golaszewski
Date: Fri Feb 08 2019 - 07:35:08 EST


pt., 8 lut 2019 o 13:06 Sekhar Nori <nsekhar@xxxxxx> napisaÅ(a):
>
> On 04/02/19 10:47 PM, Bartosz Golaszewski wrote:
> > +static const struct davinci_timer_cfg da850_timer_cfg = {
> > + .reg = {
> > + .start = DA8XX_TIMER64P0_BASE,
> > + .end = DA8XX_TIMER64P0_BASE + SZ_4K,
>
> SZ_4K - 1
>
> This should have prevented watchdog timer from getting registered.
>
> Thanks,
> Sekhar

My clocksource driver doesn't call request_region() so a subsequent
devm_ioremap_resource() in the watchdog driver would still succeed. I
now fixed both the missing call and the value here.

Bart