On 11/08/2017 10:29 AM, Geert Uytterhoeven wrote: > Hi Marek, Hi,
> On Wed, Nov 8, 2017 at 10:15 AM, Marek Vasut <[email protected]> wrote: >> From: Harunobu Kurokawa <[email protected]> >> >> Add compatible string for the RCar M3 R8A7796 SoC. >> >> Signed-off-by: Harunobu Kurokawa <[email protected]> >> Signed-off-by: Marek Vasut <[email protected]> > > This patch is not needed, as the DT bindings specify that the DTS should use > both "renesas,pcie-r8a7796" and "renesas,pcie-rcar-gen3", and the driver > already matches against the latter. OK, then please just drop this one. I'll keep this in mind next time I add redundant bindings, thanks :) >> --- a/drivers/pci/host/pcie-rcar.c >> +++ b/drivers/pci/host/pcie-rcar.c >> @@ -1062,6 +1062,7 @@ static const struct of_device_id rcar_pcie_of_match[] >> = { >> { .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 >> }, > > 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 > -- Best regards, Marek Vasut
