Re: [PATCH] fixup! efi: clocksoure: add EFI event timer
On Mon, Mar 13, 2017 at 02:16:06PM +0100, Michael Olbrich wrote: > --- > > I played with the numbers a bit. This hardware has a 18 Hz tick :-/. > That's not really useful so just skipping the clocksource is probably the > best solution. > > Michael > > drivers/clocksource/efi.c | 4 > 1 file changed, 4 insertions(+) > > diff --git a/drivers/clocksource/efi.c b/drivers/clocksource/efi.c > index 89906c452eb7..fb5b7ca63de9 100644 > --- a/drivers/clocksource/efi.c > +++ b/drivers/clocksource/efi.c > @@ -71,6 +71,10 @@ static int efi_cs_init(struct clocksource *cs) > uint64_t nb_100ns; > > freq = ticks_freq_x86(); > + if (freq == 0) { > + BS->close_event(efi_cs_evt); > + return -ENODEV; > + } > nb_100ns = 10 * 1000 * 1000 / freq; > pr_warn("EFI Event timer too slow freq = %llu Hz\n", freq); > efiret = BS->set_timer(efi_cs_evt, EFI_TIMER_PERIODIC, > nb_100ns); Since the patch this fixup is for is already in master: Can you provide a proper, non-fixup patch? Sascha -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0| Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917- | ___ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox
Re: [PATCH] fixup! efi: clocksoure: add EFI event timer
> On 13 Mar 2017, at 9:16 PM, Michael Olbrich wrote: > > --- > > I played with the numbers a bit. This hardware has a 18 Hz tick :-/. 18Hz wow it’s unusable how can they implement this > That's not really useful so just skipping the clocksource is probably the > best solution. > Acked-by: Jean-Christophe PLAGNIOL-VILLARD Best Regards, J. ___ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox
[PATCH] fixup! efi: clocksoure: add EFI event timer
--- I played with the numbers a bit. This hardware has a 18 Hz tick :-/. That's not really useful so just skipping the clocksource is probably the best solution. Michael drivers/clocksource/efi.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/clocksource/efi.c b/drivers/clocksource/efi.c index 89906c452eb7..fb5b7ca63de9 100644 --- a/drivers/clocksource/efi.c +++ b/drivers/clocksource/efi.c @@ -71,6 +71,10 @@ static int efi_cs_init(struct clocksource *cs) uint64_t nb_100ns; freq = ticks_freq_x86(); + if (freq == 0) { + BS->close_event(efi_cs_evt); + return -ENODEV; + } nb_100ns = 10 * 1000 * 1000 / freq; pr_warn("EFI Event timer too slow freq = %llu Hz\n", freq); efiret = BS->set_timer(efi_cs_evt, EFI_TIMER_PERIODIC, nb_100ns); -- 2.11.0 ___ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox