On Tue, 28 Jun 2016, SF Markus Elfring wrote: > >> @@ -222,19 +222,21 @@ static struct device *add_child(struct i2c_client *client, const char *name, > >> status = platform_device_add_resources(pdev, &r, 1); > >> if (status < 0) { > >> dev_dbg(&pdev->dev, "can't add irq\n"); > >> - goto err; > >> + goto put_device; > >> } > >> } > >> > >> status = platform_device_add(pdev); > >> > > > > Remove this line too. > > Do you propose the deletion of a blank line here? Yes. > Did you skip this update suggestion while the other patches were finally accepted? I don't know what this means. The other patches in the set have been accepted. Please fix this one and send it again on its own. -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html