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

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 at 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

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 at 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

On Wed, Jul 8, 2020 at 3:43 PM Kever Yang <kever.yang at 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.