Re: [PATCH v1] clk: tegra: Fix duplicated SE clock entry

Thierry Reding
2020-12-10T08:03:47Z

[-- Attachment #1: Type: text/plain, Size: 817 bytes --]

On Wed, Dec 09, 2020 at 05:27:10PM -0800, Stephen Boyd wrote:
> Quoting Dmitry Osipenko (2020-10-25 15:42:12)
> > The periph_clks[] array contains duplicated entry for Security Engine
> > clock which was meant to be defined for T210, but it wasn't added
> > properly. This patch corrects the T210 SE entry and fixes the following
> > error message on T114/T124: "Tegra clk 127: register failed with -17".
> >
> > Fixes: dc37fec48314 ("clk: tegra: periph: Add new periph clks and muxes for Tegra210")
> > Tested-by Nicolas Chauvet <kwizart@gmail.com>
> > Reported-by Nicolas Chauvet <kwizart@gmail.com>
> > Signed-off-by: Dmitry Osipenko <digetx@gmail.com>
> > ---
>
> Looks correct. Should I pick this up Thierry?

Yes, please. Thanks for catching this, it had slipped through my filter.

Thierry

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]