lkml.org 
[lkml]   [2018]   [Oct]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH v4 2/4] crypto: mxs-dcp - Add support for dcp clk
Date
On Wed, 2018-10-17 at 09:48 -0300, Fabio Estevam wrote:
> On Wed, Oct 17, 2018 at 9:38 AM Leonard Crestez <leonard.crestez@nxp.com> wrote:
> >
> > On 6ull and 6sll the DCP block has a clock which needs to be explicitly
> > enabled.
> >
> > Add minimal handling for this at probe/remove time.
> >
> > Signed-off-by: Leonard Crestez <leonard.crestez@nxp.com>
>
> Please always explain what changed from the previous patch version.

There is a changelog in the cover letter.

> > + /* Restart the DCP block. */
> > + ret = stmp_reset_block(sdcp->base);
> > + if (ret) {
> > + dev_err(dev, "Failed reset\n");
> > + goto err_disable_unprepare_clk;
> > + }
>
> This seems like an unrelated change that should be in a separate patch.
>
> Also, this was not present in v3, so it is better to explain that this
> has been introduced in v4.

This only looks slightly odd in git diff but it's not unrelated.

I placed clk get/prepare/enable just before stmp_reset_block and made
stmp_reset_block print a message and goto err_disable_unprepare_clk on
failure instead of just "if (ret) return ret;"
\
 
 \ /
  Last update: 2018-10-17 15:00    [W:0.046 / U:3.508 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site