[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <ZvJnC7hHISkbOo2n@yilunxu-OptiPlex-7050>
Date: Tue, 24 Sep 2024 15:15:23 +0800
From: Xu Yilun <yilun.xu@...ux.intel.com>
To: Peter Colberg <peter.colberg@...el.com>
Cc: Wu Hao <hao.wu@...el.com>, Tom Rix <trix@...hat.com>,
Moritz Fischer <mdf@...nel.org>, Xu Yilun <yilun.xu@...el.com>,
linux-fpga@...r.kernel.org, linux-kernel@...r.kernel.org,
Russ Weight <russ.weight@...ux.dev>,
Marco Pagani <marpagan@...hat.com>,
Matthew Gerlach <matthew.gerlach@...ux.intel.com>
Subject: Re: [PATCH v3 6/9] fpga: dfl: factor out feature data creation from
build_info_commit_dev()
On Thu, Sep 19, 2024 at 04:34:27PM -0400, Peter Colberg wrote:
> Add a separate function binfo_create_feature_dev_data() which allocates
> and populates the feature platform data, and call the function from
> build_info_commit_dev() which registers the feature platform device.
>
> Signed-off-by: Peter Colberg <peter.colberg@...el.com>
> Reviewed-by: Matthew Gerlach <matthew.gerlach@...ux.intel.com>
> ---
> drivers/fpga/dfl.c | 74 ++++++++++++++++++++++++++--------------------
> 1 file changed, 42 insertions(+), 32 deletions(-)
>
> diff --git a/drivers/fpga/dfl.c b/drivers/fpga/dfl.c
> index 4c79d433d216..e644eb9fde39 100644
> --- a/drivers/fpga/dfl.c
> +++ b/drivers/fpga/dfl.c
> @@ -749,12 +749,8 @@ static void dfl_fpga_cdev_add_port_data(struct dfl_fpga_cdev *cdev,
> mutex_unlock(&cdev->lock);
> }
>
> -/*
> - * register current feature device, it is called when we need to switch to
> - * another feature parsing or we have parsed all features on given device
> - * feature list.
> - */
> -static int build_info_commit_dev(struct build_feature_devs_info *binfo)
> +static struct dfl_feature_platform_data *
> +binfo_create_feature_dev_data(struct build_feature_devs_info *binfo)
> {
> struct platform_device *fdev = binfo->feature_dev;
> struct dfl_feature_platform_data *pdata;
> @@ -764,7 +760,7 @@ static int build_info_commit_dev(struct build_feature_devs_info *binfo)
>
> type = feature_dev_id_type(fdev);
> if (WARN_ON_ONCE(type >= DFL_ID_MAX))
> - return -EINVAL;
> + return ERR_PTR(-EINVAL);
>
> /*
> * we do not need to care for the memory which is associated with
> @@ -774,7 +770,7 @@ static int build_info_commit_dev(struct build_feature_devs_info *binfo)
> */
> pdata = kzalloc(struct_size(pdata, features, binfo->feature_num), GFP_KERNEL);
> if (!pdata)
> - return -ENOMEM;
> + return ERR_PTR(-ENOMEM);
>
> pdata->dev = fdev;
> pdata->num = binfo->feature_num;
> @@ -799,7 +795,7 @@ static int build_info_commit_dev(struct build_feature_devs_info *binfo)
> fdev->resource = kcalloc(binfo->feature_num, sizeof(*fdev->resource),
> GFP_KERNEL);
> if (!fdev->resource)
> - return -ENOMEM;
> + return ERR_PTR(-ENOMEM);
>
> /* fill features and resource information for feature dev */
> list_for_each_entry_safe(finfo, p, &binfo->sub_features, node) {
> @@ -818,7 +814,7 @@ static int build_info_commit_dev(struct build_feature_devs_info *binfo)
> finfo->params, finfo->param_size,
> GFP_KERNEL);
> if (!feature->params)
> - return -ENOMEM;
> + return ERR_PTR(-ENOMEM);
>
> feature->param_size = finfo->param_size;
> }
> @@ -834,8 +830,10 @@ static int build_info_commit_dev(struct build_feature_devs_info *binfo)
> feature->ioaddr =
> devm_ioremap_resource(binfo->dev,
> &finfo->mmio_res);
> - if (IS_ERR(feature->ioaddr))
> - return PTR_ERR(feature->ioaddr);
> + if (IS_ERR(feature->ioaddr)) {
> + ret = PTR_ERR(feature->ioaddr);
> + return ERR_PTR(ret);
Why change the type back and forth?
return feature->ioaddr;
is it OK?
Thanks,
Yilun
Powered by blists - more mailing lists