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] [day] [month] [year] [list]
Message-ID: <89792a78-95f7-e334-632c-9b0baf217024@microchip.com>
Date:   Tue, 24 Mar 2020 15:57:12 +0000
From:   <Nicolas.Ferre@...rochip.com>
To:     <alexandre.belloni@...tlin.com>, <Ludovic.Desroches@...rochip.com>
CC:     <linux-arm-kernel@...ts.infradead.org>,
        <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] ARM: dts: at91: rm9200: switch to new clock bindings

On 24/03/2020 at 13:41, Alexandre Belloni wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> 
> Switch sama5d3 boards to the new PMC clock bindings.

must be at91rm9200 there.

> 
> Signed-off-by: Alexandre Belloni <alexandre.belloni@...tlin.com>
> ---
>   arch/arm/boot/dts/at91rm9200.dtsi | 296 +++---------------------------
>   1 file changed, 23 insertions(+), 273 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/at91rm9200.dtsi b/arch/arm/boot/dts/at91rm9200.dtsi
> index 44385718d9d4..a5040f5ea641 100644
> --- a/arch/arm/boot/dts/at91rm9200.dtsi
> +++ b/arch/arm/boot/dts/at91rm9200.dtsi
> @@ -101,259 +101,9 @@ pmc: pmc@...ffc00 {
>                                  compatible = "atmel,at91rm9200-pmc", "syscon";
>                                  reg = <0xfffffc00 0x100>;
>                                  interrupts = <1 IRQ_TYPE_LEVEL_HIGH 7>;
> -                               interrupt-controller;
> -                               #address-cells = <1>;
> -                               #size-cells = <0>;
> -                               #interrupt-cells = <1>;
> -
> -                               main_osc: main_osc {
> -                                       compatible = "atmel,at91rm9200-clk-main-osc";
> -                                       #clock-cells = <0>;
> -                                       interrupts-extended = <&pmc AT91_PMC_MOSCS>;
> -                                       clocks = <&main_xtal>;
> -                               };
> -
> -                               main: mainck {
> -                                       compatible = "atmel,at91rm9200-clk-main";
> -                                       #clock-cells = <0>;
> -                                       clocks = <&main_osc>;
> -                               };
> -
> -                               plla: pllack {
> -                                       compatible = "atmel,at91rm9200-clk-pll";
> -                                       #clock-cells = <0>;
> -                                       interrupts-extended = <&pmc AT91_PMC_LOCKA>;
> -                                       clocks = <&main>;
> -                                       reg = <0>;
> -                                       atmel,clk-input-range = <1000000 32000000>;
> -                                       #atmel,pll-clk-output-range-cells = <3>;
> -                                       atmel,pll-clk-output-ranges = <80000000 160000000 0>,
> -                                                               <150000000 180000000 2>;
> -                               };
> -
> -                               pllb: pllbck {
> -                                       compatible = "atmel,at91rm9200-clk-pll";
> -                                       #clock-cells = <0>;
> -                                       interrupts-extended = <&pmc AT91_PMC_LOCKB>;
> -                                       clocks = <&main>;
> -                                       reg = <1>;
> -                                       atmel,clk-input-range = <1000000 32000000>;
> -                                       #atmel,pll-clk-output-range-cells = <3>;
> -                                       atmel,pll-clk-output-ranges = <80000000 160000000 0>,
> -                                                               <150000000 180000000 2>;
> -                               };
> -
> -                               mck: masterck {
> -                                       compatible = "atmel,at91rm9200-clk-master";
> -                                       #clock-cells = <0>;
> -                                       interrupts-extended = <&pmc AT91_PMC_MCKRDY>;
> -                                       clocks = <&slow_xtal>, <&main>, <&plla>, <&pllb>;
> -                                       atmel,clk-output-range = <0 80000000>;
> -                                       atmel,clk-divisors = <1 2 3 4>;
> -                               };
> -
> -                               usb: usbck {
> -                                       compatible = "atmel,at91rm9200-clk-usb";
> -                                       #clock-cells = <0>;
> -                                       atmel,clk-divisors = <1 2 0 0>;
> -                                       clocks = <&pllb>;
> -                               };
> -
> -                               prog: progck {
> -                                       compatible = "atmel,at91rm9200-clk-programmable";
> -                                       #address-cells = <1>;
> -                                       #size-cells = <0>;
> -                                       interrupt-parent = <&pmc>;
> -                                       clocks = <&slow_xtal>, <&main>, <&plla>, <&pllb>;
> -
> -                                       prog0: prog0 {
> -                                               #clock-cells = <0>;
> -                                               reg = <0>;
> -                                               interrupts = <AT91_PMC_PCKRDY(0)>;
> -                                       };
> -
> -                                       prog1: prog1 {
> -                                               #clock-cells = <0>;
> -                                               reg = <1>;
> -                                               interrupts = <AT91_PMC_PCKRDY(1)>;
> -                                       };
> -
> -                                       prog2: prog2 {
> -                                               #clock-cells = <0>;
> -                                               reg = <2>;
> -                                               interrupts = <AT91_PMC_PCKRDY(2)>;
> -                                       };
> -
> -                                       prog3: prog3 {
> -                                               #clock-cells = <0>;
> -                                               reg = <3>;
> -                                               interrupts = <AT91_PMC_PCKRDY(3)>;
> -                                       };
> -                               };
> -
> -                               systemck {
> -                                       compatible = "atmel,at91rm9200-clk-system";
> -                                       #address-cells = <1>;
> -                                       #size-cells = <0>;
> -
> -                                       udpck: udpck {
> -                                               #clock-cells = <0>;
> -                                               reg = <2>;
> -                                               clocks = <&usb>;
> -                                       };
> -
> -                                       uhpck: uhpck {
> -                                               #clock-cells = <0>;
> -                                               reg = <4>;
> -                                               clocks = <&usb>;
> -                                       };
> -
> -                                       pck0: pck0 {
> -                                               #clock-cells = <0>;
> -                                               reg = <8>;
> -                                               clocks = <&prog0>;
> -                                       };
> -
> -                                       pck1: pck1 {
> -                                               #clock-cells = <0>;
> -                                               reg = <9>;
> -                                               clocks = <&prog1>;
> -                                       };
> -
> -                                       pck2: pck2 {
> -                                               #clock-cells = <0>;
> -                                               reg = <10>;
> -                                               clocks = <&prog2>;
> -                                       };
> -
> -                                       pck3: pck3 {
> -                                               #clock-cells = <0>;
> -                                               reg = <11>;
> -                                               clocks = <&prog3>;
> -                                       };
> -                               };
> -
> -                               periphck {
> -                                       compatible = "atmel,at91rm9200-clk-peripheral";
> -                                       #address-cells = <1>;
> -                                       #size-cells = <0>;
> -                                       clocks = <&mck>;
> -
> -                                       pioA_clk: pioA_clk {
> -                                               #clock-cells = <0>;
> -                                               reg = <2>;
> -                                       };
> -
> -                                       pioB_clk: pioB_clk {
> -                                               #clock-cells = <0>;
> -                                               reg = <3>;
> -                                       };
> -
> -                                       pioC_clk: pioC_clk {
> -                                               #clock-cells = <0>;
> -                                               reg = <4>;
> -                                       };
> -
> -                                       pioD_clk: pioD_clk {
> -                                               #clock-cells = <0>;
> -                                               reg = <5>;
> -                                       };
> -
> -                                       usart0_clk: usart0_clk {
> -                                               #clock-cells = <0>;
> -                                               reg = <6>;
> -                                       };
> -
> -                                       usart1_clk: usart1_clk {
> -                                               #clock-cells = <0>;
> -                                               reg = <7>;
> -                                       };
> -
> -                                       usart2_clk: usart2_clk {
> -                                               #clock-cells = <0>;
> -                                               reg = <8>;
> -                                       };
> -
> -                                       usart3_clk: usart3_clk {
> -                                               #clock-cells = <0>;
> -                                               reg = <9>;
> -                                       };
> -
> -                                       mci0_clk: mci0_clk {
> -                                               #clock-cells = <0>;
> -                                               reg = <10>;
> -                                       };
> -
> -                                       udc_clk: udc_clk {
> -                                               #clock-cells = <0>;
> -                                               reg = <11>;
> -                                       };
> -
> -                                       twi0_clk: twi0_clk {
> -                                               reg = <12>;
> -                                               #clock-cells = <0>;
> -                                       };
> -
> -                                       spi0_clk: spi0_clk {
> -                                               #clock-cells = <0>;
> -                                               reg = <13>;
> -                                       };
> -
> -                                       ssc0_clk: ssc0_clk {
> -                                               #clock-cells = <0>;
> -                                               reg = <14>;
> -                                       };
> -
> -                                       ssc1_clk: ssc1_clk {
> -                                               #clock-cells = <0>;
> -                                               reg = <15>;
> -                                       };
> -
> -                                       ssc2_clk: ssc2_clk {
> -                                               #clock-cells = <0>;
> -                                               reg = <16>;
> -                                       };
> -
> -                                       tc0_clk: tc0_clk {
> -                                               #clock-cells = <0>;
> -                                               reg = <17>;
> -                                       };
> -
> -                                       tc1_clk: tc1_clk {
> -                                               #clock-cells = <0>;
> -                                               reg = <18>;
> -                                       };
> -
> -                                       tc2_clk: tc2_clk {
> -                                               #clock-cells = <0>;
> -                                               reg = <19>;
> -                                       };
> -
> -                                       tc3_clk: tc3_clk {
> -                                               #clock-cells = <0>;
> -                                               reg = <20>;
> -                                       };
> -
> -                                       tc4_clk: tc4_clk {
> -                                               #clock-cells = <0>;
> -                                               reg = <21>;
> -                                       };
> -
> -                                       tc5_clk: tc5_clk {
> -                                               #clock-cells = <0>;
> -                                               reg = <22>;
> -                                       };
> -
> -                                       ohci_clk: ohci_clk {
> -                                               #clock-cells = <0>;
> -                                               reg = <23>;
> -                                       };
> -
> -                                       macb0_clk: macb0_clk {
> -                                               #clock-cells = <0>;
> -                                               reg = <24>;
> -                                       };
> -                               };
> +                               #clock-cells = <2>;
> +                               clocks = <&slow_xtal>, <&main_xtal>;
> +                               clock-names = "slow_xtal", "main_xtal";
>                          };
> 
>                          st: timer@...ffd00 {
> @@ -383,7 +133,7 @@ tcb0: timer@...a0000 {
>                                  interrupts = <17 IRQ_TYPE_LEVEL_HIGH 0
>                                                18 IRQ_TYPE_LEVEL_HIGH 0
>                                                19 IRQ_TYPE_LEVEL_HIGH 0>;
> -                               clocks = <&tc0_clk>, <&tc1_clk>, <&tc2_clk>, <&slow_xtal>;
> +                               clocks = <&pmc PMC_TYPE_PERIPHERAL 17>, <&pmc PMC_TYPE_PERIPHERAL 18>, <&pmc PMC_TYPE_PERIPHERAL 19>, <&slow_xtal>;
>                                  clock-names = "t0_clk", "t1_clk", "t2_clk", "slow_clk";
>                          };
> 
> @@ -395,7 +145,7 @@ tcb1: timer@...a4000 {
>                                  interrupts = <20 IRQ_TYPE_LEVEL_HIGH 0
>                                                21 IRQ_TYPE_LEVEL_HIGH 0
>                                                22 IRQ_TYPE_LEVEL_HIGH 0>;
> -                               clocks = <&tc3_clk>, <&tc4_clk>, <&tc5_clk>, <&slow_xtal>;
> +                               clocks = <&pmc PMC_TYPE_PERIPHERAL 20>, <&pmc PMC_TYPE_PERIPHERAL 21>, <&pmc PMC_TYPE_PERIPHERAL 22>, <&slow_xtal>;
>                                  clock-names = "t0_clk", "t1_clk", "t2_clk", "slow_clk";
>                          };
> 
> @@ -405,7 +155,7 @@ i2c0: i2c@...b8000 {
>                                  interrupts = <12 IRQ_TYPE_LEVEL_HIGH 6>;
>                                  pinctrl-names = "default";
>                                  pinctrl-0 = <&pinctrl_twi>;
> -                               clocks = <&twi0_clk>;
> +                               clocks = <&pmc PMC_TYPE_PERIPHERAL 12>;
>                                  #address-cells = <1>;
>                                  #size-cells = <0>;
>                                  status = "disabled";
> @@ -415,7 +165,7 @@ mmc0: mmc@...b4000 {
>                                  compatible = "atmel,hsmci";
>                                  reg = <0xfffb4000 0x4000>;
>                                  interrupts = <10 IRQ_TYPE_LEVEL_HIGH 0>;
> -                               clocks = <&mci0_clk>;
> +                               clocks = <&pmc PMC_TYPE_PERIPHERAL 10>;
>                                  clock-names = "mci_clk";
>                                  #address-cells = <1>;
>                                  #size-cells = <0>;
> @@ -429,7 +179,7 @@ ssc0: ssc@...d0000 {
>                                  interrupts = <14 IRQ_TYPE_LEVEL_HIGH 5>;
>                                  pinctrl-names = "default";
>                                  pinctrl-0 = <&pinctrl_ssc0_tx &pinctrl_ssc0_rx>;
> -                               clocks = <&ssc0_clk>;
> +                               clocks = <&pmc PMC_TYPE_PERIPHERAL 14>;
>                                  clock-names = "pclk";
>                                  status = "disabled";
>                          };
> @@ -440,7 +190,7 @@ ssc1: ssc@...d4000 {
>                                  interrupts = <15 IRQ_TYPE_LEVEL_HIGH 5>;
>                                  pinctrl-names = "default";
>                                  pinctrl-0 = <&pinctrl_ssc1_tx &pinctrl_ssc1_rx>;
> -                               clocks = <&ssc1_clk>;
> +                               clocks = <&pmc PMC_TYPE_PERIPHERAL 15>;
>                                  clock-names = "pclk";
>                                  status = "disabled";
>                          };
> @@ -451,7 +201,7 @@ ssc2: ssc@...d8000 {
>                                  interrupts = <16 IRQ_TYPE_LEVEL_HIGH 5>;
>                                  pinctrl-names = "default";
>                                  pinctrl-0 = <&pinctrl_ssc2_tx &pinctrl_ssc2_rx>;
> -                               clocks = <&ssc2_clk>;
> +                               clocks = <&pmc PMC_TYPE_PERIPHERAL 16>;
>                                  clock-names = "pclk";
>                                  status = "disabled";
>                          };
> @@ -463,7 +213,7 @@ macb0: ethernet@...bc000 {
>                                  phy-mode = "rmii";
>                                  pinctrl-names = "default";
>                                  pinctrl-0 = <&pinctrl_macb_rmii>;
> -                               clocks = <&macb0_clk>;
> +                               clocks = <&pmc PMC_TYPE_PERIPHERAL 24>;
>                                  clock-names = "ether_clk";
>                                  status = "disabled";
>                          };
> @@ -803,7 +553,7 @@ pioA: gpio@...ff400 {
>                                          gpio-controller;
>                                          interrupt-controller;
>                                          #interrupt-cells = <2>;
> -                                       clocks = <&pioA_clk>;
> +                                       clocks = <&pmc PMC_TYPE_PERIPHERAL 2>;
>                                  };
> 
>                                  pioB: gpio@...ff600 {
> @@ -814,7 +564,7 @@ pioB: gpio@...ff600 {
>                                          gpio-controller;
>                                          interrupt-controller;
>                                          #interrupt-cells = <2>;
> -                                       clocks = <&pioB_clk>;
> +                                       clocks = <&pmc PMC_TYPE_PERIPHERAL 3>;
>                                  };
> 
>                                  pioC: gpio@...ff800 {
> @@ -825,7 +575,7 @@ pioC: gpio@...ff800 {
>                                          gpio-controller;
>                                          interrupt-controller;
>                                          #interrupt-cells = <2>;
> -                                       clocks = <&pioC_clk>;
> +                                       clocks = <&pmc PMC_TYPE_PERIPHERAL 4>;
>                                  };
> 
>                                  pioD: gpio@...ffa00 {
> @@ -836,7 +586,7 @@ pioD: gpio@...ffa00 {
>                                          gpio-controller;
>                                          interrupt-controller;
>                                          #interrupt-cells = <2>;
> -                                       clocks = <&pioD_clk>;
> +                                       clocks = <&pmc PMC_TYPE_PERIPHERAL 5>;
>                                  };
>                          };
> 
> @@ -846,7 +596,7 @@ dbgu: serial@...ff200 {
>                                  interrupts = <1 IRQ_TYPE_LEVEL_HIGH 7>;
>                                  pinctrl-names = "default";
>                                  pinctrl-0 = <&pinctrl_dbgu>;
> -                               clocks = <&mck>;
> +                               clocks = <&pmc PMC_TYPE_CORE PMC_MCK>;
>                                  clock-names = "usart";
>                                  status = "disabled";
>                          };
> @@ -859,7 +609,7 @@ usart0: serial@...c0000 {
>                                  atmel,use-dma-tx;
>                                  pinctrl-names = "default";
>                                  pinctrl-0 = <&pinctrl_uart0>;
> -                               clocks = <&usart0_clk>;
> +                               clocks = <&pmc PMC_TYPE_PERIPHERAL 6>;
>                                  clock-names = "usart";
>                                  status = "disabled";
>                          };
> @@ -872,7 +622,7 @@ usart1: serial@...c4000 {
>                                  atmel,use-dma-tx;
>                                  pinctrl-names = "default";
>                                  pinctrl-0 = <&pinctrl_uart1>;
> -                               clocks = <&usart1_clk>;
> +                               clocks = <&pmc PMC_TYPE_PERIPHERAL 7>;
>                                  clock-names = "usart";
>                                  status = "disabled";
>                          };
> @@ -885,7 +635,7 @@ usart2: serial@...c8000 {
>                                  atmel,use-dma-tx;
>                                  pinctrl-names = "default";
>                                  pinctrl-0 = <&pinctrl_uart2>;
> -                               clocks = <&usart2_clk>;
> +                               clocks = <&pmc PMC_TYPE_PERIPHERAL 8>;
>                                  clock-names = "usart";
>                                  status = "disabled";
>                          };
> @@ -898,7 +648,7 @@ usart3: serial@...cc000 {
>                                  atmel,use-dma-tx;
>                                  pinctrl-names = "default";
>                                  pinctrl-0 = <&pinctrl_uart3>;
> -                               clocks = <&usart3_clk>;
> +                               clocks = <&pmc PMC_TYPE_PERIPHERAL 9>;
>                                  clock-names = "usart";
>                                  status = "disabled";
>                          };
> @@ -907,7 +657,7 @@ usb1: gadget@...b0000 {
>                                  compatible = "atmel,at91rm9200-udc";
>                                  reg = <0xfffb0000 0x4000>;
>                                  interrupts = <11 IRQ_TYPE_LEVEL_HIGH 2>;
> -                               clocks = <&udc_clk>, <&udpck>;
> +                               clocks = <&pmc PMC_TYPE_PERIPHERAL 11>, <&pmc PMC_TYPE_SYSTEM 2>;
>                                  clock-names = "pclk", "hclk";
>                                  status = "disabled";
>                          };
> @@ -920,7 +670,7 @@ spi0: spi@...e0000 {
>                                  interrupts = <13 IRQ_TYPE_LEVEL_HIGH 3>;
>                                  pinctrl-names = "default";
>                                  pinctrl-0 = <&pinctrl_spi0>;
> -                               clocks = <&spi0_clk>;
> +                               clocks = <&pmc PMC_TYPE_PERIPHERAL 13>;
>                                  clock-names = "spi_clk";
>                                  status = "disabled";
>                          };
> @@ -947,7 +697,7 @@ usb0: ohci@...000 {
>                          compatible = "atmel,at91rm9200-ohci", "usb-ohci";
>                          reg = <0x00300000 0x100000>;
>                          interrupts = <23 IRQ_TYPE_LEVEL_HIGH 2>;
> -                       clocks = <&ohci_clk>, <&ohci_clk>, <&uhpck>;
> +                       clocks = <&pmc PMC_TYPE_PERIPHERAL 23>, <&pmc PMC_TYPE_PERIPHERAL 23>, <&pmc PMC_TYPE_SYSTEM 4>;
>                          clock-names = "ohci_clk", "hclk", "uhpck";
>                          status = "disabled";
>                  };
> --
> 2.25.1
> 

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ