From: Chris Paterson <Chris.Paterson2@renesas.com> To: Geert Uytterhoeven <geert@linux-m68k.org>, Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> Cc: Rob Herring <robh+dt@kernel.org>, Mark Rutland <mark.rutland@arm.com>, Russell King <linux@armlinux.org.uk>, Biju Das <biju.das@bp.renesas.com>, Simon Horman <horms@verge.net.au>, Magnus Damm <magnus.damm@gmail.com>, "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>, Linux-Renesas <linux-renesas-soc@vger.kernel.org>, "linux-arm-kernel@lists.infradead.org" <linux-arm-kernel@lists.infradead.org>, "netdev@vger.kernel.org" <netdev@vger.kernel.org> Subject: RE: [PATCH v 1/2] ravb: Add support for r8a7743 SoC Date: Wed, 5 Jul 2017 13:51:50 +0000 [thread overview] Message-ID: <TY1PR06MB0702B8D1C817F6C3CBD93D99B7D40@TY1PR06MB0702.apcprd06.prod.outlook.com> (raw) In-Reply-To: <CAMuHMdVMunZ0trcNmMg4KkEd+d88foep7C0kU=yLQNk5JojosA@mail.gmail.com> Hello Geert, Sergei, > From: geert.uytterhoeven@gmail.com > [mailto:geert.uytterhoeven@gmail.com] On Behalf Of Geert Uytterhoeven > Sent: 05 July 2017 14:47 > > Hi Biju, > > On Wed, Jul 5, 2017 at 3:01 PM, Biju Das <biju.das@bp.renesas.com> wrote: > > Add support for Gigabit Ethernet E-MAC on r8a7743 (RZ/G1M) SoC. > > Renesas RZ/G1M (R8A7743) SoC Ethernet AVB IP is identical to the R-Car > > Gen2 family. > > > > Signed-off-by: Biju Das <biju.das@bp.renesas.com> > > Reviewed-by: Chris Paterson <chris.paterson2@renesas.com> > > Thanks for your patch! > > > --- a/drivers/net/ethernet/renesas/ravb_main.c > > +++ b/drivers/net/ethernet/renesas/ravb_main.c > > @@ -1869,6 +1869,7 @@ static int ravb_mdio_release(struct ravb_private > > *priv) } > > > > static const struct of_device_id ravb_match_table[] = { > > + { .compatible = "renesas,etheravb-r8a7743", .data = (void > > + *)RCAR_GEN2 }, > > { .compatible = "renesas,etheravb-r8a7790", .data = (void > *)RCAR_GEN2 }, > > { .compatible = "renesas,etheravb-r8a7794", .data = (void > *)RCAR_GEN2 }, > > { .compatible = "renesas,etheravb-rcar-gen2", .data = (void > > *)RCAR_GEN2 }, > > As Sergei already mentioned, a driver update is not needed due to the > presence of family-specific compatible values. > Please drop that part, and you can add my: > Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> Should the other compatible values (r8a7790, 94 etc) be removed then? Or are they needed for backwards compatibility? Kind regards, Chris > > Gr{oetje,eeting}s, > > Geert > > -- > Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux- > m68k.org > > 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
WARNING: multiple messages have this Message-ID (diff)
From: Chris.Paterson2@renesas.com (Chris Paterson) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH v 1/2] ravb: Add support for r8a7743 SoC Date: Wed, 5 Jul 2017 13:51:50 +0000 [thread overview] Message-ID: <TY1PR06MB0702B8D1C817F6C3CBD93D99B7D40@TY1PR06MB0702.apcprd06.prod.outlook.com> (raw) In-Reply-To: <CAMuHMdVMunZ0trcNmMg4KkEd+d88foep7C0kU=yLQNk5JojosA@mail.gmail.com> Hello Geert, Sergei, > From: geert.uytterhoeven at gmail.com > [mailto:geert.uytterhoeven at gmail.com] On Behalf Of Geert Uytterhoeven > Sent: 05 July 2017 14:47 > > Hi Biju, > > On Wed, Jul 5, 2017 at 3:01 PM, Biju Das <biju.das@bp.renesas.com> wrote: > > Add support for Gigabit Ethernet E-MAC on r8a7743 (RZ/G1M) SoC. > > Renesas RZ/G1M (R8A7743) SoC Ethernet AVB IP is identical to the R-Car > > Gen2 family. > > > > Signed-off-by: Biju Das <biju.das@bp.renesas.com> > > Reviewed-by: Chris Paterson <chris.paterson2@renesas.com> > > Thanks for your patch! > > > --- a/drivers/net/ethernet/renesas/ravb_main.c > > +++ b/drivers/net/ethernet/renesas/ravb_main.c > > @@ -1869,6 +1869,7 @@ static int ravb_mdio_release(struct ravb_private > > *priv) } > > > > static const struct of_device_id ravb_match_table[] = { > > + { .compatible = "renesas,etheravb-r8a7743", .data = (void > > + *)RCAR_GEN2 }, > > { .compatible = "renesas,etheravb-r8a7790", .data = (void > *)RCAR_GEN2 }, > > { .compatible = "renesas,etheravb-r8a7794", .data = (void > *)RCAR_GEN2 }, > > { .compatible = "renesas,etheravb-rcar-gen2", .data = (void > > *)RCAR_GEN2 }, > > As Sergei already mentioned, a driver update is not needed due to the > presence of family-specific compatible values. > Please drop that part, and you can add my: > Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> Should the other compatible values (r8a7790, 94 etc) be removed then? Or are they needed for backwards compatibility? Kind regards, Chris > > Gr{oetje,eeting}s, > > Geert > > -- > Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at linux- > m68k.org > > 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
next prev parent reply other threads:[~2017-07-05 13:51 UTC|newest] Thread overview: 66+ messages / expand[flat|nested] mbox.gz Atom feed top 2017-07-05 13:01 [PATCH v 0/2] Add RZ/G1M (R8A7743) Ethernet AVB support Biju Das 2017-07-05 13:01 ` Biju Das 2017-07-05 13:01 ` [PATCH v 1/2] ravb: Add support for r8a7743 SoC Biju Das 2017-07-05 13:01 ` Biju Das 2017-07-05 13:20 ` Sergei Shtylyov 2017-07-05 13:20 ` Sergei Shtylyov [not found] ` <afcd181c-c805-8f3c-955f-652d001a03d4-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8@public.gmane.org> 2017-07-05 15:02 ` Biju Das 2017-07-05 15:02 ` Biju Das 2017-07-05 15:02 ` Biju Das 2017-07-05 15:56 ` [PATCH v2 0/2] Add RZ/G1M (R8A7743) Ethernet AVB support Biju Das 2017-07-05 15:56 ` Biju Das 2017-07-05 15:56 ` [PATCH v2 1/2] ravb: Document binding for r8a7743 SoC Biju Das 2017-07-05 15:56 ` Biju Das [not found] ` <1499270189-29945-1-git-send-email-biju.das-kTT6dE0pTRh9uiUsa/gSgQ@public.gmane.org> 2017-07-05 15:56 ` [PATCH v2 2/2] ARM: shmobile: defconfig: Enable Ethernet AVB Biju Das 2017-07-05 15:56 ` Biju Das 2017-07-05 15:56 ` Biju Das [not found] ` <1499270189-29945-3-git-send-email-biju.das-kTT6dE0pTRh9uiUsa/gSgQ@public.gmane.org> 2017-07-05 16:13 ` Sergei Shtylyov 2017-07-05 16:13 ` Sergei Shtylyov 2017-07-05 16:13 ` Sergei Shtylyov 2017-07-06 8:05 ` Chris Paterson 2017-07-06 8:05 ` Chris Paterson 2017-07-06 8:05 ` Chris Paterson 2017-07-10 8:38 ` Simon Horman 2017-07-10 8:38 ` Simon Horman 2017-07-05 13:47 ` [PATCH v 1/2] ravb: Add support for r8a7743 SoC Geert Uytterhoeven 2017-07-05 13:47 ` Geert Uytterhoeven 2017-07-05 13:47 ` Geert Uytterhoeven 2017-07-05 13:51 ` Chris Paterson [this message] 2017-07-05 13:51 ` Chris Paterson 2017-07-05 14:55 ` Geert Uytterhoeven 2017-07-05 14:55 ` Geert Uytterhoeven [not found] ` <CAMuHMdUXxHU8JaO0riJ7h7hJpCwAw27titTfo3Ox6CqO3i8E9A-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2017-07-05 14:59 ` Chris Paterson 2017-07-05 14:59 ` Chris Paterson 2017-07-05 14:59 ` Chris Paterson [not found] ` <1499259711-56175-2-git-send-email-biju.das-kTT6dE0pTRh9uiUsa/gSgQ@public.gmane.org> 2017-07-10 1:20 ` Rob Herring 2017-07-10 1:20 ` Rob Herring 2017-07-10 1:20 ` Rob Herring 2017-07-10 8:20 ` Biju Das 2017-07-10 8:20 ` Biju Das 2017-07-10 9:31 ` Sergei Shtylyov 2017-07-10 9:31 ` Sergei Shtylyov 2017-07-10 15:32 ` [PATCH v2] dt-bindings: net: ravb : " Biju Das 2017-07-10 15:32 ` Biju Das [not found] ` <1499700776-54504-1-git-send-email-biju.das-kTT6dE0pTRh9uiUsa/gSgQ@public.gmane.org> 2017-07-10 15:51 ` Geert Uytterhoeven 2017-07-10 15:51 ` Geert Uytterhoeven 2017-07-10 15:51 ` Geert Uytterhoeven 2017-07-11 9:07 ` Simon Horman 2017-07-11 9:07 ` Simon Horman 2017-07-11 12:12 ` Sergei Shtylyov 2017-07-11 12:12 ` Sergei Shtylyov [not found] ` <52eb0b92-5d25-e1f4-df54-e4fcf648c90e-M4DtvfQ/ZS1MRgGoP+s0PdBPR1lH4CV8@public.gmane.org> 2017-07-11 12:20 ` Chris Paterson 2017-07-11 12:20 ` Chris Paterson 2017-07-11 12:20 ` Chris Paterson 2017-07-11 12:55 ` Sergei Shtylyov 2017-07-11 12:55 ` Sergei Shtylyov 2017-07-11 12:21 ` Simon Horman 2017-07-11 12:21 ` Simon Horman [not found] ` <20170711122100.GM4517-/R6kz+dDXgpPR4JQBCEnsQ@public.gmane.org> 2017-07-11 12:51 ` Sergei Shtylyov 2017-07-11 12:51 ` Sergei Shtylyov 2017-07-11 12:51 ` Sergei Shtylyov 2017-07-12 5:52 ` Simon Horman 2017-07-12 5:52 ` Simon Horman 2017-07-05 13:01 ` [PATCH v 2/2] ARM: shmobile: defconfig: Enable Ethernet AVB Biju Das 2017-07-05 13:01 ` Biju Das 2017-07-05 13:45 ` Geert Uytterhoeven 2017-07-05 13:45 ` Geert Uytterhoeven
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=TY1PR06MB0702B8D1C817F6C3CBD93D99B7D40@TY1PR06MB0702.apcprd06.prod.outlook.com \ --to=chris.paterson2@renesas.com \ --cc=biju.das@bp.renesas.com \ --cc=devicetree@vger.kernel.org \ --cc=geert@linux-m68k.org \ --cc=horms@verge.net.au \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-renesas-soc@vger.kernel.org \ --cc=linux@armlinux.org.uk \ --cc=magnus.damm@gmail.com \ --cc=mark.rutland@arm.com \ --cc=netdev@vger.kernel.org \ --cc=robh+dt@kernel.org \ --cc=sergei.shtylyov@cogentembedded.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.