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 BDE95C3DA4A for ; Mon, 5 Aug 2024 15:51:24 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:In-Reply-To:From:References:Cc:To:Subject:MIME-Version:Date: Message-ID:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=/vZFiGEaRCK52F8dlOLL/ikJJjWqQULFOAlwfCZtB2U=; b=gdssYjGtlBA9FVCnqtYeJWOntt ZCoYvyG2gFlEAH+TITQszfkGhb/3TmqdONc/RKQZkXciqfSA4V4OvlzaNcTYMIj0svBOf2b9UEIUN 0JjmOZqdIvYZ4PeXiGx9Blls3K/WxD0uxYrnzCKbmSJXMlB6xvWlhKmOudta/0OS+b97qzSMbAWv8 PLipnSI7Q5yXzBQSsZisM6WftZFhy78bTOsDkQnb/X3fqW8s9CaPF49cmSG3hC2cUhIttqlG2T4Pa YpeClLeTSfbi1H3Ns6OMf/OHDYvpcAi9sRva4AdYIauFuYkgIp6MCUyuncvzj5pbbe/SqPGgeVcsu /DOl1fSw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sazzK-0000000GOJD-0VlJ; Mon, 05 Aug 2024 15:51:10 +0000 Received: from linux.microsoft.com ([13.77.154.182]) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sazy7-0000000GNqj-3JBY for linux-arm-kernel@lists.infradead.org; Mon, 05 Aug 2024 15:49:57 +0000 Received: from [10.137.186.190] (unknown [131.107.159.62]) by linux.microsoft.com (Postfix) with ESMTPSA id C5C8520B7177; Mon, 5 Aug 2024 08:49:53 -0700 (PDT) DKIM-Filter: OpenDKIM Filter v2.11.0 linux.microsoft.com C5C8520B7177 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.microsoft.com; s=default; t=1722872994; bh=/vZFiGEaRCK52F8dlOLL/ikJJjWqQULFOAlwfCZtB2U=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=WAZUEdvvplp9Kz9DnN3K1N7AnKVUJSClgxvtv1KR134wKpshpUYQPaVLBnCg5d4s9 6dtcZ71qsNM6vRC7M/WJ5MuXFXCAtc80nH4ZX3BUKjCch7p1HYqnKS+IZ/LSHT77RN m8PBDBuvrEe9ZdT5NtLDT0zvyY4AQld/+GxIX8Xs= Message-ID: <2db03d90-55d8-440c-b90a-de421b6929ff@linux.microsoft.com> Date: Mon, 5 Aug 2024 08:49:54 -0700 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v3 6/7] Drivers: hv: vmbus: Get the IRQ number from DT To: Michael Kelley , Saurabh Singh Sengar Cc: "arnd@arndb.de" , "bhelgaas@google.com" , "bp@alien8.de" , "catalin.marinas@arm.com" , "dave.hansen@linux.intel.com" , "decui@microsoft.com" , "haiyangz@microsoft.com" , "hpa@zytor.com" , "kw@linux.com" , "kys@microsoft.com" , "lenb@kernel.org" , "lpieralisi@kernel.org" , "mingo@redhat.com" , "rafael@kernel.org" , "robh@kernel.org" , "tglx@linutronix.de" , "wei.liu@kernel.org" , "will@kernel.org" , "linux-acpi@vger.kernel.org" , "linux-arch@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-hyperv@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "linux-pci@vger.kernel.org" , "x86@kernel.org" , "apais@microsoft.com" , "benhill@microsoft.com" , "ssengar@microsoft.com" , "sunilmut@microsoft.com" , "vdso@hexbites.dev" References: <20240726225910.1912537-1-romank@linux.microsoft.com> <20240726225910.1912537-7-romank@linux.microsoft.com> <20240805083024.GB31897@linuxonhyperv3.guj3yctzbm1etfxqx2vob5hsef.xx.internal.cloudapp.net> Content-Language: en-US From: Roman Kisel In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240805_084956_063065_9965360D X-CRM114-Status: GOOD ( 15.74 ) 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: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 8/5/2024 7:12 AM, Michael Kelley wrote: > From: Saurabh Singh Sengar Sent: Monday, August 5, 2024 1:30 AM >> >> On Fri, Jul 26, 2024 at 03:59:09PM -0700, Roman Kisel wrote: >>> The VMBus driver uses ACPI for interrupt assignment on >>> arm64 hence it won't function in the VTL mode where only >>> DeviceTree can be used. >>> >>> Update the VMBus driver to discover interrupt configuration >>> via DeviceTree and indicate DMA cache coherency. >>> >>> Signed-off-by: Roman Kisel >>> --- >>> drivers/hv/vmbus_drv.c | 49 ++++++++++++++++++++++++++++++++++++++++++ >>> 1 file changed, 49 insertions(+) >>> >>> diff --git a/drivers/hv/vmbus_drv.c b/drivers/hv/vmbus_drv.c >>> index 12a707ab73f8..7eee7caff5f6 100644 >>> --- a/drivers/hv/vmbus_drv.c >>> +++ b/drivers/hv/vmbus_drv.c >>> @@ -2306,6 +2306,34 @@ static int vmbus_acpi_add(struct platform_device *pdev) >>> } >>> #endif >>> >>> +static int __maybe_unused vmbus_set_irq(struct platform_device *pdev) >>> +{ >>> + struct irq_desc *desc; >>> + int irq; >>> + >>> + irq = platform_get_irq(pdev, 0); >>> + if (irq == 0) { >>> + pr_err("VMBus interrupt mapping failure\n"); >>> + return -EINVAL; >>> + } >>> + if (irq < 0) { >>> + pr_err("VMBus interrupt data can't be read from DeviceTree, error %d\n", irq); >>> + return irq; >>> + } >>> + >>> + desc = irq_to_desc(irq); >> >> irq_to_desc is not an exported symbol if CONFIG_SPARSE_IRQ is enabled. This will >> break the builds for HYPERV as module. >> > > Instead, use irq_get_irq_data(), then irqd_to_hwirq(). > Couldn't appreciate enough your indispensable advice, folks! > Michael -- Thank you, Roman