Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1346413

Re: [PATCH v2 13/13] ARM: sunxi: Introduce MACH_SUN8I_A83T option

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

Hi,

On Sun, Feb 28, 2016 at 11:18:58PM +0800, Vishnu Patekar wrote:
> A83T has CPUS clock similar to A80's. currently, a80 cpus clock only
> compiled for A80. So, Introduce MACH_SUN8I_A83T to compile it for
> A83T as well.
> 
> Signed-off-by: Vishnu Patekar <vishnupatekar0510@gmail.com>
> ---
>  arch/arm/mach-sunxi/Kconfig | 5 +++++
>  drivers/clk/sunxi/Makefile  | 3 +++
>  2 files changed, 8 insertions(+)
> 
> diff --git a/arch/arm/mach-sunxi/Kconfig b/arch/arm/mach-sunxi/Kconfig
> index c124d65..424b76d 100644
> --- a/arch/arm/mach-sunxi/Kconfig
> +++ b/arch/arm/mach-sunxi/Kconfig
> @@ -41,6 +41,11 @@ config MACH_SUN8I
>  	select ARM_GIC
>  	select MFD_SUN6I_PRCM
>  
> +config MACH_SUN8I_A83T
> +	bool "Allwinner A83T (sun8i) SoCs support"
> +	default ARCH_SUNXI
> +	select ARM_GIC
> +
>  config MACH_SUN9I
>  	bool "Allwinner (sun9i) SoCs support"
>  	default ARCH_SUNXI
> diff --git a/drivers/clk/sunxi/Makefile b/drivers/clk/sunxi/Makefile
> index 3fd7901..2aabfc4 100644
> --- a/drivers/clk/sunxi/Makefile
> +++ b/drivers/clk/sunxi/Makefile
> @@ -20,6 +20,9 @@ obj-y += clk-usb.o
>  obj-$(CONFIG_MACH_SUN9I) += clk-sun8i-apb0.o
>  obj-$(CONFIG_MACH_SUN9I) += clk-sun9i-cpus.o
>  
> +obj-$(CONFIG_MACH_SUN8I_A83T) += clk-sun8i-apb0.o
> +obj-$(CONFIG_MACH_SUN8I_A83T) += clk-sun9i-cpus.o
> +

This is not how it should be done. You should rather create a Kconfig
option for these clocks, and use def_bool CONFIG_MACH_SUN8I || CONFIG_MACH_SUN9I

Switching to Kconfig for the clocks has been on my todo-list for quite
some time, I guess these two clocks would be a good start.

Thanks!
Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v2 00/13]  Add A83T clk, r_pio, mmc rsb support Vishnu Patekar <vishnupatekar0510@gmail.com> - 2016-02-28 16:20 +0100
  [PATCH v2 01/13] pinctrl: sunxi: Add A83T R_PIO controller support Vishnu Patekar <vishnupatekar0510@gmail.com> - 2016-02-28 16:20 +0100
    Re: [PATCH v2 01/13] pinctrl: sunxi: Add A83T R_PIO controller  support Maxime Ripard <maxime.ripard@free-electrons.com> - 2016-03-01 08:10 +0100
  [PATCH v2 04/13] ARM: dts: sun8i-a83t: Add basic clocks and resets Vishnu Patekar <vishnupatekar0510@gmail.com> - 2016-02-28 16:20 +0100
    Re: [PATCH v2 04/13] ARM: dts: sun8i-a83t: Add basic clocks and resets Chen-Yu Tsai <wens@csie.org> - 2016-02-28 18:30 +0100
      Re: [PATCH v2 04/13] ARM: dts: sun8i-a83t: Add basic clocks and resets Vishnu Patekar <vishnupatekar0510@gmail.com> - 2016-02-29 15:50 +0100
  [PATCH v2 05/13] ARM: dts: sun8i-a83t: add mmc clock nodes Vishnu Patekar <vishnupatekar0510@gmail.com> - 2016-02-28 16:20 +0100
  [PATCH v2 02/13] clk: sunxi: add ahb1 clock for A83T Vishnu Patekar <vishnupatekar0510@gmail.com> - 2016-02-28 16:20 +0100
    Re: [PATCH v2 02/13] clk: sunxi: add ahb1 clock for A83T Chen-Yu Tsai <wens@csie.org> - 2016-02-29 05:20 +0100
  [PATCH v2 06/13] ARM: dts: sun8i-a83t: Add mmc controller nodes Vishnu Patekar <vishnupatekar0510@gmail.com> - 2016-02-28 16:20 +0100
  [PATCH v2 03/13] clk: sunxi: Add APB1 clock for A83T Vishnu Patekar <vishnupatekar0510@gmail.com> - 2016-02-28 16:20 +0100
  [PATCH v2 08/13] ARM: dts: sun8i-a83t: Add R_PIO controller node to the dtsi Vishnu Patekar <vishnupatekar0510@gmail.com> - 2016-02-28 16:30 +0100
  [PATCH v2 11/13] ARM: dts: sun8i: enable mmc for H8Homlet Board. Vishnu Patekar <vishnupatekar0510@gmail.com> - 2016-02-28 16:30 +0100
  [PATCH v2 10/13] ARM: dts: sun8i-a83t: add mmc0 CD pin Vishnu Patekar <vishnupatekar0510@gmail.com> - 2016-02-28 16:30 +0100
  [PATCH v2 09/13] ARM: dts: sun8i-a83t: Add RSB controller device node to dtsi Vishnu Patekar <vishnupatekar0510@gmail.com> - 2016-02-28 16:30 +0100
  [PATCH v2 07/13] ARM: dts: sun8i-a83t: Add PRCM related clocks and resets Vishnu Patekar <vishnupatekar0510@gmail.com> - 2016-02-28 16:30 +0100
  [PATCH v2 12/13] ARM: dts: sun8i: Add A83T based Sinovoip Bpi-M3 Board Vishnu Patekar <vishnupatekar0510@gmail.com> - 2016-02-28 16:30 +0100
  [PATCH v2 13/13] ARM: sunxi: Introduce MACH_SUN8I_A83T option Vishnu Patekar <vishnupatekar0510@gmail.com> - 2016-02-28 16:30 +0100
    Re: [PATCH v2 13/13] ARM: sunxi: Introduce MACH_SUN8I_A83T option Maxime Ripard <maxime.ripard@free-electrons.com> - 2016-03-01 08:20 +0100
      Re: [PATCH v2 13/13] ARM: sunxi: Introduce MACH_SUN8I_A83T option Vishnu Patekar <vishnupatekar0510@gmail.com> - 2016-03-02 17:10 +0100

csiph-web