Thanks, Shawn. All comments will be adopted in v2.
Peter
>> +"pll4_audio_div", "pll3_pfd2_508m", "pll5_video_div", "pll3_usb_otg",
>> +}; static const char *epdc_sels[] = { "epdc_podf", "ipp_di0",
>> +"ipp_di1", "ldb_di0", "ldb_di1", };
>> +
>
>The new line is not needed. There is already one below.
>
>>
>> static struct clk *clks[IMX6UL_CLK_END]; static struct
>> clk_onecell_data clk_data;
>
>> -#define IMX6UL_CLK_END 225
>> +/* For i.MX6ULL */
>> +#define IMX6UL_CLK_ESAI_PRED 225
>> +#define IMX6UL_CLK_ESAI_PODF 226
>> +#define IMX6UL_CLK_ESAI_EXTAL 227
>> +#define IMX6UL_CLK_ESAI_MEM 228
>> +#define IMX6UL_CLK_ESAI_IPG 229
>> +#define IMX6UL_CLK_DCP_CLK 230
>> +#define IMX6UL_CLK_EPDC_PRE_SEL 231
>> +#define IMX6UL_CLK_EPDC_SEL 232
>> +#define IMX6UL_CLK_EPDC_PODF 233
>> +#define IMX6UL_CLK_EPDC_ACLK 234
>> +#define IMX6UL_CLK_EPDC_PIX 235
>> +#define IMX6UL_CLK_ESAI_SEL 236
>
>Can we have these imx6ull only clocks named after IMX6ULL_xxx?
>
>Shawn
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
