Hi Scott,

-----Original Message-----
From: Scott Wood
Sent: Thursday, November 24, 2016 4:21 PM
To: Y.T. Tang <redacted>; mturquette at baylibre.com
Cc: sboyd at codeaurora.org; linux-clk at vger.kernel.org; linux-
kernel at vger.kernel.org; linux-arm-kernel at lists.infradead.org
Subject: Re: [PATCH v3] clk: qoriq: added ls1012a clock configuration

On 11/23/2016 08:50 PM, yuantian.tang at nxp.com wrote:

quoted

From: Tang Yuantian <redacted>

Signed-off-by: Tang Yuantian <redacted>
---
v3:
  - rebased to latest kernel and re-sorted the code

[snip]

quoted

@@ -1316,6 +1334,7 @@ CLK_OF_DECLARE(qoriq_clockgen_2,

"fsl,qoriq-clockgen-2.0", clockgen_init);
CLK_OF_DECLARE(qoriq_clockgen_ls1021a, "fsl,ls1021a-clockgen",
clockgen_init);  CLK_OF_DECLARE(qoriq_clockgen_ls1043a,
"fsl,ls1043a-clockgen", clockgen_init);
CLK_OF_DECLARE(qoriq_clockgen_ls1046a, "fsl,ls1046a-clockgen",
clockgen_init);
+CLK_OF_DECLARE(qoriq_clockgen_ls1012a, "fsl,ls1012a-clockgen",
+clockgen_init);
 CLK_OF_DECLARE(qoriq_clockgen_ls2080a, "fsl,ls2080a-clockgen",
clockgen_init);

You need a better sorting algorithm. :-P

I don't think so. This time, each code snip goes right after ls1046a.
The sorting 'algorithm' is best ever. :)

Regards,
Yuantian
 

-Scott