Re: [Freedreno] [DPU PATCH] msm/hdcp: Remove redundant stubs/CONFIG

2018-03-13 Thread abhinavk
On 2018-03-12 13:12, Sean Paul wrote: On Tue, Feb 27, 2018 at 11:24:31AM -0500, Sean Paul wrote: On Mon, Feb 26, 2018 at 03:01:14PM -0800, abhin...@codeaurora.org wrote: > The change itself is okay. So, Reviewed-by? Sean > However I am planning to do a bigger cleanup here > ( removing the

Re: [Freedreno] [DPU PATCH] msm/hdcp: Remove redundant stubs/CONFIG

2018-03-12 Thread Sean Paul
On Tue, Feb 27, 2018 at 11:24:31AM -0500, Sean Paul wrote: > On Mon, Feb 26, 2018 at 03:01:14PM -0800, abhin...@codeaurora.org wrote: > > The change itself is okay. So, Reviewed-by? Sean > > However I am planning to do a bigger cleanup here > > ( removing the entire hdmi_hdcp.c ). > > > > We

Re: [Freedreno] [DPU PATCH] msm/hdcp: Remove redundant stubs/CONFIG

2018-02-27 Thread Sean Paul
On Mon, Feb 26, 2018 at 03:01:14PM -0800, abhin...@codeaurora.org wrote: > The change itself is okay. However I am planning to do a bigger cleanup here > ( removing the entire hdmi_hdcp.c ). > > We dont use this file as we have our equivalent sde_hdcp_1x.c. Yes, we definitely need to rationalize

Re: [Freedreno] [DPU PATCH] msm/hdcp: Remove redundant stubs/CONFIG

2018-02-26 Thread abhinavk
The change itself is okay. However I am planning to do a bigger cleanup here ( removing the entire hdmi_hdcp.c ). We dont use this file as we have our equivalent sde_hdcp_1x.c. Was planning this cleanup as part of the HDCP 1x requirements. If we want to push this as as separate change, I am

[Freedreno] [DPU PATCH] msm/hdcp: Remove redundant stubs/CONFIG

2018-02-26 Thread Sean Paul
We already have CONFIG_DRM_MSM_HDMI_HDCP, with accompanying stubs in hdmi/hdmi.h. Signed-off-by: Sean Paul --- drivers/gpu/drm/msm/hdmi/hdmi_hdcp.c | 24 1 file changed, 24 deletions(-) diff --git a/drivers/gpu/drm/msm/hdmi/hdmi_hdcp.c