Re: [PATCH v3 1/6] powerpc/powernv: must include hvcall.h to get PAPR defines

From: Michael Ellerman
Date: Thu Jul 09 2020 - 06:03:16 EST


Nicholas Piggin <npiggin@xxxxxxxxx> writes:
> An include goes away in future patches which breaks compilation
> without this.
>
> Signed-off-by: Nicholas Piggin <npiggin@xxxxxxxxx>
> ---
> arch/powerpc/platforms/powernv/pci-ioda-tce.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/powerpc/platforms/powernv/pci-ioda-tce.c b/arch/powerpc/platforms/powernv/pci-ioda-tce.c
> index f923359d8afc..8eba6ece7808 100644
> --- a/arch/powerpc/platforms/powernv/pci-ioda-tce.c
> +++ b/arch/powerpc/platforms/powernv/pci-ioda-tce.c
> @@ -15,6 +15,7 @@
>
> #include <asm/iommu.h>
> #include <asm/tce.h>
> +#include <asm/hvcall.h> /* share error returns with PAPR */
> #include "pci.h"
>
> unsigned long pnv_ioda_parse_tce_sizes(struct pnv_phb *phb)
> --
> 2.23.0

This isn't needed anymore AFAICS, since:

5f202c1a1d42 ("powerpc/powernv/ioda: Return correct error if TCE level allocation failed")

cheers