Hello Joonyoung, On 09/23/2015 08:48 AM, Joonyoung Shim wrote: > The sysmmu_fimd1_1 should bind the clock CLK_SMMU_FIMD1M1, not the clock > CLK_SMMU_FIMD1M0. CLK_SMMU_FIMD1M0 is a clock for the sysmmu_fimd1_0. > Can you please mention in the commit message if this patch is solving an issue on a given machine or if it is just for the sake of correctness. Git blame shows that this was added by commit b70045167815 ("ARM: dts: add sysmmu nodes for exynos5420") so please add a Fixes: tag. > Signed-off-by: Joonyoung Shim <jy0922.shim@xxxxxxxxxxx> > Cc: <stable@xxxxxxxxxxxxxxx> # v4.2 > --- Patch looks good to me. Reviewed-by: Javier Martinez Canillas <javier@xxxxxxxxxxxxxxx> Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html