Re: [PATCH v4] of: replace Asahi Kasei Corp venter prefix
Hello. On 11/17/2014 3:51 AM, Kuninori Morimoto wrote: From: Kuninori Morimoto s/venter/vendor/ in the subject. Sorry for a late comment. Current vendor-prefixes.txt already has "ak" prefix for Asahi Kasei Corp by ae8c4209af2c(of: Add vendor prefix for Asahi Kasei Corp.) It went through the appropriate review process. But, almost all Asahi Kasei chip drivers are using "asahi-kasei" prefix today. (arch/arm/boot/dts/tegra20-seaboard.dts only is using "ak,ak8975", but there are instances of "asahi-kasei,ak8975" in other dts files. And drivers/iio/magnetometer/ak8975.c doesn't support "ak,ak8975") So, we made a mistake there. In addition, checkpatch.pl reports WARNING if it is using "asahi-kasei" prerfix in DT file. (DT compatible string vendor "asahi-kasei" appears un-documented) Marking it deprecated and warning with checkpatch is certainly preferable. So, this patch replace "ak" to "asahi-kasei" in vendor-prefixes.txt. (and fixup tegra20-seaboard) OTOH, Asahi Kasei is usually referred to as "AKM", but this patch doesn't care about it. Because no DT is using that today. Acked-by: Thierry Reding Acked-by: Alexandre Courbot Signed-off-by: Kuninori Morimoto One normally places ACKs after his own sign-off. WBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH v4] of: replace Asahi Kasei Corp venter prefix
Hello. On 11/17/2014 3:51 AM, Kuninori Morimoto wrote: From: Kuninori Morimoto kuninori.morimoto...@renesas.com s/venter/vendor/ in the subject. Sorry for a late comment. Current vendor-prefixes.txt already has ak prefix for Asahi Kasei Corp by ae8c4209af2c(of: Add vendor prefix for Asahi Kasei Corp.) It went through the appropriate review process. But, almost all Asahi Kasei chip drivers are using asahi-kasei prefix today. (arch/arm/boot/dts/tegra20-seaboard.dts only is using ak,ak8975, but there are instances of asahi-kasei,ak8975 in other dts files. And drivers/iio/magnetometer/ak8975.c doesn't support ak,ak8975) So, we made a mistake there. In addition, checkpatch.pl reports WARNING if it is using asahi-kasei prerfix in DT file. (DT compatible string vendor asahi-kasei appears un-documented) Marking it deprecated and warning with checkpatch is certainly preferable. So, this patch replace ak to asahi-kasei in vendor-prefixes.txt. (and fixup tegra20-seaboard) OTOH, Asahi Kasei is usually referred to as AKM, but this patch doesn't care about it. Because no DT is using that today. Acked-by: Thierry Reding tred...@nvidia.com Acked-by: Alexandre Courbot acour...@nvidia.com Signed-off-by: Kuninori Morimoto kuninori.morimoto...@renesas.com One normally places ACKs after his own sign-off. WBR, Sergei -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/