From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.3 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED,USER_AGENT_MUTT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id F001DC43613 for ; Mon, 24 Jun 2019 11:19:08 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 85AFF212F5 for ; Mon, 24 Jun 2019 11:19:08 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=ideasonboard.com header.i=@ideasonboard.com header.b="UZF7L0Ot" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729183AbfFXLTH (ORCPT ); Mon, 24 Jun 2019 07:19:07 -0400 Received: from perceval.ideasonboard.com ([213.167.242.64]:60808 "EHLO perceval.ideasonboard.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727953AbfFXLTH (ORCPT ); Mon, 24 Jun 2019 07:19:07 -0400 Received: from pendragon.ideasonboard.com (81-175-216-236.bb.dnainternet.fi [81.175.216.236]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id BFAF6323; Mon, 24 Jun 2019 13:19:04 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1561375144; bh=B6Kyd/rtGoZ/rniG3OdECkB1xfXOxJ86TGPqPhSRnd0=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=UZF7L0OtkFEm3c6C+yr3wAtoXGqgyjsN+/jO8N8+hVRxpmtdm0UeHhiq9u/KmNJ53 tf5FCstcqBXEsD1X9/2hGyciTHm324pnEmUb6nl1k81izWnQWdxWrOro6z+BInLdNX RsH6bRzDPzqY1b5vj7Nmpm6VW2dI+bpQdY6UkrM4= Date: Mon, 24 Jun 2019 14:16:36 +0300 From: Laurent Pinchart To: Neil Armstrong Cc: Jonas Karlman , "a.hajda@samsung.com" , "jernej.skrabec@siol.net" , "maxime.ripard@bootlin.com" , Daniel Vetter , "khilman@baylibre.com" , "dri-devel@lists.freedesktop.org" , "linux-kernel@vger.kernel.org" , "wens@csie.org" , "zhengyang@rock-chips.com" Subject: Re: [PATCH 0/4] drm/bridge: dw-hdmi: Add support for HDR metadata Message-ID: <20190624111636.GB5737@pendragon.ideasonboard.com> References: <085dc3be-20e5-b2fe-4c02-bf4a4d1473da@baylibre.com> <20190621090125.GX12905@phenom.ffwll.local> <20190623233017.GI6124@pendragon.ideasonboard.com> <58243243-fbd8-e67b-a050-baa9757be43e@baylibre.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <58243243-fbd8-e67b-a050-baa9757be43e@baylibre.com> User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Neil, On Mon, Jun 24, 2019 at 10:19:34AM +0200, Neil Armstrong wrote: > Hi Daniel, Laurent, Andrzej, > > On 24/06/2019 01:30, Laurent Pinchart wrote: > > On Fri, Jun 21, 2019 at 11:01:25AM +0200, Daniel Vetter wrote: > >> On Thu, Jun 20, 2019 at 04:40:12PM +0200, Neil Armstrong wrote: > >>> Hi Andrzej, > >>> > >>> Gentle ping, could you review the dw-hdmi changes here ? > >> > >> btw not sure you absolutely need review from Andrzej, we're currently a > >> bit undersupplied with bridge reviewers I think ... Better to ramp up > >> more. > > > > I try to review DRM bridge patches when possible, but dw-hdmi is a > > special case. I was told by the supplier of an SoC datasheet that > > contains the HDMI encoder IP core documentation that Synopsys required > > them to route all contributions made based on that documentation through > > Synopsys' internal legal review before publishing them. I thus decided > > to not contribute to the driver anymore, at least for areas that require > > access to documentation. > > I'd like to propose myself as co-maintainer of the DRM bridge subsystem if > everybody agrees, following the excellent work Laurent and Andrzej did. > I have a very little knowledge of DSI, & other bridge drivers, but I'll do > my best. > > For the dw-hdmi driver, we have a big roadmap including : > - HDR (this patchset) > - HDCP 1/2 > - YUV420, YUV422, YUV44, 10bit/12bit/16bit HDMI output > - Enhanced audio support and ELD notification to ASoC > ... You're more than welcome as a DRM bridge maintainer, especially given that you have just volunteered to implement bridge states and format negotiation, right ? ;-) > Having a more active maintainer/reviewer team would be needed, at least for > the dw-hdmi bridge. > > I'll also propose Jonas Karlman as reviewer since he is very > active for the multimedia support on RockChip, Allwinner and Amlogic SoCs. > I'll also propose Jernej Skrabec@siol.net , if he wants, > as reviewer since he is very active on the Allwinner SoCs side. > > >>> On 26/05/2019 23:18, Jonas Karlman wrote: > >>>> Add support for HDR metadata using the hdr_output_metadata connector property, > >>>> configure Dynamic Range and Mastering InfoFrame accordingly. > >>>> > >>>> A drm_infoframe flag is added to dw_hdmi_plat_data that platform drivers > >>>> can use to signal when Dynamic Range and Mastering infoframes is supported. > >>>> This flag is needed because Amlogic GXBB and GXL report same DW-HDMI version, > >>>> and only GXL support DRM InfoFrame. > >>>> > >>>> The first patch add functionality to configure DRM InfoFrame based on the > >>>> hdr_output_metadata connector property. > >>>> > >>>> The remaining patches sets the drm_infoframe flag on some SoCs supporting > >>>> Dynamic Range and Mastering InfoFrame. > >>>> > >>>> Note that this was based on top of drm-misc-next and Neil Armstrong's > >>>> "drm/meson: Add support for HDMI2.0 YUV420 4k60" series at [1] > >>>> > >>>> [1] https://patchwork.freedesktop.org/series/58725/#rev2 > >>>> > >>>> Jonas Karlman (4): > >>>> drm/bridge: dw-hdmi: Add Dynamic Range and Mastering InfoFrame support > >>>> drm/rockchip: Enable DRM InfoFrame support on RK3328 and RK3399 > >>>> drm/meson: Enable DRM InfoFrame support on GXL, GXM and G12A > >>>> drm/sun4i: Enable DRM InfoFrame support on H6 > >>>> > >>>> drivers/gpu/drm/bridge/synopsys/dw-hdmi.c | 109 ++++++++++++++++++++ > >>>> drivers/gpu/drm/bridge/synopsys/dw-hdmi.h | 37 +++++++ > >>>> drivers/gpu/drm/meson/meson_dw_hdmi.c | 5 + > >>>> drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c | 2 + > >>>> drivers/gpu/drm/sun4i/sun8i_dw_hdmi.c | 2 + > >>>> drivers/gpu/drm/sun4i/sun8i_dw_hdmi.h | 1 + > >>>> include/drm/bridge/dw_hdmi.h | 1 + > >>>> 7 files changed, 157 insertions(+) -- Regards, Laurent Pinchart