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 mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 6870CC433F5 for ; Mon, 11 Oct 2021 11:11:53 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id 2EFD060EE2 for ; Mon, 11 Oct 2021 11:11:53 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 2EFD060EE2 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.infradead.org 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:References: 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: List-Owner; bh=F6iEhMX0zJ5hmipxK6DsIjFiodcOHeyqyq2BUC871QU=; b=Suq5K1Fli6vOUI z15R225ze/hTmgjNFbElGR18283PWxWZtkedtFc2ZLGtvjtZL7Llb4tffJWQki3SOycfa4gOQQLRC RxtqDuX0IP6Kp5uoN7UDc1Tm1EENRnErkBuVfoLGEJF+58S1sRh2c85YYFQ8Ur0KT1bMiz/3sOX7j xwPqbmPF7e65DsV4mR7+vmpkUi4lEKhd8hY6QqqVwK6zKCBoOqW71Xmxl+YT+9qVcAE1T4YkYAjtH uwRF+7kYnfPkKescvVXkjhEXSnjZm+kpXq7xv4dhR6Ky+Dl72rUi59h7Wfb7vLD3Fqa3WExJWPbDk pNslwNTD0l6KOjybTIbg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mZtBn-0090y2-UX; Mon, 11 Oct 2021 11:09:52 +0000 Received: from mail.kernel.org ([198.145.29.99]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1mZtBj-0090wp-7k for linux-arm-kernel@lists.infradead.org; Mon, 11 Oct 2021 11:09:48 +0000 Received: by mail.kernel.org (Postfix) with ESMTPSA id 7952460EB4; Mon, 11 Oct 2021 11:09:45 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1633950586; bh=aU/ORD9aTN28OfqMBJVt9VUK/rhaDiAzfDUP8h089Q0=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=GouTvkCeDkPzQvUYCbuKjINzgc5ybPb58t45woq27gFjR2efiF6CiDdzr3QUBXKQt vz/l1uEv5esSH6QiodIueAf2dUEu62OagO4oRWLTD68dZUs/l4TJ7VCb+7urslXc9u QaHwO5wnsfo7lNa2yH45RW1f+d461Do6FhHRYXnv7oL9GU/TcZfjpZ+Wh83oq5k+xe zXUo1MXLdgW8xfLJvYhvJ5t/Y5EE5i1HxdVnzIg6QHXo2HQQm++SiZyOP+xf+8GI0I WcHD7KXegeH6Nv+Tk09vOspkvzEJHPjyp+7tg6LXSmLKCImuEgOh3x9qKqIrlQXqp7 6i+5j6uZUnBIQ== Date: Mon, 11 Oct 2021 12:09:41 +0100 From: Will Deacon To: Lorenzo Pieralisi Cc: yajun.deng@linux.dev, catalin.marinas@arm.com, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH linux-next v2] arm64: PCI: Introduce pcibios_free_irq() helper function Message-ID: <20211011110940.GC4068@willie-the-truck> References: <20210921154731.GA2756@lpieralisi> <20210826060406.12571-1-yajun.deng@linux.dev> <20210922162201.GA24429@lpieralisi> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20210922162201.GA24429@lpieralisi> User-Agent: Mutt/1.10.1 (2018-07-13) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20211011_040947_316626_876A8374 X-CRM114-Status: GOOD ( 21.00 ) 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="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Wed, Sep 22, 2021 at 05:22:01PM +0100, Lorenzo Pieralisi wrote: > On Wed, Sep 22, 2021 at 02:49:13AM +0000, yajun.deng@linux.dev wrote: > > September 21, 2021 11:47 PM, "Lorenzo Pieralisi" wrote: > > > > > On Thu, Aug 26, 2021 at 02:04:06PM +0800, Yajun Deng wrote: > > > > > >> pcibios_alloc_irq() will be called in pci_device_probe(), but there > > >> hasn't pcibios_free_irq() in arm64 architecture correspond it. > > >> pcibios_free_irq() is an empty weak function in drivers/pci/pci-driver.c. > > > > > > "pcibios_alloc_irq() is a weak function called to allocate IRQs for > > > a device in pci_device_probe(); arm64 implements it with > > > ACPI specific code to enable IRQs for a device. > > > > > > When a device is removed (pci_device_remove()) the pcibios_free_irq() > > > counterpart is called. > > > > > > Current arm64 code does not implement a pcibios_free_irq() function, > > > and therefore, the weak empty stub is executed, which means that the > > > IRQ for a device are not properly disabled when a device is removed. > > > > > > Add an arm64 pcibios_free_irq() to undo the actions carried out in > > > pcibios_alloc_irq()." > > > > > > This is a stub commit log. Then you need to describe the bug you > > > are fixing (if any, or it is just code inspection ?) > > > > > It is just code inspection. > > Before merging it it must be tested, which would prove it is needed. Ok, I'll ignore this patch for now then. Please yell if you want me to pick it up. Will _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel