Hi Geert,

Thanks for your feedback.

On 2019-02-18 10:07:04 +0100, Geert Uytterhoeven wrote:
> Hi Niklas,
> 
> On Sun, Feb 17, 2019 at 8:54 AM Niklas Söderlund
> <niklas.soderlund+rene...@ragnatech.se> wrote:
> > Later versions of the datasheet updates the reset procedure to more
> > closely resemble the standby mode. Update the driver to enter and exit
> > the standby mode instead of resetting the hardware before and after
> > streaming is started and stopped.
> >
> > While at it break out the full start and stop procedures from
> > rcsi2_s_stream() into the existing helper functions.
> >
> > Signed-off-by: Niklas Söderlund <niklas.soderlund+rene...@ragnatech.se>
> 
> Thanks for your patch!
> 
> > --- a/drivers/media/platform/rcar-vin/rcar-csi2.c
> > +++ b/drivers/media/platform/rcar-vin/rcar-csi2.c
> 
> > @@ -936,6 +947,10 @@ static int rcsi2_probe_resources(struct rcar_csi2 
> > *priv,
> >         if (irq < 0)
> >                 return irq;
> >
> > +       priv->rstc = devm_reset_control_get(&pdev->dev, NULL);
> > +       if (IS_ERR(priv->rstc))
> > +               return PTR_ERR(priv->rstc);
> > +
> >         return 0;
> 
> Does the driver Kconfig option need "select RESET_CONTROLLER"?
> If the option is not enabled, devm_reset_control_get() will return -ENOTSUPP.

Yes it does, thanks for pointing this out, will add this in v2.

> 
> Gr{oetje,eeting}s,
> 
>                         Geert
> 
> -- 
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- 
> ge...@linux-m68k.org
> 
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like 
> that.
>                                 -- Linus Torvalds

-- 
Regards,
Niklas Söderlund

Reply via email to