On Sat, 2014-11-29 at 18:40 +0100, Beniamino Galvani wrote:
> On Tue, Nov 25, 2014 at 04:04:20PM +0800, Yingjoe Chen wrote:
> > Mediatek SoCs have interrupt polarity support in sysirq which
> > allows to invert polarity for given interrupt. Add this support
> > using hierarchy irq domain.
> >
> > [...]
> >
> > +static int __init mtk_sysirq_of_init(struct device_node *node,
> > + struct device_node *parent)
> > +{
> > + struct irq_domain *domain, *domain_parent;
> > + struct mtk_sysirq_chip_data *chip_data;
> > + int ret = 0;
> > +
> > + domain_parent = irq_find_host(parent);
> > + if (!domain_parent) {
> > + pr_err("mtk_sysirq: interrupt-parent not found\n");
> > + return -EINVAL;
> > + }
> > +
> > + chip_data = kzalloc(sizeof(*chip_data), GFP_KERNEL);
> > + if (!chip_data)
> > + return -ENOMEM;
> > +
> > + chip_data->intpol_base = of_io_request_and_map(node, 0, "intpol");
> > + if (!chip_data->intpol_base) {
>
> Hi,
>
> you should use IS_ERR() to check the return value here.
Thanks for catching this. I think this is merged, so I'll prepare a new
patch to fix this.
Joe.C
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
