[Intel-gfx] [PATCH v4] drm: Fix downstream dev count read
Sean Paul
seanpaul at chromium.org
Wed Apr 4 19:23:57 UTC 2018
On Wed, Apr 04, 2018 at 12:07:41PM -0700, Rodrigo Vivi wrote:
> On Wed, Apr 04, 2018 at 11:57:42PM +0530, Ramalingam C wrote:
> > In both HDMI and DP, device count is represented by 6:0 bits of a
> > register(BInfo/Bstatus)
> >
> > So macro for bitmasking the device_count is fixed(0x3F->0x7F).
> >
> > v3:
> > Retained the Rb-ed.
> > v4:
> > %s/drm\/i915/drm [rodrigo]
> >
>
> Shouldn't this patch have a "Fixes:" ?
Yes, I think that'd be good.
> cc: stable?
It couldn't hurt.
>
> I pushed first 3 patches on the series to dinq.
> I believe this one here could be there with Dave's ack or
> maybe on drm-misc-fixes?
Meh. The severity of this isn't too big, given that I doubt people care _too_
much about plugging in more than 64 HDCP-enabled devices. If you want to drop it
in -misc-next-fixes, I can send it out next week.
While we're asking for a respin, could we add HDCP somewhere in the subject?
Sean
>
> > Signed-off-by: Ramalingam C <ramalingam.c at intel.com>
> > cc: Sean Paul <seanpaul at chromium.org>
> > Reviewed-by: Sean Paul <seanpaul at chromium.org>
> > ---
> > include/drm/drm_hdcp.h | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/include/drm/drm_hdcp.h b/include/drm/drm_hdcp.h
> > index 562fa7df2637..98e63d870139 100644
> > --- a/include/drm/drm_hdcp.h
> > +++ b/include/drm/drm_hdcp.h
> > @@ -19,7 +19,7 @@
> > #define DRM_HDCP_RI_LEN 2
> > #define DRM_HDCP_V_PRIME_PART_LEN 4
> > #define DRM_HDCP_V_PRIME_NUM_PARTS 5
> > -#define DRM_HDCP_NUM_DOWNSTREAM(x) (x & 0x3f)
> > +#define DRM_HDCP_NUM_DOWNSTREAM(x) (x & 0x7f)
> > #define DRM_HDCP_MAX_CASCADE_EXCEEDED(x) (x & BIT(3))
> > #define DRM_HDCP_MAX_DEVICE_EXCEEDED(x) (x & BIT(7))
> >
> > --
> > 2.7.4
> >
--
Sean Paul, Software Engineer, Google / Chromium OS
More information about the Intel-gfx
mailing list