From mboxrd@z Thu Jan  1 00:00:00 1970
From: Tony Lindgren <tony@atomide.com>
Subject: Re: Nokia N900: Broken lirc ir-rx51 driver
Date: Fri, 22 Apr 2016 15:21:11 -0700
Message-ID: <20160422222110.GM5995@atomide.com>
References: <201601021545.35201@pali>
 <20160102170657.GB12622@atomide.com>
 <20160105101841.GI22661@pali>
 <20160106011249.GH12777@atomide.com>
 <20160422134929.GG29406@pali>
Mime-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: QUOTED-PRINTABLE
Return-path: <linux-kernel-owner@vger.kernel.org>
Content-Disposition: inline
In-Reply-To: <20160422134929.GG29406@pali>
Sender: linux-kernel-owner@vger.kernel.org
To: Pali =?utf-8?B?Um9ow6Fy?= <pali.rohar@gmail.com>
Cc: Aaro Koskinen <aaro.koskinen@iki.fi>, Ivaylo Dimitrov <ivo.g.dimitrov.75@gmail.com>, Sebastian Reichel <sre@kernel.org>, Pavel Machel <pavel@ucw.cz>, Nishanth Menon <nm@ti.com>, Timo Kokkonen <timo.t.kokkonen@iki.fi>, linux-omap@vger.kernel.org, linux-kernel@vger.kernel.org
List-Id: linux-omap@vger.kernel.org

* Pali Roh=C3=A1r <pali.rohar@gmail.com> [160422 06:50]:
> On Tuesday 05 January 2016 17:12:50 Tony Lindgren wrote:
> > * Pali Roh=C3=A1r <pali.rohar@gmail.com> [160105 02:19]:
> > > On Saturday 02 January 2016 09:06:57 Tony Lindgren wrote:
> > > >
> > > > Yup please take a look at thread "[PATCH 0/3] pwm: omap: Add PW=
M support
> > > > using dual-mode timers". Chances are we still need to set up th=
e dmtimer
> > > > code to provide also irqchip functions. That way ir-rx51.c can =
just do
> > > > request_irq on the selected dmtimer for interrupts.
> > >=20
> > > No I see that patch from that thread uses dmtimer.h from plat-oma=
p. So
> > > it is really OK?
> >=20
> > It's using the header to populate the platform data in mach-omap2 s=
o
> > that's fine. But we do not want to directly expose the dmtimer func=
tions
> > to device drivers as they are not Linux generic at this point.
>
> Hi Tony! Is there any progress for ir-rx51 driver? Months ago we were
> waiting for some omap pwm patches... What is current state?

Sorry it will be slow going for this one.. I think we should initially
just make it use include/linux/platform_data/pwm_omap_dmtimer.h and
pdata-quirks.c. It seems we need to add few new functions there.

Then once it's working again, we can continue to make it use hrtimer
for one of the gptimers. Then after that it should be easy to change
it to use the PWM framework with the pwm_omap_dmtimer.

Regards,

Tony