Re: [PATCH 7/8] media: vivid: add CEC support to display present ctrl

2019-06-11 Thread Hans Verkuil
On 6/11/19 10:25 PM, Johan Korsnes wrote: > Thank you for the review Hans. My comments are inline. A v2 of this > series is under way. > > On 5/29/19 12:39 AM, Hans Verkuil wrote: >> Hi Johan, >> >> Thanks for this patch series! >> >> The previous patches all look good, but this one needs a bit mo

Re: [PATCH 7/8] media: vivid: add CEC support to display present ctrl

2019-06-11 Thread Johan Korsnes
Thank you for the review Hans. My comments are inline. A v2 of this series is under way. On 5/29/19 12:39 AM, Hans Verkuil wrote: > Hi Johan, > > Thanks for this patch series! > > The previous patches all look good, but this one needs a bit more work: > > On 5/28/19 7:19 PM, johan.kors...@gmail

Re: [PATCH 7/8] media: vivid: add CEC support to display present ctrl

2019-05-29 Thread Hans Verkuil
Hi Johan, Thanks for this patch series! The previous patches all look good, but this one needs a bit more work: On 5/28/19 7:19 PM, johan.kors...@gmail.com wrote: > From: Johan Korsnes > > Set/invalidate physical addresses based on the configuration of the > display present control. This is re

[PATCH 7/8] media: vivid: add CEC support to display present ctrl

2019-05-28 Thread johan . korsnes
From: Johan Korsnes Set/invalidate physical addresses based on the configuration of the display present control. This is relevant not only when the display present control is modified, but also when the Vivid instance EDID is set/cleared. Signed-off-by: Johan Korsnes --- drivers/media/platform