Messages in this thread | | | From | "R, Sricharan" <> | Subject | RE: [PATCH V2 2/7] DRIVERS: IRQCHIP: CROSSBAR: Add support for Crossbar IP | Date | Thu, 31 Oct 2013 10:57:47 +0000 |
| |
Hi Thomas, Again sorry for top post.
I agree and will fix both of the comments below.
Thanks for the reviews.
Regards, Sricharan ________________________________________ From: Thomas Gleixner [tglx@linutronix.de] Sent: Wednesday, October 30, 2013 9:00 PM To: R, Sricharan Cc: linux-kernel@vger.kernel.org; devicetree@vger.kernel.org; linux-doc@vger.kernel.org; linux-arm-kernel@lists.infradead.org; linux-omap@vger.kernel.org; linus.walleij@linaro.org; linux@arm.linux.org.uk; tony@atomide.com; Nayak, Rajendra; marc.zyngier@arm.com; grant.likely@linaro.org; mark.rutland@arm.com; robherring2@gmail.com; Shilimkar, Santosh; Rob Herring Subject: Re: [PATCH V2 2/7] DRIVERS: IRQCHIP: CROSSBAR: Add support for Crossbar IP
On Wed, 30 Oct 2013, Sricharan R wrote: > +static inline const u32 allocate_free_irq(int cb_no)
I understand the "static inline" part, but "const u32" is more than fishy. What's wrong with "static inline int" ?
> +{ > + int i; > + > + for (i = 0; i < cb->int_max; i++) { > + if (cb->irq_map[i] == IRQ_FREE) { > + cb->irq_map[i] = cb_no; > + return i; > + } > + } > + > + return -ENODEV; > +}
> +static int crossbar_domain_xlate(struct irq_domain *d, > + struct device_node *controller, > + const u32 *intspec, unsigned int intsize, > + unsigned long *out_hwirq, > + unsigned int *out_type) > +{ > + unsigned long ret = 0;
Why do you need to initialize ret when you assign a value to it in the next line?
Thanks,
tglx
| |