[Skiboot] [PATCH] phb4: The "AB detect" mechanism doesn't exist anymore

Oliver O'Halloran oohall at gmail.com
Mon Jan 9 11:32:04 AEDT 2017


On Mon, Jan 9, 2017 at 10:30 AM, Benjamin Herrenschmidt
<benh at kernel.crashing.org> wrote:
> Remove the code copying the property around and the corresponding
> HDAT code.
>
> Signed-off-by: Benjamin Herrenschmidt <benh at kernel.crashing.org>
> ---
>  hdata/iohub.c | 10 ----------
>  hw/phb4.c     |  2 --
>  2 files changed, 12 deletions(-)
>
> diff --git a/hdata/iohub.c b/hdata/iohub.c
> index 192e8cf..d40f644 100644
> --- a/hdata/iohub.c
> +++ b/hdata/iohub.c
> @@ -281,16 +281,6 @@ static struct dt_node *add_pec_stack(const struct cechub_io_hub *hub,
>
>         __dt_add_property_u64s(stack, "ibm,lane-eq", 8, eq);
>
> -       /* We indicate that this is an IBM setup, which means that
> -        * the presence detect A/B bits are meaningful. So far we
> -        * don't know whether they make any sense on customer setups
> -        * so we only set that when booting with HDAT
> -        *
> -        * XXX: Since we have HDAT in P9 OpenPower boxes
> -        *      is this still going to be true?
> -        */
> -       dt_add_property(stack, "ibm,use-ab-detect", NULL, 0);
> -
>         return stack;
>  }
>
> diff --git a/hw/phb4.c b/hw/phb4.c
> index 981c16d..77b5f07 100644
> --- a/hw/phb4.c
> +++ b/hw/phb4.c
> @@ -3442,8 +3442,6 @@ static void phb4_probe_stack(struct dt_node *stk_node, uint32_t pec_index,
>         dt_add_property_cells(np, "ibm,phb-stack", stk_node->phandle);
>         dt_add_property_cells(np, "ibm,phb-stack-index", stk_index);
>         dt_add_property_cells(np, "ibm,chip-id", gcid);
> -       if (dt_has_node_property(stk_node, "ibm,use-ab-detect", NULL))
> -               dt_add_property(np, "ibm,use-ab-detect", NULL, 0);
>         if (dt_has_node_property(stk_node, "ibm,hub-id", NULL))
>                 dt_add_property_cells(np, "ibm,hub-id",
>                                       dt_prop_get_u32(stk_node, "ibm,hub-id"));

The patch this is based on top of isn't in master. I've folded this
into my patch and I'll post that in a series later today/tomorrow.


More information about the Skiboot mailing list