From patchwork Thu Jul 6 13:38:41 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Russell King (Oracle)" X-Patchwork-Id: 9828337 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id A2E2B602BD for ; Thu, 6 Jul 2017 13:39:20 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 924D025F3E for ; Thu, 6 Jul 2017 13:39:20 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 84F6C28653; Thu, 6 Jul 2017 13:39:20 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-1.9 required=2.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID autolearn=unavailable version=3.3.1 Received: from bombadil.infradead.org (bombadil.infradead.org [65.50.211.133]) (using TLSv1.2 with cipher AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 22B3825F3E for ; Thu, 6 Jul 2017 13:39:19 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=ytmN06P2uCVDGMzM+xa8cHzABAQ3Tv2K9A0QCJ5/73Q=; b=YN2XUMbuSdpFLZ elF7mA6QbPcIuOlymz8b8Ofbhi0DJwsZlFk5NX+sKG7rkCEs55uxHC6IlTCav6K1399fbcECvMlm8 /SD7ddleprRFF/c0vVLvqw0HWqbZHVuX/uBu86ERurb2PeGWsHoVcyrjcFy/841NZrdkjwxU/gsRJ ZWNQ46KkteUtJBg4XiickBFQquT25tcmv0fgOXAuHAeQ1PovV+WmMoOaCtxHm7Drpy2yNJBLW7/TW UV9BhPu9d5P+ouQ0WkFXEv1PdN5s64+hAIXye+NTTo3FtXyFQNK4Ssbu0OwSa2ShAZ8L2YvXJNHz7 ehisImltQo55K7pfnlKw==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux)) id 1dT703-0005tE-6B; Thu, 06 Jul 2017 13:39:19 +0000 Received: from pandora.armlinux.org.uk ([2001:4d48:ad52:3201:214:fdff:fe10:1be6]) by bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux)) id 1dT6zz-0005s0-BP for linux-amlogic@lists.infradead.org; Thu, 06 Jul 2017 13:39:18 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=armlinux.org.uk; s=pandora-2014; h=Sender:In-Reply-To:Content-Type:MIME-Version:References:Message-ID:Subject:Cc:To:From:Date; bh=NdlYBf1avzDXzCOZeZqIDy/4Ndeu6kNmPQEI4GpRqsM=; b=EX/5i2EfOjPEhmP0GhSFcQJuec86PYgpzFxIrQ7LIQU6tTPUeMA3P+wn3aMbxg2vwYVPzs6Etm+NjrSwAahV/w7AWhtb4xnditckCV6RjyVDOSD+Rg2bQFdya6qahU5CGw6xaYCokaGoLZypjAKVucNwLaT0JbUsxkZnTlccNoc=; Received: from n2100.armlinux.org.uk ([2002:4e20:1eda:1:214:fdff:fe10:4f86]:53102) by pandora.armlinux.org.uk with esmtpsa (TLSv1:DHE-RSA-AES256-SHA:256) (Exim 4.82_1-5b7a7c0-XX) (envelope-from ) id 1dT6zU-0006ym-Oo; Thu, 06 Jul 2017 14:38:44 +0100 Received: from linux by n2100.armlinux.org.uk with local (Exim 4.76) (envelope-from ) id 1dT6zR-0007S1-MK; Thu, 06 Jul 2017 14:38:41 +0100 Date: Thu, 6 Jul 2017 14:38:41 +0100 From: Russell King - ARM Linux To: Neil Armstrong Subject: Re: [PATCH] drm/bridge: dw_hdmi: add cec notifier support Message-ID: <20170706133841.GA28170@n2100.armlinux.org.uk> References: <1499337186-24429-1-git-send-email-narmstrong@baylibre.com> <20170706110553.GJ4902@n2100.armlinux.org.uk> <185aa90b-a1b2-d899-40bb-7d333f584bb5@baylibre.com> <20170706114555.GK4902@n2100.armlinux.org.uk> <20170706115643.GA24728@n2100.armlinux.org.uk> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20170706115643.GA24728@n2100.armlinux.org.uk> User-Agent: Mutt/1.5.23 (2014-03-12) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20170706_063915_773686_C388F90A X-CRM114-Status: GOOD ( 24.53 ) X-BeenThere: linux-amlogic@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: a.hajda@samsung.com, architt@codeaurora.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-amlogic@lists.infradead.org Sender: "linux-amlogic" Errors-To: linux-amlogic-bounces+patchwork-linux-amlogic=patchwork.kernel.org@lists.infradead.org X-Virus-Scanned: ClamAV using ClamSMTP On Thu, Jul 06, 2017 at 12:56:43PM +0100, Russell King - ARM Linux wrote: > On Thu, Jul 06, 2017 at 12:45:55PM +0100, Russell King - ARM Linux wrote: > > Well, from what I can see in 4.12, the cec-notifier stuff is rather > > broken (tda998x has stopped working as its stuck with a physical > > address of f.f.f.f) so I think the whole thing is rather moot right > > now. I don't yet know what's going on with that, other than the > > notifier stuff seems to not be working, despite being enabled in > > the .config. > > The problem there appears to be the changes that were made with the > way the config works - which IMHO are totally broken. > > Let's take this scenario: > > - You have a HDMI bridge, and you build that into the kernel, because you > want the display to come up early. > - You have a CEC driver, which you build as a module. > > If the HDMI bridge driver selects CEC_NOTIFIER and the CEC driver selects > both CEC_NOTIFIER and CEC_CORE, you end up with CEC_NOTIFIER=y and > CEC_CORE=m. > > We now come to this: > > #if IS_REACHABLE(CONFIG_CEC_CORE) && IS_ENABLED(CONFIG_CEC_NOTIFIER) > > The definition of IS_REACHABLE() is that it is false if the config symbol > is selected as a module. So, in this case, we end up compiling out all > the CEC notifier functions from the HDMI bridge, and building them into > the CEC driver. > > The CEC notifier also gets built as a module, meaning that there's no way > for the built-in HDMI bridge could ever call the notifier. > > The overall result of this is that such a configuration completely breaks > such a setup - a setup that worked fine before the CEC Kconfig changes. > > This isn't limited to tda998x - I'd expect the same to be true of dw-hdmi. Fixing this so cec-notifier is built-in isn't sufficient, because we also need the cec-edid parsing code as well, which is currently part of cec-core, and that function gets stubbed out if cec-core is not built-in... The patch below works for me. drivers/media/Makefile | 2 +- drivers/media/cec/Makefile | 6 +++--- drivers/media/cec/cec-core.c | 2 +- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/media/Makefile b/drivers/media/Makefile index 044503aa8801..0c02fbe4b9c7 100644 --- a/drivers/media/Makefile +++ b/drivers/media/Makefile @@ -24,7 +24,7 @@ obj-$(CONFIG_DVB_CORE) += dvb-core/ # There are both core and drivers at RC subtree - merge before drivers obj-y += rc/ -obj-$(CONFIG_CEC_CORE) += cec/ +obj-y += cec/ # # Finally, merge the drivers that require the core diff --git a/drivers/media/cec/Makefile b/drivers/media/cec/Makefile index eaf408e64669..58394b77a328 100644 --- a/drivers/media/cec/Makefile +++ b/drivers/media/cec/Makefile @@ -1,7 +1,7 @@ cec-objs := cec-core.o cec-adap.o cec-api.o cec-edid.o -ifeq ($(CONFIG_CEC_NOTIFIER),y) - cec-objs += cec-notifier.o -endif +obj-$(CONFIG_CEC_NOTIFIER) += cec-notifier.o cec-edid.o +# Remove cec-edid from cec-core if the notifier is enabled +cec-objs := $(filter-out $(obj-y) $(obj-m), $(cec-objs)) obj-$(CONFIG_CEC_CORE) += cec.o diff --git a/drivers/media/cec/cec-core.c b/drivers/media/cec/cec-core.c index 2f87748ba4fc..bce26b94c348 100644 --- a/drivers/media/cec/cec-core.c +++ b/drivers/media/cec/cec-core.c @@ -187,7 +187,7 @@ static void cec_devnode_unregister(struct cec_devnode *devnode) put_device(&devnode->dev); } -#ifdef CONFIG_CEC_NOTIFIER +#if IS_ENABLED(CONFIG_CEC_NOTIFIER) static void cec_cec_notify(struct cec_adapter *adap, u16 pa) { cec_s_phys_addr(adap, pa, false);