On 08/20/2017 03:11 PM, Benjamin Gaignard wrote: > 2017-08-04 12:41 GMT+02:00 Hans Verkuil <hverkuil@xxxxxxxxx>: >> From: Hans Verkuil <hans.verkuil@xxxxxxxxx> >> >> Use the new CEC_CAP_DEFAULTS define in this driver. >> This also adds the CEC_CAP_RC capability which was missing here >> (and this is also the reason for this new define, to avoid missing >> such capabilities). >> >> Signed-off-by: Hans Verkuil <hans.verkuil@xxxxxxxxx> > > Acked-by: Benjamin Gaignard <benjamin.gaignard@xxxxxxxxxx> Thanks! Just in time for 4.14. Will make another pull request for cec fixes tomorrow. Regards, Hans > >> --- >> drivers/media/platform/sti/cec/stih-cec.c | 4 +--- >> 1 file changed, 1 insertion(+), 3 deletions(-) >> >> diff --git a/drivers/media/platform/sti/cec/stih-cec.c b/drivers/media/platform/sti/cec/stih-cec.c >> index ce7964c71b50..dc221527fd05 100644 >> --- a/drivers/media/platform/sti/cec/stih-cec.c >> +++ b/drivers/media/platform/sti/cec/stih-cec.c >> @@ -351,9 +351,7 @@ static int stih_cec_probe(struct platform_device *pdev) >> } >> >> cec->adap = cec_allocate_adapter(&sti_cec_adap_ops, cec, >> - CEC_NAME, >> - CEC_CAP_LOG_ADDRS | CEC_CAP_PASSTHROUGH | >> - CEC_CAP_TRANSMIT, CEC_MAX_LOG_ADDRS); >> + CEC_NAME, CEC_CAP_DEFAULTS, CEC_MAX_LOG_ADDRS); >> if (IS_ERR(cec->adap)) >> return PTR_ERR(cec->adap); >> >> -- >> 2.13.2 >> > > >