On 07/02/2015 09:01 AM, Krzysztof Kozlowski wrote: > On 01.07.2015 22:21, Chanwoo Choi wrote: >> This patch add CPU operating points which include CPU frequency and regulator >> voltage to use generic cpufreq drivers. >> >> Cc: Kukjin Kim <kgene@xxxxxxxxxx> >> Cc: Krzysztof Kozlowski <k.kozlowski@xxxxxxxxxxx> >> Signed-off-by: Chanwoo Choi <cw00.choi@xxxxxxxxxxx> >> Acked-by: Kyungmin Park <kyungmin.park@xxxxxxxxxxx> >> --- >> arch/arm/boot/dts/exynos3250-monk.dts | 4 ++++ >> arch/arm/boot/dts/exynos3250-rinato.dts | 4 ++++ >> arch/arm/boot/dts/exynos3250.dtsi | 15 +++++++++++++++ >> 3 files changed, 23 insertions(+) > > The patch looks good. Could you put the 'cpu' nodes in alphabetical > order? The order for Rinato board is already broken but still putting it > after 'adc' node would be better. > > With the change above: > Reviewed-by: Krzysztof Kozlowski <k.kozlowski@xxxxxxxxxxx> OK. I'll resend it. Thanks, Chanwoo Choi -- 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