From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from lelnx193.ext.ti.com ([198.47.27.77]:27763 "EHLO lelnx193.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755563AbeDWOLo (ORCPT ); Mon, 23 Apr 2018 10:11:44 -0400 Subject: Re: [PATCH 5/7] omapfb: omapfb_dss.h: add stubs to build with COMPILE_TEST && DRM_OMAP To: Bartlomiej Zolnierkiewicz , Mauro Carvalho Chehab CC: Linux Media Mailing List , Mauro Carvalho Chehab , , , Laurent Pinchart References: <2542100.cElVns0SR0@amdc3058> <5379683.QunLsIS18Z@amdc3058> From: Tomi Valkeinen Message-ID: Date: Mon, 23 Apr 2018 17:11:14 +0300 MIME-Version: 1.0 In-Reply-To: <5379683.QunLsIS18Z@amdc3058> Content-Type: text/plain; charset="utf-8" Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-media-owner@vger.kernel.org List-ID: On 23/04/18 16:56, Bartlomiej Zolnierkiewicz wrote: > Ideally we should be able to build both drivers in the same kernel > (especially as modules). > > I was hoping that it could be fixed easily but then I discovered > the root source of the problem: > > drivers/gpu/drm/omapdrm/dss/display.o: In function `omapdss_unregister_display': > display.c:(.text+0x2c): multiple definition of `omapdss_unregister_display' > drivers/video/fbdev/omap2/omapfb/dss/display.o:display.c:(.text+0x198): first defined here The main problem is that omapdrm and omapfb are two different drivers for the same HW. You need to pick one, even if we would change those functions and fix the link issue. At some point in time we could compile both as modules (but not built-in), but the only use for that was development/testing and in the end made our life more difficult. So, now you must fully disable one of them to enable the other. And, actually, we even have boot-time code, not included in the module itself, which gets enabled when omapdrm is enabled. While it's of course good to support COMPILE_TEST, if using COMPILE_TEST with omapfb is problematic, I'm not sure if it's worth to spend time on that. We should be moving away from omapfb to omapdrm. Tomi -- Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki