Instead of hardcoding the UI Command key values, use the new
defines.

Signed-off-by: Hans Verkuil <hverkuil-ci...@xs4all.nl>
---
 drivers/media/cec/cec-adap.c | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/drivers/media/cec/cec-adap.c b/drivers/media/cec/cec-adap.c
index 5ef7daeb8cbd..284f9b845161 100644
--- a/drivers/media/cec/cec-adap.c
+++ b/drivers/media/cec/cec-adap.c
@@ -1976,7 +1976,7 @@ static int cec_receive_notify(struct cec_adapter *adap, 
struct cec_msg *msg,
                 * Play function, this message can have variable length
                 * depending on the specific play function that is used.
                 */
-               case 0x60:
+               case CEC_OP_UI_CMD_PLAY_FUNCTION:
                        if (msg->len == 2)
                                rc_keydown(adap->rc, RC_PROTO_CEC,
                                           msg->msg[2], 0);
@@ -1993,8 +1993,12 @@ static int cec_receive_notify(struct cec_adapter *adap, 
struct cec_msg *msg,
                 * For the time being these messages are not processed by the
                 * framework and are simply forwarded to the user space.
                 */
-               case 0x56: case 0x57:
-               case 0x67: case 0x68: case 0x69: case 0x6a:
+               case CEC_OP_UI_CMD_SELECT_BROADCAST_TYPE:
+               case CEC_OP_UI_CMD_SELECT_SOUND_PRESENTATION:
+               case CEC_OP_UI_CMD_TUNE_FUNCTION:
+               case CEC_OP_UI_CMD_SELECT_MEDIA_FUNCTION:
+               case CEC_OP_UI_CMD_SELECT_AV_INPUT_FUNCTION:
+               case CEC_OP_UI_CMD_SELECT_AUDIO_INPUT_FUNCTION:
                        break;
                default:
                        rc_keydown(adap->rc, RC_PROTO_CEC, msg->msg[2], 0);
-- 
2.23.0.rc1

Reply via email to