lkml.org 
[lkml]   [2018]   [Mar]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH] memory: brcmstb: dpfe: Remove need for dpfe_dev
On 27 March 2018 at 16:40, Florian Fainelli <f.fainelli@gmail.com> wrote:
> We can hook sysfs objects to the parent platform device that we are
> created from, no need to have a synthetic dpfe_dev just for that. This
> incidentally removes the need for having an index, since we are
> guaranteed to have an unique path in the sysfs hiearchy.
>
> Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>

Looks good. Thanks.

Acked-by: Markus Mayer <mmayer@broadcom.com>

> ---
> drivers/memory/brcmstb_dpfe.c | 42 ++++++++++--------------------------------
> 1 file changed, 10 insertions(+), 32 deletions(-)
>
> diff --git a/drivers/memory/brcmstb_dpfe.c b/drivers/memory/brcmstb_dpfe.c
> index 0a7bdbed3a6f..5f8d5aff6040 100644
> --- a/drivers/memory/brcmstb_dpfe.c
> +++ b/drivers/memory/brcmstb_dpfe.c
> @@ -168,7 +168,6 @@ struct private_data {
> void __iomem *dmem;
> void __iomem *imem;
> struct device *dev;
> - unsigned int index;
> struct mutex lock;
> };
>
> @@ -628,10 +627,8 @@ static int brcmstb_dpfe_probe(struct platform_device *pdev)
> {
> struct device *dev = &pdev->dev;
> struct private_data *priv;
> - struct device *dpfe_dev;
> struct init_data init;
> struct resource *res;
> - u32 index;
> int ret;
>
> priv = devm_kzalloc(dev, sizeof(*priv), GFP_KERNEL);
> @@ -641,11 +638,6 @@ static int brcmstb_dpfe_probe(struct platform_device *pdev)
> mutex_init(&priv->lock);
> platform_set_drvdata(pdev, priv);
>
> - /* Cell index is optional; default to 0 if not present. */
> - ret = of_property_read_u32(dev->of_node, "cell-index", &index);
> - if (ret)
> - index = 0;
> -
> res = platform_get_resource_byname(pdev, IORESOURCE_MEM, "dpfe-cpu");
> priv->regs = devm_ioremap_resource(dev, res);
> if (IS_ERR(priv->regs)) {
> @@ -669,35 +661,20 @@ static int brcmstb_dpfe_probe(struct platform_device *pdev)
>
> ret = brcmstb_dpfe_download_firmware(pdev, &init);
> if (ret)
> - goto err;
> -
> - dpfe_dev = devm_kzalloc(dev, sizeof(*dpfe_dev), GFP_KERNEL);
> - if (!dpfe_dev) {
> - ret = -ENOMEM;
> - goto err;
> - }
> -
> - priv->dev = dpfe_dev;
> - priv->index = index;
> + return ret;
>
> - dpfe_dev->parent = dev;
> - dpfe_dev->groups = dpfe_groups;
> - dpfe_dev->of_node = dev->of_node;
> - dev_set_drvdata(dpfe_dev, priv);
> - dev_set_name(dpfe_dev, "dpfe%u", index);
> + ret = sysfs_create_groups(&pdev->dev.kobj, dpfe_groups);
> + if (!ret)
> + dev_info(dev, "registered.\n");
>
> - ret = device_register(dpfe_dev);
> - if (ret)
> - goto err;
> + return ret;
> +}
>
> - dev_info(dev, "registered.\n");
> +static int brcmstb_dpfe_remove(struct platform_device *pdev)
> +{
> + sysfs_remove_groups(&pdev->dev.kobj, dpfe_groups);
>
> return 0;
> -
> -err:
> - dev_err(dev, "failed to initialize -- error %d\n", ret);
> -
> - return ret;
> }
>
> static const struct of_device_id brcmstb_dpfe_of_match[] = {
> @@ -712,6 +689,7 @@ static struct platform_driver brcmstb_dpfe_driver = {
> .of_match_table = brcmstb_dpfe_of_match,
> },
> .probe = brcmstb_dpfe_probe,
> + .remove = brcmstb_dpfe_remove,
> .resume = brcmstb_dpfe_resume,
> };
>
> --
> 2.14.1
>

\
 
 \ /
  Last update: 2018-03-28 03:57    [W:0.072 / U:1.104 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site