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 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 3ADA0C77B7A for ; Thu, 1 Jun 2023 09:27:21 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232326AbjFAJ1U (ORCPT ); Thu, 1 Jun 2023 05:27:20 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:46896 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231607AbjFAJ1S (ORCPT ); Thu, 1 Jun 2023 05:27:18 -0400 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id EA79297; Thu, 1 Jun 2023 02:27:17 -0700 (PDT) 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 dfw.source.kernel.org (Postfix) with ESMTPS id 8A18C6427A; Thu, 1 Jun 2023 09:27:17 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id BB2E9C433EF; Thu, 1 Jun 2023 09:27:16 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1685611636; bh=F71V0LqFvJaV2Dh0oD8DhGsDszrCckaBPOEhZorcm1s=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=tKtcfVQYOggy9UmUERLxPqYbXoYAVO62e6tciqk8bQfUD9g9DsX6CeQzsTHnYHHWN R2xyBP+dPyeT/OPUJVDwUMVQNjVCeCeVMBPbS+enDc9j/zBifPwcO28qN3fCXvnFkX j+szQYX8g+2CJLy7wH3nFl+CqazYnAfFfSedTtUug9WXQzRizLAnp0pahsdjUXBthu /X3DPx2ctwAqmJ24mfV0liEwtRTFGKnWFxN4CWgb6HS/0urFc5XHHOo/OpAhnPVC+K Om1gF8C8GVY6pNLvNvzUu0jRrr6oErLt1lOvxMa6H3m0WlbWp1qo+TQ6JPs82Uh5OC grnybOg/2o/fw== Date: Thu, 1 Jun 2023 05:27:15 -0400 From: Sasha Levin To: Arnd Bergmann Cc: linux-kernel@vger.kernel.org, stable@vger.kernel.org, Jakub Kicinski , Andrew Lunn , Heiner Kallweit , Netdev Subject: Re: [PATCH AUTOSEL 4.14 20/20] mdio_bus: unhide mdio_bus_init prototype Message-ID: References: <20230525184520.2004878-1-sashal@kernel.org> <20230525184520.2004878-20-sashal@kernel.org> <7f46f5c0-a5cb-4e4f-9201-2fd06e92e1ef@app.fastmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii; format=flowed Content-Disposition: inline In-Reply-To: <7f46f5c0-a5cb-4e4f-9201-2fd06e92e1ef@app.fastmail.com> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, May 25, 2023 at 10:05:01PM +0200, Arnd Bergmann wrote: >On Thu, May 25, 2023, at 20:45, Sasha Levin wrote: >> From: Arnd Bergmann >> >> [ Upstream commit 2e9f8ab68f42b059e80db71266c1675c07c664bd ] >> >> mdio_bus_init() is either used as a local module_init() entry, >> or it gets called in phy_device.c. In the former case, there >> is no declaration, which causes a warning: >> >> drivers/net/phy/mdio_bus.c:1371:12: error: no previous prototype for >> 'mdio_bus_init' [-Werror=missing-prototypes] >> >> Remove the #ifdef around the declaration to avoid the warning.. > >Hi Sasha, > >While there is nothing wrong with backporting the -Wmissing-prototypes >warning fixes I sent, there is also really no point unless you >want to backport all 140 of them and then also turn on that warning >during testing. > >The option is only enabled at the W=1 level or when using sparse (C=1). >I hope to get these all done in 6.5 for the most common architectures, >but I wouldn't bother putting them into stable kernels. I'll go drop it. In general, we've been trying to avoid W=1 fixes but sometimes they end up sneaking in, which is also okay... -- Thanks, Sasha