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=-7.7 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,MENTIONS_GIT_HOSTING, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED 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 9E7E3C4708F for ; Wed, 2 Jun 2021 18:09:08 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 431C861090 for ; Wed, 2 Jun 2021 18:09:08 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 431C861090 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 9C18C6B0036; Wed, 2 Jun 2021 14:09:07 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 94AA16B006C; Wed, 2 Jun 2021 14:09:07 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 79CCC6B0070; Wed, 2 Jun 2021 14:09:07 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0162.hostedemail.com [216.40.44.162]) by kanga.kvack.org (Postfix) with ESMTP id 43B096B0036 for ; Wed, 2 Jun 2021 14:09:07 -0400 (EDT) Received: from smtpin25.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay05.hostedemail.com (Postfix) with ESMTP id CCAD9181AC9C6 for ; Wed, 2 Jun 2021 18:09:06 +0000 (UTC) X-FDA: 78209570292.25.310B65F Received: from mail-ot1-f50.google.com (mail-ot1-f50.google.com [209.85.210.50]) by imf16.hostedemail.com (Postfix) with ESMTP id B1A3D8019364 for ; Wed, 2 Jun 2021 18:08:53 +0000 (UTC) Received: by mail-ot1-f50.google.com with SMTP id 69-20020a9d0a4b0000b02902ed42f141e1so3279977otg.2 for ; Wed, 02 Jun 2021 11:09:06 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=ZhGtSb18ge2sD/kgR/xLamgIUWfRdZt+iX2nxECTX1s=; b=Q5fFXrhc2ocUnUVbOxhAt2M2xzs8Wf4wPRj4AhJD3zM8hSWwNcs77XGN5GJY56XsCn JzZhgPlBLG/qiH9P06+8mpY4BbtI78Auqzbx9nJTD4ygDpoyil99KbhzSbM/cXkvm9/z VkTweaE+KvEUsKTN9M1+YZzoWdRrd3wnXtB3w6ck5AhwyMCcMWRCNB7XiCp4o293q31p N3QUZDtjh0A2KFjL1j78A6t+CwKTJcPf9Y51CJCECTxyggUNMjeoqmYTjnseNshhmPfX ejGPxy5FyvVf3yaQDtefUmE1WHW5m6NfXYnON3qRqMhqkaXNYXVGHuo/6CLEHrwmDi75 yiPA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=ZhGtSb18ge2sD/kgR/xLamgIUWfRdZt+iX2nxECTX1s=; b=Sv3p96WbNHyA9lSBdakQhEDT6c6TkLOOFjX5g0Gqa1k/qr1VzOCWCbzdmhgA1lyaU2 8bRg3/p9ukVNuDS7fDCBoBjhyByMfJnrCiBQ2W1RVhG0NJB5S4AWA0eHA9e6dChyDs1+ dM5ILUThcuU/89RqrjDxwHC/j5nyhzdu/Qug0/p6ytZ/uVmsJl/Wjw6HUPvh9L9ywUtT zNot64jKf3yO5c5MaT/cFJUenrleO6opixCDkeFwWTWhC9kiQ+zm3VfWXMrqu/u3YAK+ phBwiU1Mjfb3lyHJupsRif4iScPjcBQ1kDvveXo4yK44gmc0KR8BsSfqjNc+t9Yo5BYF ZC3w== X-Gm-Message-State: AOAM530DQGghfcFL9Hdn+huimeUjzt2vCP4OheWOmolzbCl1VB8N4X+0 56iVS/90lFPS+APEETRa4kJCmag4m50LqVw3vFU= X-Google-Smtp-Source: ABdhPJx8mUt77pr4XjIQI1ngXgyLIkXSGk3YC3aWOEClXhFjsHtUK54JHsHbIkVetrqD38hgjpx5nJA2/dw9I6o0uKA= X-Received: by 2002:a9d:4f19:: with SMTP id d25mr1304290otl.72.1622657345719; Wed, 02 Jun 2021 11:09:05 -0700 (PDT) MIME-Version: 1.0 References: <202106021822.Do0Ih08d-lkp@intel.com> In-Reply-To: From: Sergio Paracuellos Date: Wed, 2 Jun 2021 20:08:54 +0200 Message-ID: Subject: Re: [linux-next:master 1932/6331] drivers/phy/ralink/phy-mt7621-pci.c:341:34: warning: unused variable 'mt7621_pci_phy_ids' To: Nick Desaulniers Cc: kernel test robot , kbuild-all@lists.01.org, clang-built-linux , Linux Memory Management List , Vinod Koul Content-Type: text/plain; charset="UTF-8" Authentication-Results: imf16.hostedemail.com; dkim=pass header.d=gmail.com header.s=20161025 header.b=Q5fFXrhc; dmarc=pass (policy=none) header.from=gmail.com; spf=pass (imf16.hostedemail.com: domain of sergioparacuellos@gmail.com designates 209.85.210.50 as permitted sender) smtp.mailfrom=sergioparacuellos@gmail.com X-Stat-Signature: 8o55wooyz81bqyt6me8ygioy4bj8myg4 X-Rspamd-Queue-Id: B1A3D8019364 X-Rspamd-Server: rspam02 X-HE-Tag: 1622657333-68213 X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: Hi Nick, On Wed, Jun 2, 2021 at 6:47 PM Nick Desaulniers wrote: > > On Wed, Jun 2, 2021 at 3:38 AM Sergio Paracuellos > wrote: > > > > Hi, > > > > On Wed, Jun 2, 2021 at 12:27 PM kernel test robot wrote: > > > > > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master > > > head: ed5d0667a6540293c9485dd95babb5f1e241226a > > > commit: 28dcfba1a0d622b0330ae3f4a9d7c7f2c245de7a [1932/6331] phy: ralink: Kconfig: enable COMPILE_TEST on mt7621-pci-phy driver > > > config: x86_64-randconfig-r016-20210602 (attached as .config) > > ^ randconfig > > > > compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project db26cd30b6dd65e88d786e97a1e453af5cd48966) > > > reproduce (this is a W=1 build): > > > wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross > > > chmod +x ~/bin/make.cross > > > # install x86_64 cross compiling tool for clang build > > > # apt-get install binutils-x86-64-linux-gnu > > > # https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?id=28dcfba1a0d622b0330ae3f4a9d7c7f2c245de7a > > > git remote add linux-next https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git > > > git fetch --no-tags linux-next master > > > git checkout 28dcfba1a0d622b0330ae3f4a9d7c7f2c245de7a > > > # save the attached .config to linux build tree > > > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64 > > > > > > If you fix the issue, kindly add following tag as appropriate > > > Reported-by: kernel test robot > > > > > > All warnings (new ones prefixed by >>): > > > > > > drivers/phy/ralink/phy-mt7621-pci.c:277:4: warning: cast to smaller integer type 'unsigned int' from 'void *' [-Wvoid-pointer-to-int-cast] > > > (unsigned int)mt7621_phy->port_base, mt7621_phy->has_dual_port); > > > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > > > include/linux/dev_printk.h:118:33: note: expanded from macro 'dev_info' > > > _dev_info(dev, dev_fmt(fmt), ##__VA_ARGS__) > > > > Already fixed in: > > https://git.kernel.org/pub/scm/linux/kernel/git/phy/linux-phy.git/commit/drivers/phy/ralink/phy-mt7621-pci.c?h=next&id=652a6a2e3824ce2ebf79a2d5326940d05c4db036 > > > > > ^~~~~~~~~~~ > > > >> drivers/phy/ralink/phy-mt7621-pci.c:341:34: warning: unused variable 'mt7621_pci_phy_ids' [-Wunused-const-variable] > > > static const struct of_device_id mt7621_pci_phy_ids[] = { > > > ^ > > > > Mmmm... This one is used through MODULE_DEVICE_TABLE macro... Am I > > missing something?? > > Pretty sure there's a config in MODULE_DEVICE_TABLE that's being > disabled by the randconfig; the definition of mt7621_pci_phy_ids > should also be behind preprocessor checks for that. So, the driver uses 'mt7621_pci_phy_ids' in two places : static const struct of_device_id mt7621_pci_phy_ids[] = { { .compatible = "mediatek,mt7621-pci-phy" }, {}, }; MODULE_DEVICE_TABLE(of, mt7621_pci_phy_ids); ^ This one is inside the 'MODULE' preprocessor directive. static struct platform_driver mt7621_pci_phy_driver = { .probe = mt7621_pci_phy_probe, .driver = { .name = "mt7621-pci-phy", .of_match_table = of_match_ptr(mt7621_pci_phy_ids), ^ This one is inside 'CONFIG_OF' }, }; randconfig is not setting 'CONFIG_OF' nor defining MODULE. I've been looking into different drivers code and all of them follow the same pattern and nothing is behind preprocessor checks at all. So I wonder if this is really something that needs to be fixed. Thanks in advance for clarifications. Best regards, Sergio Paracuellos > > > > > > 2 warnings generated. > > > > > > > > > vim +/mt7621_pci_phy_ids +341 drivers/phy/ralink/phy-mt7621-pci.c > > > > > > d87da32372a03c Sergio Paracuellos 2020-11-21 340 > > > d87da32372a03c Sergio Paracuellos 2020-11-21 @341 static const struct of_device_id mt7621_pci_phy_ids[] = { > > > d87da32372a03c Sergio Paracuellos 2020-11-21 342 { .compatible = "mediatek,mt7621-pci-phy" }, > > > d87da32372a03c Sergio Paracuellos 2020-11-21 343 {}, > > > d87da32372a03c Sergio Paracuellos 2020-11-21 344 }; > > > 8145dcb07d0c8b Sergio Paracuellos 2020-12-01 345 MODULE_DEVICE_TABLE(of, mt7621_pci_phy_ids); > > > d87da32372a03c Sergio Paracuellos 2020-11-21 346 > > > > > > :::::: The code at line 341 was first introduced by commit > > > :::::: d87da32372a03ce121fc65ccd2c9a43edf56b364 phy: ralink: Add PHY driver for MT7621 PCIe PHY > > > > > > :::::: TO: Sergio Paracuellos > > > :::::: CC: Vinod Koul > > > > > > --- > > > 0-DAY CI Kernel Test Service, Intel Corporation > > > https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org > > > > Best regards, > > Sergio Paracuellos > > > > -- > > You received this message because you are subscribed to the Google Groups "Clang Built Linux" group. > > To unsubscribe from this group and stop receiving emails from it, send an email to clang-built-linux+unsubscribe@googlegroups.com. > > To view this discussion on the web visit https://groups.google.com/d/msgid/clang-built-linux/CAMhs-H8qyOKQmPtChYO%2BjfC%2BZmj8U9yGeZJORUx14MzyLeT5QA%40mail.gmail.com. > > > > -- > Thanks, > ~Nick Desaulniers