[Pdbg] [PATCH 1/2] Add checkstop FIR support via DTS
Alistair Popple
alistair at popple.id.au
Wed Feb 14 17:07:22 AEDT 2018
Hi Balbir,
> +mc {
> + fir_mask=<0x1800000 0x0>;
> + instances=<0x0 0x1>;
> + compatible="ibm,chiplet-fir";
> +};
There are plans to add each chiplet as it's own node in the device-tree with an
associated ibm,chiplet-id = <0x??> to idenitfy it. Rather than having to have a
seperate chiplet representation for a chiplets top-level FIR register I think it
makes more sense to link the bits set in global_fir with device tree nodes based
on the chiplet-id property. So for example the above would become:
eg:
mc01 {
reg = <0xc0ffee>;
ibm,chiplet-id = <0x7>;
};
mc23 {
reg = <0xdeadbeef>;
ibm,chiplet-id = <0x8>;
};
Admittedly this results in each chiplet having to explicitly exist in the
device-tree, but that will be required anyway for the "reg = <...>" properties.
Anyway let me know your thoughts.
- Alistair
> +
> +ob {
> + fir_mask=<0x780000 0x0>;
> + instances=<0x0 0x3>;
> + compatible="ibm,chiplet-fir";
> +};
> +
> +pci0 {
> + fir_mask=<0x40000 0x0>;
> + instances=<0x0 0x0>;
> + compatible="ibm,chiplet-fir";
> +};
> +
> +pci1 {
> + fir_mask=<0x20000 0x0>;
> + instances=<0x1 0x1>;
> + compatible="ibm,chiplet-fir";
> +};
> +
> +pci2 {
> + fir_mask=<0x10000 0x0>;
> + instances=<0x2 0x2>;
> + compatible="ibm,chiplet-fir";
> +};
> +
> +eq {
> + fir_mask=<0xfc00 0x0>;
> + instances=<0x0 0x5>;
> + compatible="ibm,chiplet-fir";
> +};
> +
> +ec {
> + fir_mask=<0x0 0xffffff00>;
> + instances=<0x0 0x17>;
> + compatible="ibm,chiplet-fir";
> +};
> +
> diff --git a/p9-fsi.dtsi.m4 b/p9-fsi.dtsi.m4
> index fde71e4..bea7b9f 100644
> --- a/p9-fsi.dtsi.m4
> +++ b/p9-fsi.dtsi.m4
> @@ -32,6 +32,7 @@
> include(p9-pib.dts.m4)dnl
> };
> };
> + include(p9-fir.dtsi.m4)
>
> };
> };
> diff --git a/p9-host.dts.m4 b/p9-host.dts.m4
> index 5566acd..31ebed2 100644
> --- a/p9-host.dts.m4
> +++ b/p9-host.dts.m4
> @@ -18,4 +18,5 @@
> index = <0x1>;
> include(p9-pib.dts.m4)dnl
> };
> + include(p9-fir.dtsi.m4)
> };
> diff --git a/p9-kernel.dts.m4 b/p9-kernel.dts.m4
> index ff19779..b30eaa9 100644
> --- a/p9-kernel.dts.m4
> +++ b/p9-kernel.dts.m4
> @@ -33,5 +33,6 @@
> include(p9-pib.dts.m4)dnl
> };
> };
> + include(p9-fir.dtsi.m4)
> };
> };
>
More information about the Pdbg
mailing list