Hi, * Pali Rohár <pali.rohar@xxxxxxxxx> [160102 06:46]: > --- a/drivers/media/rc/ir-rx51.c > +++ b/drivers/media/rc/ir-rx51.c > @@ -25,9 +25,9 @@ > #include <linux/platform_device.h> > #include <linux/sched.h> > #include <linux/wait.h> > +#include <linux/clk.h> > > -#include <plat/dmtimer.h> > -#include <plat/clock.h> > +#include "../../../arch/arm/plat-omap/include/plat/dmtimer.h" Well we don't want to export the dmtimer functions to drivers..But we now have the PWM driver that can be already used for most of the ir-rx51.c. > #include <media/lirc.h> > #include <media/lirc_dev.h> > @@ -208,7 +208,7 @@ static int lirc_rx51_init_port(struct lirc_rx51 *lirc_rx51) > } > > clk_fclk = omap_dm_timer_get_fclk(lirc_rx51->pwm_timer); > - lirc_rx51->fclk_khz = clk_fclk->rate / 1000; > + lirc_rx51->fclk_khz = clk_get_rate(clk_fclk) / 1000; > > return 0; > > > So Tony, you are author of that commit (a62a6e98c3) which broke ir-rx51 > module for Nokia N900. Do you know how to fix this driver for upstream > kernel? It would be great to have driver working and not to have it in > this dead state... Yup please take a look at thread "[PATCH 0/3] pwm: omap: Add PWM support using dual-mode timers". Chances are we still need to set up the dmtimer code to provide also irqchip functions. That way ir-rx51.c can just do request_irq on the selected dmtimer for interrupts. > Also platform data for this driver are only in legacy board code. > Support in DTS is missing, so driver (after fixing above problem) cannot > be used on DT booted kernel. Yeah those parts should be already doable with the PWM timer code AFAIK. Regards, Tony -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html