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=-6.0 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_PASS,URIBL_BLOCKED,USER_AGENT_NEOMUTT 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 24BCAC282CE for ; Tue, 9 Apr 2019 07:45:37 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id E89C220857 for ; Tue, 9 Apr 2019 07:45:36 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="g1rdH0AE" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org E89C220857 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=bootlin.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender:Content-Type: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject:To:From:Date:Reply-To: Cc:Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=2kbdnFG9vZzZL1RGRjaTIlVT2c7/pNS9t9RYOgBpTB0=; b=g1rdH0AEBEmn5Fk5hWdErrziO IbmfRDLWWdGly5NRzSN2bDBkrNS2jxOhXNJRLZPXw5C1ZN2hSg1pux8WTD6qhaOveOK8hp8hbV+ux 5s+xpHwRUZoRQv4MQF3cSOgTC1fsg58j7ghW5hf8DVlHWcJ8Gv0VDsznA+swwWT2RPBE2E9EnRlNi jsVOumfbkDKfYF4r2sGDX2RYCx/IekfDJ66kC6i5rGJlyZfiN/76RgXQuQpYvGWjOLi5bdYGDeD7K gZtUtlGf7Nlzxs8Sk7S3s+08cJ2he3iSgxJ4xkF2I0sts9hSoxrYGiIZsH4EbC7SQ79y8cTHA50Kw Abk995RHQ==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1hDlRi-0005qT-Dx; Tue, 09 Apr 2019 07:45:30 +0000 Received: from relay7-d.mail.gandi.net ([217.70.183.200]) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1hDlRe-0005q0-5H for linux-arm-kernel@lists.infradead.org; Tue, 09 Apr 2019 07:45:28 +0000 X-Originating-IP: 90.88.30.125 Received: from localhost (aaubervilliers-681-1-89-125.w90-88.abo.wanadoo.fr [90.88.30.125]) (Authenticated sender: maxime.ripard@bootlin.com) by relay7-d.mail.gandi.net (Postfix) with ESMTPSA id 300C02000A; Tue, 9 Apr 2019 07:45:18 +0000 (UTC) Date: Tue, 9 Apr 2019 09:45:17 +0200 From: Maxime Ripard To: Chen-Yu Tsai , linux-sunxi , Rob Herring , Linus Walleij , David Airlie , Daniel Vetter , Mark Rutland , Giuseppe Cavallaro , Alexandre Torgue , Jose Abreu , "David S. Miller" , Maxime Coquelin , Arend van Spriel , Franky Lin , Hante Meuleman , Chi-Hsien Lin , Wright Feng , Kalle Valo , Naveen Gupta , dri-devel , devicetree , linux-arm-kernel , linux-kernel , netdev , linux-stm32@st-md-mailman.stormreply.com, linux-wireless , brcm80211-dev-list.pdl@broadcom.com, brcm80211-dev-list@cypress.com, "open list:GPIO SUBSYSTEM" Subject: Re: [linux-sunxi] Re: [PATCH 02/12] drm: sun4i: Add support for enabling DDC I2C bus to dw_hdmi glue Message-ID: <20190409074517.lu4gsmyq2qcunqg4@flea> References: <20190405234514.6183-1-megous@megous.com> <20190405234514.6183-3-megous@megous.com> <20190408072338.3urcotemju3qi2un@flea> <20190408084714.b6xdi2pxbjvzwsnu@flea> <20190408121727.4vdqdokr6w5thck6@core.my.home> MIME-Version: 1.0 In-Reply-To: <20190408121727.4vdqdokr6w5thck6@core.my.home> User-Agent: NeoMutt/20180716 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190409_004526_499749_D9F410D0 X-CRM114-Status: GOOD ( 18.64 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: multipart/mixed; boundary="===============0669109469930598407==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============0669109469930598407== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="ex55duqs5l5l627l" Content-Disposition: inline --ex55duqs5l5l627l Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Mon, Apr 08, 2019 at 02:17:27PM +0200, Ond=C5=99ej Jirman wrote: > On Mon, Apr 08, 2019 at 10:47:14AM +0200, Maxime Ripard wrote: > > On Mon, Apr 08, 2019 at 03:28:24PM +0800, Chen-Yu Tsai wrote: > > > On Mon, Apr 8, 2019 at 3:23 PM Maxime Ripard wrote: > > > > > > > > On Sat, Apr 06, 2019 at 01:45:04AM +0200, megous@megous.com wrote: > > > > > From: Ondrej Jirman > > > > > > > > > > Orange Pi 3 board requires enabling DDC I2C bus via some GPIO con= nected > > > > > transistors, before it can be used. Model this as a power supply = for DDC > > > > > (via regulator framework). > > > > > > > > > > Signed-off-by: Ondrej Jirman > > > > > > > > The DDC bus itself is usually attached to the HDMI connector, so it > > > > would make sense to make the supply also a property of the connecto= r. > > > > > > I believe these are separate things. What this patch covers is power = for > > > a voltage shifter between the SoC and HDMI DDC pins. The HDMI connect= or's > > > 5V supply to power the remote DDC chip is something else. And on the > > > Orange Pi 3 they are indeed separate supplies. > > > > Then maybe the endpoint link between the two would be the best place > > to put this? > > Interestingly &hdmi node configures the DDC bus pins via pinctrl on the S= oC > side, so I put this there too, because it's related to those pins. I'm no= t sure > if that changes anything in the discussion. It's kind of different though. The DDC controller is within the HDMI controller, which is inside the SoC. Just like the pin muxer. As far as the hardware goes, even on your board, you don't need that supply so that the signal gets out of the SoC. If the regulator is to power up some component on the path between the SoC and the connector, then it should be attached there. Maxime -- Maxime Ripard, Bootlin Embedded Linux and Kernel engineering https://bootlin.com --ex55duqs5l5l627l Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRcEzekXsqa64kGDp7j7w1vZxhRxQUCXKxNjQAKCRDj7w1vZxhR xd1YAQCbZIs2Pp7jd1cm9rrokDi5iMPRnQBKcSOdLIRXgkvLTAEAjNSL9iiFRvXv J1tkS7/bj/Wsm4X0Ihv29bnFFGddWwQ= =LLjI -----END PGP SIGNATURE----- --ex55duqs5l5l627l-- --===============0669109469930598407== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============0669109469930598407==--