From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758373AbdEXPqJ (ORCPT ); Wed, 24 May 2017 11:46:09 -0400 Received: from gloria.sntech.de ([95.129.55.99]:54318 "EHLO gloria.sntech.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751170AbdEXPqF (ORCPT ); Wed, 24 May 2017 11:46:05 -0400 From: Heiko Stuebner To: Randy Li Cc: devicetree@vger.kernel.org, robh+dt@kernel.org, mark.rutland@arm.com, linux@armlinux.org.uk, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2] ARM: dts: rockchip: add eDP panel support for Firefly Date: Wed, 24 May 2017 17:45:51 +0200 Message-ID: <1982494.aeoIls14E7@phil> User-Agent: KMail/5.2.3 (Linux/4.9.0-2-amd64; KDE/5.28.0; x86_64; ; ) In-Reply-To: <20170508155833.19652-1-ayaka@soulik.info> References: <20170508155833.19652-1-ayaka@soulik.info> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Randy, Am Montag, 8. Mai 2017, 23:58:33 CEST schrieb Randy Li: > This patch adds the supporting to the eDP panel sold by > the T-CHIP for the Firefly RK3288. I assign the VOP lite > for the eDP panel and VOP big to HDMI, as the HDMI supports > 4K resolution. With a different VOP device, eDP panel > and HDMI could display a different contents. > > The InvenSense MPU6050 sensor at the botton of the panel > is also enabled. > > The Firefly RK3288 Reload use a different GPIO pin to enable > the power of the eDP panel. > > Signed-off-by: Randy Li Sorry, I'm not yet sure, what to make of this patch. On the firefly the edp pins are actual part of the general pin header and while the edp pins themself do not have multiple functions, for example the pin you claim as interrupt for the mpu6050 (gpio5_b4) also is part of the spi0 set of pins and that pin also seems pretty randomly selected. Same with the enable-gpio of the display. Also claiming that every firefly board has this display connected also feels strange and the whole display thing would make more sense as a devicetree overlay [0], though I don't know how these should get handled in-kernel - if at all. And finally, please don't try to use the devicetree to configure outputs (forcing edp to one and hdmi to the other). The devicetree is supposed to describe the hardware and of course both vops can drive the edp and if necessary the drm driver should handle differences and decide on the vop to use. Heiko [0] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/devicetree/overlay-notes.txt