Re: [PATCH 2/3] smiapp: Read link-frequencies property from the endpoint node

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Sakari,

Thanks for the patch.

On Tue, Mar 10, 2015 at 1:18 AM, Sakari Ailus <sakari.ailus@xxxxxx> wrote:
> The documentation stated that the link-frequencies property belongs to the
> endpoint node, not to the device's of_node. Fix this.
>
> There are no DT board descriptions using the driver yet, so a fix in the
> driver is sufficient.
>
> Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxx>

Tested-by: Lad, Prabhakar <prabhakar.csengg@xxxxxxxxx>

Cheers,
--Prabhakar Lad

> ---
>  drivers/media/i2c/smiapp/smiapp-core.c |    5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/media/i2c/smiapp/smiapp-core.c b/drivers/media/i2c/smiapp/smiapp-core.c
> index 565a00c..ecae76b 100644
> --- a/drivers/media/i2c/smiapp/smiapp-core.c
> +++ b/drivers/media/i2c/smiapp/smiapp-core.c
> @@ -3022,8 +3022,7 @@ static struct smiapp_platform_data *smiapp_get_pdata(struct device *dev)
>         dev_dbg(dev, "reset %d, nvm %d, clk %d, csi %d\n", pdata->xshutdown,
>                 pdata->nvm_size, pdata->ext_clk, pdata->csi_signalling_mode);
>
> -       rval = of_get_property(
> -               dev->of_node, "link-frequencies", &asize) ? 0 : -ENOENT;
> +       rval = of_get_property(ep, "link-frequencies", &asize) ? 0 : -ENOENT;
>         if (rval) {
>                 dev_warn(dev, "can't get link-frequencies array size\n");
>                 goto out_err;
> @@ -3037,7 +3036,7 @@ static struct smiapp_platform_data *smiapp_get_pdata(struct device *dev)
>
>         asize /= sizeof(*pdata->op_sys_clock);
>         rval = of_property_read_u64_array(
> -               dev->of_node, "link-frequencies", pdata->op_sys_clock, asize);
> +               ep, "link-frequencies", pdata->op_sys_clock, asize);
>         if (rval) {
>                 dev_warn(dev, "can't get link-frequencies\n");
>                 goto out_err;
> --
> 1.7.10.4
>
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux