lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <525555D0.4020506@atmel.com>
Date:	Wed, 9 Oct 2013 15:10:40 +0200
From:	Nicolas Ferre <nicolas.ferre@...el.com>
To:	Boris BREZILLON <b.brezillon@...rkiz.com>,
	Russell King <linux@....linux.org.uk>,
	Andrew Victor <linux@...im.org.za>,
	"Jean-Christophe Plagniol-Villard" <plagnioj@...osoft.com>,
	Ludovic Desroches <ludovic.desroches@...el.com>,
	Lorenzo Pieralisi <lorenzo.pieralisi@....com>
CC:	<linux-arm-kernel@...ts.infradead.org>,
	<linux-kernel@...r.kernel.org>, <devicetree@...r.kernel.org>
Subject: Re: [PATCH 3/6] ARM: at91/dt: define sama5d3 clocks

On 08/08/2013 11:31, Boris BREZILLON :
> Define sama5d3 clocks in sama5d3 device tree.
> Add references to the appropriate clocks in each peripheral.
>
> Signed-off-by: Boris BREZILLON <b.brezillon@...rkiz.com>

Do not forget to remove the macros and change file paths...

> ---
>   arch/arm/boot/dts/sama5d3.dtsi      |  331 ++++++++++++++++++++++++++++++++++-
>   arch/arm/boot/dts/sama5d3_can.dtsi  |   19 ++
>   arch/arm/boot/dts/sama5d3_emac.dtsi |   12 ++
>   arch/arm/boot/dts/sama5d3_gmac.dtsi |   12 ++
>   arch/arm/boot/dts/sama5d3_lcd.dtsi  |   17 ++
>   arch/arm/boot/dts/sama5d3_mci2.dtsi |   11 ++
>   arch/arm/boot/dts/sama5d3_tcb1.dtsi |   12 ++
>   arch/arm/boot/dts/sama5d3_uart.dtsi |   19 ++
>   8 files changed, 431 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/boot/dts/sama5d3.dtsi b/arch/arm/boot/dts/sama5d3.dtsi
> index f0ca1ba..e2791c5 100644
> --- a/arch/arm/boot/dts/sama5d3.dtsi
> +++ b/arch/arm/boot/dts/sama5d3.dtsi
> @@ -14,6 +14,10 @@
>   #include <dt-bindings/interrupt-controller/irq.h>
>   #include <dt-bindings/gpio/gpio.h>
>   #include <dt-bindings/at91/sama5d3/peripherals.h>
> +#include <dt-bindings/clk/at91/common/clk-system.h>
> +#include <dt-bindings/clk/at91/common/pmc.h>
> +#include <dt-bindings/clk/at91/at91sam9x5/clk-peripheral.h>
> +#include <dt-bindings/clk/at91/common/clk-pll.h>
>
>   / {
>   	model = "Atmel SAMA5D3 family SoC";
> @@ -52,6 +56,14 @@
>   		reg = <0x20000000 0x8000000>;
>   	};
>
> +	clocks {
> +		adc_op_clk: adc_op_clk{
> +			compatible = "fixed-clock";
> +			#clock-cells = <0>;
> +			clock-frequency = <20000000>;
> +		};
> +	};
> +
>   	ahb {
>   		compatible = "simple-bus";
>   		#address-cells = <1>;
> @@ -75,6 +87,8 @@
>   				status = "disabled";
>   				#address-cells = <1>;
>   				#size-cells = <0>;
> +				clocks = <&periph SAMA5D3_ID_HSMCI0>;
> +				clock-names = "mci_clk";
>   			};
>
>   			spi0: spi@...04000 {
> @@ -88,6 +102,8 @@
>   				dma-names = "tx", "rx";
>   				pinctrl-names = "default";
>   				pinctrl-0 = <&pinctrl_spi0>;
> +				clocks = <&periph SAMA5D3_ID_SPI0>;
> +				clock-names = "spi_clk";
>   				status = "disabled";
>   			};
>
> @@ -97,6 +113,8 @@
>   				interrupts = <SAMA5D3_ID_SSC0 IRQ_TYPE_LEVEL_HIGH 4>;
>   				pinctrl-names = "default";
>   				pinctrl-0 = <&pinctrl_ssc0_tx &pinctrl_ssc0_rx>;
> +				clocks = <&periph SAMA5D3_ID_SSC0>;
> +				clock-names = "pclk";
>   				status = "disabled";
>   			};
>
> @@ -104,6 +122,8 @@
>   				compatible = "atmel,at91sam9x5-tcb";
>   				reg = <0xf0010000 0x100>;
>   				interrupts = <SAMA5D3_ID_TC0 IRQ_TYPE_LEVEL_HIGH 0>;
> +				clocks = <&periph SAMA5D3_ID_TC0>;
> +				clock-names = "t0_clk";
>   			};
>
>   			i2c0: i2c@...14000 {
> @@ -117,6 +137,7 @@
>   				pinctrl-0 = <&pinctrl_i2c0>;
>   				#address-cells = <1>;
>   				#size-cells = <0>;
> +				clocks = <&periph SAMA5D3_ID_TWI0>;
>   				status = "disabled";
>   			};
>
> @@ -131,6 +152,7 @@
>   				pinctrl-0 = <&pinctrl_i2c1>;
>   				#address-cells = <1>;
>   				#size-cells = <0>;
> +				clocks = <&periph SAMA5D3_ID_TWI1>;
>   				status = "disabled";
>   			};
>
> @@ -140,6 +162,8 @@
>   				interrupts = <SAMA5D3_ID_USART0 IRQ_TYPE_LEVEL_HIGH 5>;
>   				pinctrl-names = "default";
>   				pinctrl-0 = <&pinctrl_usart0>;
> +				clocks = <&periph SAMA5D3_ID_USART0>;
> +				clock-names = "usart";
>   				status = "disabled";
>   			};
>
> @@ -149,6 +173,8 @@
>   				interrupts = <SAMA5D3_ID_USART1 IRQ_TYPE_LEVEL_HIGH 5>;
>   				pinctrl-names = "default";
>   				pinctrl-0 = <&pinctrl_usart1>;
> +				clocks = <&periph SAMA5D3_ID_USART1>;
> +				clock-names = "usart";
>   				status = "disabled";
>   			};
>
> @@ -170,6 +196,8 @@
>   				status = "disabled";
>   				#address-cells = <1>;
>   				#size-cells = <0>;
> +				clocks = <&periph SAMA5D3_ID_HSMCI1>;
> +				clock-names = "mci_clk";
>   			};
>
>   			spi1: spi@...08000 {
> @@ -183,6 +211,8 @@
>   				dma-names = "tx", "rx";
>   				pinctrl-names = "default";
>   				pinctrl-0 = <&pinctrl_spi1>;
> +				clocks = <&periph SAMA5D3_ID_SPI1>;
> +				clock-names = "spi_clk";
>   				status = "disabled";
>   			};
>
> @@ -192,6 +222,8 @@
>   				interrupts = <SAMA5D3_ID_SSC1 IRQ_TYPE_LEVEL_HIGH 4>;
>   				pinctrl-names = "default";
>   				pinctrl-0 = <&pinctrl_ssc1_tx &pinctrl_ssc1_rx>;
> +				clocks = <&periph SAMA5D3_ID_SSC1>;
> +				clock-names = "pclk";
>   				status = "disabled";
>   			};
>
> @@ -215,6 +247,9 @@
>   					&pinctrl_adc0_ad10
>   					&pinctrl_adc0_ad11
>   					>;
> +				clocks = <&periph SAMA5D3_ID_ADC>,
> +					 <&adc_op_clk>;
> +				clock-names = "adc_clk", "adc_op_clk";
>   				atmel,adc-channel-base = <0x50>;
>   				atmel,adc-channels-used = <0xfff>;
>   				atmel,adc-drdy-mask = <0x1000000>;
> @@ -270,6 +305,7 @@
>   				dma-names = "tx", "rx";
>   				#address-cells = <1>;
>   				#size-cells = <0>;
> +				clocks = <&periph SAMA5D3_ID_TWI2>;
>   				status = "disabled";
>   			};
>
> @@ -279,6 +315,8 @@
>   				interrupts = <SAMA5D3_ID_USART2 IRQ_TYPE_LEVEL_HIGH 5>;
>   				pinctrl-names = "default";
>   				pinctrl-0 = <&pinctrl_usart2>;
> +				clocks = <&periph SAMA5D3_ID_USART2>;
> +				clock-names = "usart";
>   				status = "disabled";
>   			};
>
> @@ -288,6 +326,8 @@
>   				interrupts = <SAMA5D3_ID_USART3 IRQ_TYPE_LEVEL_HIGH 5>;
>   				pinctrl-names = "default";
>   				pinctrl-0 = <&pinctrl_usart3>;
> +				clocks = <&periph SAMA5D3_ID_USART3>;
> +				clock-names = "usart";
>   				status = "disabled";
>   			};
>
> @@ -300,7 +340,7 @@
>   			aes@...38000 {
>   				compatible = "atmel,sam9g46-aes";
>   				reg = <0xf8038000 0x100>;
> -				interrupts = <SAMA5D3_ID_AES 4 0>;
> +				interrupts = <SAMA5D3_ID_AES IRQ_TYPE_LEVEL_HIGH 0>;
>   			};
>
>   			tdes@...3c000 {
> @@ -314,6 +354,8 @@
>   				reg = <0xffffe600 0x200>;
>   				interrupts = <SAMA5D3_ID_DMA0 IRQ_TYPE_LEVEL_HIGH 0>;
>   				#dma-cells = <2>;
> +				clocks = <&periph SAMA5D3_ID_DMA0>;
> +				clock-names = "dma_clk";
>   			};
>
>   			dma1: dma-controller@...fe800 {
> @@ -321,6 +363,8 @@
>   				reg = <0xffffe800 0x200>;
>   				interrupts = <SAMA5D3_ID_DMA1 IRQ_TYPE_LEVEL_HIGH 0>;
>   				#dma-cells = <2>;
> +				clocks = <&periph SAMA5D3_ID_DMA1>;
> +				clock-names = "dma_clk";
>   			};
>
>   			ramc0: ramc@...fea00 {
> @@ -334,6 +378,8 @@
>   				interrupts = <SAMA5D3_ID_DBGU IRQ_TYPE_LEVEL_HIGH 7>;
>   				pinctrl-names = "default";
>   				pinctrl-0 = <&pinctrl_dbgu>;
> +				clocks = <&periph 2>;
> +				clock-names = "usart";
>   				status = "disabled";
>   			};
>
> @@ -622,6 +668,7 @@
>   					gpio-controller;
>   					interrupt-controller;
>   					#interrupt-cells = <2>;
> +					clocks = <&periph 6>;
>   				};
>
>   				pioB: gpio@...ff400 {
> @@ -632,6 +679,7 @@
>   					gpio-controller;
>   					interrupt-controller;
>   					#interrupt-cells = <2>;
> +					clocks = <&periph 7>;
>   				};
>
>   				pioC: gpio@...ff600 {
> @@ -642,6 +690,7 @@
>   					gpio-controller;
>   					interrupt-controller;
>   					#interrupt-cells = <2>;
> +					clocks = <&periph 8>;
>   				};
>
>   				pioD: gpio@...ff800 {
> @@ -652,6 +701,7 @@
>   					gpio-controller;
>   					interrupt-controller;
>   					#interrupt-cells = <2>;
> +					clocks = <&periph 9>;
>   				};
>
>   				pioE: gpio@...ffa00 {
> @@ -662,12 +712,279 @@
>   					gpio-controller;
>   					interrupt-controller;
>   					#interrupt-cells = <2>;
> +					clocks = <&periph 10>;
>   				};
>   			};
>
>   			pmc: pmc@...ffc00 {
> -				compatible = "atmel,at91rm9200-pmc";
> +				compatible = "atmel,sama5d3-pmc";
>   				reg = <0xfffffc00 0x120>;
> +				interrupts = <AT91_ID_SYS IRQ_TYPE_LEVEL_HIGH 7>;
> +				interrupt-controller;
> +				#interrupt-cells = <2>;
> +
> +				clk32k: slck {
> +					compatible = "fixed-clock";
> +					#clock-cells = <0>;
> +					clock-frequency = <32768>;
> +				};
> +
> +				main: mainck {
> +					compatible = "atmel,at91rm9200-clk-main";
> +					interrupt-parent = <&pmc>;
> +					interrupts = <AT91_PMC_MOSCS IRQ_TYPE_LEVEL_HIGH>;
> +					#clock-cells = <0>;
> +					clocks = <&clk32k>;
> +				};
> +
> +				plla: pllack {
> +					compatible = "atmel,sama5d3-clk-pll";
> +					interrupt-parent = <&pmc>;
> +					interrupts = <AT91_PMC_LOCKA IRQ_TYPE_LEVEL_HIGH>;
> +					#clock-cells = <0>;
> +					clocks = <&main>;
> +					atmel,clk-id = <AT91_PLLA_CLK>;
> +					atmel,clk-input-range = <8000000 50000000>;
> +					#atmel,pll-clk-output-range-cells = <4>;
> +					atmel,pll-clk-output-ranges = <400000000 1000000000 0 0>;
> +				};
> +
> +				plladiv: plladivck {
> +					compatible = "atmel,at91sam9x5-clk-plldiv";
> +					#clock-cells = <0>;
> +					clocks = <&plla>;
> +				};
> +
> +				utmi: utmick {
> +					compatible = "atmel,at91sam9x5-clk-utmi";
> +					interrupt-parent = <&pmc>;
> +					interrupts = <AT91_PMC_LOCKU IRQ_TYPE_LEVEL_HIGH>;
> +					#clock-cells = <0>;
> +					clocks = <&main>;
> +				};
> +
> +				mck: masterck {
> +					compatible = "atmel,at91sam9x5-clk-master";
> +					interrupt-parent = <&pmc>;
> +					interrupts = <AT91_PMC_MCKRDY IRQ_TYPE_LEVEL_HIGH>;
> +					#clock-cells = <0>;
> +					clocks = <&clk32k>, <&main>, <&plladiv>, <&utmi>;
> +					atmel,clk-output-range = <0 166000000>;
> +					atmel,clk-divisors = <1 2 4 3>;
> +				};
> +
> +				usb: usbck {
> +					compatible = "atmel,at91sam9x5-clk-usb";
> +					#clock-cells = <0>;
> +					clocks = <&plladiv>, <&utmi>;
> +				};
> +
> +				prog: progck {
> +					compatible = "atmel,at91sam9x5-clk-programmable";
> +					interrupt-parent = <&pmc>;
> +					#clock-cells = <1>;
> +					clocks = <&clk32k>, <&main>, <&plladiv>, <&utmi>, <&mck>;
> +
> +					prog0 {
> +						atmel,clk-id = <0>;
> +						interrupts = <AT91_PMC_PCKRDY(0) IRQ_TYPE_LEVEL_HIGH>;
> +					};
> +
> +					prog1 {
> +						atmel,clk-id = <1>;
> +						interrupts = <AT91_PMC_PCKRDY(1) IRQ_TYPE_LEVEL_HIGH>;
> +					};
> +
> +					prog2 {
> +						atmel,clk-id = <2>;
> +						interrupts = <AT91_PMC_PCKRDY(2) IRQ_TYPE_LEVEL_HIGH>;
> +					};
> +				};
> +
> +				smd: smdclk {
> +					compatible = "atmel,at91sam9x5-clk-smd";
> +					#clock-cells = <0>;
> +					clocks = <&plladiv>, <&utmi>;
> +				};
> +
> +				system: systemck {
> +					compatible = "atmel,at91rm9200-clk-system";
> +					#clock-cells = <1>;
> +
> +					ddrck {
> +						atmel,clk-id = <AT91_DDRCK_SYS_CLK>;
> +					};
> +
> +					smdck {
> +						atmel,clk-id = <AT91_SMDCK_SYS_CLK>;
> +					};
> +
> +					uhpck {
> +						atmel,clk-id = <AT91_UHP_SYS_CLK>;
> +					};
> +
> +					udpck {
> +						atmel,clk-id = <AT91_UDP_SYS_CLK>;
> +					};
> +
> +					pck0 {
> +						atmel,clk-id = <AT91_PROG_SYS_CLK(0)>;
> +					};
> +
> +					pck1 {
> +						atmel,clk-id = <AT91_PROG_SYS_CLK(1)>;
> +					};
> +
> +					pck2 {
> +						atmel,clk-id = <AT91_PROG_SYS_CLK(2)>;
> +					};
> +				};
> +
> +				periph: periphck {
> +					compatible = "atmel,at91sam9x5-clk-peripheral";
> +					#clock-cells = <1>;
> +					clocks = <&mck>;
> +
> +					dbgu_clk {
> +						atmel,clk-id = <SAMA5D3_ID_DBGU>;
> +					};
> +
> +					pioA_clk {
> +						atmel,clk-id = <SAMA5D3_ID_PIOA>;
> +					};
> +
> +					pioB_clk {
> +						atmel,clk-id = <SAMA5D3_ID_PIOB>;
> +					};
> +
> +					pioC_clk {
> +						atmel,clk-id = <SAMA5D3_ID_PIOC>;
> +					};
> +
> +					pioD_clk {
> +						atmel,clk-id = <SAMA5D3_ID_PIOD>;
> +					};
> +
> +					pioE_clk {
> +						atmel,clk-id = <SAMA5D3_ID_PIOE>;
> +					};
> +
> +					usart0_clk {
> +						atmel,clk-id = <SAMA5D3_ID_USART0>;
> +						atmel,clk-default-divisor = <AT91SAM9X5_PERIPH_CLK_DIV2>;
> +					};
> +
> +					usart1_clk {
> +						atmel,clk-id = <SAMA5D3_ID_USART1>;
> +						atmel,clk-default-divisor = <AT91SAM9X5_PERIPH_CLK_DIV2>;
> +					};
> +
> +					usart2_clk {
> +						atmel,clk-id = <SAMA5D3_ID_USART2>;
> +						atmel,clk-default-divisor = <AT91SAM9X5_PERIPH_CLK_DIV2>;
> +					};
> +
> +					usart3_clk {
> +						atmel,clk-id = <SAMA5D3_ID_USART3>;
> +						atmel,clk-default-divisor = <AT91SAM9X5_PERIPH_CLK_DIV2>;
> +					};
> +
> +					twi0_clk {
> +						atmel,clk-id = <SAMA5D3_ID_TWI0>;
> +						atmel,clk-default-divisor = <AT91SAM9X5_PERIPH_CLK_DIV2>;
> +					};
> +
> +					twi1_clk {
> +						atmel,clk-id = <SAMA5D3_ID_TWI1>;
> +						atmel,clk-default-divisor = <AT91SAM9X5_PERIPH_CLK_DIV2>;
> +					};
> +
> +					twi2_clk {
> +						atmel,clk-id = <SAMA5D3_ID_TWI2>;
> +						atmel,clk-default-divisor = <AT91SAM9X5_PERIPH_CLK_DIV2>;
> +					};
> +
> +					mci0_clk {
> +						atmel,clk-id = <SAMA5D3_ID_HSMCI0>;
> +					};
> +
> +					mci1_clk {
> +						atmel,clk-id = <SAMA5D3_ID_HSMCI1>;
> +					};
> +
> +					spi0_clk {
> +						atmel,clk-id = <SAMA5D3_ID_SPI0>;
> +					};
> +
> +					spi1_clk {
> +						atmel,clk-id = <SAMA5D3_ID_SPI1>;
> +					};
> +
> +					tcb0_clk {
> +						atmel,clk-id = <SAMA5D3_ID_TC0>;
> +						atmel,clk-default-divisor = <AT91SAM9X5_PERIPH_CLK_DIV2>;
> +					};
> +
> +					pwm_clk {
> +						atmel,clk-id = <SAMA5D3_ID_PWM>;
> +					};
> +
> +					adc_clk {
> +						atmel,clk-id = <SAMA5D3_ID_ADC>;
> +						atmel,clk-default-divisor = <AT91SAM9X5_PERIPH_CLK_DIV2>;
> +					};
> +
> +					dma0_clk {
> +						atmel,clk-id = <SAMA5D3_ID_DMA0>;
> +					};
> +
> +					dma1_clk {
> +						atmel,clk-id = <SAMA5D3_ID_DMA1>;
> +					};
> +
> +					uhphs_clk {
> +						atmel,clk-id = <SAMA5D3_ID_UHPHS>;
> +					};
> +
> +					udphs_clk {
> +						atmel,clk-id = <SAMA5D3_ID_UDPHS>;
> +					};
> +
> +					isi_clk {
> +						atmel,clk-id = <SAMA5D3_ID_ISI>;
> +					};
> +
> +					ssc0_clk {
> +						atmel,clk-id = <SAMA5D3_ID_SSC0>;
> +						atmel,clk-default-divisor = <AT91SAM9X5_PERIPH_CLK_DIV2>;
> +					};
> +
> +					ssc1_clk {
> +						atmel,clk-id = <SAMA5D3_ID_SSC1>;
> +						atmel,clk-default-divisor = <AT91SAM9X5_PERIPH_CLK_DIV2>;
> +					};
> +
> +					sha_clk {
> +						atmel,clk-id = <SAMA5D3_ID_SHA>;
> +						atmel,clk-default-divisor = <AT91SAM9X5_PERIPH_CLK_DIV8>;
> +					};
> +
> +					aes_clk {
> +						atmel,clk-id = <SAMA5D3_ID_AES>;
> +					};
> +
> +					tdes_clk {
> +						atmel,clk-id = <SAMA5D3_ID_TDES>;
> +					};
> +
> +					trng_clk {
> +						atmel,clk-id = <SAMA5D3_ID_TRNG>;
> +					};
> +
> +					fuse_clk {
> +						atmel,clk-id = <SAMA5D3_ID_FUSE>;
> +					};
> +				};
>   			};
>
>   			rstc@...ffe00 {
> @@ -679,6 +996,7 @@
>   				compatible = "atmel,at91sam9260-pit";
>   				reg = <0xfffffe30 0xf>;
>   				interrupts = <AT91_ID_PIT IRQ_TYPE_LEVEL_HIGH 5>;
> +				clocks = <&mck>;
>   			};
>
>   			watchdog@...ffe40 {
> @@ -701,6 +1019,8 @@
>   			reg = <0x00500000 0x100000
>   			       0xf8030000 0x4000>;
>   			interrupts = <SAMA5D3_ID_UDPHS IRQ_TYPE_LEVEL_HIGH 2>;
> +			clocks = <&periph SAMA5D3_ID_UDPHS>, <&utmi>;
> +			clock-names = "pclk", "hclk";
>   			status = "disabled";
>
>   			ep0 {
> @@ -813,6 +1133,10 @@
>   			compatible = "atmel,at91rm9200-ohci", "usb-ohci";
>   			reg = <0x00600000 0x100000>;
>   			interrupts = <SAMA5D3_ID_UHPHS IRQ_TYPE_LEVEL_HIGH 2>;
> +			clocks = <&usb>, <&periph SAMA5D3_ID_UHPHS>,
> +				 <&periph SAMA5D3_ID_UHPHS>,
> +				 <&system AT91_UHP_SYS_CLK>;
> +			clock-names = "usb_clk", "ohci_clk", "hclk", "uhpck";
>   			status = "disabled";
>   		};
>
> @@ -820,6 +1144,9 @@
>   			compatible = "atmel,at91sam9g45-ehci", "usb-ehci";
>   			reg = <0x00700000 0x100000>;
>   			interrupts = <SAMA5D3_ID_UHPHS IRQ_TYPE_LEVEL_HIGH 2>;
> +			clocks = <&periph SAMA5D3_ID_UHPHS>,
> +				 <&system AT91_UHP_SYS_CLK>;
> +			clock-names = "ehci_clk", "uhpck";
>   			status = "disabled";
>   		};
>
> diff --git a/arch/arm/boot/dts/sama5d3_can.dtsi b/arch/arm/boot/dts/sama5d3_can.dtsi
> index 55b2d9f..0c3fa82 100644
> --- a/arch/arm/boot/dts/sama5d3_can.dtsi
> +++ b/arch/arm/boot/dts/sama5d3_can.dtsi
> @@ -10,6 +10,7 @@
>   #include <dt-bindings/pinctrl/at91.h>
>   #include <dt-bindings/interrupt-controller/irq.h>
>   #include <dt-bindings/at91/sama5d3/peripherals.h>
> +#include <dt-bindings/clk/at91/at91sam9x5/clk-peripheral.h>
>
>   / {
>   	ahb {
> @@ -33,12 +34,28 @@
>
>   			};
>
> +			pmc: pmc@...ffc00 {
> +				periph: periphck {
> +					can0_clk {
> +						atmel,clk-id = <SAMA5D3_ID_CAN0>;
> +						atmel,clk-default-divisor = <AT91SAM9X5_PERIPH_CLK_DIV2>;
> +					};
> +
> +					can1_clk {
> +						atmel,clk-id = <SAMA5D3_ID_CAN1>;
> +						atmel,clk-default-divisor = <AT91SAM9X5_PERIPH_CLK_DIV2>;
> +					};
> +				};
> +			};
> +
>   			can0: can@...0c000 {
>   				compatible = "atmel,at91sam9x5-can";
>   				reg = <0xf000c000 0x300>;
>   				interrupts = <SAMA5D3_ID_CAN0 IRQ_TYPE_LEVEL_HIGH 3>;
>   				pinctrl-names = "default";
>   				pinctrl-0 = <&pinctrl_can0_rx_tx>;
> +				clocks = <&periph SAMA5D3_ID_CAN0>;
> +				clock-names = "can_clk";
>   				status = "disabled";
>   			};
>
> @@ -48,6 +65,8 @@
>   				interrupts = <SAMA5D3_ID_CAN1 IRQ_TYPE_LEVEL_HIGH 3>;
>   				pinctrl-names = "default";
>   				pinctrl-0 = <&pinctrl_can1_rx_tx>;
> +				clocks = <&periph SAMA5D3_ID_CAN1>;
> +				clock-names = "can_clk";
>   				status = "disabled";
>   			};
>   		};
> diff --git a/arch/arm/boot/dts/sama5d3_emac.dtsi b/arch/arm/boot/dts/sama5d3_emac.dtsi
> index fb4628d..d4a9556 100644
> --- a/arch/arm/boot/dts/sama5d3_emac.dtsi
> +++ b/arch/arm/boot/dts/sama5d3_emac.dtsi
> @@ -10,6 +10,7 @@
>   #include <dt-bindings/pinctrl/at91.h>
>   #include <dt-bindings/interrupt-controller/irq.h>
>   #include <dt-bindings/at91/sama5d3/peripherals.h>
> +#include <dt-bindings/clk/at91/at91sam9x5/clk-peripheral.h>
>
>   / {
>   	ahb {
> @@ -32,12 +33,23 @@
>   				};
>   			};
>
> +			pmc: pmc@...ffc00 {
> +				periph: periphck {
> +					macb1_clk {
> +						atmel,clk-id = <SAMA5D3_ID_EMAC>;
> +					};
> +				};
> +			};
> +
>   			macb1: ethernet@...2c000 {
>   				compatible = "cdns,at32ap7000-macb", "cdns,macb";
>   				reg = <0xf802c000 0x100>;
>   				interrupts = <SAMA5D3_ID_EMAC IRQ_TYPE_LEVEL_HIGH 3>;
>   				pinctrl-names = "default";
>   				pinctrl-0 = <&pinctrl_macb1_rmii>;
> +				clocks = <&periph SAMA5D3_ID_EMAC>,
> +					 <&periph SAMA5D3_ID_EMAC>;
> +				clock-names = "hclk", "pclk";
>   				status = "disabled";
>   			};
>   		};
> diff --git a/arch/arm/boot/dts/sama5d3_gmac.dtsi b/arch/arm/boot/dts/sama5d3_gmac.dtsi
> index f92c721..8082021 100644
> --- a/arch/arm/boot/dts/sama5d3_gmac.dtsi
> +++ b/arch/arm/boot/dts/sama5d3_gmac.dtsi
> @@ -10,6 +10,7 @@
>   #include <dt-bindings/pinctrl/at91.h>
>   #include <dt-bindings/interrupt-controller/irq.h>
>   #include <dt-bindings/at91/sama5d3/peripherals.h>
> +#include <dt-bindings/clk/at91/at91sam9x5/clk-peripheral.h>
>
>   / {
>   	ahb {
> @@ -65,12 +66,23 @@
>   				};
>   			};
>
> +			pmc: pmc@...ffc00 {
> +				periph: periphck {
> +					macb0_clk {
> +						atmel,clk-id = <SAMA5D3_ID_GMAC>;
> +					};
> +				};
> +			};
> +
>   			macb0: ethernet@...28000 {
>   				compatible = "cdns,pc302-gem", "cdns,gem";
>   				reg = <0xf0028000 0x100>;
>   				interrupts = <SAMA5D3_ID_GMAC IRQ_TYPE_LEVEL_HIGH 3>;
>   				pinctrl-names = "default";
>   				pinctrl-0 = <&pinctrl_macb0_data_rgmii &pinctrl_macb0_signal_rgmii>;
> +				clocks = <&periph SAMA5D3_ID_GMAC>,
> +					 <&periph SAMA5D3_ID_GMAC>;
> +				clock-names = "hclk", "pclk";
>   				status = "disabled";
>   			};
>   		};
> diff --git a/arch/arm/boot/dts/sama5d3_lcd.dtsi b/arch/arm/boot/dts/sama5d3_lcd.dtsi
> index 01f52a7..15ff059 100644
> --- a/arch/arm/boot/dts/sama5d3_lcd.dtsi
> +++ b/arch/arm/boot/dts/sama5d3_lcd.dtsi
> @@ -9,6 +9,9 @@
>
>   #include <dt-bindings/pinctrl/at91.h>
>   #include <dt-bindings/interrupt-controller/irq.h>
> +#include <dt-bindings/at91/sama5d3/peripherals.h>
> +#include <dt-bindings/clk/at91/common/clk-system.h>
> +#include <dt-bindings/clk/at91/at91sam9x5/clk-peripheral.h>
>
>   / {
>   	ahb {
> @@ -50,6 +53,20 @@
>   					};
>   				};
>   			};
> +
> +			pmc: pmc@...ffc00 {
> +				periph: periphck {
> +					lcdc_clk {
> +						atmel,clk-id = <SAMA5D3_ID_LCDC>;
> +					};
> +				};
> +
> +				system: systemck {
> +					lcdck {
> +						atmel,clk-id = <AT91_LCDCK_SYS_CLK>;
> +					};
> +				};
> +			};
>   		};
>   	};
>   };
> diff --git a/arch/arm/boot/dts/sama5d3_mci2.dtsi b/arch/arm/boot/dts/sama5d3_mci2.dtsi
> index 6471610..7cef03e 100644
> --- a/arch/arm/boot/dts/sama5d3_mci2.dtsi
> +++ b/arch/arm/boot/dts/sama5d3_mci2.dtsi
> @@ -10,6 +10,7 @@
>   #include <dt-bindings/pinctrl/at91.h>
>   #include <dt-bindings/interrupt-controller/irq.h>
>   #include <dt-bindings/at91/sama5d3/peripherals.h>
> +#include <dt-bindings/clk/at91/at91sam9x5/clk-peripheral.h>
>
>   / {
>   	ahb {
> @@ -31,6 +32,14 @@
>   				};
>   			};
>
> +			pmc: pmc@...ffc00 {
> +				periph: periphck {
> +					mci2_clk {
> +						atmel,clk-id = <SAMA5D3_ID_HSMCI2>;
> +					};
> +				};
> +			};
> +
>   			mmc2: mmc@...04000 {
>   				compatible = "atmel,hsmci";
>   				reg = <0xf8004000 0x600>;
> @@ -39,6 +48,8 @@
>   				dma-names = "rxtx";
>   				pinctrl-names = "default";
>   				pinctrl-0 = <&pinctrl_mmc2_clk_cmd_dat0 &pinctrl_mmc2_dat1_3>;
> +				clocks = <&periph SAMA5D3_ID_HSMCI2>;
> +				clock-names = "mci_clk";
>   				status = "disabled";
>   				#address-cells = <1>;
>   				#size-cells = <0>;
> diff --git a/arch/arm/boot/dts/sama5d3_tcb1.dtsi b/arch/arm/boot/dts/sama5d3_tcb1.dtsi
> index bd26568..fb00a22 100644
> --- a/arch/arm/boot/dts/sama5d3_tcb1.dtsi
> +++ b/arch/arm/boot/dts/sama5d3_tcb1.dtsi
> @@ -10,6 +10,7 @@
>   #include <dt-bindings/pinctrl/at91.h>
>   #include <dt-bindings/interrupt-controller/irq.h>
>   #include <dt-bindings/at91/sama5d3/peripherals.h>
> +#include <dt-bindings/clk/at91/at91sam9x5/clk-peripheral.h>
>
>   / {
>   	aliases {
> @@ -18,10 +19,21 @@
>
>   	ahb {
>   		apb {
> +			pmc: pmc@...ffc00 {
> +				periph: periphck {
> +					tcb1_clk {
> +						atmel,clk-id = <SAMA5D3_ID_TC1>;
> +						atmel,clk-default-divisor = <AT91SAM9X5_PERIPH_CLK_DIV2>;
> +					};
> +				};
> +			};
> +
>   			tcb1: timer@...14000 {
>   				compatible = "atmel,at91sam9x5-tcb";
>   				reg = <0xf8014000 0x100>;
>   				interrupts = <SAMA5D3_ID_TC1 IRQ_TYPE_LEVEL_HIGH 0>;
> +				clocks = <&periph SAMA5D3_ID_TC1>;
> +				clock-names = "t0_clk";
>   			};
>   		};
>   	};
> diff --git a/arch/arm/boot/dts/sama5d3_uart.dtsi b/arch/arm/boot/dts/sama5d3_uart.dtsi
> index 1d89110..675b04d 100644
> --- a/arch/arm/boot/dts/sama5d3_uart.dtsi
> +++ b/arch/arm/boot/dts/sama5d3_uart.dtsi
> @@ -10,6 +10,7 @@
>   #include <dt-bindings/pinctrl/at91.h>
>   #include <dt-bindings/interrupt-controller/irq.h>
>   #include <dt-bindings/at91/sama5d3/peripherals.h>
> +#include <dt-bindings/clk/at91/at91sam9x5/clk-peripheral.h>
>
>   / {
>   	ahb {
> @@ -32,12 +33,28 @@
>   				};
>   			};
>
> +			pmc: pmc@...ffc00 {
> +				periph: periphck {
> +					uart0_clk {
> +						atmel,clk-id = <SAMA5D3_ID_UART0>;
> +						atmel,clk-default-divisor = <AT91SAM9X5_PERIPH_CLK_DIV2>;
> +					};
> +
> +					uart1_clk {
> +						atmel,clk-id = <SAMA5D3_ID_UART1>;
> +						atmel,clk-default-divisor = <AT91SAM9X5_PERIPH_CLK_DIV2>;
> +					};
> +				};
> +			};
> +
>   			uart0: serial@...24000 {
>   				compatible = "atmel,at91sam9260-usart";
>   				reg = <0xf0024000 0x200>;
>   				interrupts = <SAMA5D3_ID_UART0 IRQ_TYPE_LEVEL_HIGH 5>;
>   				pinctrl-names = "default";
>   				pinctrl-0 = <&pinctrl_uart0>;
> +				clocks = <&periph SAMA5D3_ID_UART0>;
> +				clock-names = "usart";
>   				status = "disabled";
>   			};
>
> @@ -47,6 +64,8 @@
>   				interrupts = <SAMA5D3_ID_UART1 IRQ_TYPE_LEVEL_HIGH 5>;
>   				pinctrl-names = "default";
>   				pinctrl-0 = <&pinctrl_uart1>;
> +				clocks = <&periph SAMA5D3_ID_UART1>;
> +				clock-names = "usart";
>   				status = "disabled";
>   			};
>   		};
>

... otherwise, it looks good.

Acked-by: Nicolas Ferre <nicolas.ferre@...el.com>

-- 
Nicolas Ferre
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ