Re: [PATCH v3 1/2] media: rc: remove ir-rx51 in favour of generic pwm-ir-tx

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]


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.



[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux