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

From: Uwe Kleine-König
Date: Wed Sep 06 2023 - 12:05:56 EST


Hello,

On Fri, Sep 01, 2023 at 03:18:56PM +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.
>
> Cc: Timo Kokkonen <timo.t.kokkonen@xxxxxx>
> Cc: Pali Rohár <pali.rohar@xxxxxxxxx>
> Cc: Ivaylo Dimitrov <ivo.g.dimitrov.75@xxxxxxxxx>
> Tested-by: Sicelo A. Mhlongo <absicsz@xxxxxxxxx>
> Signed-off-by: Sean Young <sean@xxxxxxxx>

I just commented on v3 of this patch. It was an

Acked-by: Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxxxx>

which I forward hereby to this v5. As with the previous revisions I'll
mark this as handled-elsewhere in the pwm patchwork in the assumption
that the pwm tree isn't involved to get this into the mainline.

Best regards
Uwe

--
Pengutronix e.K. | Uwe Kleine-König |
Industrial Linux Solutions | https://www.pengutronix.de/ |

Attachment: signature.asc
Description: PGP signature