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 009EFCD4F35 for ; Wed, 13 May 2026 10:46:00 +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=Jnb2bxZ/lnKlTGg+igX+necaPDsE1yQq4AeJ4mWz2ec=; b=dGnqHJkVNglaVdEfAOj+5C0Mt9 tqGRt6i/01VAkCiXjTbr8wvdKzK1EIPQxq0vB0Rdc7PnZGB1ATId6fSwTNDsERmJcVl/QCB887IXK +/2wAZH5FrkqR8kYQF+q6NnIDl2E+Whv3uOjHssUz9syn/G5/BbLrQHfDAOdC6F99Yg5dP9O4FP0H /4OWelkvHZQkJxzHj9ZtNTV18FtJtxl25Vj+ARzBExJILmO7YO9pQrBWAvPLIfxCtiINB6jonLHAb EPF24Qyc4ym3woSQwj9Z+/K8rCvRVeSoCU3KvmOXNmjIsGs8Xndxjvhn+ysE3+7WhbYIvjOzKrP5f pt5uadSA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1wN76A-00000002FEV-0wQ3; Wed, 13 May 2026 10:45:55 +0000 Received: from sea.source.kernel.org ([172.234.252.31]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1wN767-00000002FDO-2KF5 for linux-arm-kernel@lists.infradead.org; Wed, 13 May 2026 10:45:52 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by sea.source.kernel.org (Postfix) with ESMTP id 16089429E0; Wed, 13 May 2026 10:45:51 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 6FA74C2BCFA; Wed, 13 May 2026 10:45:50 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1778669151; bh=muuwARazTjEAvbXIOfzW/1TAyB+3XXA0p2SK256ykVk=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=AvMpum+0pqdXm9vGudPLtAmsoqsFbzz3XrQU7cRtMb7mU9DP5Qs70kk0AUdyyxdtb qDueFOICre4yQUeyMTywlSc0DEO7wUbtjUDMwDtFiFAX2WrJCHWb2rb66fr7lhj5DX mYezn7FnP8jK3rB1dUlu8+ZOE8H55IjMgrtoZTm/wMiUmdkbLP+hV+ePbOt9aIKcGX JqKbc66ny+KzcrVHmaxsGazj9jl3zcJBCQFk4IQ52v27paYFx40W27WPUMMevX1gMW DDXHG1TU0D8EM524vv5a0Q1sTqpBJRlb6AnTH1vg2RtR6qvXFJm3dgX7qH8w3CYZLG g5nJfhI4pDtaQ== Message-ID: Date: Wed, 13 May 2026 05:45:48 -0500 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH] drivers: altera_edac: Guard SDRAM irq2 retrieval for Arria10 only Content-Language: en-US To: "Nazle Asmade, Muhammad Nazim Amirul" , "bp@alien8.de" , "tony.luck@intel.com" Cc: "linux-edac@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" References: <20260508075240.23638-1-muhammad.nazim.amirul.nazle.asmade@altera.com> <275595ca-6044-4882-a9fd-ca9d496a27a5@altera.com> <4f80ef77-a38b-4652-a1f1-1bea3e88de74@altera.com> From: Dinh Nguyen In-Reply-To: <4f80ef77-a38b-4652-a1f1-1bea3e88de74@altera.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.9.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20260513_034551_639494_94D2ABEC X-CRM114-Status: GOOD ( 14.19 ) 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 5/12/26 06:51, Nazle Asmade, Muhammad Nazim Amirul wrote: > On 12/5/2026 7:25 pm, Dinh Nguyen wrote: >> >> >> On 5/11/26 20:37, Nazle Asmade, Muhammad Nazim Amirul wrote: >>> On 11/5/2026 7:54 pm, Dinh Nguyen wrote: >>>> >>>> >>>> On 5/8/26 02:52, muhammad.nazim.amirul.nazle.asmade@altera.com wrote: >>>>> From: Nazim Amirul >>>>> >>>>> Guard the irq2 retrieval with an of_machine_is_compatible() check so >>>>> that platform_get_irq(pdev, 1) is only called on Arria10 platforms. >>>>> >>>>> Signed-off-by: Nazim Amirul >>>>> >>>>> Signed-off-by: Niravkumar L Rabara >>>>> --- >>>>>    drivers/edac/altera_edac.c | 3 ++- >>>>>    1 file changed, 2 insertions(+), 1 deletion(-) >>>>> >>>>> diff --git a/drivers/edac/altera_edac.c b/drivers/edac/altera_edac.c >>>>> index 4edd2088c2db..b30302198cd4 100644 >>>>> --- a/drivers/edac/altera_edac.c >>>>> +++ b/drivers/edac/altera_edac.c >>>>> @@ -348,7 +348,8 @@ static int altr_sdram_probe(struct platform_device >>>>> *pdev) >>>>>        } >>>>>        /* Arria10 has a 2nd IRQ */ >>>>> -    irq2 = platform_get_irq(pdev, 1); >>>>> +    if (of_machine_is_compatible("altr,socfpga-arria10")) >>>>> +        irq2 = platform_get_irq(pdev, 1); >>>>>        layers[0].type = EDAC_MC_LAYER_CHIP_SELECT; >>>>>        layers[0].size = 1; >>>> >>>> Why? We already switch on arria10 later in the same function. >>>> >>>> Sorry, but NAK. >>>> >>>> Dinh >>> This driver were used by cyclone5 and arria10. Cyclone5 only has one >>> interrupt whereby arria10 has 2 interrupt. That is the reason why the >>> interrupt was guard by (of_machine_is_compatible("altr,socfpga-arria10")) >>> >> >> Yes, but look at line 397, >> >>           /* Only the Arria10 has separate IRQs */ >>         if (of_machine_is_compatible("altr,socfpga-arria10")) { >>                 /* Arria10 specific initialization */ >> >> Dinh >> >> > Hi Dinh, That is true, but the one that we looking at now is at line 352 > which enabling the second interrupt and it is not required by cyclone5. > Perhaps are you saying we should move the irq2 at line 352 under this > line 397? Yes, that would be fine. Dinh