Hi Kaneko-san,

On Sun, Dec 18, 2016 at 1:27 PM, Yoshihiro Kaneko <[email protected]> wrote:
> From: Harunobu Kurokawa <[email protected]>
>
> This patch adds support for r8a7796.
>
> Signed-off-by: Harunobu Kurokawa <[email protected]>
> Signed-off-by: Yoshihiro Kaneko <[email protected]>
> ---
>
> This patch is based on the next branch of the pci tree.
>
>  Documentation/devicetree/bindings/pci/rcar-pci.txt | 1 +
>  drivers/pci/host/pcie-rcar.c                       | 1 +
>  2 files changed, 2 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/pci/rcar-pci.txt 
> b/Documentation/devicetree/bindings/pci/rcar-pci.txt
> index eee518d..34712d6 100644
> --- a/Documentation/devicetree/bindings/pci/rcar-pci.txt
> +++ b/Documentation/devicetree/bindings/pci/rcar-pci.txt
> @@ -6,6 +6,7 @@ compatible: "renesas,pcie-r8a7779" for the R8A7779 SoC;
>             "renesas,pcie-r8a7791" for the R8A7791 SoC;
>             "renesas,pcie-r8a7793" for the R8A7793 SoC;
>             "renesas,pcie-r8a7795" for the R8A7795 SoC;
> +           "renesas,pcie-r8a7796" for the R8A7796 SoC;
>             "renesas,pcie-rcar-gen2" for a generic R-Car Gen2 compatible 
> device.
>             "renesas,pcie-rcar-gen3" for a generic R-Car Gen3 compatible 
> device.
>
> diff --git a/drivers/pci/host/pcie-rcar.c b/drivers/pci/host/pcie-rcar.c
> index aca85be..02e5777 100644
> --- a/drivers/pci/host/pcie-rcar.c
> +++ b/drivers/pci/host/pcie-rcar.c
> @@ -1078,6 +1078,7 @@ static int rcar_pcie_parse_map_dma_ranges(struct 
> rcar_pcie *pcie,
>         { .compatible = "renesas,pcie-rcar-gen2",
>           .data = rcar_pcie_hw_init_gen2 },
>         { .compatible = "renesas,pcie-r8a7795", .data = rcar_pcie_hw_init },
> +       { .compatible = "renesas,pcie-r8a7796", .data = rcar_pcie_hw_init },
>         { .compatible = "renesas,pcie-rcar-gen3", .data = rcar_pcie_hw_init },
>         {},

Given the driver already matches against "renesas,pcie-rcar-gen3",
and mainline arch/arm64/boot/dts/renesas/r8a7796.dtsi doesn't have pcie
nodes yet, I think there's no need to update the driver, only the bindings.

Hence if you drop the last chunk, you can add my
Reviewed-by: Geert Uytterhoeven <[email protected]>

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

Reply via email to