On 11/08/2017 10:29 AM, Geert Uytterhoeven wrote:
> Hi Marek,
Hi,
> On Wed, Nov 8, 2017 at 10:15 AM, Marek Vasut <marek.vasut@xxxxxxxxx> wrote:
>> From: Harunobu Kurokawa <harunobu.kurokawa.dn@xxxxxxxxxxx>
>>
>> Add compatible string for the RCar M3 R8A7796 SoC.
>>
>> Signed-off-by: Harunobu Kurokawa <harunobu.kurokawa.dn@xxxxxxxxxxx>
>> Signed-off-by: Marek Vasut <marek.vasut+renesas@xxxxxxxxx>
>
> 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 -- geert@xxxxxxxxxxxxxx
>
> 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
