Received: by 2002:a25:1985:0:0:0:0:0 with SMTP id 127csp710183ybz; Wed, 15 Apr 2020 17:14:23 -0700 (PDT) X-Google-Smtp-Source: APiQypLuB/Sb0gTh3d2tc4vy0UmbNBQqKQyU+cI37+i32avp9CQryC7DPwJ5TpFxU20uNVe2J46N X-Received: by 2002:a50:c054:: with SMTP id u20mr23528561edd.302.1586996063531; Wed, 15 Apr 2020 17:14:23 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1586996063; cv=none; d=google.com; s=arc-20160816; b=kyqY6oXc1tMhR0LQuv6w45fhQ0GLhty3HKrqfhJGOYuNEMPmsNdEaqNSo8krl8mYe/ 2U27ZJVJPjOMuo2WgNqiqcPUyg0E08+iZuhn/jCqcGMOpMCfHUeoFPz/6s8/nw6I1+pN 0iU1bLLxn+ZJ21WoagmO7WKrAJNbirAprO/jNzdAWLKQU3x/YeLk5oyiAGVvn4OIs8T2 8tKFmPsjexsbQ8sU1kxXTsTyuiOXgPVx5dVTzfO0u2gjh7LLsU8XOUzcUt9OvsYLyfZ3 Cwb3gVJAr3RRSQ7oLOb2E69n72v6mJWjhMw6ZurY74bPEr9wwkodxLRPljJODc29dNal 2CfQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version; bh=iD0wQUJMdwElrMJqbg1Kvg72fur5xT7g1q6QIK06fgA=; b=Dvwjy2uZ0qgA8Lfik1SUFxK03Nu96xLZMaD+PQklEUYxJFQxUcvwksVfMUPlEJJVle JeHUH7BnR3fPuYsYDA+QrMN4JtwwNRogP6ObJjvt0HLqFaYT9336fDSCMeuzt7i7o+Yh idzxQkrTkrR2UHbRZGUQ0LjPxiHIPI/Oy/YFyzMRtiQS3SQiaTQiMJxepsBeSfM78jbo CoGzC9h0Q8ywNBGsypFx/0Kd3dVn3/PIkJX9dHscqcpJKn2MzeIxnvWCs1kqhPC/e/wS NZ7WhZ65GKwC3al67JW/K0dvPohKSeikEDh37OVMwDmjBlozm4cPnIgWz+2/0rt1CJ+s nFgw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id w16si2187542edq.89.2020.04.15.17.14.00; Wed, 15 Apr 2020 17:14:23 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1414665AbgDOPSo (ORCPT + 99 others); Wed, 15 Apr 2020 11:18:44 -0400 Received: from mout.kundenserver.de ([212.227.17.13]:56625 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1414655AbgDOPSl (ORCPT ); Wed, 15 Apr 2020 11:18:41 -0400 Received: from mail-qk1-f181.google.com ([209.85.222.181]) by mrelayeu.kundenserver.de (mreue107 [212.227.15.145]) with ESMTPSA (Nemesis) id 1Mi23L-1il1DD0uZt-00e37A; Wed, 15 Apr 2020 17:18:38 +0200 Received: by mail-qk1-f181.google.com with SMTP id c63so17606965qke.2; Wed, 15 Apr 2020 08:18:37 -0700 (PDT) X-Gm-Message-State: AGi0PuYQ7iQhk/BikuN6BKO+BL3lPwjdOlvL37MtOvCFqqOG4wui5OGn 0cIjB+fYfemUFnWfZ5APRYOtoJstcXL1jjQMy9o= X-Received: by 2002:a37:ba47:: with SMTP id k68mr15682834qkf.394.1586963916750; Wed, 15 Apr 2020 08:18:36 -0700 (PDT) MIME-Version: 1.0 References: <20200408202711.1198966-1-arnd@arndb.de> <20200408202711.1198966-6-arnd@arndb.de> <20200414201739.GJ19819@pendragon.ideasonboard.com> <20200414205158.GM19819@pendragon.ideasonboard.com> In-Reply-To: From: Arnd Bergmann Date: Wed, 15 Apr 2020 17:18:20 +0200 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [RFC 5/6] drm/rcar-du: fix selection of CMM driver To: Geert Uytterhoeven Cc: Laurent Pinchart , "linux-kernel@vger.kernel.org" , Masahiro Yamada , Nicolas Pitre , Andrzej Hajda , Neil Armstrong , Jonas Karlman , Jernej Skrabec , David Airlie , Daniel Vetter , Kieran Bingham , "David S. Miller" , Saeed Mahameed , Leon Romanovsky , dri-devel , Linux-Renesas , Networking , linux-rdma Content-Type: text/plain; charset="UTF-8" X-Provags-ID: V03:K1:LcT7bwGznqc8gtfgF2bfiglIsjD0HReWu/nHchEwDxrf+9vm5BH rmdjTk0uOUNkmKoP4TP/efPp6Sd4lGmapqbt1l63xHODdE9k8x8ilzOIS4zVIE7rJAwPgP9 nv14ZKTD64pSJNEWvMGnfnTFEl7qLpGkCgk1VSBIzOAG5FvyjqcFClevQUtWagaxdqe70Vi puR68X7pu8N6lETJMiAIw== X-Spam-Flag: NO X-UI-Out-Filterresults: notjunk:1;V03:K0:rJ5/p8u5JGQ=:gVqEX27W8WlhhlVVuppYhw JyqF3pScGR9U4BjvZeVFBDID+rUE2rQ0KuPuwkLy+zaw8dkwKT5TBF2enfoMg5vPY7VwZWvP9 vfupHu/mPvuaiLi1AKTCDKPzUSIdaiApiIOh3cEMybo46gLRfZmBqZ/J/hZSqRsIS+vLmPuwg sMXQ7c2s00DUvGHbdT3j7a7Rgn0/+wtM0hti3hxDOh+gKrjqemtVQKhCFesKpkAKm1YdytiX0 0hhKtlq0AQmQdMh1tQ7lU1agyr75LwPnoXOO9VBvFEe3k1xMXZDde8z9s7EQSyBan6D4m2tVf 6L1x0HOk2oRxloiZEceJ4uwAnbk2RyNEa52eKzpKWAIjAElLRCIM+vt8eQx7KsVVGoT+v/LfR ed2ShkBR9f75Jdr81/tnXf9pVh6q8aG3m+0qvRHX4Bl0vVLFJO08jbmWUqOkNLEF7zMTXcajM upeHDmBLR0mWFlBJ3j6dNwuzla/nOE5IC0pjp9HzM7062i2ioS8SK6Ymzs/QcOrJ9mq8aAH+h AHk8r/ZfXxAZpMf8LASeNLo+0cluDkYIgGv2USXQEYAF8Dd1/ke3ZE+YhDbXlm2r5HDaDJHzx pfnDCCgLnHJFCfiuJeoydljGjjVg6Ke3S9NWPk74dJgsTLy4SfNVJ6GHEwKVtmu6fcmV/+KGv YiEUm4b0yF9GbqOytoeONWGo/bbkaH4BkXZjHbHSGjbKv6YLJKWFoghy7b/Hwd6TZNdOUH8nh L9XmRJrRDca56ZO3dVfd8ZYxPOMI0iCDISUZ4RCvho8beBrx9uaeU/i+QvlIcN3QhP25d1+Aj YWRg42iGcwguhIE/rxMHGkxFeTHXMtCrANnjYw5JzNGBBef0rg= Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Apr 15, 2020 at 4:13 PM Geert Uytterhoeven wrote: > On Wed, Apr 15, 2020 at 3:47 PM Arnd Bergmann wrote: > > On Tue, Apr 14, 2020 at 10:52 PM Laurent Pinchart wrote: > > > Doesn't "imply" mean it gets selected by default but can be manually > > > disabled ? > > > > That may be what it means now (I still don't understand how it's defined > > as of v5.7-rc1), but traditionally it was more like a 'select if all > > dependencies are met'. > > That's still what it is supposed to mean right now ;-) > Except that now it should correctly handle the modular case, too. Then there is a bug. If I run 'make menuconfig' now on a mainline kernel and enable CONFIG_DRM_RCAR_DU, I can set DRM_RCAR_CMM and DRM_RCAR_LVDS to 'y', 'n' or 'm' regardless of whether CONFIG_DRM_RCAR_DU is 'm' or 'y'. The 'implies' statement seems to be ignored entirely, except as reverse 'default' setting. > > > > In that case, a Makefile trick could also work, doing > > > > ifdef CONFIG_DRM_RCAR_CMM > > obj-$(CONFIG_DRM_RCAR_DU) += rcar-cmm.o > > endif > > > > Thereby making the cmm module have the same state (y or m) as > > the du module whenever the option is enabled. > > What about dropping the "imply DRM_RCAR_CMM", but defaulting to > enable CMM if DU is enabled? > > config DRM_RCAR_CMM > tristate "R-Car DU Color Management Module (CMM) Support" > depends on DRM_RCAR_DU && OF > default DRM_RCAR_DU That doesn't work because it allows DRM_RCAR_DU=y with DRM_RCAR_CMM=m, which causes a link failure. Arnd