[v2,1/2] rockchip: Add SoC detection helper

Message ID 20200704091503.71121-1-jagan@amarulasolutions.com
State New
Headers show
Series
  • [v2,1/2] rockchip: Add SoC detection helper
Related show

Commit Message

Jagan Teki July 4, 2020, 9:15 a.m. UTC
Rockchip SoC's has a new revision chip for rk3288, rk3308
based SoCs.

RK3288 has a new revision chip called RK3288W which is relevantly
similar but different hclk_vio clock and working ohci host.

Add common rockchip SoC detection helper to support this rk3288w
detection.

Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>
---
Changes for v2:
- new patch

 arch/arm/include/asm/arch-rockchip/cpu.h | 49 ++++++++++++++++++++++++
 1 file changed, 49 insertions(+)
 create mode 100644 arch/arm/include/asm/arch-rockchip/cpu.h

Comments

Kever Yang July 8, 2020, 10:13 a.m. UTC | #1
Hi Jagan,


     This is all for rk3288, please just put it into rk3288.c instead of 
make a new cpu.h.


Thanks,

- Kever

On 2020/7/4 下午5:15, Jagan Teki wrote:
> Rockchip SoC's has a new revision chip for rk3288, rk3308
> based SoCs.
>
> RK3288 has a new revision chip called RK3288W which is relevantly
> similar but different hclk_vio clock and working ohci host.
>
> Add common rockchip SoC detection helper to support this rk3288w
> detection.
>
> Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>
> ---
> Changes for v2:
> - new patch
>
>   arch/arm/include/asm/arch-rockchip/cpu.h | 49 ++++++++++++++++++++++++
>   1 file changed, 49 insertions(+)
>   create mode 100644 arch/arm/include/asm/arch-rockchip/cpu.h
>
> diff --git a/arch/arm/include/asm/arch-rockchip/cpu.h b/arch/arm/include/asm/arch-rockchip/cpu.h
> new file mode 100644
> index 0000000000..d0d22e4967
> --- /dev/null
> +++ b/arch/arm/include/asm/arch-rockchip/cpu.h
> @@ -0,0 +1,49 @@
> +/* SPDX-License-Identifier: GPL-2.0+  */
> +/*
> + * Rockchip Electronics Co., Ltd.
> + */
> +
> +#ifndef __ASM_ARCH_CPU_H
> +#define __ASM_ARCH_CPU_H
> +
> +#include <asm/io.h>
> +
> +#define ROCKCHIP_CPU_MASK       0xffff0000
> +#define ROCKCHIP_CPU_RK3288     0x32880000
> +
> +#define ROCKCHIP_SOC_MASK	(ROCKCHIP_CPU_MASK | 0xff)
> +#define ROCKCHIP_SOC_RK3288     (ROCKCHIP_CPU_RK3288 | 0x00)
> +#define ROCKCHIP_SOC_RK3288W    (ROCKCHIP_CPU_RK3288 | 0x01)
> +
> +#define RK3288_HDMI_PHYS	0xff980000
> +#define HDMI_CONFIG0_ID		0x4
> +#define RK3288W_HDMI_REVID	0x1a
> +
> +static inline int rockchip_soc_id(void)
> +{
> +	u8 reg;
> +
> +#if defined(CONFIG_ROCKCHIP_RK3288)
> +	reg = readb(RK3288_HDMI_PHYS + HDMI_CONFIG0_ID);
> +	if (reg == RK3288W_HDMI_REVID)
> +		return ROCKCHIP_SOC_RK3288W;
> +	else
> +		return ROCKCHIP_SOC_RK3288;
> +#else
> +	return 0;
> +#endif
> +}
> +
> +#define ROCKCHIP_SOC(id, ID) \
> +static inline bool soc_is_##id(void) \
> +{ \
> +	int soc_id = rockchip_soc_id(); \
> +	if (soc_id) \
> +		return ((soc_id & ROCKCHIP_SOC_MASK) == ROCKCHIP_SOC_ ##ID); \
> +	return false; \
> +}
> +
> +ROCKCHIP_SOC(rk3288, RK3288)
> +ROCKCHIP_SOC(rk3288w, RK3288W)
> +
> +#endif
Jagan Teki July 8, 2020, 10:29 a.m. UTC | #2
On Wed, Jul 8, 2020 at 3:43 PM Kever Yang <kever.yang@rock-chips.com> wrote:
>
> Hi Jagan,
>
>
>      This is all for rk3288, please just put it into rk3288.c instead of
> make a new cpu.h.

Yes, I have used cpu.h since rk3308 also has revision, if not to
consider than I will move to rk3288 specific.

Jagan.
Kever Yang July 18, 2020, 3:18 a.m. UTC | #3
On 2020/7/8 下午6:29, Jagan Teki wrote:
> On Wed, Jul 8, 2020 at 3:43 PM Kever Yang <kever.yang@rock-chips.com> wrote:
>> Hi Jagan,
>>
>>
>>       This is all for rk3288, please just put it into rk3288.c instead of
>> make a new cpu.h.
> Yes, I have used cpu.h since rk3308 also has revision, if not to
> consider than I will move to rk3288 specific.

Please do it, thanks.


Kever

>
> Jagan.
>
>

Patch

diff --git a/arch/arm/include/asm/arch-rockchip/cpu.h b/arch/arm/include/asm/arch-rockchip/cpu.h
new file mode 100644
index 0000000000..d0d22e4967
--- /dev/null
+++ b/arch/arm/include/asm/arch-rockchip/cpu.h
@@ -0,0 +1,49 @@ 
+/* SPDX-License-Identifier: GPL-2.0+  */
+/*
+ * Rockchip Electronics Co., Ltd.
+ */
+
+#ifndef __ASM_ARCH_CPU_H
+#define __ASM_ARCH_CPU_H
+
+#include <asm/io.h>
+
+#define ROCKCHIP_CPU_MASK       0xffff0000
+#define ROCKCHIP_CPU_RK3288     0x32880000
+
+#define ROCKCHIP_SOC_MASK	(ROCKCHIP_CPU_MASK | 0xff)
+#define ROCKCHIP_SOC_RK3288     (ROCKCHIP_CPU_RK3288 | 0x00)
+#define ROCKCHIP_SOC_RK3288W    (ROCKCHIP_CPU_RK3288 | 0x01)
+
+#define RK3288_HDMI_PHYS	0xff980000
+#define HDMI_CONFIG0_ID		0x4
+#define RK3288W_HDMI_REVID	0x1a
+
+static inline int rockchip_soc_id(void)
+{
+	u8 reg;
+
+#if defined(CONFIG_ROCKCHIP_RK3288)
+	reg = readb(RK3288_HDMI_PHYS + HDMI_CONFIG0_ID);
+	if (reg == RK3288W_HDMI_REVID)
+		return ROCKCHIP_SOC_RK3288W;
+	else
+		return ROCKCHIP_SOC_RK3288;
+#else
+	return 0;
+#endif
+}
+
+#define ROCKCHIP_SOC(id, ID) \
+static inline bool soc_is_##id(void) \
+{ \
+	int soc_id = rockchip_soc_id(); \
+	if (soc_id) \
+		return ((soc_id & ROCKCHIP_SOC_MASK) == ROCKCHIP_SOC_ ##ID); \
+	return false; \
+}
+
+ROCKCHIP_SOC(rk3288, RK3288)
+ROCKCHIP_SOC(rk3288w, RK3288W)
+
+#endif