* [PATCH] arm64: irq: include <linux/cpumask.h>
@ 2024-01-09 14:04 Tudor Ambarus
2024-01-09 14:46 ` Mark Rutland
0 siblings, 1 reply; 3+ messages in thread
From: Tudor Ambarus @ 2024-01-09 14:04 UTC (permalink / raw)
To: catalin.marinas, will
Cc: sumit.garg, dianders, mark.rutland, linux-arm-kernel,
linux-kernel, kernel-team, andre.draszik, willmcvicker,
peter.griffin, Tudor Ambarus
Sorting include files in alphabetic order in
drivers/tty/serial/samsung.c revealed the following error:
In file included from drivers/tty/serial/samsung_tty.c:24:
./arch/arm64/include/asm/irq.h:9:43: error: unknown type name ‘cpumask_t’
9 | void arch_trigger_cpumask_backtrace(const cpumask_t *mask, int exclude_cpu);
| ^~~~~~~~~
Include cpumask.h to avod unknown type errors for parents of irq.h that
don't include cpumask.h.
Signed-off-by: Tudor Ambarus <tudor.ambarus@linaro.org>
---
arch/arm64/include/asm/irq.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm64/include/asm/irq.h b/arch/arm64/include/asm/irq.h
index 50ce8b697ff3..d5612bc770da 100644
--- a/arch/arm64/include/asm/irq.h
+++ b/arch/arm64/include/asm/irq.h
@@ -5,6 +5,7 @@
#ifndef __ASSEMBLER__
#include <asm-generic/irq.h>
+#include <linux/cpumask.h>
void arch_trigger_cpumask_backtrace(const cpumask_t *mask, int exclude_cpu);
#define arch_trigger_cpumask_backtrace arch_trigger_cpumask_backtrace
--
2.43.0.472.g3155946c3a-goog
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] arm64: irq: include <linux/cpumask.h>
2024-01-09 14:04 [PATCH] arm64: irq: include <linux/cpumask.h> Tudor Ambarus
@ 2024-01-09 14:46 ` Mark Rutland
2024-01-09 16:16 ` Tudor Ambarus
0 siblings, 1 reply; 3+ messages in thread
From: Mark Rutland @ 2024-01-09 14:46 UTC (permalink / raw)
To: Tudor Ambarus
Cc: catalin.marinas, will, sumit.garg, dianders, linux-arm-kernel,
linux-kernel, kernel-team, andre.draszik, willmcvicker,
peter.griffin
On Tue, Jan 09, 2024 at 02:04:37PM +0000, Tudor Ambarus wrote:
> Sorting include files in alphabetic order in
> drivers/tty/serial/samsung.c revealed the following error:
>
> In file included from drivers/tty/serial/samsung_tty.c:24:
> ./arch/arm64/include/asm/irq.h:9:43: error: unknown type name 'cpumask_t'
> 9 | void arch_trigger_cpumask_backtrace(const cpumask_t *mask, int exclude_cpu);
> | ^~~~~~~~~
>
> Include cpumask.h to avod unknown type errors for parents of irq.h that
> don't include cpumask.h.
s/avod/avoid/
> Signed-off-by: Tudor Ambarus <tudor.ambarus@linaro.org>
> ---
> arch/arm64/include/asm/irq.h | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm64/include/asm/irq.h b/arch/arm64/include/asm/irq.h
> index 50ce8b697ff3..d5612bc770da 100644
> --- a/arch/arm64/include/asm/irq.h
> +++ b/arch/arm64/include/asm/irq.h
> @@ -5,6 +5,7 @@
> #ifndef __ASSEMBLER__
>
> #include <asm-generic/irq.h>
> +#include <linux/cpumask.h>
Minor nit: we usually have the <linux/*.h> headers first, then a line space,
then the <asm/*.h> headers, e.g.
| #include <linux/cpumask.h>
|
| #include <asm-generic/irq.h>
With those changes, the patch itself looks good to me:
Acked-by: Mark Rutland <mark.rutland@arm.com>
Mark.
> void arch_trigger_cpumask_backtrace(const cpumask_t *mask, int exclude_cpu);
> #define arch_trigger_cpumask_backtrace arch_trigger_cpumask_backtrace
> --
> 2.43.0.472.g3155946c3a-goog
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] arm64: irq: include <linux/cpumask.h>
2024-01-09 14:46 ` Mark Rutland
@ 2024-01-09 16:16 ` Tudor Ambarus
0 siblings, 0 replies; 3+ messages in thread
From: Tudor Ambarus @ 2024-01-09 16:16 UTC (permalink / raw)
To: Mark Rutland
Cc: catalin.marinas, will, sumit.garg, dianders, linux-arm-kernel,
linux-kernel, kernel-team, andre.draszik, willmcvicker,
peter.griffin
On 1/9/24 14:46, Mark Rutland wrote:
> On Tue, Jan 09, 2024 at 02:04:37PM +0000, Tudor Ambarus wrote:
>> Sorting include files in alphabetic order in
>> drivers/tty/serial/samsung.c revealed the following error:
>>
>> In file included from drivers/tty/serial/samsung_tty.c:24:
>> ./arch/arm64/include/asm/irq.h:9:43: error: unknown type name 'cpumask_t'
>> 9 | void arch_trigger_cpumask_backtrace(const cpumask_t *mask, int exclude_cpu);
>> | ^~~~~~~~~
>>
>> Include cpumask.h to avod unknown type errors for parents of irq.h that
>> don't include cpumask.h.
>
> s/avod/avoid/
ah, will fix, thanks.
>
>> Signed-off-by: Tudor Ambarus <tudor.ambarus@linaro.org>
>> ---
>> arch/arm64/include/asm/irq.h | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/arch/arm64/include/asm/irq.h b/arch/arm64/include/asm/irq.h
>> index 50ce8b697ff3..d5612bc770da 100644
>> --- a/arch/arm64/include/asm/irq.h
>> +++ b/arch/arm64/include/asm/irq.h
>> @@ -5,6 +5,7 @@
>> #ifndef __ASSEMBLER__
>>
>> #include <asm-generic/irq.h>
>> +#include <linux/cpumask.h>
>
> Minor nit: we usually have the <linux/*.h> headers first, then a line space,
> then the <asm/*.h> headers, e.g.
I wasn't aware of this habit. Will update accordingly in v2.
Thanks!
ta
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2024-01-09 16:16 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-01-09 14:04 [PATCH] arm64: irq: include <linux/cpumask.h> Tudor Ambarus
2024-01-09 14:46 ` Mark Rutland
2024-01-09 16:16 ` Tudor Ambarus
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).