Hi Helen,
Just spotted a small thing.
On Fri, 2019-12-27 at 17:01 -0300, Helen Koike wrote:
[..]
> +static int rkisp1_probe(struct platform_device *pdev)
> +{
> + struct device_node *node = pdev->dev.of_node;
> + const struct rkisp1_match_data *clk_data;
> + const struct of_device_id *match;
> + struct device *dev = &pdev->dev;
> + struct rkisp1_device *rkisp1;
> + struct v4l2_device *v4l2_dev;
> + unsigned int i;
> + int ret, irq;
> +
> + match = of_match_node(rkisp1_of_match, node);
> + rkisp1 = devm_kzalloc(dev, sizeof(*rkisp1), GFP_KERNEL);
> + if (!rkisp1)
> + return -ENOMEM;
> +
> + dev_set_drvdata(dev, rkisp1);
> + rkisp1->dev = dev;
> +
> + rkisp1_debug_init(rkisp1);
> +
> + rkisp1->base_addr = devm_platform_ioremap_resource(pdev, 0);
> + if (IS_ERR(rkisp1->base_addr))
> + return PTR_ERR(rkisp1->base_addr);
> +
> + irq = platform_get_irq(pdev, 0);
> + if (irq < 0)
> + return irq;
> +
> + ret = devm_request_irq(dev, irq, rkisp1_isr, IRQF_SHARED,
> + dev_driver_string(dev), dev);
> + if (ret) {
> + dev_err(dev, "request irq failed: %d\n", ret);
> + return ret;
> + }
> +
> + rkisp1->irq = irq;
> + clk_data = match->data;
> +
> + for (i = 0; i < clk_data->size; i++)
> + rkisp1->clks[i].id = clk_data->clks[i];
> + ret = devm_clk_bulk_get(dev, clk_data->size, rkisp1->clks);
> + if (ret)
> + return ret;
> + rkisp1->clk_size = clk_data->size;
> +
> + pm_runtime_enable(&pdev->dev);
> +
> + strscpy(rkisp1->media_dev.model, RKISP1_DRIVER_NAME,
> + sizeof(rkisp1->media_dev.model));
> + rkisp1->media_dev.dev = &pdev->dev;
> + strscpy(rkisp1->media_dev.bus_info,
> + "platform: " RKISP1_DRIVER_NAME,
> + sizeof(rkisp1->media_dev.bus_info));
> + media_device_init(&rkisp1->media_dev);
> +
> + v4l2_dev = &rkisp1->v4l2_dev;
> + v4l2_dev->mdev = &rkisp1->media_dev;
> + strscpy(v4l2_dev->name, RKISP1_DRIVER_NAME, sizeof(v4l2_dev->name));
> +
> + ret = v4l2_device_register(rkisp1->dev, &rkisp1->v4l2_dev);
> + if (ret)
> + return ret;
> +
> + ret = media_device_register(&rkisp1->media_dev);
> + if (ret) {
> + dev_err(dev, "Failed to register media device: %d\n", ret);
> + goto err_unreg_v4l2_dev;
> + }
> +
> + ret = rkisp1_entities_register(rkisp1);
> + if (ret)
> + goto err_unreg_media_dev;
> +
> + return 0;
> +
> +err_unreg_media_dev:
> + media_device_unregister(&rkisp1->media_dev);
> +err_unreg_v4l2_dev:
> + v4l2_device_unregister(&rkisp1->v4l2_dev);
> + pm_runtime_disable(&pdev->dev);
There's a missing call to debugfs_remove_recursive here.
> + return ret;
> +}
> +
> +static int rkisp1_remove(struct platform_device *pdev)
> +{
> + struct rkisp1_device *rkisp1 = platform_get_drvdata(pdev);
> +
> + v4l2_async_notifier_unregister(&rkisp1->notifier);
> + v4l2_async_notifier_cleanup(&rkisp1->notifier);
> +
> + rkisp1_isp_unregister(rkisp1);
> +
> + media_device_unregister(&rkisp1->media_dev);
> + v4l2_device_unregister(&rkisp1->v4l2_dev);
> +
> + pm_runtime_disable(&pdev->dev);
> +
> + debugfs_remove_recursive(rkisp1->debug.debugfs_dir);
> + return 0;
> +}
> +
Thanks,
Ezequiel
