From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from lindbergh.monkeyblade.net (lindbergh.monkeyblade.net [23.128.96.19]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id DDF1B7C for ; Mon, 29 May 2023 02:07:47 +0000 (UTC) Received: from smtpbg156.qq.com (smtpbg156.qq.com [15.184.82.18]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 2ACF0BB for ; Sun, 28 May 2023 19:07:44 -0700 (PDT) X-QQ-mid:Yeas48t1685325967t907t14719 Received: from 3DB253DBDE8942B29385B9DFB0B7E889 (jiawenwu@trustnetic.com [183.159.96.128]) X-QQ-SSF:00400000000000F0FOF000000000000 From: =?utf-8?b?Smlhd2VuIFd1?= X-BIZMAIL-ID: 14564377025718723905 To: "'Russell King \(Oracle\)'" , "'kernel test robot'" Cc: , , , , , , , , , , , , "'Piotr Raczynski'" References: <20230524091722.522118-6-jiawenwu@trustnetic.com> <202305261959.mnGUW17n-lkp@intel.com> In-Reply-To: Subject: RE: [PATCH net-next v9 5/9] net: txgbe: Add SFP module identify Date: Mon, 29 May 2023 10:06:06 +0800 Message-ID: <02dd01d991d2$2120fcf0$6362f6d0$@trustnetic.com> Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit X-Mailer: Microsoft Outlook 16.0 Thread-Index: AQFlCTXCD0V13/nxYaH3lJLB+zCPMgFguorBARt7XSiwRa6NsA== Content-Language: zh-cn X-QQ-SENDSIZE: 520 Feedback-ID: Yeas:trustnetic.com:qybglogicsvrgz:qybglogicsvrgz5a-1 X-Spam-Status: No, score=-2.9 required=5.0 tests=BAYES_00,FROM_EXCESS_BASE64, RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H5,RCVD_IN_MSPIKE_WL,SPF_HELO_PASS, SPF_PASS,T_SCC_BODY_TEXT_LINE,UNPARSEABLE_RELAY autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net On Friday, May 26, 2023 7:37 PM, Russell King (Oracle) wrote: > On Fri, May 26, 2023 at 07:30:45PM +0800, kernel test robot wrote: > > Kconfig warnings: (for reference only) > > WARNING: unmet direct dependencies detected for I2C_DESIGNWARE_PLATFORM > > Depends on [n]: I2C [=n] && HAS_IOMEM [=y] && (ACPI && COMMON_CLK [=y] || !ACPI) > > Selected by [y]: > > - TXGBE [=y] && NETDEVICES [=y] && ETHERNET [=y] && NET_VENDOR_WANGXUN [=y] && PCI [=y] > > WARNING: unmet direct dependencies detected for SFP > > Depends on [n]: NETDEVICES [=y] && PHYLIB [=y] && I2C [=n] && PHYLINK [=y] && (HWMON [=n] || HWMON [=n]=n) > > Selected by [y]: > > - TXGBE [=y] && NETDEVICES [=y] && ETHERNET [=y] && NET_VENDOR_WANGXUN [=y] && PCI [=y] > > ... and is basically caused by "select SFP". No. Do not do this unless > you look at the dependencies for SFP and ensure that those are also > satisfied - because if you don't you create messes like the above > build errors. So how do I make sure that the module I need compiles and loads correctly, rely on the user to manually select it?