Skip to content

Commit

Permalink
powerpc/85xx: clean up FPGA device tree nodes for Freecsale QorIQ boards
Browse files Browse the repository at this point in the history
Standarize and document the FPGA nodes used on Freescale QorIQ reference
boards.  There are different kinds of FPGAs used on the boards, but
only two are currently standard: "pixis", "ngpixis", and "qixis".  Although
there are minor differences among the boards that have one kind of FPGA, most
of the functionality is the same, so it makes sense to create common
compatibility strings.

We also need to update the P1022DS platform file, because the compatible
string for its PIXIS node has changed.  This means that older kernels are
not compatible with newer device trees.  This is not a real problem, however,
since that particular function doesn't work anyway.  When the DIU is active,
the PIXIS is in "indirect mode", and so cannot be accessed as a memory-mapped
device.

Signed-off-by: Timur Tabi <timur@freescale.com>
Signed-off-by: Kumar Gala <galak@kernel.crashing.org>
  • Loading branch information
Timur Tabi authored and Kumar Gala committed Oct 12, 2011
1 parent 878e3cb commit 499ccb2
Show file tree
Hide file tree
Showing 7 changed files with 42 additions and 22 deletions.
30 changes: 18 additions & 12 deletions Documentation/devicetree/bindings/powerpc/fsl/board.txt
Original file line number Diff line number Diff line change
@@ -1,3 +1,8 @@
Freescale Reference Board Bindings

This document describes device tree bindings for various devices that
exist on some Freescale reference boards.

* Board Control and Status (BCSR)

Required properties:
Expand All @@ -12,25 +17,26 @@ Example:
reg = <f8000000 8000>;
};

* Freescale on board FPGA
* Freescale on-board FPGA

This is the memory-mapped registers for on board FPGA.

Required properities:
- compatible : should be "fsl,fpga-pixis".
- reg : should contain the address and the length of the FPPGA register
set.
- compatible: should be a board-specific string followed by a string
indicating the type of FPGA. Example:
"fsl,<board>-fpga", "fsl,fpga-pixis"
- reg: should contain the address and the length of the FPGA register set.
- interrupt-parent: should specify phandle for the interrupt controller.
- interrupts : should specify event (wakeup) IRQ.
- interrupts: should specify event (wakeup) IRQ.

Example (MPC8610HPCD):
Example (P1022DS):

board-control@e8000000 {
compatible = "fsl,fpga-pixis";
reg = <0xe8000000 32>;
interrupt-parent = <&mpic>;
interrupts = <8 8>;
};
board-control@3,0 {
compatible = "fsl,p1022ds-fpga", "fsl,fpga-ngpixis";
reg = <3 0 0x30>;
interrupt-parent = <&mpic>;
interrupts = <8 8 0 0>;
};

* Freescale BCSR GPIO banks

Expand Down
2 changes: 1 addition & 1 deletion arch/powerpc/boot/dts/p1022ds.dts
Original file line number Diff line number Diff line change
Expand Up @@ -150,7 +150,7 @@
};

board-control@3,0 {
compatible = "fsl,p1022ds-pixis";
compatible = "fsl,p1022ds-fpga", "fsl,fpga-ngpixis";
reg = <3 0 0x30>;
interrupt-parent = <&mpic>;
/*
Expand Down
5 changes: 5 additions & 0 deletions arch/powerpc/boot/dts/p2020ds.dts
Original file line number Diff line number Diff line change
Expand Up @@ -118,6 +118,11 @@
};
};

board-control@3,0 {
compatible = "fsl,p2020ds-fpga", "fsl,fpga-ngpixis";
reg = <0x3 0x0 0x30>;
};

nand@4,0 {
compatible = "fsl,elbc-fcm-nand";
reg = <0x4 0x0 0x40000>;
Expand Down
4 changes: 2 additions & 2 deletions arch/powerpc/boot/dts/p3041ds.dts
Original file line number Diff line number Diff line change
Expand Up @@ -147,8 +147,8 @@
};

board-control@3,0 {
compatible = "fsl,p3041ds-pixis";
reg = <3 0 0x20>;
compatible = "fsl,p3041ds-fpga", "fsl,fpga-ngpixis";
reg = <3 0 0x30>;
};
};

Expand Down
8 changes: 7 additions & 1 deletion arch/powerpc/boot/dts/p4080ds.dts
Original file line number Diff line number Diff line change
Expand Up @@ -108,14 +108,20 @@

localbus@ffe124000 {
reg = <0xf 0xfe124000 0 0x1000>;
ranges = <0 0 0xf 0xe8000000 0x08000000>;
ranges = <0 0 0xf 0xe8000000 0x08000000
3 0 0xf 0xffdf0000 0x00008000>;

flash@0,0 {
compatible = "cfi-flash";
reg = <0 0 0x08000000>;
bank-width = <2>;
device-width = <2>;
};

board-control@3,0 {
compatible = "fsl,p4080ds-fpga", "fsl,fpga-ngpixis";
reg = <3 0 0x30>;
};
};

pci0: pcie@ffe200000 {
Expand Down
4 changes: 2 additions & 2 deletions arch/powerpc/boot/dts/p5020ds.dts
Original file line number Diff line number Diff line change
Expand Up @@ -147,8 +147,8 @@
};

board-control@3,0 {
compatible = "fsl,p5020ds-pixis";
reg = <3 0 0x20>;
compatible = "fsl,p5020ds-fpga", "fsl,fpga-ngpixis";
reg = <3 0 0x30>;
};
};

Expand Down
11 changes: 7 additions & 4 deletions arch/powerpc/platforms/85xx/p1022_ds.c
Original file line number Diff line number Diff line change
Expand Up @@ -128,17 +128,20 @@ static void p1022ds_set_gamma_table(int monitor_port, char *gamma_table_base)
*/
static void p1022ds_set_monitor_port(int monitor_port)
{
struct device_node *pixis_node;
struct device_node *np;
void __iomem *pixis;
u8 __iomem *brdcfg1;

pixis_node = of_find_compatible_node(NULL, NULL, "fsl,p1022ds-pixis");
if (!pixis_node) {
np = of_find_compatible_node(NULL, NULL, "fsl,p1022ds-fpga");
if (!np)
/* older device trees used "fsl,p1022ds-pixis" */
np = of_find_compatible_node(NULL, NULL, "fsl,p1022ds-pixis");
if (!np) {
pr_err("p1022ds: missing ngPIXIS node\n");
return;
}

pixis = of_iomap(pixis_node, 0);
pixis = of_iomap(np, 0);
if (!pixis) {
pr_err("p1022ds: could not map ngPIXIS registers\n");
return;
Expand Down

0 comments on commit 499ccb2

Please sign in to comment.