[PATCH 6/7] [POWERPC] booting-without-of: add FHCI USB, FSL MCU, FSL UPM and GPIO LEDs bindings
This patch adds few bindings for the new drivers to be submitted through appropriate maintainers. Signed-off-by: Anton Vorontsov [EMAIL PROTECTED] --- Documentation/powerpc/booting-without-of.txt | 125 ++ 1 files changed, 125 insertions(+), 0 deletions(-) diff --git a/Documentation/powerpc/booting-without-of.txt b/Documentation/powerpc/booting-without-of.txt index 1ad903a..8b788a1 100644 --- a/Documentation/powerpc/booting-without-of.txt +++ b/Documentation/powerpc/booting-without-of.txt @@ -62,6 +62,10 @@ Table of Contents s) Freescale on board FPGA t) Freescael MSI interrupt controller u) Freescale General-purpose Timers Module + v) Freescale QUICC Engine USB Controller + w) Freescale MCU with MPC8349E-mITX compatible firmware + x) Freescale Localbus UPM programmed to work with NAND flash + y) LEDs on GPIOs VII - Marvell Discovery mv64[345]6x System Controller chips 1) The /system-controller node @@ -2970,6 +2974,127 @@ platforms are moved over to use the flattened-device-tree model. clock-frequency = 0; }; +v) Freescale QUICC Engine USB Controller + +Required properties: + - compatible : should be fsl,chip-qe-usb, fsl,mpc8323-qe-usb; + - reg : the first two cells should contain gtm registers location and +length, the next two two cells should contain PRAM location and +length. + - interrupts : should contain USB interrupt. + - interrupt-parent : interrupt source phandle. + - fsl,fullspeed-clock : specifies the full speed USB clock source in +clknum or brgnum format. + - fsl,lowspeed-clock : specifies the low speed USB clock source in +clknum or brgnum format. + - fsl,usb-mode : should be host. + - linux,hub-power-budget : optional, USB power budget for the root hub +in mA. + - gpios : should specify GPIOs in this order: USBOE, USBTP, USBTN, USBRP, +USBRN, SPEED (optional), and POWER (optional). + +Example: + + [EMAIL PROTECTED] { + compatible = fsl,mpc8360-qe-usb, fsl,mpc8323-qe-usb; + reg = 0x6c0 0x40 0x8b00 0x100; + interrupts = 11; + interrupt-parent = qeic; + fsl,fullspeed-clock = clk21; + fsl,usb-mode = host; + gpios = qe_pio_b 2 0 /* USBOE */ +qe_pio_b 3 0 /* USBTP */ +qe_pio_b 8 0 /* USBTN */ +qe_pio_b 9 0 /* USBRP */ +qe_pio_b 11 0 /* USBRN */ +qe_pio_e 20 0 /* SPEED */ +qe_pio_e 21 0 /* POWER */; + }; + +w) Freescale MCU with MPC8349E-mITX compatible firmware + +Required properties: + - compatible : fsl,mcu-chip-board, fsl,mcu-mpc8349emitx; + - reg : should specify I2C address (0x0a). + - #address-cells : should be 0. + - #size-cells : should be 0. + - #gpio-cells : should be 2. + - gpio-controller : should be present; + +Example: + + mcu_pio: [EMAIL PROTECTED] { + #address-cells = 0; + #size-cells = 0; + #gpio-cells = 2; + compatible = fsl,mc9s08qg8-mpc8349emitx, +fsl,mcu-mpc8349emitx; + reg = 0x0a; + gpio-controller; + }; + +x) Freescale Localbus UPM programmed to work with NAND flash + + Required properties: + - #address-cells : should be 0; + - #size-cells : should be 0; + - compatible : fsl,upm-nand. + - reg : should specify localbus chip select and size used for the chip. + - fsl,upm-addr-offset : UPM pattern offset for the address latch. + - fsl,upm-cmd-offset : UPM pattern offset for the command latch. + - gpios : may specify optional GPIO connected to the Ready-Not-Busy pin. + + Example: + + [EMAIL PROTECTED],0 { + #address-cells = 0; + #size-cells = 0; + compatible = fsl,upm-nand; + reg = 1 0 1; + fsl,upm-addr-offset = 16; + fsl,upm-cmd-offset = 8; + gpios = qe_pio_e 18 0; + + flash { + #address-cells = 1; + #size-cells = 1; + compatible = stmicro,NAND512W3A2BN6E; + + [EMAIL PROTECTED] { + ... + }; + }; + }; + +y) LEDs on GPIOs + +Required properties: + - compatible : should be linux,gpio-led. + - linux,name : LED name. + - linux,active-low : property should be present if LED wired as +active-low. + - linux,default-trigger : Linux default trigger for this LED. + - linux,brightness : default brightness. + - gpios : should specify LED GPIO. + +Example: + + [EMAIL PROTECTED] { +
Re: [PATCH 6/7] [POWERPC] booting-without-of: add FHCI USB, FSL MCU, FSL UPM and GPIO LEDs bindings
+ - fsl,fullspeed-clock : specifies the full speed USB clock source in +clknum or brgnum format. + - fsl,lowspeed-clock : specifies the low speed USB clock source in +clknum or brgnum format. What format is num in? + - fsl,usb-mode : should be host. If that's the only possible value, this property is unnecessary, no? It probably would make sense to make this optional (and default to host) anyway. + - linux,hub-power-budget : optional, USB power budget for the root hub +in mA. Why is this linux-specific? +w) Freescale MCU with MPC8349E-mITX compatible firmware + +Required properties: + - compatible : fsl,mcu-chip-board, fsl,mcu-mpc8349emitx; + - reg : should specify I2C address (0x0a). + - #address-cells : should be 0. + - #size-cells : should be 0. +x) Freescale Localbus UPM programmed to work with NAND flash Similar here, except this one is never a GPIO controller. If the point to have #a = #s = 0 is to not have a unit-address in the child nodes: you should do that simply by not specifying a reg in the child nodes. +y) LEDs on GPIOs This one is so full of linux, stuff that I won't review it -- I wouldn't know where to start, sorry. Segher ___ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev
Re: [PATCH 6/7] [POWERPC] booting-without-of: add FHCI USB, FSL MCU, FSL UPM and GPIO LEDs bindings
On Sat, May 24, 2008 at 12:10:56AM +0200, Segher Boessenkool wrote: + - fsl,fullspeed-clock : specifies the full speed USB clock source in +clknum or brgnum format. + - fsl,lowspeed-clock : specifies the low speed USB clock source in +clknum or brgnum format. What format is num in? + - fsl,usb-mode : should be host. If that's the only possible value, This could be peripheral and otg in future. this property is unnecessary, no? It probably would make sense to make this optional (and default to host) anyway. + - linux,hub-power-budget : optional, USB power budget for the root hub +in mA. Why is this linux-specific? +w) Freescale MCU with MPC8349E-mITX compatible firmware + +Required properties: + - compatible : fsl,mcu-chip-board, fsl,mcu-mpc8349emitx; + - reg : should specify I2C address (0x0a). + - #address-cells : should be 0. + - #size-cells : should be 0. +x) Freescale Localbus UPM programmed to work with NAND flash Similar here, except this one is never a GPIO controller. If the point to have #a = #s = 0 is to not have a unit-address in the child nodes: you should do that simply by not specifying a reg in the child nodes. +y) LEDs on GPIOs This one is so full of linux, stuff that I won't review it -- I wouldn't know where to start, sorry. How about this version? (diff against the previous version is here, and new full version will follow) Doubts: 1. I removed linux,name and now Linux will use node's name. Is this okay, or name property would be better? 2. I can't thinkout replacement for the linux,default-trigger, this is really linux-specific. The value could be anything Linux implements currently or will implement in the future... diff --git a/Documentation/powerpc/booting-without-of.txt b/Documentation/powerpc/booting-without-of.txt index 07aad16..8e3f743 100644 --- a/Documentation/powerpc/booting-without-of.txt +++ b/Documentation/powerpc/booting-without-of.txt @@ -2951,18 +2951,22 @@ platforms are moved over to use the flattened-device-tree model. Required properties: - compatible : should be fsl,chip-qe-usb, fsl,mpc8323-qe-usb; - - reg : the first two cells should contain gtm registers location and + - reg : the first two cells should contain usb registers location and length, the next two two cells should contain PRAM location and length. - interrupts : should contain USB interrupt. - interrupt-parent : interrupt source phandle. - - fsl,fullspeed-clock : specifies the full speed USB clock source in -clknum or brgnum format. - - fsl,lowspeed-clock : specifies the low speed USB clock source in -clknum or brgnum format. - - fsl,usb-mode : should be host. - - linux,hub-power-budget : optional, USB power budget for the root hub -in mA. + - fsl,fullspeed-clock : specifies the full speed USB clock source: +none: clock source is disabled +brg1 through brg16: clock source is BRG1-BRG16, respectively +clk1 through clk24: clock source is CLK1-CLK24, respectively + - fsl,lowspeed-clock : specifies the low speed USB clock source: +none: clock source is disabled +brg1 through brg16: clock source is BRG1-BRG16, respectively +clk1 through clk24: clock source is CLK1-CLK24, respectively + - fsl,usb-mode : (optional) so far acceptable value is host only +(this is default mode if property is absent). + - hub-power-budget : optional, USB power budget for the root hub, in mA. - gpios : should specify GPIOs in this order: USBOE, USBTP, USBTN, USBRP, USBRN, SPEED (optional), and POWER (optional). @@ -2974,7 +2978,6 @@ platforms are moved over to use the flattened-device-tree model. interrupts = 11; interrupt-parent = qeic; fsl,fullspeed-clock = clk21; - fsl,usb-mode = host; gpios = qe_pio_b 2 0 /* USBOE */ qe_pio_b 3 0 /* USBTP */ qe_pio_b 8 0 /* USBTN */ @@ -2989,16 +2992,12 @@ platforms are moved over to use the flattened-device-tree model. Required properties: - compatible : fsl,mcu-chip-board, fsl,mcu-mpc8349emitx; - reg : should specify I2C address (0x0a). - - #address-cells : should be 0. - - #size-cells : should be 0. - #gpio-cells : should be 2. - gpio-controller : should be present; Example: mcu_pio: [EMAIL PROTECTED] { - #address-cells = 0; - #size-cells = 0; #gpio-cells = 2; compatible = fsl,mc9s08qg8-mpc8349emitx, fsl,mcu-mpc8349emitx; @@ -3009,8 +3008,6 @@ platforms are moved over to use the flattened-device-tree model. x) Freescale Localbus UPM programmed to work with NAND flash
[PATCH 6/7] [POWERPC] booting-without-of: add FHCI USB, FSL MCU, FSL UPM and GPIO LEDs bindings
This patch adds few bindings for the new drivers to be submitted through appropriate maintainers. Signed-off-by: Anton Vorontsov [EMAIL PROTECTED] --- Documentation/powerpc/booting-without-of.txt | 114 ++ 1 files changed, 114 insertions(+), 0 deletions(-) diff --git a/Documentation/powerpc/booting-without-of.txt b/Documentation/powerpc/booting-without-of.txt index 7c01730..8e3f743 100644 --- a/Documentation/powerpc/booting-without-of.txt +++ b/Documentation/powerpc/booting-without-of.txt @@ -62,6 +62,10 @@ Table of Contents s) Freescale on board FPGA t) Freescael MSI interrupt controller u) Freescale General-purpose Timers Module + v) Freescale QUICC Engine USB Controller + w) Freescale MCU with MPC8349E-mITX compatible firmware + x) Freescale Localbus UPM programmed to work with NAND flash + y) LEDs on GPIOs VII - Marvell Discovery mv64[345]6x System Controller chips 1) The /system-controller node @@ -2943,6 +2947,116 @@ platforms are moved over to use the flattened-device-tree model. clock-frequency = 0; }; +v) Freescale QUICC Engine USB Controller + +Required properties: + - compatible : should be fsl,chip-qe-usb, fsl,mpc8323-qe-usb; + - reg : the first two cells should contain usb registers location and +length, the next two two cells should contain PRAM location and +length. + - interrupts : should contain USB interrupt. + - interrupt-parent : interrupt source phandle. + - fsl,fullspeed-clock : specifies the full speed USB clock source: +none: clock source is disabled +brg1 through brg16: clock source is BRG1-BRG16, respectively +clk1 through clk24: clock source is CLK1-CLK24, respectively + - fsl,lowspeed-clock : specifies the low speed USB clock source: +none: clock source is disabled +brg1 through brg16: clock source is BRG1-BRG16, respectively +clk1 through clk24: clock source is CLK1-CLK24, respectively + - fsl,usb-mode : (optional) so far acceptable value is host only +(this is default mode if property is absent). + - hub-power-budget : optional, USB power budget for the root hub, in mA. + - gpios : should specify GPIOs in this order: USBOE, USBTP, USBTN, USBRP, +USBRN, SPEED (optional), and POWER (optional). + +Example: + + [EMAIL PROTECTED] { + compatible = fsl,mpc8360-qe-usb, fsl,mpc8323-qe-usb; + reg = 0x6c0 0x40 0x8b00 0x100; + interrupts = 11; + interrupt-parent = qeic; + fsl,fullspeed-clock = clk21; + gpios = qe_pio_b 2 0 /* USBOE */ +qe_pio_b 3 0 /* USBTP */ +qe_pio_b 8 0 /* USBTN */ +qe_pio_b 9 0 /* USBRP */ +qe_pio_b 11 0 /* USBRN */ +qe_pio_e 20 0 /* SPEED */ +qe_pio_e 21 0 /* POWER */; + }; + +w) Freescale MCU with MPC8349E-mITX compatible firmware + +Required properties: + - compatible : fsl,mcu-chip-board, fsl,mcu-mpc8349emitx; + - reg : should specify I2C address (0x0a). + - #gpio-cells : should be 2. + - gpio-controller : should be present; + +Example: + + mcu_pio: [EMAIL PROTECTED] { + #gpio-cells = 2; + compatible = fsl,mc9s08qg8-mpc8349emitx, +fsl,mcu-mpc8349emitx; + reg = 0x0a; + gpio-controller; + }; + +x) Freescale Localbus UPM programmed to work with NAND flash + + Required properties: + - compatible : fsl,upm-nand. + - reg : should specify localbus chip select and size used for the chip. + - fsl,upm-addr-offset : UPM pattern offset for the address latch. + - fsl,upm-cmd-offset : UPM pattern offset for the command latch. + - gpios : may specify optional GPIO connected to the Ready-Not-Busy pin. + + Example: + + [EMAIL PROTECTED],0 { + compatible = fsl,upm-nand; + reg = 1 0 1; + fsl,upm-addr-offset = 16; + fsl,upm-cmd-offset = 8; + gpios = qe_pio_e 18 0; + + flash { + #address-cells = 1; + #size-cells = 1; + compatible = stmicro,NAND512W3A2BN6E; + + [EMAIL PROTECTED] { + ... + }; + }; + }; + +y) LEDs on GPIOs + +Required properties: + - compatible : should be gpio-led. + - default-brightness : (optional) default brightness: 1 on, 0 off. +Assumed off if property doesn't present. + - linux,default-trigger : (optional) Linux default trigger for this LED. + - gpios : should specify LED GPIO. + +Example: + + [EMAIL PROTECTED] { + compatible
[PATCH 6/7] [POWERPC] booting-without-of: add FHCI USB, FSL MCU, FSL UPM and GPIO LEDs bindings
This patch adds few bindings for the new drivers to be submitted through appropriate maintainers. Signed-off-by: Anton Vorontsov [EMAIL PROTECTED] --- Documentation/powerpc/booting-without-of.txt | 125 ++ 1 files changed, 125 insertions(+), 0 deletions(-) diff --git a/Documentation/powerpc/booting-without-of.txt b/Documentation/powerpc/booting-without-of.txt index c1044ee..4e15c13 100644 --- a/Documentation/powerpc/booting-without-of.txt +++ b/Documentation/powerpc/booting-without-of.txt @@ -61,6 +61,10 @@ Table of Contents r) Freescale Display Interface Unit s) Freescale on board FPGA t) Freescale General-purpose Timers Module + u) Freescale QUICC Engine USB Controller + v) LEDs on GPIOs + w) Freescale MCU with MPC8349E-mITX compatible firmware + x) Freescale Localbus UPM programmed to work with NAND flash VII - Marvell Discovery mv64[345]6x System Controller chips 1) The /system-controller node @@ -2932,6 +2936,127 @@ platforms are moved over to use the flattened-device-tree model. clock-frequency = 0; }; +u) Freescale QUICC Engine USB Controller + +Required properties: + - compatible : should be fsl,chip-qe-usb, fsl,mpc8323-qe-usb; + - reg : the first two cells should contain gtm registers location and +length, the next two two cells should contain PRAM location and +length. + - interrupts : should contain USB interrupt. + - interrupt-parent : interrupt source phandle. + - fsl,fullspeed-clock : specifies the full speed USB clock source in +clknum or brgnum format. + - fsl,lowspeed-clock : specifies the low speed USB clock source in +clknum or brgnum format. + - fsl,usb-mode : should be host. + - linux,hub-power-budget : optional, USB power budget for the root hub +in mA. + - gpios : should specify GPIOs in this order: USBOE, USBTP, USBTN, USBRP, +USBRN, SPEED (optional), and POWER (optional). + +Example: + + [EMAIL PROTECTED] { + compatible = fsl,mpc8360-qe-usb, fsl,mpc8323-qe-usb; + reg = 0x6c0 0x40 0x8b00 0x100; + interrupts = 11; + interrupt-parent = qeic; + fsl,fullspeed-clock = clk21; + fsl,usb-mode = host; + gpios = qe_pio_b 2 0 /* USBOE */ +qe_pio_b 3 0 /* USBTP */ +qe_pio_b 8 0 /* USBTN */ +qe_pio_b 9 0 /* USBRP */ +qe_pio_b 11 0 /* USBRN */ +qe_pio_e 20 0 /* SPEED */ +qe_pio_e 21 0 /* POWER */; + }; + +v) LEDs on GPIOs + +Required properties: + - compatible : should be linux,gpio-led. + - linux,name : LED name. + - linux,active-low : property should be present if LED wired as +active-low. + - linux,default-trigger : Linux default trigger for this LED. + - linux,brightness : default brightness. + - gpios : should specify LED GPIO. + +Example: + + [EMAIL PROTECTED] { + compatible = linux,gpio-led; + linux,name = pwr; + linux,brightness = 1; + linux,active-low; + gpios = mcu_pio 0; + }; + + [EMAIL PROTECTED] { + compatible = linux,gpio-led; + linux,name = hdd; + linux,default-trigger = ide-disk; + linux,active-low; + gpios = mcu_pio 1; + }; + +w) Freescale MCU with MPC8349E-mITX compatible firmware + +Required properties: + - compatible : fsl,mcu-chip-board, fsl,mcu-mpc8349emitx; + - reg : should specify I2C address (0x0a). + - #address-cells : should be 0. + - #size-cells : should be 0. + - #gpio-cells : should be 2. + - gpio-controller : should be present; + +Example: + + mcu_pio: [EMAIL PROTECTED] { + #address-cells = 0; + #size-cells = 0; + #gpio-cells = 2; + compatible = fsl,mc9s08qg8-mpc8349emitx, +fsl,mcu-mpc8349emitx; + reg = 0x0a; + gpio-controller; + }; + +x) Freescale Localbus UPM programmed to work with NAND flash + + Required properties: + - #address-cells : should be 0; + - #size-cells : should be 0; + - compatible : fsl,upm-nand. + - reg : should specify localbus chip select and size used for the chip. + - fsl,upm-addr-offset : UPM pattern offset for the address latch. + - fsl,upm-cmd-offset : UPM pattern offset for the command latch. + - gpios : may specify optional GPIO connected to the Ready-Not-Busy pin. + + Example: + + [EMAIL PROTECTED],0 { + #address-cells = 0; + #size-cells = 0; + compatible = fsl,upm-nand; + reg = 1 0 1; +