Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752104AbdLAS1p (ORCPT ); Fri, 1 Dec 2017 13:27:45 -0500 Received: from smtp.domeneshop.no ([194.63.252.55]:49616 "EHLO smtp.domeneshop.no" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750872AbdLAS1n (ORCPT ); Fri, 1 Dec 2017 13:27:43 -0500 Subject: Re: [PATCH v1 2/2] drm/tinydrm: add driver for ILI9225 panels To: Linus Walleij , David Lechner , Stefano Babic Cc: Mark Rutland , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , Rob Herring , "linux-kernel@vger.kernel.org" , "open list:DRM PANEL DRIVERS" , Thierry Reding References: <1510113136-6788-1-git-send-email-david@lechnology.com> <1510113136-6788-3-git-send-email-david@lechnology.com> From: =?UTF-8?Q?Noralf_Tr=c3=b8nnes?= Message-ID: <81afd9e7-4c84-697a-0963-834494ca77a5@tronnes.org> Date: Fri, 1 Dec 2017 19:27:37 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.5.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3131 Lines: 77 (cc: Thierry) Den 01.12.2017 15.03, skrev Linus Walleij: > On Wed, Nov 8, 2017 at 4:52 AM, David Lechner wrote: > >> This adds a new driver for display panels based on the Ilitek ILI9225 >> controller. >> >> This was developed for a no-name panel with a red PCB that is commonly >> marketed for Arduino. See . >> >> I really did try very hard to find a make and model for this panel, but >> there doesn't seem to be one, so the best I can do is offer the picture >> in the link above for identification. >> >> Signed-off-by: David Lechner > Can you explain why tinydrm is not putting its panel drivers in > drivers/gpu/drm/panel? The short answer is that tinydrm sends pixel data over the panel controller's control interface whereas drm/panel leaves that to a dedicated pixel interface fed by another driver. tinydrm is used with controllers that have onboard GRAM which is scanned out by the panel controller itself. Many of these controllers support multiple interfaces, like MIPI DSI/DPI/DBI. A MIPI DPI compatible controller has a control bus, usually SPI, to operate the panel. This same control bus can be used in MIPI DBI mode on some controllers to push pixels. The MIPI standard documents isn't open, but some are available if you do a search. We also have MIPI DCS which is the command set shared by the MIPI compatible controllers. So how do we deal with controllers that can operate in many modes? I raised the question when a panel driver was reviewed earlier this year, but nothing really came out of it. I suppose that displays that are used with DSI/DPI doesn't have a controller with onboard GRAM, since that would just increase the price. The MIPI standard defines different controller types which has no, partial or full framebuffer. So in reality it's not that likely that we will see the same controller used both in tinydrm and drm/panel. But I'm hardly an expert on these matters, I've only used the DBI mode. I've cc'ed the drm/panel maintainer, maybe he can shed some more light. Noralf. > I guess everybody knows except me, it's usually like that :( > > I am anyways working on a driver for Ilitek 9322 that I want > to land in drivers/gpu/drm/panel. Here is the last iteration: > https://lists.freedesktop.org/archives/dri-devel/2017-August/150205.html > Yeah I got sidetracked. OK I will get to it now. > > There are some similarities with the code I'm seeing here > but I believe they are essentially different. But it will be hard > to share code if you put the driver in the tinydrm framework. > > I guess you have also seen: > drivers/video/backlight/ili922x.c > ? > > Stefano Babic who wrote the backlight driver is available for > reviewing, so includ him in follow-ups (added to To: line). > > I'm putting you on CC as I'm rewriting it a bit after the DT > maintainers review, will try to repost ASAP. > > Yours, > Linus Walleij > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel >