On 05/02/2014 02:29 PM, Kukjin Kim wrote: > Beomho Seo wrote: >> >> Hi all, >> >> Didn't have Any other comment for this patch set? >> >> On 04/25/2014 11:04 AM, Beomho Seo wrote: >>> This patchset add some device node for exynos4412-trats2. >>> It is based on v3.15-next/dt-samsung-2 branch. >>> >>> exynos4412-trats2.dts >>> - Fix incorrect compatible. Compatible of AK8975 are "ak8975" or "asahi- >> kasei,ak8975". >>> - Add cm36651 light/proximity sensor device node. >>> - Change gpio-key device node. fix incorrect gpio property, and then add >> "ok-key" node. >>> >>> Beomho Seo (3): >>> ARM: dts: fix incorrect compatible for exynos4412 >>> ARM: dts: add cm36651 light/proximity sensor node >>> ARM: dts: fixed gpio key node for exynos4412 >>> >>> arch/arm/boot/dts/exynos4412-trats2.dts | 50 >> ++++++++++++++++++++++++++----- >>> 1 file changed, 43 insertions(+), 7 deletions(-) >>> >> > I have no objection on this series but need to get ack from dt guys. > > BTW, shouldn't it include vendor name in compatible?... > > And according to dt bindings(iio/magnetometer/ak8975.txt), it should be: > > "compatible = "asahi-kasei,ak8975";" > > even though "ak8975" also included in the driver "drivers/iio/magnetometer/ak8975.c"... > > Thanks, > Kukjin > Thank you for comment. Compatible of ak8975 will be fixed according to dt bindings(iio/magnetometer/ak8975.txt). I will fix it and send v2 patch soon. Thanks, Beomho > -- > To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- 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