Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965675AbcDLTbO (ORCPT ); Tue, 12 Apr 2016 15:31:14 -0400 Received: from mail-wm0-f67.google.com ([74.125.82.67]:34172 "EHLO mail-wm0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933789AbcDLTbM (ORCPT ); Tue, 12 Apr 2016 15:31:12 -0400 Date: Tue, 12 Apr 2016 21:31:28 +0200 From: Daniel Vetter To: Liviu Dudau Cc: Dave Airlie , Daniel Stone , David Brown , Brian Starkey , devicetree@vger.kernel.org, LKML , DRI devel Subject: Re: [RFC][PATCH 2/2] drm/arm: Add support for Mali Display Processors Message-ID: <20160412193128.GO2510@phenom.ffwll.local> Mail-Followup-To: Liviu Dudau , Dave Airlie , Daniel Stone , David Brown , Brian Starkey , devicetree@vger.kernel.org, LKML , DRI devel References: <1459527712-9488-1-git-send-email-Liviu.Dudau@arm.com> <1459527712-9488-3-git-send-email-Liviu.Dudau@arm.com> <20160412155810.GH2510@phenom.ffwll.local> <20160412171654.GL16063@e106497-lin.cambridge.arm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20160412171654.GL16063@e106497-lin.cambridge.arm.com> X-Operating-System: Linux phenom 4.4.0-1-amd64 User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1366 Lines: 31 On Tue, Apr 12, 2016 at 06:16:54PM +0100, Liviu Dudau wrote: > On Tue, Apr 12, 2016 at 05:58:11PM +0200, Daniel Vetter wrote: > > On Fri, Apr 01, 2016 at 05:21:52PM +0100, Liviu Dudau wrote: > > > Add support for the new family of Display Processors from ARM Ltd. > > > This commit adds basic support for Mali DP500, DP550 and DP650 > > > parts, with only the display engine being supported at the moment. > > > > > > Cc: David Brown > > > Cc: Brian Starkey > > > > > > Signed-off-by: Liviu Dudau > > > > Ok, on 2nd look something puzzling: Where are the > > drm_encoder/drm_connectors in this driver? Somehow I think just these > > parts here won't light up a lot ... > > The magic of component based drivers, heh :) > > On my test system I'm using the NXP TDA19988 HDMI and the driver works > out of box once the DT describes the proper port/endpoint dance. For > models/qemu style environment I have a generic DRM encoder driver that I'm > going to post for review that only reads display timings from DT and behaves > as if it connected to a real monitor. I didn't know that magic worked so well already. Seriously impressed, and happy that it doesn't take anything in addition to make it all just work. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch