[RFC PATCH] arm: vt8500: Convert irq.c for multiplatform integration

Tony Prisk linux at prisktech.co.nz
Wed Oct 10 13:51:35 EDT 2012


On Wed, 2012-10-10 at 08:35 +0000, Arnd Bergmann wrote:
> On Wednesday 10 October 2012, Tony Prisk wrote:
> > This patch converts arch-vt8500/irq.c to MULTI_IRQ_HANDLER and
> > SPARSE_IRQ. IRQ domain is changed from legacy to linear.
> > 
> > Also, remove legacy code in include/mach/entry-macro.S and
> > include/mach/irq.h to prepare for multiplatform.
> > 
> > Signed-off-by: Tony Prisk <linux at prisktech.co.nz>
> 
> Excellent!
> 
> Acked-by: Arnd Bergmann <arnd at arndb.de>
> 
> One small style comment for a preexisting issue I had not noticed before:
> 
> >  static void vt8500_irq_mask(struct irq_data *d)
> >  {
> > -	struct vt8500_irq_priv *priv =
> > -			(struct vt8500_irq_priv *)(d->domain->host_data);
> > +	struct vt8500_irq_data *priv = 
> > +			(struct vt8500_irq_data *)(d->domain->host_data);
> 
> host_data is a void pointer, so you don't need the type cast. Writing this
> as 
> 
> 	struct vt8500_irq_priv *priv = d->domain->host_data;
> 
> is both the common convention and easier to read.
> 
> 	Arnd
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

I seem to have a thing about casting pointers - this must be the 3rd-4th
time you've mentioned different occasions where I've unnecessarily cast
a void pointer.

Must do better! :)

Changes made locally as requested - Should have the rest of the
multiplatform change done shortly.. then roll on -rc1....

Regards
Tony P




More information about the linux-arm-kernel mailing list