From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefan Wahren Subject: Re: [PATCH 0/7] drm/vc4: Allow for more boot-time configuration Date: Mon, 4 Mar 2019 16:44:30 +0100 Message-ID: <61aa0ea1-9e2e-f512-a4f5-8d98e1752442@i2se.com> References: Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Content-Language: de-DE List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Maxime Ripard , Maarten Lankhorst , Sean Paul , Daniel Vetter , David Airlie Cc: eben@raspberrypi.org, dri-devel@lists.freedesktop.org, Paul Kocialkowski , Eric Anholt , Thomas Petazzoni , linux-arm-kernel@lists.infradead.org List-Id: dri-devel@lists.freedesktop.org Hi Maxime, Am 04.03.2019 um 15:52 schrieb Maxime Ripard: > Hi, > > The proprietary stack for the RaspberryPi allows for a number of video > parameters widely used by their users, but yet don't have any equivalents > in the mainline kernel. > > Those options are detailed here: > https://www.raspberrypi.org/documentation/configuration/config-txt/video.md > > While not all of them are desirable to have in the mainline kernel, some of > them still have value, such as properties to initialise the overscan or > rotation parameters, or the one to deal with broken displays. > > This series is an attempt to support those, and is based on a rewrite of > the command line parser I did a couple of years ago that never reached > upstream (due to a lack of time on my side). While this parser was > initially made to deal with named modes (in order to support TV modes), it > also allowed to extend it more easily, which is why it's resurrected. > > Since a change of the command line parser can pretty easily get things > wrong and introduce regressions, I also worked with a number of unit tests > that you can find here: http://code.bulix.org/tpo7dg-607264?raw > > Eventually, I guess those tests should be part of the kernel somewhere, but > I haven't found a suitable place for them to be included yet. > > Let me know what you think, > Maxime > thanks for this series. In case you want some feedback from the Foundation, please add Phil Elwell instead of Eben to CC. Stefan