Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1644923

Re: [PATCH 7/8] clocksource/drivers/fttmr010: Merge Moxa into FTTMR010

Show all headers | View raw

On 05/18/2017 10:20 PM, Linus Walleij wrote:
> On Thu, May 18, 2017 at 2:43 PM, Linus Walleij <linus.walleij@linaro.org> wrote:
>> On Thu, May 18, 2017 at 9:22 AM, Joel Stanley <joel@jms.id.au> wrote:
>>> On Wed, May 17, 2017 at 10:05 PM, Linus Walleij
>>> <linus.walleij@linaro.org> wrote:
>>>> This merges the Moxa Art timer driver into the Faraday FTTMR010
>>>> driver and replaces all Kconfig symbols to use the Faraday
>>>> driver instead. We are now so similar that the drivers can
>>>> be merged by just adding a few lines to the Faraday timer.
>>>
>>> Nice work!
>>>
>>> I gave this a spin on hardware and it didn't work :(
>>
>> How typical.
> 
> I sent a v2 patch set.
> 
> I couldn't get qemu to work because Fedora's QEMU is not
> up-to-date and I didn't want to venture into compiling from source...
> 
> I did the second best and tested Gemini with downward counting
> timers, after just adding that as general code path in the driver
> in the commit merging the two drivers.
> 
> Please test it, and I hope I'm not wasting too much of your time :/
> 
> If it still doesn't work I guess I have to try to get qemu going.
> 
> The fttmr010 branch in my git is updated:
> https://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-nomadik.git/

So a real AST2500 evb board boot fines with this branch but the timer 
model in QEMU crashes miserably :

	[    0.000000] NR_IRQS:16 nr_irqs:16 16
	Floating point exception (core dumped)

that is just before :

	[    0.000000] clocksource: FTTMR010-TIMER2: mask: 0xffffffff max_cycles: 0xffffffff, max_idle_ns: 77222644334 ns
	[    0.000025] sched_clock: 32 bits at 24MHz, resolution 40ns, wraps every 86767015915ns

Something to look at in QEMU.

Joel, do you confirm ? 

Thanks,

C.  

	

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 0/8] clocksource: Generalize Faraday timer Linus Walleij <linus.walleij@linaro.org> - 2017-05-17 16:10 +0200
  [PATCH 8/8] ARM: dts: augment Moxa and Aspeed DTS for FTTMR010 Linus Walleij <linus.walleij@linaro.org> - 2017-05-17 16:10 +0200
  [PATCH 5/8] clocksource/drivers/fttmr010: Switch to use bitops Linus Walleij <linus.walleij@linaro.org> - 2017-05-17 16:10 +0200
  [PATCH 2/8] clocksource/drivers/fttmr010: Merge FTTMR010 DT bindings Linus Walleij <linus.walleij@linaro.org> - 2017-05-17 16:10 +0200
    Re: [PATCH 2/8] clocksource/drivers/fttmr010: Merge FTTMR010 DT  bindings Rob Herring <robh@kernel.org> - 2017-05-23 03:00 +0200
  [PATCH 3/8] clocksource/drivers/fttmr010: Drop Gemini specifics Linus Walleij <linus.walleij@linaro.org> - 2017-05-17 16:10 +0200
  [PATCH 6/8] clocksource/drivers/fttmr010: Switch to use TIMER2 src Linus Walleij <linus.walleij@linaro.org> - 2017-05-17 16:10 +0200
  [PATCH 7/8] clocksource/drivers/fttmr010: Merge Moxa into FTTMR010 Linus Walleij <linus.walleij@linaro.org> - 2017-05-17 16:10 +0200
    Re: [PATCH 7/8] clocksource/drivers/fttmr010: Merge Moxa into FTTMR010 Joel Stanley <joel@jms.id.au> - 2017-05-18 09:30 +0200
      Re: [PATCH 7/8] clocksource/drivers/fttmr010: Merge Moxa into FTTMR010 Linus Walleij <linus.walleij@linaro.org> - 2017-05-18 14:50 +0200
        Re: [PATCH 7/8] clocksource/drivers/fttmr010: Merge Moxa into  FTTMR010 Cédric Le Goater <clg@kaod.org> - 2017-05-18 16:40 +0200
        Re: [PATCH 7/8] clocksource/drivers/fttmr010: Merge Moxa into FTTMR010 Linus Walleij <linus.walleij@linaro.org> - 2017-05-18 22:30 +0200
          Re: [PATCH 7/8] clocksource/drivers/fttmr010: Merge Moxa into  FTTMR010 Cédric Le Goater <clg@kaod.org> - 2017-05-18 23:10 +0200
            Re: [PATCH 7/8] clocksource/drivers/fttmr010: Merge Moxa into  FTTMR010 Cédric Le Goater <clg@kaod.org> - 2017-05-19 10:00 +0200
  [PATCH 4/8] clocksource/drivers/fttmr010: Use state container Linus Walleij <linus.walleij@linaro.org> - 2017-05-17 16:10 +0200
  [PATCH 1/8] clocksource/drivers/fttmr010: Fix the clock handling Linus Walleij <linus.walleij@linaro.org> - 2017-05-17 16:10 +0200
  Re: [PATCH 0/8] clocksource: Generalize Faraday timer Jonas Jensen <jonas.jensen@gmail.com> - 2017-05-18 12:40 +0200
    Re: [PATCH 0/8] clocksource: Generalize Faraday timer Linus Walleij <linus.walleij@linaro.org> - 2017-05-18 14:20 +0200
      Re: [PATCH 0/8] clocksource: Generalize Faraday timer Daniel Lezcano <daniel.lezcano@linaro.org> - 2017-05-18 15:40 +0200

csiph-web