On Sun, Sep 22, 2013 at 04:08:26PM -0500, Rob Herring wrote:
> On Wed, Sep 18, 2013 at 8:24 AM, Thierry Reding
> <thierry.reding@xxxxxxxxx> wrote:
> > Now that all helpers return precise error codes, this function can
> > propagate these errors to the caller properly.
> >
> > Signed-off-by: Thierry Reding <treding@xxxxxxxxxx>
> > ---
> > 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
Attachment:
pgpO4ufgTZNlg.pgp
Description: PGP signature
