Re: [V2][PATCH 2/2] drm: xlnx: consolidate the functions which programming AUDIO_VIDEO_SELECT register

2021-05-20 Thread quanyang.wang
Hi Laurent, Thank you for your review. On 5/20/21 5:56 PM, Laurent Pinchart wrote: Hi Quanyang, Thank you for the patch. On Tue, May 18, 2021 at 05:50:19PM +0800, quanyang.w...@windriver.com wrote: From: Quanyang Wang For now, the functions zynqmp_disp_avbuf_enable/disable_audio and

Re: [V2][PATCH 2/2] drm: xlnx: consolidate the functions which programming AUDIO_VIDEO_SELECT register

2021-05-20 Thread Laurent Pinchart
Hi Quanyang, Thank you for the patch. On Tue, May 18, 2021 at 05:50:19PM +0800, quanyang.w...@windriver.com wrote: > From: Quanyang Wang > > For now, the functions zynqmp_disp_avbuf_enable/disable_audio and > zynqmp_disp_avbuf_enable/disable_video are all programming the register >

Re: [V2][PATCH 2/2] drm: xlnx: consolidate the functions which programming AUDIO_VIDEO_SELECT register

2021-05-19 Thread Paul Cercueil
Hi, Le mar., mai 18 2021 at 17:50:19 +0800, quanyang.w...@windriver.com a écrit : From: Quanyang Wang For now, the functions zynqmp_disp_avbuf_enable/disable_audio and zynqmp_disp_avbuf_enable/disable_video are all programming the register AV_BUF_OUTPUT_AUDIO_VIDEO_SELECT to select the

[V2][PATCH 2/2] drm: xlnx: consolidate the functions which programming AUDIO_VIDEO_SELECT register

2021-05-18 Thread quanyang . wang
From: Quanyang Wang For now, the functions zynqmp_disp_avbuf_enable/disable_audio and zynqmp_disp_avbuf_enable/disable_video are all programming the register AV_BUF_OUTPUT_AUDIO_VIDEO_SELECT to select the output for audio or video. And in the future, many drm properties (like video_tpg,