]> git.baikalelectronics.ru Git - kernel.git/commit
media: rc: nuvoton: remove rudimentary transmit functionality
authorHeiner Kallweit <hkallweit1@gmail.com>
Sat, 15 Jul 2017 21:13:14 +0000 (17:13 -0400)
committerMauro Carvalho Chehab <mchehab@s-opensource.com>
Sun, 20 Aug 2017 13:47:57 +0000 (09:47 -0400)
commit2cbf8e8730e23cda2cb1be17850b174b18f6c91c
tree6ccb804cf6afea507b181b220768f34d6143f07d
parent62ad68f213ba233b37297c574b985e61081eb249
media: rc: nuvoton: remove rudimentary transmit functionality

Transmit support in this driver was never tested and based on the code
it can't work. Just one example:
The buffer provided to nvt_tx_ir holds unsigned int values in
micro seconds: First value is for a pulse, second for a pause, etc.
Bytes in this buffer are copied as-is to the chip FIFO what can't work
as the chip-internal format is totally different. See also conversion
done in nvt_process_rx_ir_data.

Even if we would try to fix this we have the issue that we can't test
it. There seems to be no device on the market using IR transmit with
one of the chips supported by this driver.

To facilitate maintenance of the driver I'd propose to remove the
rudimentary transmit support.

Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
Signed-off-by: Sean Young <sean@mess.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
drivers/media/rc/nuvoton-cir.c
drivers/media/rc/nuvoton-cir.h