All of lore.kernel.org
 help / color / mirror / Atom feed
* [DPU PATCH] msm/hdcp: Remove redundant stubs/CONFIG
@ 2018-02-26 20:48 Sean Paul
       [not found] ` <20180226204821.149179-1-seanpaul-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>
  0 siblings, 1 reply; 5+ messages in thread
From: Sean Paul @ 2018-02-26 20:48 UTC (permalink / raw)
  To: freedreno-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW,
	linux-arm-msm-u79uwXL29TY76Z2rM5mHXA
  Cc: jsanka-sgV2jX0FEOL9JmXXK+q4OQ, abhinavk-jfJNa2p1gH1BDgjK7y7TUQ,
	robdclark-Re5JQEeQqe8AvxtiuMwx3w, Sean Paul,
	hoegsberg-F7+t8E8rja9g9hUCZPvPmw

We already have CONFIG_DRM_MSM_HDMI_HDCP, with accompanying stubs in
hdmi/hdmi.h.

Signed-off-by: Sean Paul <seanpaul@chromium.org>
---
 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 b/drivers/gpu/drm/msm/hdmi/hdmi_hdcp.c
index d24527468284..87e3acb3a259 100644
--- a/drivers/gpu/drm/msm/hdmi/hdmi_hdcp.c
+++ b/drivers/gpu/drm/msm/hdmi/hdmi_hdcp.c
@@ -14,7 +14,6 @@
 #include "hdmi.h"
 #include <linux/qcom_scm.h>
 
-#ifdef CONFIG_DRM_MSM_HDCP
 #define HDCP_REG_ENABLE 0x01
 #define HDCP_REG_DISABLE 0x00
 #define HDCP_PORT_ADDR 0x74
@@ -1436,26 +1435,3 @@ void msm_hdmi_hdcp_destroy(struct hdmi *hdmi)
 		hdmi->hdcp_ctrl = NULL;
 	}
 }
-
-#else
-struct hdmi_hdcp_ctrl *msm_hdmi_hdcp_init(struct hdmi *hdmi)
-{
-	return NULL;
-}
-
-void msm_hdmi_hdcp_destroy(struct hdmi *hdmi)
-{
-}
-
-void msm_hdmi_hdcp_on(struct hdmi_hdcp_ctrl *hdcp_ctrl)
-{
-}
-
-void msm_hdmi_hdcp_off(struct hdmi_hdcp_ctrl *hdcp_ctrl)
-{
-}
-
-void msm_hdmi_hdcp_irq(struct hdmi_hdcp_ctrl *hdcp_ctrl)
-{
-}
-#endif
-- 
2.16.1.291.g4437f3f132-goog

_______________________________________________
Freedreno mailing list
Freedreno@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/freedreno

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [DPU PATCH] msm/hdcp: Remove redundant stubs/CONFIG
       [not found] ` <20180226204821.149179-1-seanpaul-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>
@ 2018-02-26 23:01   ` abhinavk-sgV2jX0FEOL9JmXXK+q4OQ
       [not found]     ` <7094f760c7720331cfa9a3cf6e1f32af-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>
  0 siblings, 1 reply; 5+ messages in thread
From: abhinavk-sgV2jX0FEOL9JmXXK+q4OQ @ 2018-02-26 23:01 UTC (permalink / raw)
  To: Sean Paul
  Cc: linux-arm-msm-u79uwXL29TY76Z2rM5mHXA,
	jsanka-sgV2jX0FEOL9JmXXK+q4OQ,
	freedreno-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW,
	robdclark-Re5JQEeQqe8AvxtiuMwx3w,
	hoegsberg-F7+t8E8rja9g9hUCZPvPmw

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 okay with it but 
would prefer to wait ...

Abhinav
On 2018-02-26 12:48, Sean Paul wrote:
> We already have CONFIG_DRM_MSM_HDMI_HDCP, with accompanying stubs in
> hdmi/hdmi.h.
> 
> Signed-off-by: Sean Paul <seanpaul@chromium.org>
> ---
>  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
> b/drivers/gpu/drm/msm/hdmi/hdmi_hdcp.c
> index d24527468284..87e3acb3a259 100644
> --- a/drivers/gpu/drm/msm/hdmi/hdmi_hdcp.c
> +++ b/drivers/gpu/drm/msm/hdmi/hdmi_hdcp.c
> @@ -14,7 +14,6 @@
>  #include "hdmi.h"
>  #include <linux/qcom_scm.h>
> 
> -#ifdef CONFIG_DRM_MSM_HDCP
>  #define HDCP_REG_ENABLE 0x01
>  #define HDCP_REG_DISABLE 0x00
>  #define HDCP_PORT_ADDR 0x74
> @@ -1436,26 +1435,3 @@ void msm_hdmi_hdcp_destroy(struct hdmi *hdmi)
>  		hdmi->hdcp_ctrl = NULL;
>  	}
>  }
> -
> -#else
> -struct hdmi_hdcp_ctrl *msm_hdmi_hdcp_init(struct hdmi *hdmi)
> -{
> -	return NULL;
> -}
> -
> -void msm_hdmi_hdcp_destroy(struct hdmi *hdmi)
> -{
> -}
> -
> -void msm_hdmi_hdcp_on(struct hdmi_hdcp_ctrl *hdcp_ctrl)
> -{
> -}
> -
> -void msm_hdmi_hdcp_off(struct hdmi_hdcp_ctrl *hdcp_ctrl)
> -{
> -}
> -
> -void msm_hdmi_hdcp_irq(struct hdmi_hdcp_ctrl *hdcp_ctrl)
> -{
> -}
> -#endif
_______________________________________________
Freedreno mailing list
Freedreno@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/freedreno

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [DPU PATCH] msm/hdcp: Remove redundant stubs/CONFIG
       [not found]     ` <7094f760c7720331cfa9a3cf6e1f32af-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>
@ 2018-02-27 16:24       ` Sean Paul
  2018-03-12 20:12         ` Sean Paul
  0 siblings, 1 reply; 5+ messages in thread
From: Sean Paul @ 2018-02-27 16:24 UTC (permalink / raw)
  To: abhinavk-sgV2jX0FEOL9JmXXK+q4OQ
  Cc: linux-arm-msm-u79uwXL29TY76Z2rM5mHXA,
	robdclark-Re5JQEeQqe8AvxtiuMwx3w, Sean Paul,
	jsanka-sgV2jX0FEOL9JmXXK+q4OQ,
	freedreno-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW,
	hoegsberg-F7+t8E8rja9g9hUCZPvPmw

On Mon, Feb 26, 2018 at 03:01:14PM -0800, abhinavk@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 the 2 versions. 

There's going to be a fair amount of work to get the sde/dpu version working
with the mainline property, backwards compatible with the hdmi version that
exists, as well as figuring out what key injection is going look like. So let's
not disable the current thing before the next thing is ready :-)

> 
> Was planning this cleanup as part of the HDCP 1x requirements.
> 
> If we want to push this as as separate change, I am okay with it but would
> prefer to wait ...

git revert is cheap, we can always overturn it, no need to wait.

Sean



> 
> Abhinav
> On 2018-02-26 12:48, Sean Paul wrote:
> > We already have CONFIG_DRM_MSM_HDMI_HDCP, with accompanying stubs in
> > hdmi/hdmi.h.
> > 
> > Signed-off-by: Sean Paul <seanpaul@chromium.org>
> > ---
> >  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
> > b/drivers/gpu/drm/msm/hdmi/hdmi_hdcp.c
> > index d24527468284..87e3acb3a259 100644
> > --- a/drivers/gpu/drm/msm/hdmi/hdmi_hdcp.c
> > +++ b/drivers/gpu/drm/msm/hdmi/hdmi_hdcp.c
> > @@ -14,7 +14,6 @@
> >  #include "hdmi.h"
> >  #include <linux/qcom_scm.h>
> > 
> > -#ifdef CONFIG_DRM_MSM_HDCP
> >  #define HDCP_REG_ENABLE 0x01
> >  #define HDCP_REG_DISABLE 0x00
> >  #define HDCP_PORT_ADDR 0x74
> > @@ -1436,26 +1435,3 @@ void msm_hdmi_hdcp_destroy(struct hdmi *hdmi)
> >  		hdmi->hdcp_ctrl = NULL;
> >  	}
> >  }
> > -
> > -#else
> > -struct hdmi_hdcp_ctrl *msm_hdmi_hdcp_init(struct hdmi *hdmi)
> > -{
> > -	return NULL;
> > -}
> > -
> > -void msm_hdmi_hdcp_destroy(struct hdmi *hdmi)
> > -{
> > -}
> > -
> > -void msm_hdmi_hdcp_on(struct hdmi_hdcp_ctrl *hdcp_ctrl)
> > -{
> > -}
> > -
> > -void msm_hdmi_hdcp_off(struct hdmi_hdcp_ctrl *hdcp_ctrl)
> > -{
> > -}
> > -
> > -void msm_hdmi_hdcp_irq(struct hdmi_hdcp_ctrl *hdcp_ctrl)
> > -{
> > -}
> > -#endif

-- 
Sean Paul, Software Engineer, Google / Chromium OS
_______________________________________________
Freedreno mailing list
Freedreno@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/freedreno

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [DPU PATCH] msm/hdcp: Remove redundant stubs/CONFIG
  2018-02-27 16:24       ` Sean Paul
@ 2018-03-12 20:12         ` Sean Paul
  2018-03-13 19:30           ` abhinavk-sgV2jX0FEOL9JmXXK+q4OQ
  0 siblings, 1 reply; 5+ messages in thread
From: Sean Paul @ 2018-03-12 20:12 UTC (permalink / raw)
  To: abhinavk-sgV2jX0FEOL9JmXXK+q4OQ
  Cc: linux-arm-msm-u79uwXL29TY76Z2rM5mHXA,
	robdclark-Re5JQEeQqe8AvxtiuMwx3w, Sean Paul,
	jsanka-sgV2jX0FEOL9JmXXK+q4OQ,
	freedreno-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW,
	hoegsberg-F7+t8E8rja9g9hUCZPvPmw

On Tue, Feb 27, 2018 at 11:24:31AM -0500, Sean Paul wrote:
> On Mon, Feb 26, 2018 at 03:01:14PM -0800, abhinavk@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 dont use this file as we have our equivalent sde_hdcp_1x.c.
> 
> Yes, we definitely need to rationalize the 2 versions. 
> 
> There's going to be a fair amount of work to get the sde/dpu version working
> with the mainline property, backwards compatible with the hdmi version that
> exists, as well as figuring out what key injection is going look like. So let's
> not disable the current thing before the next thing is ready :-)
> 
> > 
> > Was planning this cleanup as part of the HDCP 1x requirements.
> > 
> > If we want to push this as as separate change, I am okay with it but would
> > prefer to wait ...
> 
> git revert is cheap, we can always overturn it, no need to wait.
> 
> Sean
> 
> 
> 
> > 
> > Abhinav
> > On 2018-02-26 12:48, Sean Paul wrote:
> > > We already have CONFIG_DRM_MSM_HDMI_HDCP, with accompanying stubs in
> > > hdmi/hdmi.h.
> > > 
> > > Signed-off-by: Sean Paul <seanpaul@chromium.org>
> > > ---
> > >  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
> > > b/drivers/gpu/drm/msm/hdmi/hdmi_hdcp.c
> > > index d24527468284..87e3acb3a259 100644
> > > --- a/drivers/gpu/drm/msm/hdmi/hdmi_hdcp.c
> > > +++ b/drivers/gpu/drm/msm/hdmi/hdmi_hdcp.c
> > > @@ -14,7 +14,6 @@
> > >  #include "hdmi.h"
> > >  #include <linux/qcom_scm.h>
> > > 
> > > -#ifdef CONFIG_DRM_MSM_HDCP
> > >  #define HDCP_REG_ENABLE 0x01
> > >  #define HDCP_REG_DISABLE 0x00
> > >  #define HDCP_PORT_ADDR 0x74
> > > @@ -1436,26 +1435,3 @@ void msm_hdmi_hdcp_destroy(struct hdmi *hdmi)
> > >  		hdmi->hdcp_ctrl = NULL;
> > >  	}
> > >  }
> > > -
> > > -#else
> > > -struct hdmi_hdcp_ctrl *msm_hdmi_hdcp_init(struct hdmi *hdmi)
> > > -{
> > > -	return NULL;
> > > -}
> > > -
> > > -void msm_hdmi_hdcp_destroy(struct hdmi *hdmi)
> > > -{
> > > -}
> > > -
> > > -void msm_hdmi_hdcp_on(struct hdmi_hdcp_ctrl *hdcp_ctrl)
> > > -{
> > > -}
> > > -
> > > -void msm_hdmi_hdcp_off(struct hdmi_hdcp_ctrl *hdcp_ctrl)
> > > -{
> > > -}
> > > -
> > > -void msm_hdmi_hdcp_irq(struct hdmi_hdcp_ctrl *hdcp_ctrl)
> > > -{
> > > -}
> > > -#endif
> 
> -- 
> Sean Paul, Software Engineer, Google / Chromium OS

-- 
Sean Paul, Software Engineer, Google / Chromium OS
_______________________________________________
Freedreno mailing list
Freedreno@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/freedreno

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [DPU PATCH] msm/hdcp: Remove redundant stubs/CONFIG
  2018-03-12 20:12         ` Sean Paul
@ 2018-03-13 19:30           ` abhinavk-sgV2jX0FEOL9JmXXK+q4OQ
  0 siblings, 0 replies; 5+ messages in thread
From: abhinavk-sgV2jX0FEOL9JmXXK+q4OQ @ 2018-03-13 19:30 UTC (permalink / raw)
  To: Sean Paul
  Cc: linux-arm-msm-u79uwXL29TY76Z2rM5mHXA,
	linux-arm-msm-owner-u79uwXL29TY76Z2rM5mHXA,
	robdclark-Re5JQEeQqe8AvxtiuMwx3w, jsanka-sgV2jX0FEOL9JmXXK+q4OQ,
	freedreno-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW,
	hoegsberg-F7+t8E8rja9g9hUCZPvPmw

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, abhinavk@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 dont use this file as we have our equivalent sde_hdcp_1x.c.
>> 
>> Yes, we definitely need to rationalize the 2 versions.
>> 
>> There's going to be a fair amount of work to get the sde/dpu version 
>> working
>> with the mainline property, backwards compatible with the hdmi version 
>> that
>> exists, as well as figuring out what key injection is going look like. 
>> So let's
>> not disable the current thing before the next thing is ready :-)
>> 
>> >
>> > Was planning this cleanup as part of the HDCP 1x requirements.
>> >
>> > If we want to push this as as separate change, I am okay with it but would
>> > prefer to wait ...
>> 
>> git revert is cheap, we can always overturn it, no need to wait.
>> 
>> Sean
>> 
>> 
>> 
>> >
>> > Abhinav
>> > On 2018-02-26 12:48, Sean Paul wrote:
>> > > We already have CONFIG_DRM_MSM_HDMI_HDCP, with accompanying stubs in
>> > > hdmi/hdmi.h.
>> > >
>> > > Signed-off-by: Sean Paul <seanpaul@chromium.org>
Reviewed-by: Abhinav Kumar <abhinavk@codeaurora.org>
>> > > ---
>> > >  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
>> > > b/drivers/gpu/drm/msm/hdmi/hdmi_hdcp.c
>> > > index d24527468284..87e3acb3a259 100644
>> > > --- a/drivers/gpu/drm/msm/hdmi/hdmi_hdcp.c
>> > > +++ b/drivers/gpu/drm/msm/hdmi/hdmi_hdcp.c
>> > > @@ -14,7 +14,6 @@
>> > >  #include "hdmi.h"
>> > >  #include <linux/qcom_scm.h>
>> > >
>> > > -#ifdef CONFIG_DRM_MSM_HDCP
>> > >  #define HDCP_REG_ENABLE 0x01
>> > >  #define HDCP_REG_DISABLE 0x00
>> > >  #define HDCP_PORT_ADDR 0x74
>> > > @@ -1436,26 +1435,3 @@ void msm_hdmi_hdcp_destroy(struct hdmi *hdmi)
>> > >  		hdmi->hdcp_ctrl = NULL;
>> > >  	}
>> > >  }
>> > > -
>> > > -#else
>> > > -struct hdmi_hdcp_ctrl *msm_hdmi_hdcp_init(struct hdmi *hdmi)
>> > > -{
>> > > -	return NULL;
>> > > -}
>> > > -
>> > > -void msm_hdmi_hdcp_destroy(struct hdmi *hdmi)
>> > > -{
>> > > -}
>> > > -
>> > > -void msm_hdmi_hdcp_on(struct hdmi_hdcp_ctrl *hdcp_ctrl)
>> > > -{
>> > > -}
>> > > -
>> > > -void msm_hdmi_hdcp_off(struct hdmi_hdcp_ctrl *hdcp_ctrl)
>> > > -{
>> > > -}
>> > > -
>> > > -void msm_hdmi_hdcp_irq(struct hdmi_hdcp_ctrl *hdcp_ctrl)
>> > > -{
>> > > -}
>> > > -#endif
>> 
>> --
>> Sean Paul, Software Engineer, Google / Chromium OS
_______________________________________________
Freedreno mailing list
Freedreno@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/freedreno

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2018-03-13 19:30 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-02-26 20:48 [DPU PATCH] msm/hdcp: Remove redundant stubs/CONFIG Sean Paul
     [not found] ` <20180226204821.149179-1-seanpaul-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>
2018-02-26 23:01   ` abhinavk-sgV2jX0FEOL9JmXXK+q4OQ
     [not found]     ` <7094f760c7720331cfa9a3cf6e1f32af-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>
2018-02-27 16:24       ` Sean Paul
2018-03-12 20:12         ` Sean Paul
2018-03-13 19:30           ` abhinavk-sgV2jX0FEOL9JmXXK+q4OQ

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.