* [PATCH] clocksource/drivers/npcm: fix GENMASK and timer operation
@ 2019-07-15 10:05 Avi Fishman
2019-07-15 12:37 ` Thomas Gleixner
2019-07-15 15:19 ` [PATCH] [v2] " Avi Fishman
0 siblings, 2 replies; 9+ messages in thread
From: Avi Fishman @ 2019-07-15 10:05 UTC (permalink / raw)
To: Tomer Maimon, Tali Perry, Patrick Venture, Nancy Yuen,
Benjamin Fair, Daniel Lezcano, Thomas Gleixner
Cc: OpenBMC Maillist, Linux Kernel Mailing List
NPCM7XX_Tx_OPER GENMASK was wrong,
npcm7xx_timer_oneshot() did wrong calculation
Signed-off-by: Avi Fishman <avifishman70@gmail.com>
---
drivers/clocksource/timer-npcm7xx.c | 9 +++------
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/drivers/clocksource/timer-npcm7xx.c
b/drivers/clocksource/timer-npcm7xx.c
index 8a30da7f083b..9780ffd8010e 100644
--- a/drivers/clocksource/timer-npcm7xx.c
+++ b/drivers/clocksource/timer-npcm7xx.c
@@ -32,7 +32,7 @@
#define NPCM7XX_Tx_INTEN BIT(29)
#define NPCM7XX_Tx_COUNTEN BIT(30)
#define NPCM7XX_Tx_ONESHOT 0x0
-#define NPCM7XX_Tx_OPER GENMASK(27, 3)
+#define NPCM7XX_Tx_OPER GENMASK(28, 27)
#define NPCM7XX_Tx_MIN_PRESCALE 0x1
#define NPCM7XX_Tx_TDR_MASK_BITS 24
#define NPCM7XX_Tx_MAX_CNT 0xFFFFFF
@@ -84,8 +84,6 @@ static int npcm7xx_timer_oneshot(struct
clock_event_device *evt)
val = readl(timer_of_base(to) + NPCM7XX_REG_TCSR0);
val &= ~NPCM7XX_Tx_OPER;
-
- val = readl(timer_of_base(to) + NPCM7XX_REG_TCSR0);
val |= NPCM7XX_START_ONESHOT_Tx;
writel(val, timer_of_base(to) + NPCM7XX_REG_TCSR0);
@@ -97,12 +95,11 @@ static int npcm7xx_timer_periodic(struct
clock_event_device *evt)
struct timer_of *to = to_timer_of(evt);
u32 val;
+ writel(timer_of_period(to), timer_of_base(to) + NPCM7XX_REG_TICR0);
+
val = readl(timer_of_base(to) + NPCM7XX_REG_TCSR0);
val &= ~NPCM7XX_Tx_OPER;
-
- writel(timer_of_period(to), timer_of_base(to) + NPCM7XX_REG_TICR0);
val |= NPCM7XX_START_PERIODIC_Tx;
-
writel(val, timer_of_base(to) + NPCM7XX_REG_TCSR0);
return 0;
--
2.18.0
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH] clocksource/drivers/npcm: fix GENMASK and timer operation
2019-07-15 10:05 [PATCH] clocksource/drivers/npcm: fix GENMASK and timer operation Avi Fishman
@ 2019-07-15 12:37 ` Thomas Gleixner
2019-07-15 15:01 ` Avi Fishman
2019-07-15 15:19 ` [PATCH] [v2] " Avi Fishman
1 sibling, 1 reply; 9+ messages in thread
From: Thomas Gleixner @ 2019-07-15 12:37 UTC (permalink / raw)
To: Avi Fishman
Cc: Tomer Maimon, Tali Perry, Patrick Venture, Nancy Yuen,
Benjamin Fair, Daniel Lezcano, OpenBMC Maillist,
Linux Kernel Mailing List
Avi,
On Mon, 15 Jul 2019, Avi Fishman wrote:
> NPCM7XX_Tx_OPER GENMASK was wrong,
That part is already fixed upstream:
9bdd7bb3a844 ("clocksource/drivers/npcm: Fix misuse of GENMASK macro")
> npcm7xx_timer_oneshot() did wrong calculation
That changelog is pretty unspecific. It does not tell what is wrong and
which consequences that has. Please be a bit more specific.
Thanks,
tglx
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] clocksource/drivers/npcm: fix GENMASK and timer operation
2019-07-15 12:37 ` Thomas Gleixner
@ 2019-07-15 15:01 ` Avi Fishman
2019-07-15 15:08 ` Thomas Gleixner
0 siblings, 1 reply; 9+ messages in thread
From: Avi Fishman @ 2019-07-15 15:01 UTC (permalink / raw)
To: Thomas Gleixner
Cc: Tomer Maimon, Tali Perry, Patrick Venture, Nancy Yuen,
Benjamin Fair, Daniel Lezcano, OpenBMC Maillist,
Linux Kernel Mailing List
Hi Thomas,
Thanks,
Avi
On Mon, Jul 15, 2019 at 3:37 PM Thomas Gleixner <tglx@linutronix.de> wrote:
>
> Avi,
>
> On Mon, 15 Jul 2019, Avi Fishman wrote:
>
> > NPCM7XX_Tx_OPER GENMASK was wrong,
>
> That part is already fixed upstream:
>
> 9bdd7bb3a844 ("clocksource/drivers/npcm: Fix misuse of GENMASK macro")
The automatic fix changed from
GENMASK(3, 27) to
GENMASK(27, 3)
I reviewd again the code to check how it worked so far and saw that it
should have been
GENMASK(28, 27) - this is a different value than 9bdd7bb3a844
For our fortune this wrong value didn't effect the our final write to
the register.
But still this should be fixed.
>
> > npcm7xx_timer_oneshot() did wrong calculation
>
> That changelog is pretty unspecific. It does not tell what is wrong and
> which consequences that has. Please be a bit more specific.
OK I will fix
>
> Thanks,
>
> tglx
--
Regards,
Avi
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] clocksource/drivers/npcm: fix GENMASK and timer operation
2019-07-15 15:01 ` Avi Fishman
@ 2019-07-15 15:08 ` Thomas Gleixner
0 siblings, 0 replies; 9+ messages in thread
From: Thomas Gleixner @ 2019-07-15 15:08 UTC (permalink / raw)
To: Avi Fishman
Cc: Tomer Maimon, Tali Perry, Patrick Venture, Nancy Yuen,
Benjamin Fair, Daniel Lezcano, OpenBMC Maillist,
Linux Kernel Mailing List
Avi,
On Mon, 15 Jul 2019, Avi Fishman wrote:
> On Mon, Jul 15, 2019 at 3:37 PM Thomas Gleixner <tglx@linutronix.de> wrote:
> > 9bdd7bb3a844 ("clocksource/drivers/npcm: Fix misuse of GENMASK macro")
>
> The automatic fix changed from
> GENMASK(3, 27) to
> GENMASK(27, 3)
> I reviewd again the code to check how it worked so far and saw that it
> should have been
> GENMASK(28, 27) - this is a different value than 9bdd7bb3a844
> For our fortune this wrong value didn't effect the our final write to
> the register.
> But still this should be fixed.
Fair enough. Please explain it proper in the changelog.
Thanks,
tglx
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH] [v2] clocksource/drivers/npcm: fix GENMASK and timer operation
2019-07-15 10:05 [PATCH] clocksource/drivers/npcm: fix GENMASK and timer operation Avi Fishman
2019-07-15 12:37 ` Thomas Gleixner
@ 2019-07-15 15:19 ` Avi Fishman
2019-07-15 15:25 ` Joe Perches
2019-07-15 15:42 ` [PATCH] [v3] " Avi Fishman
1 sibling, 2 replies; 9+ messages in thread
From: Avi Fishman @ 2019-07-15 15:19 UTC (permalink / raw)
To: Tomer Maimon, Tali Perry, Patrick Venture, Nancy Yuen,
Benjamin Fair, Daniel Lezcano, Thomas Gleixner, Avi Fishman
Cc: OpenBMC Maillist, Linux Kernel Mailing List
clocksource/drivers/npcm: fix GENMASK and timer operation
NPCM7XX_Tx_OPER GENMASK() changed from (27, 3) to (28, 27)
in npcm7xx_timer_oneshot() the original NPCM7XX_REG_TCSR0 register was
read again after masking it with ~NPCM7XX_Tx_OPER so the masking didn't
take effect.
npcm7xx_timer_periodic() was not wrong but it wrote to NPCM7XX_REG_TICR0
in a middle of read modify write to NPCM7XX_REG_TCSR0 which is
confusing.
Signed-off-by: Avi Fishman <avifishman70@gmail.com>
---
drivers/clocksource/timer-npcm7xx.c | 9 +++------
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/drivers/clocksource/timer-npcm7xx.c
b/drivers/clocksource/timer-npcm7xx.c
index 8a30da7f083b..9780ffd8010e 100644
--- a/drivers/clocksource/timer-npcm7xx.c
+++ b/drivers/clocksource/timer-npcm7xx.c
@@ -32,7 +32,7 @@
#define NPCM7XX_Tx_INTEN BIT(29)
#define NPCM7XX_Tx_COUNTEN BIT(30)
#define NPCM7XX_Tx_ONESHOT 0x0
-#define NPCM7XX_Tx_OPER GENMASK(27, 3)
+#define NPCM7XX_Tx_OPER GENMASK(28, 27)
#define NPCM7XX_Tx_MIN_PRESCALE 0x1
#define NPCM7XX_Tx_TDR_MASK_BITS 24
#define NPCM7XX_Tx_MAX_CNT 0xFFFFFF
@@ -84,8 +84,6 @@ static int npcm7xx_timer_oneshot(struct
clock_event_device *evt)
val = readl(timer_of_base(to) + NPCM7XX_REG_TCSR0);
val &= ~NPCM7XX_Tx_OPER;
-
- val = readl(timer_of_base(to) + NPCM7XX_REG_TCSR0);
val |= NPCM7XX_START_ONESHOT_Tx;
writel(val, timer_of_base(to) + NPCM7XX_REG_TCSR0);
@@ -97,12 +95,11 @@ static int npcm7xx_timer_periodic(struct
clock_event_device *evt)
struct timer_of *to = to_timer_of(evt);
u32 val;
+ writel(timer_of_period(to), timer_of_base(to) + NPCM7XX_REG_TICR0);
+
val = readl(timer_of_base(to) + NPCM7XX_REG_TCSR0);
val &= ~NPCM7XX_Tx_OPER;
-
- writel(timer_of_period(to), timer_of_base(to) + NPCM7XX_REG_TICR0);
val |= NPCM7XX_START_PERIODIC_Tx;
-
writel(val, timer_of_base(to) + NPCM7XX_REG_TCSR0);
return 0;
--
2.18.0
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH] [v2] clocksource/drivers/npcm: fix GENMASK and timer operation
2019-07-15 15:19 ` [PATCH] [v2] " Avi Fishman
@ 2019-07-15 15:25 ` Joe Perches
2019-07-15 15:48 ` Avi Fishman
2019-07-15 15:42 ` [PATCH] [v3] " Avi Fishman
1 sibling, 1 reply; 9+ messages in thread
From: Joe Perches @ 2019-07-15 15:25 UTC (permalink / raw)
To: Avi Fishman, Tomer Maimon, Tali Perry, Patrick Venture,
Nancy Yuen, Benjamin Fair, Daniel Lezcano, Thomas Gleixner
Cc: OpenBMC Maillist, Linux Kernel Mailing List
On Mon, 2019-07-15 at 18:19 +0300, Avi Fishman wrote:
> clocksource/drivers/npcm: fix GENMASK and timer operation
>
> NPCM7XX_Tx_OPER GENMASK() changed from (27, 3) to (28, 27)
>
> in npcm7xx_timer_oneshot() the original NPCM7XX_REG_TCSR0 register was
> read again after masking it with ~NPCM7XX_Tx_OPER so the masking didn't
> take effect.
>
> npcm7xx_timer_periodic() was not wrong but it wrote to NPCM7XX_REG_TICR0
> in a middle of read modify write to NPCM7XX_REG_TCSR0 which is
> confusing.
You might mention how the original use of GENMASK(3, 27)
was defective or correct without effect.
> Signed-off-by: Avi Fishman <avifishman70@gmail.com>
> ---
> drivers/clocksource/timer-npcm7xx.c | 9 +++------
> 1 file changed, 3 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/clocksource/timer-npcm7xx.c
> b/drivers/clocksource/timer-npcm7xx.c
> index 8a30da7f083b..9780ffd8010e 100644
> --- a/drivers/clocksource/timer-npcm7xx.c
> +++ b/drivers/clocksource/timer-npcm7xx.c
> @@ -32,7 +32,7 @@
> #define NPCM7XX_Tx_INTEN BIT(29)
> #define NPCM7XX_Tx_COUNTEN BIT(30)
> #define NPCM7XX_Tx_ONESHOT 0x0
> -#define NPCM7XX_Tx_OPER GENMASK(27, 3)
> +#define NPCM7XX_Tx_OPER GENMASK(28, 27)
> #define NPCM7XX_Tx_MIN_PRESCALE 0x1
> #define NPCM7XX_Tx_TDR_MASK_BITS 24
> #define NPCM7XX_Tx_MAX_CNT 0xFFFFFF
> @@ -84,8 +84,6 @@ static int npcm7xx_timer_oneshot(struct
> clock_event_device *evt)
>
> val = readl(timer_of_base(to) + NPCM7XX_REG_TCSR0);
> val &= ~NPCM7XX_Tx_OPER;
> -
> - val = readl(timer_of_base(to) + NPCM7XX_REG_TCSR0);
> val |= NPCM7XX_START_ONESHOT_Tx;
> writel(val, timer_of_base(to) + NPCM7XX_REG_TCSR0);
>
> @@ -97,12 +95,11 @@ static int npcm7xx_timer_periodic(struct
> clock_event_device *evt)
> struct timer_of *to = to_timer_of(evt);
> u32 val;
>
> + writel(timer_of_period(to), timer_of_base(to) + NPCM7XX_REG_TICR0);
> +
> val = readl(timer_of_base(to) + NPCM7XX_REG_TCSR0);
> val &= ~NPCM7XX_Tx_OPER;
> -
> - writel(timer_of_period(to), timer_of_base(to) + NPCM7XX_REG_TICR0);
> val |= NPCM7XX_START_PERIODIC_Tx;
> -
> writel(val, timer_of_base(to) + NPCM7XX_REG_TCSR0);
>
> return 0;
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH] [v3] clocksource/drivers/npcm: fix GENMASK and timer operation
2019-07-15 15:19 ` [PATCH] [v2] " Avi Fishman
2019-07-15 15:25 ` Joe Perches
@ 2019-07-15 15:42 ` Avi Fishman
2019-07-22 9:15 ` Thomas Gleixner
1 sibling, 1 reply; 9+ messages in thread
From: Avi Fishman @ 2019-07-15 15:42 UTC (permalink / raw)
To: Tomer Maimon, Tali Perry, Patrick Venture, Nancy Yuen,
Benjamin Fair, Daniel Lezcano, Thomas Gleixner, Avi Fishman
Cc: OpenBMC Maillist, Linux Kernel Mailing List
clocksource/drivers/npcm: fix GENMASK and timer operation
NPCM7XX_Tx_OPER GENMASK() changed from (27, 3) to (28, 27)
Since NPCM7XX_REG_TICR0 register reset value of those bits was 0,
it did not cause an issue
in npcm7xx_timer_oneshot() the original NPCM7XX_REG_TCSR0 register was
read again after masking it with ~NPCM7XX_Tx_OPER so the masking didn't
take effect.
npcm7xx_timer_periodic() was not wrong but it wrote to NPCM7XX_REG_TICR0
in a middle of read modify write to NPCM7XX_REG_TCSR0 which is
confusing.
Signed-off-by: Avi Fishman <avifishman70@gmail.com>
---
drivers/clocksource/timer-npcm7xx.c | 9 +++------
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/drivers/clocksource/timer-npcm7xx.c
b/drivers/clocksource/timer-npcm7xx.c
index 8a30da7f083b..9780ffd8010e 100644
--- a/drivers/clocksource/timer-npcm7xx.c
+++ b/drivers/clocksource/timer-npcm7xx.c
@@ -32,7 +32,7 @@
#define NPCM7XX_Tx_INTEN BIT(29)
#define NPCM7XX_Tx_COUNTEN BIT(30)
#define NPCM7XX_Tx_ONESHOT 0x0
-#define NPCM7XX_Tx_OPER GENMASK(27, 3)
+#define NPCM7XX_Tx_OPER GENMASK(28, 27)
#define NPCM7XX_Tx_MIN_PRESCALE 0x1
#define NPCM7XX_Tx_TDR_MASK_BITS 24
#define NPCM7XX_Tx_MAX_CNT 0xFFFFFF
@@ -84,8 +84,6 @@ static int npcm7xx_timer_oneshot(struct
clock_event_device *evt)
val = readl(timer_of_base(to) + NPCM7XX_REG_TCSR0);
val &= ~NPCM7XX_Tx_OPER;
-
- val = readl(timer_of_base(to) + NPCM7XX_REG_TCSR0);
val |= NPCM7XX_START_ONESHOT_Tx;
writel(val, timer_of_base(to) + NPCM7XX_REG_TCSR0);
@@ -97,12 +95,11 @@ static int npcm7xx_timer_periodic(struct
clock_event_device *evt)
struct timer_of *to = to_timer_of(evt);
u32 val;
+ writel(timer_of_period(to), timer_of_base(to) + NPCM7XX_REG_TICR0);
+
val = readl(timer_of_base(to) + NPCM7XX_REG_TCSR0);
val &= ~NPCM7XX_Tx_OPER;
-
- writel(timer_of_period(to), timer_of_base(to) + NPCM7XX_REG_TICR0);
val |= NPCM7XX_START_PERIODIC_Tx;
-
writel(val, timer_of_base(to) + NPCM7XX_REG_TCSR0);
return 0;
--
2.18.0
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH] [v2] clocksource/drivers/npcm: fix GENMASK and timer operation
2019-07-15 15:25 ` Joe Perches
@ 2019-07-15 15:48 ` Avi Fishman
0 siblings, 0 replies; 9+ messages in thread
From: Avi Fishman @ 2019-07-15 15:48 UTC (permalink / raw)
To: Joe Perches
Cc: Tomer Maimon, Tali Perry, Patrick Venture, Nancy Yuen,
Benjamin Fair, Daniel Lezcano, Thomas Gleixner, OpenBMC Maillist,
Linux Kernel Mailing List
On Mon, Jul 15, 2019 at 6:25 PM Joe Perches <joe@perches.com> wrote:
>
> On Mon, 2019-07-15 at 18:19 +0300, Avi Fishman wrote:
> > clocksource/drivers/npcm: fix GENMASK and timer operation
> >
> > NPCM7XX_Tx_OPER GENMASK() changed from (27, 3) to (28, 27)
> >
> > in npcm7xx_timer_oneshot() the original NPCM7XX_REG_TCSR0 register was
> > read again after masking it with ~NPCM7XX_Tx_OPER so the masking didn't
> > take effect.
> >
> > npcm7xx_timer_periodic() was not wrong but it wrote to NPCM7XX_REG_TICR0
> > in a middle of read modify write to NPCM7XX_REG_TCSR0 which is
> > confusing.
>
> You might mention how the original use of GENMASK(3, 27)
> was defective or correct without effect.
Done, see v3 of this patch.
>
> > Signed-off-by: Avi Fishman <avifishman70@gmail.com>
> > ---
> > drivers/clocksource/timer-npcm7xx.c | 9 +++------
> > 1 file changed, 3 insertions(+), 6 deletions(-)
> >
> > diff --git a/drivers/clocksource/timer-npcm7xx.c
> > b/drivers/clocksource/timer-npcm7xx.c
> > index 8a30da7f083b..9780ffd8010e 100644
> > --- a/drivers/clocksource/timer-npcm7xx.c
> > +++ b/drivers/clocksource/timer-npcm7xx.c
> > @@ -32,7 +32,7 @@
> > #define NPCM7XX_Tx_INTEN BIT(29)
> > #define NPCM7XX_Tx_COUNTEN BIT(30)
> > #define NPCM7XX_Tx_ONESHOT 0x0
> > -#define NPCM7XX_Tx_OPER GENMASK(27, 3)
> > +#define NPCM7XX_Tx_OPER GENMASK(28, 27)
> > #define NPCM7XX_Tx_MIN_PRESCALE 0x1
> > #define NPCM7XX_Tx_TDR_MASK_BITS 24
> > #define NPCM7XX_Tx_MAX_CNT 0xFFFFFF
> > @@ -84,8 +84,6 @@ static int npcm7xx_timer_oneshot(struct
> > clock_event_device *evt)
> >
> > val = readl(timer_of_base(to) + NPCM7XX_REG_TCSR0);
> > val &= ~NPCM7XX_Tx_OPER;
> > -
> > - val = readl(timer_of_base(to) + NPCM7XX_REG_TCSR0);
> > val |= NPCM7XX_START_ONESHOT_Tx;
> > writel(val, timer_of_base(to) + NPCM7XX_REG_TCSR0);
> >
> > @@ -97,12 +95,11 @@ static int npcm7xx_timer_periodic(struct
> > clock_event_device *evt)
> > struct timer_of *to = to_timer_of(evt);
> > u32 val;
> >
> > + writel(timer_of_period(to), timer_of_base(to) + NPCM7XX_REG_TICR0);
> > +
> > val = readl(timer_of_base(to) + NPCM7XX_REG_TCSR0);
> > val &= ~NPCM7XX_Tx_OPER;
> > -
> > - writel(timer_of_period(to), timer_of_base(to) + NPCM7XX_REG_TICR0);
> > val |= NPCM7XX_START_PERIODIC_Tx;
> > -
> > writel(val, timer_of_base(to) + NPCM7XX_REG_TCSR0);
> >
> > return 0;
> >
>
--
Regards,
Avi
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] [v3] clocksource/drivers/npcm: fix GENMASK and timer operation
2019-07-15 15:42 ` [PATCH] [v3] " Avi Fishman
@ 2019-07-22 9:15 ` Thomas Gleixner
0 siblings, 0 replies; 9+ messages in thread
From: Thomas Gleixner @ 2019-07-22 9:15 UTC (permalink / raw)
To: Avi Fishman
Cc: Tomer Maimon, Tali Perry, Patrick Venture, Nancy Yuen,
Benjamin Fair, Daniel Lezcano, OpenBMC Maillist,
Linux Kernel Mailing List
On Mon, 15 Jul 2019, Avi Fishman wrote:
> clocksource/drivers/npcm: fix GENMASK and timer operation
Don't repeat the subject line please
> NPCM7XX_Tx_OPER GENMASK() changed from (27, 3) to (28, 27)
Please do not write down WHAT the patch does. That can be seen from the
patch itself. Tell why this is wrong and what's the potential problem.
> Since NPCM7XX_REG_TICR0 register reset value of those bits was 0,
> it did not cause an issue
>
> in npcm7xx_timer_oneshot() the original NPCM7XX_REG_TCSR0 register was
> read again after masking it with ~NPCM7XX_Tx_OPER so the masking didn't
> take effect.
>
> npcm7xx_timer_periodic() was not wrong but it wrote to NPCM7XX_REG_TICR0
> in a middle of read modify write to NPCM7XX_REG_TCSR0 which is
> confusing.
>
> Signed-off-by: Avi Fishman <avifishman70@gmail.com>
> ---
> drivers/clocksource/timer-npcm7xx.c | 9 +++------
> 1 file changed, 3 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/clocksource/timer-npcm7xx.c
> b/drivers/clocksource/timer-npcm7xx.c
> index 8a30da7f083b..9780ffd8010e 100644
> --- a/drivers/clocksource/timer-npcm7xx.c
> +++ b/drivers/clocksource/timer-npcm7xx.c
> @@ -32,7 +32,7 @@
> #define NPCM7XX_Tx_INTEN BIT(29)
> #define NPCM7XX_Tx_COUNTEN BIT(30)
> #define NPCM7XX_Tx_ONESHOT 0x0
> -#define NPCM7XX_Tx_OPER GENMASK(27, 3)
> +#define NPCM7XX_Tx_OPER GENMASK(28, 27)
> #define NPCM7XX_Tx_MIN_PRESCALE 0x1
> #define NPCM7XX_Tx_TDR_MASK_BITS 24
> #define NPCM7XX_Tx_MAX_CNT 0xFFFFFF
> @@ -84,8 +84,6 @@ static int npcm7xx_timer_oneshot(struct
> clock_event_device *evt)
>
> val = readl(timer_of_base(to) + NPCM7XX_REG_TCSR0);
> val &= ~NPCM7XX_Tx_OPER;
> -
> - val = readl(timer_of_base(to) + NPCM7XX_REG_TCSR0);
> val |= NPCM7XX_START_ONESHOT_Tx;
> writel(val, timer_of_base(to) + NPCM7XX_REG_TCSR0);
>
> @@ -97,12 +95,11 @@ static int npcm7xx_timer_periodic(struct
> clock_event_device *evt)
Your mail client mangled the patch so it does not apply:
patching file drivers/clocksource/timer-npcm7xx.c
Hunk #1 FAILED at 32.
patch: **** malformed patch at line 49: clock_event_device *evt)
See Documentation/process for hints about sending patches with various mail
clients. Send the patch to yourself first and try to apply it yourself.
Thanks,
tglx
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2019-07-22 9:15 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-07-15 10:05 [PATCH] clocksource/drivers/npcm: fix GENMASK and timer operation Avi Fishman
2019-07-15 12:37 ` Thomas Gleixner
2019-07-15 15:01 ` Avi Fishman
2019-07-15 15:08 ` Thomas Gleixner
2019-07-15 15:19 ` [PATCH] [v2] " Avi Fishman
2019-07-15 15:25 ` Joe Perches
2019-07-15 15:48 ` Avi Fishman
2019-07-15 15:42 ` [PATCH] [v3] " Avi Fishman
2019-07-22 9:15 ` Thomas Gleixner
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).