Re: [PATCH] powerpc: add FHCI USB, FSL MCU, FSL UPM and GPIO LEDs bindings
Hi Anton, Anton Vorontsov wrote: This patch adds few bindings for the new drivers to be submitted through the appropriate maintainers. Signed-off-by: Anton Vorontsov <[EMAIL PROTECTED]> --- Segher, thanks for the comments on the previous version. Do you see any issues with this one or anything I could improve further? And if there are no issues... Kumar, could you apply this for powerpc-next? As you know, I have some additions to your bindings and I posted a patch recently: http://ozlabs.org/pipermail/linuxppc-dev/2008-June/058617.html Segher did not comment yet, maybe you would like to have a look. And do you have some news about the corresponding NAND support from the MTD people? I really appreciate breaking booting-without-of.txt up. Will this happen already for powerpc-next (for 2.6.27)? Thanks, Wolfgang. ___ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev
Re: [PATCH] powerpc: add FHCI USB, FSL MCU, FSL UPM and GPIO LEDs bindings
On Jul 3, 2008, at 1:24 PM, Anton Vorontsov wrote: On Thu, Jul 03, 2008 at 12:50:01PM -0500, Kumar Gala wrote: On Jul 3, 2008, at 12:48 PM, Anton Vorontsov wrote: This patch adds few bindings for the new drivers to be submitted through the appropriate maintainers. Signed-off-by: Anton Vorontsov <[EMAIL PROTECTED]> --- Segher, thanks for the comments on the previous version. Do you see any issues with this one or anything I could improve further? And if there are no issues... Kumar, could you apply this for powerpc-next? Thanks. Documentation/powerpc/booting-without-of.txt | 111 +++ ++ + 1 files changed, 111 insertions(+), 0 deletions(-) diff --git a/Documentation/powerpc/booting-without-of.txt b/ Documentation/powerpc/booting-without-of.txt index b68684d..a7ea549 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 MCU doesn't mean much to me. Its good to expand out the acronym at least once. MCU stands for MicroController Unit, and even expanded this doesn't mean much to anybody. :-) Freescale documentation names it as boards' "MCU Subsystem", that is, the chip provides fan control, GPIOs, software halt, etc. Basically, it is boards' power management companion chip. I don't know how to name it more correctly, so I adhered to documentation's naming (they simply call it MCU). ^^ I'll add some of these words to the node description though. I agree, but MicroController lets me a know a bit more than the random letters M-C-U. Just one reference in the heading is sufficient. - k ___ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev
Re: [PATCH] powerpc: add FHCI USB, FSL MCU, FSL UPM and GPIO LEDs bindings
On Thu, Jul 03, 2008 at 12:50:01PM -0500, Kumar Gala wrote: > > On Jul 3, 2008, at 12:48 PM, Anton Vorontsov wrote: > >> This patch adds few bindings for the new drivers to be submitted >> through >> the appropriate maintainers. >> >> Signed-off-by: Anton Vorontsov <[EMAIL PROTECTED]> >> --- >> >> Segher, thanks for the comments on the previous version. Do you see >> any >> issues with this one or anything I could improve further? >> >> And if there are no issues... Kumar, could you apply this for >> powerpc-next? >> >> Thanks. >> >> Documentation/powerpc/booting-without-of.txt | 111 + >> + >> 1 files changed, 111 insertions(+), 0 deletions(-) >> >> diff --git a/Documentation/powerpc/booting-without-of.txt b/ >> Documentation/powerpc/booting-without-of.txt >> index b68684d..a7ea549 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 > > MCU doesn't mean much to me. Its good to expand out the acronym at > least once. MCU stands for MicroController Unit, and even expanded this doesn't mean much to anybody. :-) Freescale documentation names it as boards' "MCU Subsystem", that is, the chip provides fan control, GPIOs, software halt, etc. Basically, it is boards' power management companion chip. I don't know how to name it more correctly, so I adhered to documentation's naming (they simply call it MCU). ^^ I'll add some of these words to the node description though. Thanks, -- Anton Vorontsov email: [EMAIL PROTECTED] irc://irc.freenode.net/bd2 ___ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev
Re: [PATCH] powerpc: add FHCI USB, FSL MCU, FSL UPM and GPIO LEDs bindings
On Jul 3, 2008, at 12:48 PM, Anton Vorontsov wrote: This patch adds few bindings for the new drivers to be submitted through the appropriate maintainers. Signed-off-by: Anton Vorontsov <[EMAIL PROTECTED]> --- Segher, thanks for the comments on the previous version. Do you see any issues with this one or anything I could improve further? And if there are no issues... Kumar, could you apply this for powerpc-next? Thanks. Documentation/powerpc/booting-without-of.txt | 111 + + 1 files changed, 111 insertions(+), 0 deletions(-) diff --git a/Documentation/powerpc/booting-without-of.txt b/ Documentation/powerpc/booting-without-of.txt index b68684d..a7ea549 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 MCU doesn't mean much to me. Its good to expand out the acronym at least once. - k ___ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev
[PATCH] powerpc: add FHCI USB, FSL MCU, FSL UPM and GPIO LEDs bindings
This patch adds few bindings for the new drivers to be submitted through the appropriate maintainers. Signed-off-by: Anton Vorontsov <[EMAIL PROTECTED]> --- Segher, thanks for the comments on the previous version. Do you see any issues with this one or anything I could improve further? And if there are no issues... Kumar, could you apply this for powerpc-next? Thanks. Documentation/powerpc/booting-without-of.txt | 111 ++ 1 files changed, 111 insertions(+), 0 deletions(-) diff --git a/Documentation/powerpc/booting-without-of.txt b/Documentation/powerpc/booting-without-of.txt index b68684d..a7ea549 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 @@ -2967,6 +2971,113 @@ 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,-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 + - 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,-", "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 = "..."; + + [EMAIL PROTECTED] { + ... + }; + }; + }; + +y) LEDs on GPIOs + +Required properties: + - compatible : should be "gpio-led". + - label : (optional) the label for this LED. If omitted, the label is +taken from the node name (excluding the unit address). + - gpios : should specify LED GP