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 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 21E0DC7EE2F for ; Mon, 12 Jun 2023 16:19:54 +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:In-Reply-To:MIME-Version:Message-ID: Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:References: List-Owner; bh=A3GMsEum63zSfKDsf9+JfbNrtYzWSIuE3DRfAfoQ5Dw=; b=IChZrsL16pzIF+ 029Y1PpiNamBry2HCIR+mEkw6FB407bf+RNhgOBlpHeOuo6Q5YXnyhvah8PlmJNn6ATlvuPKvoPKQ srEUlN6aEgJWqW0RsgDqJywVjlGNPPiuLfXypV4I4k0WkV0Dfb7+QGa+ocNpReTZIkMUwA4ufzops TQ3olIiO6ZCmIu2XEcXCAnJPkaqI1kjxyahjE4YhqxELlvyJg8BIZUFs2iQSLqIpylPic4VrJR9nT thKvtaXgTWjIE9BWmGmXLeRKmdKUig7XXxbYIbcoS4GbZDdRHkrH0ay7YPue3+57jheXthWhE2HT7 9jAny2oPSkJ0RiPEezjw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1q8kGT-004bhd-0H; Mon, 12 Jun 2023 16:19:33 +0000 Received: from dfw.source.kernel.org ([139.178.84.217]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1q8kGQ-004bgx-2B; Mon, 12 Jun 2023 16:19:32 +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 dfw.source.kernel.org (Postfix) with ESMTPS id B73E562B70; Mon, 12 Jun 2023 16:19:29 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 369B1C4339B; Mon, 12 Jun 2023 16:19:29 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1686586769; bh=OTJAkzUB/kHEz+zDpGNc9vDTw+iMQ7TJ/wo3kRt6WNE=; h=Date:From:To:Cc:Subject:In-Reply-To:From; b=CCG/J7Hk7a7Pqayi8Dhv4V5Lo1gKzDd2gONXFYj9dZ3ZlZsjrC3VfeL99tkG8oXFv xWdp4qVMK0MopWl4sfLAXlJg2OrAm/XFsPasvOIh6LQCGt7aa7bQd7zpD1TFNkbIU9 L1jxyebl9sLVd2W9lZZTRnLyhkOJYK3XZGm6jEO0rnjypc3oze2wFCaSBtLsKc83Gs PboYwpcQKg6IbNC1xzF3lx3VIfAVL0Aa9/tP4w7+I/dUsDNefxPJmfYtREE5tTIKk7 Jn1iBRGaC/y9DoMdE+4K1As0RtgR13AwOkAC0v2pyyRAfvbyFCrNiHrhaE2JHkncKF OwBnmjtJoxSCQ== Date: Mon, 12 Jun 2023 11:19:27 -0500 From: Bjorn Helgaas To: Pali =?iso-8859-1?Q?Roh=E1r?= Cc: Uwe =?iso-8859-1?Q?Kleine-K=F6nig?= , Bjorn Helgaas , Thomas Petazzoni , Lorenzo Pieralisi , Krzysztof =?utf-8?Q?Wilczy=C5=84ski?= , Joyce Ooi , Florian Fainelli , Jim Quinlan , Nicolas Saenz Julienne , Vignesh Raghavendra , Tom Joseph , Shawn Guo , Rahul Tanwar , Manivannan Sadhasivam , Thierry Reding , Jonathan Hunter , Serge Semin , Vidya Sagar , Jisheng Zhang , Miaoqian Lin , Ray Jui , Scott Branden , Ryder Lee , Jianjun Wang , Matthias Brugger , Sergio Paracuellos , Shawn Lin , Heiko Stuebner , Toan Le , Rob Herring , linux-pci@vger.kernel.org, linux-rockchip@lists.infradead.org, Broadcom internal kernel review list , linux-rpi-kernel@lists.infradead.org, kernel@pengutronix.de, linux-arm-msm@vger.kernel.org, linux-tegra@vger.kernel.org, linux-mediatek@lists.infradead.org, linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org, AngeloGioacchino Del Regno Subject: Re: [PATCH 00/15] PCI: Convert to platform remove callback returning void Message-ID: <20230612161927.GA1335109@bhelgaas> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20230611132423.milnj2pnvjqzwino@pali> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230612_091930_810203_69248005 X-CRM114-Status: GOOD ( 24.41 ) 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: , Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Sun, Jun 11, 2023 at 03:24:23PM +0200, Pali Roh=E1r wrote: > On Friday 02 June 2023 16:37:34 Bjorn Helgaas wrote: > > On Tue, May 30, 2023 at 04:07:42PM +0200, Uwe Kleine-K=F6nig wrote: > > > On Tue, Mar 21, 2023 at 08:31:53PM +0100, Uwe Kleine-K=F6nig wrote: > > > > this series adapts the platform drivers below drivers/pci to > > > > use the .remove_new() callback. Compared to the traditional > > > > .remove() callback .remove_new() returns no value. This is a > > > > good thing because the driver core doesn't (and cannot) cope > > > > for errors during remove. The only effect of a non-zero return > > > > value in .remove() is that the driver core emits a warning. > > > > The device is removed anyhow and an early return from > > > > .remove() usually yields a resource leak. > ... > Hello Bjorn, it should be expected that other changes for PCIe drivers > sent by other people which were sent to the list before this patch > series and are still waiting for the review (because are without > comments), would be processed before and patches sent later. I don't think it's necessary to delay simple, easily-reviewed changes behind more complicated ones. > Also I would like to point out that in past I have sent fixes for PCIe > mvebu driver, which is currently in the broken state. And this is also > on waiting on the list. Thanks for this reminder. Would you mind reposting them? I poked around in patchwork and I must be doing something wrong because I can't find *any* patches from you, though obviously there are many. If you repost them at least we'll know unambiguously what is on the table. Bjorn _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel