Re: [PATCH v2 2/2] cec: add STM32 cec driver

2017-05-16 Thread Benjamin Gaignard
2017-05-16 15:09 GMT+02:00 Hans Verkuil : > Looks good, except for the logical address handling that I think is wrong: > > On 16/05/17 14:56, Benjamin Gaignard wrote: >> This patch add cec driver for STM32 platforms. >> cec hardware block isn't not always used with hdmi so >> cec notifier is not im

Re: [PATCH v2 2/2] cec: add STM32 cec driver

2017-05-16 Thread Hans Verkuil
Looks good, except for the logical address handling that I think is wrong: On 16/05/17 14:56, Benjamin Gaignard wrote: > This patch add cec driver for STM32 platforms. > cec hardware block isn't not always used with hdmi so > cec notifier is not implemented. That will be done later > when STM32 DS

[PATCH v2 2/2] cec: add STM32 cec driver

2017-05-16 Thread Benjamin Gaignard
This patch add cec driver for STM32 platforms. cec hardware block isn't not always used with hdmi so cec notifier is not implemented. That will be done later when STM32 DSI driver will be available. Driver compliance has been tested with cec-ctl and cec-compliance tools. Signed-off-by: Benjamin G