On Wed, Nov 21, 2018 at 10:27:41AM +0000, Biju Das wrote:
> Hi Simon,
> 
> Thanks for the feedback.
> 
> > -----Original Message-----
> > From: Simon Horman <[email protected]>
> > Sent: 21 November 2018 10:24
> > To: Biju Das <[email protected]>
> > Cc: Rob Herring <[email protected]>; Mark Rutland
> > <[email protected]>; Magnus Damm <[email protected]>;
> > [email protected]; [email protected]; Geert
> > Uytterhoeven <[email protected]>; Chris Paterson
> > <[email protected]>; Daniel Lezcano
> > <[email protected]>; Thomas Gleixner <[email protected]>; John
> > Stultz <[email protected]>; Fabrizio Castro
> > <[email protected]>
> > Subject: Re: [PATCH] arm64: dts: renesas: r8a7796: Add CMT device nodes
> >
> > On Fri, Oct 26, 2018 at 09:25:07AM +0100, Biju Das wrote:
> > > This patch adds CMT{0|1|2|3} device nodes for r8a7796 SoC.
> > >
> > > Signed-off-by: Biju Das <[email protected]>
> > > ---
> > > This patch is tested against renesas-dev
> > >
> > > I have executed on inconsistency-check, nanosleep and
> > > clocksource_switch selftests on this arm64 SoC. The
> > > inconsistency-check and nanosleep tests are working fine.The
> > > clocksource_switch asynchronous test is failing due to inconsistency-check
> > failure on "arch_sys_counter".
> > >
> > > But if i skip the clocksource_switching of "arch_sys_counter", the
> > > asynchronous test is passing for CMT0/1/2/3 timer.
> > >
> > > Has any one noticed this issue?
> >
> > I am reluctant to apply this patch until there is a better understanding of 
> > the
> > issue above.
> 
> If I understand correctly this patch has no issue at all. The problem is 
> related to ARM architecture  timer.
> Please correct me if you think otherwise.

Thanks for the clarification, I have applied this patch for v4.21.

Reply via email to