diff mbox series

soc: samsung: exynos-chipid: describe which SoCs go with compatibles

Message ID 20211031205212.59505-1-krzysztof.kozlowski@canonical.com (mailing list archive)
State Accepted
Commit 569e45a1135497d8dddc647bc615e26c49b070a8
Headers show
Series soc: samsung: exynos-chipid: describe which SoCs go with compatibles | expand

Commit Message

Krzysztof Kozlowski Oct. 31, 2021, 8:52 p.m. UTC
The Exynos ChipID driver, like most of the Exynos drivers, uses one
compatible for entire family of compatible devices using one devicetree
"compatible".  The compatibility is here described by programming
interface (register layout), not by actual values, so the product ID
register on one family of devices has different values for different
SoCs.

Describe which SoC goes with which compatible for documentation
purposes, if the DTS is not available in mainline.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
---
 drivers/soc/samsung/exynos-chipid.c | 2 ++
 1 file changed, 2 insertions(+)

Comments

Krzysztof Kozlowski Nov. 15, 2021, 1:41 p.m. UTC | #1
On Sun, 31 Oct 2021 21:52:12 +0100, Krzysztof Kozlowski wrote:
> The Exynos ChipID driver, like most of the Exynos drivers, uses one
> compatible for entire family of compatible devices using one devicetree
> "compatible".  The compatibility is here described by programming
> interface (register layout), not by actual values, so the product ID
> register on one family of devices has different values for different
> SoCs.
> 
> [...]

Applied, thanks!

[1/1] soc: samsung: exynos-chipid: describe which SoCs go with compatibles
      commit: 569e45a1135497d8dddc647bc615e26c49b070a8

Best regards,
diff mbox series

Patch

diff --git a/drivers/soc/samsung/exynos-chipid.c b/drivers/soc/samsung/exynos-chipid.c
index a28053ec7e6a..baf3afc69d79 100644
--- a/drivers/soc/samsung/exynos-chipid.c
+++ b/drivers/soc/samsung/exynos-chipid.c
@@ -42,6 +42,7 @@  static const struct exynos_soc_id {
 	unsigned int id;
 } soc_ids[] = {
 	/* List ordered by SoC name */
+	/* Compatible with: samsung,exynos4210-chipid */
 	{ "EXYNOS3250", 0xE3472000 },
 	{ "EXYNOS4210", 0x43200000 },	/* EVT0 revision */
 	{ "EXYNOS4210", 0x43210000 },
@@ -55,6 +56,7 @@  static const struct exynos_soc_id {
 	{ "EXYNOS5440", 0xE5440000 },
 	{ "EXYNOS5800", 0xE5422000 },
 	{ "EXYNOS7420", 0xE7420000 },
+	/* Compatible with: samsung,exynos850-chipid */
 	{ "EXYNOS850", 0xE3830000 },
 	{ "EXYNOSAUTOV9", 0xAAA80000 },
 };