From patchwork Wed Nov 16 20:50:59 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Bjorn Helgaas X-Patchwork-Id: 13045745 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id B8971C433FE for ; Wed, 16 Nov 2022 20:52:23 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-Id:Date:Subject:Cc:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=xTlAG7hrKbHx9rVPHba/zpIJu5eusXq7T5z4ccWRnGQ=; b=HLI5p3hOLRT1wZ jh0YNbW9hfDZdXKvSQ7wfastFwksSH/A1H36YemUudNzo4a9XtOram0QgnFyS/qj5YZYhWZS0Jq+d Zrb7bN0qCUIitm/YFMbNPZ59mCLMQtfsnN/Zg3b+Ietur+/39+gHgYQB1wHneMCc/YcvC+TmIhYkf 6bR0SJkd/6MqaW6tqtmkYlOzmMeeksIc/M0tAzZ22WHPsKAKceGqUtQ6h4qcu1JB3B+H7XL8Lx2s+ KhQIRpzUPJ7Tsu33lPnqIx9tfmRRhPa54si4kod+ZWhiZcqx4cjn1B86FiVTPTTXhLqVtCmN3EjbP C9L5nybtwoxkzT5www9Q==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1ovPNR-007t6m-1i; Wed, 16 Nov 2022 20:51:21 +0000 Received: from ams.source.kernel.org ([2604:1380:4601:e00::1]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1ovPNL-007t1G-GS for linux-arm-kernel@lists.infradead.org; Wed, 16 Nov 2022 20:51:17 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id 1E213B81D98; Wed, 16 Nov 2022 20:51:10 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 95817C433D6; Wed, 16 Nov 2022 20:51:08 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1668631868; bh=m6/u2vgmMBuIIdm3mXwZ8M7e/kuC3+TOJ2k7xEqGPF8=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=fzKeuOjOZxzuqYLQfd4CmqiACVuym10CfQHY3frTCsUUDGehntk6guziLR4zMHr3j +LDx7TVtM+lmr3YKsHOHTycfkFFnBi7QL6dUJV9hopZvLOzG7tY6Aes7geVnsMrT4A 4vS90YYFQsnTpgQe/50+yTHGrWdHY+M5Zoa6LUAK3IoGaIkc6u5zpiWktMH/a1yhUV n2lQlz1L2zlMgyhAAQn8wTL+mvySXUZ6Yw/HHj2b6SUub8uaSq8DHdQ+AHODumUSaf xyeOpWLRDA+6t7IU2dD+c7YnF14HUPIxeSdWMPuZ0W80sbfoY1+J12curKlUxVdG5L LiNvyu4/wWfiw== From: Bjorn Helgaas To: linux-pci@vger.kernel.org Cc: Lorenzo Pieralisi , Geert Uytterhoeven , =?utf-8?q?Pali_Roh=C3=A1r?= , Rob Herring , =?utf-8?q?Krzysztof_Wilczy=C5=84ski?= , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Bjorn Helgaas Subject: [PATCH v2 1/2] PCI: Drop of_match_ptr() to avoid unused variables Date: Wed, 16 Nov 2022 14:50:59 -0600 Message-Id: <20221116205100.1136224-2-helgaas@kernel.org> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20221116205100.1136224-1-helgaas@kernel.org> References: <20221116205100.1136224-1-helgaas@kernel.org> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20221116_125115_722185_D9894EF4 X-CRM114-Status: GOOD ( 15.56 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org From: Bjorn Helgaas We have stubs for most OF interfaces even when CONFIG_OF is not set, so we allow building of most controller drivers in that case for compile testing. When CONFIG_OF is not set, "of_match_ptr()" compiles to NULL, which leaves unused, resulting in errors like this: $ make W=1 drivers/pci/controller/pci-xgene.c:636:34: error: ‘xgene_pcie_match_table’ defined but not used [-Werror=unused-const-variable=] Drop of_match_ptr() to avoid the unused variable warning. See also 1dff012f636d ("PCI: Drop of_match_ptr() to avoid unused variables"). Link: https://lore.kernel.org/r/20221025191339.667614-2-helgaas@kernel.org Signed-off-by: Bjorn Helgaas --- drivers/pci/controller/pci-ftpci100.c | 2 +- drivers/pci/controller/pci-v3-semi.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/pci/controller/pci-ftpci100.c b/drivers/pci/controller/pci-ftpci100.c index 0cfd9d5a497c..ecd3009df586 100644 --- a/drivers/pci/controller/pci-ftpci100.c +++ b/drivers/pci/controller/pci-ftpci100.c @@ -553,7 +553,7 @@ static const struct of_device_id faraday_pci_of_match[] = { static struct platform_driver faraday_pci_driver = { .driver = { .name = "ftpci100", - .of_match_table = of_match_ptr(faraday_pci_of_match), + .of_match_table = faraday_pci_of_match, .suppress_bind_attrs = true, }, .probe = faraday_pci_probe, diff --git a/drivers/pci/controller/pci-v3-semi.c b/drivers/pci/controller/pci-v3-semi.c index 784fcf35599c..ca44b0c83d1b 100644 --- a/drivers/pci/controller/pci-v3-semi.c +++ b/drivers/pci/controller/pci-v3-semi.c @@ -901,7 +901,7 @@ static const struct of_device_id v3_pci_of_match[] = { static struct platform_driver v3_pci_driver = { .driver = { .name = "pci-v3-semi", - .of_match_table = of_match_ptr(v3_pci_of_match), + .of_match_table = v3_pci_of_match, .suppress_bind_attrs = true, }, .probe = v3_pci_probe,