On Wed, Sep 06, 2023 at 05:58:03PM +0200, Uwe Kleine-König wrote: > On Tue, Aug 22, 2023 at 03:56:43PM +0100, Sean Young wrote: > > The ir-rx51 is a pwm-based TX driver specific to the N900. This can be > > handled entirely by the generic pwm-ir-tx driver, and in fact the > > pwm-ir-tx driver has been compatible with ir-rx51 from the start. > > > > Note that the suspend code in the ir-rx51 driver is unnecessary, since > > during transmit, the process is not in interruptable sleep. The process > > is not put to sleep until the transmit completes. > > > > Tested-by: Sicelo A. Mhlongo <absicsz@xxxxxxxxx> > > Signed-off-by: Sean Young <sean@xxxxxxxx> > > Cc: Tony Lindgren <tony@xxxxxxxxxxx> > > Cc: Russell King <linux@xxxxxxxxxxxxxxx> > > Cc: Thierry Reding <thierry.reding@xxxxxxxxx> > > Cc: "Uwe Kleine-König" <u.kleine-koenig@xxxxxxxxxxxxxx> > > Cc: linux-omap@xxxxxxxxxxxxxxx > > Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx > > Cc: linux-pwm@xxxxxxxxxxxxxxx > > Cc: Ivaylo Dimitrov <ivo.g.dimitrov.75@xxxxxxxxx> > > Cc: Pali Rohár <pali.rohar@xxxxxxxxx> > > Cc: Pavel Machek <pavel@xxxxxx> > > Cc: Timo Kokkonen <timo.t.kokkonen@xxxxxx> > > Looks ok for me. I assume this will go in via the media tree and the pwm > list was only Cc:d because drivers/media/rc/pwm-ir-tx.c uses pwm > functions? That is exactly right. > I'll mark this patch as "handled-elsewhere" in the pwm > patchwork. Please tell me if you object. > > Acked-by: Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxxxx> That's perfect. Thanks, Sean
