On Tue, Oct 13, 2020 at 12:21:53PM -0500, Rob Herring wrote: > On Tue, Oct 13, 2020 at 9:19 AM Ioana Ciornei <ioana.ciornei@xxxxxxx> wrote: > > > > On Tue, Oct 13, 2020 at 08:31:32AM -0500, Rob Herring wrote: > > > On Wed, 07 Oct 2020 23:39:26 +0300, Ioana Ciornei wrote: > > > > Add a documentation entry for the DTS bindings needed and supported by > > > > the dpaa2-mac driver. > > > > > > > > Signed-off-by: Ioana Ciornei <ioana.ciornei@xxxxxxx> > > > > --- > > > > Changes in v2: > > > > - new patch > > > > Changes in v3: > > > > - renamed dpmac@x into ethernet@x > > > > - renamed the new documentation file to use the same name as the > > > > compatible > > > > - marked additionalProperties as false > > > > - added a reference to ethernet-controller.yaml > > > > > > > > .../bindings/net/fsl,qoriq-mc-dpmac.yaml | 60 +++++++++++++++++++ > > > > 1 file changed, 60 insertions(+) > > > > create mode 100644 Documentation/devicetree/bindings/net/fsl,qoriq-mc-dpmac.yaml > > > > > > > > > > Reviewed-by: Rob Herring <robh@xxxxxxxxxx> > > > > Thanks for the review. > > > > Is this going in 5.10 by any chance or should I resend these after > > the merge window with the reviewed-by tags? > > I just noticed you didn't Cc netdev as I would expect this to go via > the networking tree. So yes, resend after the merge window. > David said to an earlier version of this patch set that he would not want to pick this up through net-next since it's a sizable change and this should go through the devicetree trees. https://lore.kernel.org/netdev/20201002.161318.726844448692603677.davem@xxxxxxxxxxxxx/ This is why I didn't cc again the netdev. Ioana
