Received-SPF: pass (google.com: domain of linux-kernel+bounces-67961-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Precedence: bulk
MIME-Version: 1.0
References: <20240215140819.1910705-1-masahiroy@kernel.org> <CH3PR02MB1024753D597A02B754FEE1D85B84D2@CH3PR02MB10247.namprd02.prod.outlook.com>
In-Reply-To: <CH3PR02MB1024753D597A02B754FEE1D85B84D2@CH3PR02MB10247.namprd02.prod.outlook.com>
From: Masahiro Yamada <masahiroy@kernel.org>
Date: Fri, 16 Feb 2024 11:03:16 +0900
Message-ID: <CAK7LNATt+0nBzO+UBOZhdkjmyYyzgU+VfG4eNxMCYvFmB-z2Sw@mail.gmail.com>
Subject: Re: [PATCH] hexagon: select GENERIC_IRQ_PROBE instead of redefining it
To: Brian Cain <bcain@quicinc.com>
Cc: "linux-kbuild@vger.kernel.org" <linux-kbuild@vger.kernel.org>, 
	"linux-hexagon@vger.kernel.org" <linux-hexagon@vger.kernel.org>, 
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Fri, Feb 16, 2024 at 4:12=E2=80=AFAM Brian Cain <bcain@quicinc.com> wrot=
e:
>
>
>
> > -----Original Message-----
> > From: Masahiro Yamada <masahiroy@kernel.org>
> > Sent: Thursday, February 15, 2024 8:08 AM
> > To: linux-kbuild@vger.kernel.org
> > Cc: Masahiro Yamada <masahiroy@kernel.org>; Brian Cain
> > <bcain@quicinc.com>; linux-hexagon@vger.kernel.org; linux-
> > kernel@vger.kernel.org
> > Subject: [PATCH] hexagon: select GENERIC_IRQ_PROBE instead of redefinin=
g it
> >
> > WARNING: This email originated from outside of Qualcomm. Please be wary=
 of
> > any links or attachments, and do not enable macros.
> >
> > Select GENERIC_IRQ_PROBE, as the other architectures do.
> >
> > Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>
> > ---
> >
> >  arch/hexagon/Kconfig | 4 +---
> >  1 file changed, 1 insertion(+), 3 deletions(-)
> >
> > diff --git a/arch/hexagon/Kconfig b/arch/hexagon/Kconfig
> > index e922026fef09..a8dd2501f39f 100644
> > --- a/arch/hexagon/Kconfig
> > +++ b/arch/hexagon/Kconfig
> > @@ -21,6 +21,7 @@ config HEXAGON
> >         select HAVE_PERF_EVENTS
> >         # GENERIC_ALLOCATOR is used by dma_alloc_coherent()
> >         select GENERIC_ALLOCATOR
> > +       select GENERIC_IRQ_PROBE
> >         select GENERIC_IRQ_SHOW
> >         select HAVE_ARCH_KGDB
> >         select HAVE_ARCH_TRACEHOOK
> > @@ -60,9 +61,6 @@ config GENERIC_CSUM
> >  #
> >  # Use the generic interrupt handling code in kernel/irq/:
> >  #
> > -config GENERIC_IRQ_PROBE
> > -       def_bool y
> > -
> >  config GENERIC_HWEIGHT
> >         def_bool y
> >
> > --
> > 2.40.1
>
> Acked-by: Brian Cain <bcain@quicinc.com>
>


Thanks for the ack.


I will also delete the comment lines above:

#
# Use the generic interrupt handling code in kernel/irq/:
#


BTW, the reason why I submitted this to linux-kbuild ML
is I did not see any hexagon pull requests
in the past few years.

(I am doing the same for sparc, alpha as well)


--=20
Best Regards
Masahiro Yamada