On 11/08/2017 10:28 AM, Sergei Shtylyov wrote:
> Hello!
Hi
> On 11/8/2017 12:15 PM, Marek Vasut wrote:
>
>> From: Harunobu Kurokawa <harunobu.kurokawa.dn@xxxxxxxxxxx>
>>
>> Add compatible string for the RCar M3 R8A7796 SoC.
>
> It's M3-W.
>
>> Signed-off-by: Harunobu Kurokawa <harunobu.kurokawa.dn@xxxxxxxxxxx>
>> Signed-off-by: Marek Vasut <marek.vasut+renesas@xxxxxxxxx>
>> Cc: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
>> Cc: Simon Horman <horms+renesas@xxxxxxxxxxxx>
>> Cc: Wolfram Sang <wsa@xxxxxxxxxxxxx>
>> Cc: linux-renesas-soc@xxxxxxxxxxxxxxx
>> ---
>> drivers/pci/host/pcie-rcar.c | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/drivers/pci/host/pcie-rcar.c b/drivers/pci/host/pcie-rcar.c
>> index 12796eccb2be..8b51208875fc 100644
>> --- 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 },
>
> Why, if we already have the line below?
For the sake of completeness, but I'll reply to Geerts email about this.
> And don't you need to update the bindings?
Nope, the bindings already have this entry.
>> { .compatible = "renesas,pcie-rcar-gen3", .data =
>> rcar_pcie_hw_init },
>> {},
>> };
>
> MBR, Sergei
--
Best regards,
Marek Vasut
