From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.3 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 0D1F1C76188 for ; Sun, 21 Jul 2019 07:22:14 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id B782720838 for ; Sun, 21 Jul 2019 07:22:13 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=innovation.ch header.i=@innovation.ch header.b="Nj9KSuzh" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726285AbfGUHWM (ORCPT ); Sun, 21 Jul 2019 03:22:12 -0400 Received: from chill.innovation.ch ([216.218.245.220]:36076 "EHLO chill.innovation.ch" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725933AbfGUHWM (ORCPT ); Sun, 21 Jul 2019 03:22:12 -0400 Date: Sun, 21 Jul 2019 00:22:11 -0700 DKIM-Filter: OpenDKIM Filter v2.10.3 chill.innovation.ch F0A56640132 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=innovation.ch; s=default; t=1563693731; bh=N+DAdWoheHn2b/cFIDXbfLsH1bX5Od7/izI9Zu/NVbo=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=Nj9KSuzhGp7+oX7JgCKiiahCdEOnbp5cqo+ObVuqcoHKYc9EeCL5J4aAgDuWwYAVC KV0kTCx6IIKf/saxRnfJvExIBM11LzJJjB1dAtFzf1XGgpmdR/ERu8+o+Otxsj+KWE CiMiZE3iQhXuZXb/hkwy0xxHp6yXL9uYwhPabMCH5l7l8WsMMs4xucXcF61MUVlDPQ XJnuoZipfPSDJODibTAMTh4thhLX5IYW+aoXzKevIdUMx/zt1kXIOQf6GC/rzr5U+R tkiSrjstYJVB6U5dZiUqnTJNCv/7fifupgueFEUKmhJ1odCtgMFFyBReizqocF0ByA 14Ckbj7SMG2fw== From: "Life is hard, and then you die" To: Andrzej Hajda Cc: Arnd Bergmann , Dmitry Torokhov , Randy Dunlap , dri-devel , LKML , Inki Dae , Laurent Pinchart , "linux-input@vger.kernel.org" Subject: Re: [PATCH] drm: bridge: DRM_SIL_SII8620 should depend on, not select INPUT Message-ID: <20190721072211.GB7992@innovation.ch> References: <8baa25c0-498b-d321-4e6a-fe987a4989ba@infradead.org> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Andrzej, On Mon, Jul 01, 2019 at 11:26:25AM +0200, Andrzej Hajda wrote: > On 01.07.2019 11:23, Andrzej Hajda wrote: > > On 01.07.2019 05:39, Randy Dunlap wrote: > >> From: Randy Dunlap > >> > >> A single driver should not enable (select) an entire subsystem, > >> such as INPUT, so change the 'select' to "depends on". > >> > >> Fixes: d6abe6df706c ("drm/bridge: sil_sii8620: do not have a dependency of RC_CORE") > >> > >> Signed-off-by: Randy Dunlap > >> Cc: Inki Dae > >> Cc: Andrzej Hajda > >> Cc: Laurent Pinchart > >> Cc: dri-devel@lists.freedesktop.org > >> --- > >> Linus has written this a couple of times in the last 15 years or so, > >> but my search fu cannot find it. And there are a few drivers in the > >> kernel tree that do this, but we shouldn't be adding more that do so. > > > > The proper solution has been already posted [1], but it has not been > > applied yet to input tree? > > > > Randy are there chances your patchset will appear in ML in near future, > > or should I merge your sii8620 patch alone? > > > Ups, I used wrong surname, I meant Ronald, added him input ML to cc. Looks like the applespi patches got merged, but they were changed at the last minute to remove the imply's that lead to the Kconfig dependency on the changes here. And I'm guessing for that reason the patch [1] was not sent with that pull request in the end. In short it seems to me that it would be best now if you sent a pull request for patch [1] and Arnd Bergmann's followup patch that was discussed in the last few days [2]. Sorry for all the delay. Cheers, Ronald > > [1]: > > https://lore.kernel.org/lkml/20190419081926.13567-2-ronald@innovation.ch/ [2] https://lore.kernel.org/lkml/20190718134240.2265724-1-arnd@arndb.de/ > > Regards > > > > Andrzej > > > > > > > >> drivers/gpu/drm/bridge/Kconfig | 3 +-- > >> 1 file changed, 1 insertion(+), 2 deletions(-) > >> > >> --- lnx-52-rc7.orig/drivers/gpu/drm/bridge/Kconfig > >> +++ lnx-52-rc7/drivers/gpu/drm/bridge/Kconfig > >> @@ -83,10 +83,9 @@ config DRM_PARADE_PS8622 > >> > >> config DRM_SIL_SII8620 > >> tristate "Silicon Image SII8620 HDMI/MHL bridge" > >> - depends on OF > >> + depends on OF && INPUT > >> select DRM_KMS_HELPER > >> imply EXTCON > >> - select INPUT > >> select RC_CORE > >> help > >> Silicon Image SII8620 HDMI/MHL bridge chip driver. > >> > >> > >> > >> >