On Sun, Sep 22, 2013 at 04:08:26PM -0500, Rob Herring wrote: > On Wed, Sep 18, 2013 at 8:24 AM, Thierry Reding > <[email protected]> wrote: > > Now that all helpers return precise error codes, this function can > > propagate these errors to the caller properly. > > > > Signed-off-by: Thierry Reding <[email protected]> > > --- > > Changes in v2: > > - return 0 on success or a negative error code on failure > > - convert callers to new calling convention > > [snip] > > > diff --git a/drivers/of/irq.c b/drivers/of/irq.c > > index e4f38c0..6d7f824 100644 > > --- a/drivers/of/irq.c > > +++ b/drivers/of/irq.c > > @@ -397,18 +397,20 @@ int of_irq_count(struct device_node *dev) > > * @res: array of resources to fill in > > * @nr_irqs: the number of IRQs (and upper bound for num of @res elements) > > You are effectively changing this to require an exact match rather > than an upper bound. That seems to be okay since that is what all the > callers want, but the documentation should be updated.
Done.
> > *
> > - * Returns the size of the filled in table (up to @nr_irqs).
> > + * Returns 0 on success or a negative error code on failure.
> > */
> > int of_irq_to_resource_table(struct device_node *dev, struct resource *res,
> > int nr_irqs)
> > {
> > - int i;
> > + int i, ret;
> >
> > - for (i = 0; i < nr_irqs; i++, res++)
> > - if (!of_irq_to_resource(dev, i, res))
>
> The error handling here needs to be updated in the previous patch.
Yes, you're right.
Thanks,
Thierry
pgpZnGWakZ0KF.pgp
Description: PGP signature
