All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Henry Ptasinski" <henryp@broadcom.com>
To: "Joe Perches" <joe@perches.com>
Cc: netdev <netdev@vger.kernel.org>,
	"Brett Rudley" <brudley@broadcom.com>,
	"Dowan Kim" <dowan@broadcom.com>,
	"Henry Ptasinski" <henryp@broadcom.com>,
	arend@broadcom.com, rvossen@broadcom.com
Subject: Re: [RFC PATCH net-next] if_ether.h: Add #define ETH_P_LARQ for HPNA/LARQ
Date: Mon, 13 Dec 2010 17:49:20 -0800	[thread overview]
Message-ID: <20101214014920.GA2453@broadcom.com> (raw)
In-Reply-To: <1292287667.26970.114.camel@Joe-Laptop>

On Mon, Dec 13, 2010 at 04:47:47PM -0800, Joe Perches wrote:
> LARQ seems to be used by Broadcom's staging driver.
> Might as well add a #define for it to the normal location.
> 
> Signed-off-by: Joe Perches <joe@perches.com>
> ---
>  include/linux/if_ether.h |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/include/linux/if_ether.h b/include/linux/if_ether.h
> index f9c3df0..b257e99 100644
> --- a/include/linux/if_ether.h
> +++ b/include/linux/if_ether.h
> @@ -71,6 +71,8 @@
>  #define ETH_P_PPP_SES	0x8864		/* PPPoE session messages	*/
>  #define ETH_P_MPLS_UC	0x8847		/* MPLS Unicast traffic		*/
>  #define ETH_P_MPLS_MC	0x8848		/* MPLS Multicast traffic	*/
> +#define ETH_P_LARQ	0x886c		/* HPNA / Broadcom
> +					 * Limited Automatic Repeat reQuest */


That should be ETH_P_BRCM (or ETH_P_EPIGRAM).  HPNA/ILCP used a range of
subtypes for LARQ and other HPNA protocols.  The brcmfmac driver uses a
different subypte for encapsulating some event signals between the device and
the host, but doesn't implement any of the HPNA protocols.

- Henry



  reply	other threads:[~2010-12-14  1:49 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-12-14  0:47 [RFC PATCH net-next] if_ether.h: Add #define ETH_P_LARQ for HPNA/LARQ Joe Perches
2010-12-14  1:49 ` Henry Ptasinski [this message]
2010-12-14  3:53   ` Joe Perches
2010-12-15 21:22     ` Henry Ptasinski
2011-01-05  2:07     ` [PATCH] include/linux/if_ether.h: Add #define ETH_P_LINK_CTL for HPNA and wlan local tunnel Henry Ptasinski
2011-01-06 19:10       ` David Miller

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=20101214014920.GA2453@broadcom.com \
    --to=henryp@broadcom.com \
    --cc=arend@broadcom.com \
    --cc=brudley@broadcom.com \
    --cc=dowan@broadcom.com \
    --cc=joe@perches.com \
    --cc=netdev@vger.kernel.org \
    --cc=rvossen@broadcom.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.