linux-kernel - Re: [PATCH 05/06] sh: Let INTC set IRQF

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20100129025041.GB7396@linux-sh.org>
Date:	Fri, 29 Jan 2010 11:50:41 +0900
From:	Paul Mundt <lethal@...ux-sh.org>
To:	Magnus Damm <magnus.damm@...il.com>
Cc:	linux@....linux.org.uk, eric.y.miao@...il.com, tony@...mide.com,
	linux-kernel@...r.kernel.org, khilman@...prootsystems.com,
	ben-linux@...ff.org, linux-arm-kernel@...ts.infradead.org
Subject: Re: [PATCH 05/06] sh: Let INTC set IRQF_VALID on ARM platforms

On Thu, Jan 28, 2010 at 09:41:21PM +0900, Magnus Damm wrote:
> From: Magnus Damm <damm@...nsource.se>
> 
> Reuse the SuperH INTC code on ARM by using set_irq_flags()
> to set IRQF_VALID on ARM platforms. Reworking the shared ARM
> interrupt code and removing IRQF_VALID is another option.
> 
> Signed-off-by: Magnus Damm <damm@...nsource.se>
> ---
> 
>  drivers/sh/intc.c |    5 +++++
>  1 file changed, 5 insertions(+)
> 
> --- 0001/drivers/sh/intc.c
> +++ work/drivers/sh/intc.c	2010-01-21 19:53:15.000000000 +0900
> @@ -658,6 +658,11 @@ static void __init intc_register_irq(str
>  
>  	if (desc->ack_regs)
>  		ack_handle[irq] = intc_ack_data(desc, d, enum_id);
> +
> +	/* Set IRQF_VALID to enable IRQ on ARM systems */
> +#ifdef CONFIG_ARCH_SHMOBILE
> +	set_irq_flags(irq, IRQF_VALID);
> +#endif
>  }
>  
This should be CONFIG_ARM rather than ARCH_SHMOBILE, as it's hardly
SH-Mobile specific.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/