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=-0.6 required=3.0 tests=DKIM_INVALID,DKIM_SIGNED, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=no 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 6BFCAC10F29 for ; Tue, 17 Mar 2020 12:18:41 +0000 (UTC) Received: from lists.ozlabs.org (lists.ozlabs.org [203.11.71.2]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id B192120735 for ; Tue, 17 Mar 2020 12:18:39 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (2048-bit key) header.d=ellerman.id.au header.i=@ellerman.id.au header.b="R3suC07y" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org B192120735 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=ellerman.id.au Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=linuxppc-dev-bounces+linuxppc-dev=archiver.kernel.org@lists.ozlabs.org Received: from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3]) by lists.ozlabs.org (Postfix) with ESMTP id 48hXJW1fxmzDqRr for ; Tue, 17 Mar 2020 23:18:35 +1100 (AEDT) Received: from ozlabs.org (bilbo.ozlabs.org [IPv6:2401:3900:2:1::2]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 48hW3Z6x4qzDqTn for ; Tue, 17 Mar 2020 22:22:18 +1100 (AEDT) Authentication-Results: lists.ozlabs.org; dmarc=none (p=none dis=none) header.from=ellerman.id.au Authentication-Results: lists.ozlabs.org; dkim=pass (2048-bit key; unprotected) header.d=ellerman.id.au header.i=@ellerman.id.au header.a=rsa-sha256 header.s=201909 header.b=R3suC07y; dkim-atps=neutral Received: from authenticated.ozlabs.org (localhost [127.0.0.1]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-256) server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by mail.ozlabs.org (Postfix) with ESMTPSA id 48hW3W3wNXzB3t3; Tue, 17 Mar 2020 22:22:14 +1100 (AEDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ellerman.id.au; s=201909; t=1584444136; bh=7m6xTV6ORC9fimP591QotV0DExi6ArwAslaYAD6a8gE=; h=From:To:Cc:Subject:In-Reply-To:References:Date:From; b=R3suC07yLLownGnMiQtBzYlWQBjC2Y2Mr1fjgaMhi7I03Tw6ufjZw4WoaZXiqoW7X 9w7zSb2LsfkCq8qcUn9o6JDfZtIASBH4sFz8ntWfjzvCQz05fGey3e+m483q7aENNl WTjER2R9FHM0pwErCXzPUaOlqszJZ/dyxvLAY/7h0QKpfmECwhSvG1su6e0j9+Ibat SXe0q3bo4rMn5DbePzxlsRagSr8oOu986j8OGuQrkaCUhpM00HGe9unpetpD+aO/eV wcdxLe1dn62eA00W63BAXp3KDbgrdVnAh5ByL/QW192YlMk3FeRuzPn7et6TB3Nz1X 57yXL6Iyl5JEg== From: Michael Ellerman To: =?utf-8?B?546L5paH6JmO?= Subject: Re: [PATCH v3] powerpc/fsl-85xx: fix compile error In-Reply-To: References: Date: Tue, 17 Mar 2020 22:22:13 +1100 Message-ID: <878sjzfcpm.fsf@mpe.ellerman.id.au> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-BeenThere: linuxppc-dev@lists.ozlabs.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: trivial@kernel.org, kernel@vivo.com, linux-kernel@vger.kernel.org, stable , Richard Fontana , Paul Mackerras , Greg Kroah-Hartman , Thomas Gleixner , linuxppc-dev@lists.ozlabs.org, Allison Randal Errors-To: linuxppc-dev-bounces+linuxppc-dev=archiver.kernel.org@lists.ozlabs.org Sender: "Linuxppc-dev" =E7=8E=8B=E6=96=87=E8=99=8E writes: > From: Michael Ellerman > Date: 2020-03-16 17:41:12 > To:WANG Wenhu ,Benjamin Herrenschmidt ,Paul Mackerras ,WANG Wenhu ,Allison Randal ,Richard Fontana ,Greg Kroah-Hartman ,Thomas Gleixner ,linuxppc-dev@lists.ozlabs.org,linux-kernel@vger.kernel.org > cc: trivial@kernel.org,kernel@vivo.com,stable > Subject: Re: [PATCH v3] powerpc/fsl-85xx: fix compile error>WANG Wenhu writes: >>> Include "linux/of_address.h" to fix the compile error for >>> mpc85xx_l2ctlr_of_probe() when compiling fsl_85xx_cache_sram.c. >>> >>> CC arch/powerpc/sysdev/fsl_85xx_l2ctlr.o >>> arch/powerpc/sysdev/fsl_85xx_l2ctlr.c: In function =E2=80=98mpc85xx_l2c= tlr_of_probe=E2=80=99: >>> arch/powerpc/sysdev/fsl_85xx_l2ctlr.c:90:11: error: implicit declaratio= n of function =E2=80=98of_iomap=E2=80=99; did you mean =E2=80=98pci_iomap= =E2=80=99? [-Werror=3Dimplicit-function-declaration] >>> l2ctlr =3D of_iomap(dev->dev.of_node, 0); >>> ^~~~~~~~ >>> pci_iomap >>> arch/powerpc/sysdev/fsl_85xx_l2ctlr.c:90:9: error: assignment makes poi= nter from integer without a cast [-Werror=3Dint-conversion] >>> l2ctlr =3D of_iomap(dev->dev.of_node, 0); >>> ^ >>> cc1: all warnings being treated as errors >>> scripts/Makefile.build:267: recipe for target 'arch/powerpc/sysdev/fsl_= 85xx_l2ctlr.o' failed >>> make[2]: *** [arch/powerpc/sysdev/fsl_85xx_l2ctlr.o] Error 1 >>> >>> Fixes: commit 6db92cc9d07d ("powerpc/85xx: add cache-sram support") >> >>The syntax is: >> >>Fixes: 6db92cc9d07d ("powerpc/85xx: add cache-sram support") >> >>> Cc: stable >> >>The commit above went into v2.6.37. >> >>So no one has noticed this bug since then, how? Or did something else >>change to expose the problem? > > Actually a hard question to answer cause it also left me scratching for l= ong. > However, I can not find right or definite answer. Oh, actually it's fairly straight forward, the code can't be built at all in upstream because CONFIG_FSL_85XX_CACHE_SRAM is not selectable or selected by anything. You sent a patch previously to make it selectable, which Scott thought was a bad idea. So this whole file is dead code as far as I'm concerned, so patches for it definitely do not need to go to stable. If you want to add a user for it then please send a series doing that, and this commit can be the first. cheers