All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andrew Lunn <andrew@lunn.ch>
To: Sunil Kovvuri <sunil.kovvuri@gmail.com>
Cc: Hariprasad Kelam <hkelam@marvell.com>,
	"netdev@vger.kernel.org" <netdev@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"kuba@kernel.org" <kuba@kernel.org>,
	"davem@davemloft.net" <davem@davemloft.net>,
	Sunil Kovvuri Goutham <sgoutham@marvell.com>,
	Linu Cherian <lcherian@marvell.com>,
	Geethasowjanya Akula <gakula@marvell.com>,
	Jerin Jacob Kollanukkaran <jerinj@marvell.com>,
	Subbaraya Sundeep Bhatta <sbhatta@marvell.com>
Subject: Re: [net-next PATCH 0/8] configuration support for switch headers & phy
Date: Thu, 25 Mar 2021 13:50:50 +0100	[thread overview]
Message-ID: <YFyHKqUpG9th+F62@lunn.ch> (raw)
In-Reply-To: <CA+sq2CeT2m2QcrzSn6g5rxUfmJDVQqjYFayW+bcuopCCoYuQ6Q@mail.gmail.com>

> > So you completely skipped how this works with mv88e6xxx or
> > prestera. If you need this private flag for some out of mainline
> > Marvell SDK, it is very unlikely to be accepted.
> >
> >         Andrew
> 
> What we are trying to do here has no dependency on DSA drivers and
> neither impacts that functionality.

So this is an indirect way of saying: Yes, this is for some out of
mainline Marvell SDK.

> Here we are just notifying the HW to parse the packets properly.

But the correct way for this to happen is probably some kernel
internal API between the MAC and the DSA driver. Mainline probably has
no need for this private flag.

   Andrew

  reply	other threads:[~2021-03-25 12:51 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-03-23 18:13 [net-next PATCH 0/8] configuration support for switch headers & phy Hariprasad Kelam
2021-03-23 20:26 ` Andrew Lunn
2021-03-25 10:44   ` Sunil Kovvuri
2021-03-25 12:50     ` Andrew Lunn [this message]
2021-03-25 13:02       ` Sunil Kovvuri
2021-03-25 13:21         ` Andrew Lunn
2021-03-28 17:00           ` Sunil Kovvuri
2021-03-28 17:47             ` Andrew Lunn
  -- strict thread matches above, loose matches on Subject: below --
2021-03-31 14:36 Hariprasad Kelam
2021-03-21 12:09 Hariprasad Kelam
2021-03-21 14:15 ` Andrew Lunn

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=YFyHKqUpG9th+F62@lunn.ch \
    --to=andrew@lunn.ch \
    --cc=davem@davemloft.net \
    --cc=gakula@marvell.com \
    --cc=hkelam@marvell.com \
    --cc=jerinj@marvell.com \
    --cc=kuba@kernel.org \
    --cc=lcherian@marvell.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=sbhatta@marvell.com \
    --cc=sgoutham@marvell.com \
    --cc=sunil.kovvuri@gmail.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: link
Be 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.