From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH v5 00/11] Add DRM for stih4xx platforms Date: Wed, 30 Jul 2014 14:55:23 +0200 Message-ID: <20140730125523.GP4747@phenom.ffwll.local> References: Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-wg0-f50.google.com (mail-wg0-f50.google.com [74.125.82.50]) by gabe.freedesktop.org (Postfix) with ESMTP id 60B746E601 for ; Wed, 30 Jul 2014 05:55:15 -0700 (PDT) Received: by mail-wg0-f50.google.com with SMTP id n12so1171641wgh.9 for ; Wed, 30 Jul 2014 05:55:14 -0700 (PDT) Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Benjamin Gaignard Cc: Russell King - ARM Linux , Linux Kernel Mailing List , "dri-devel@lists.freedesktop.org" , "linaro-mm-sig@lists.linaro.org" , Laurent Pinchart List-Id: dri-devel@lists.freedesktop.org On Wed, Jul 30, 2014 at 12:43:33PM +0200, Benjamin Gaignard wrote: > I have push two additional patches on drm_kms_for_next-v7 branch to > make the driver build as a module: > - "gpu: drm: export drm_sysfs_connector_{add,remove} function" Nope, you're using the wrong functions. Instead you should use drm_connector_register/unregister. Please don't just export randome functions without digging into the history a bit. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch