netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH ethtool] ethtool: Report Backplane as supported port
@ 2014-02-20 17:48 Ivan Vecera
  2014-02-21 10:40 ` Ivan Vecera
  2014-02-27 22:55 ` Ben Hutchings
  0 siblings, 2 replies; 5+ messages in thread
From: Ivan Vecera @ 2014-02-20 17:48 UTC (permalink / raw)
  To: netdev; +Cc: bhutchings, agospoda

Signed-off-by: Ivan Vecera <ivecera@redhat.com>
---
 ethtool.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/ethtool.c b/ethtool.c
index 315d00f..acb4397 100644
--- a/ethtool.c
+++ b/ethtool.c
@@ -496,6 +496,8 @@ static void dump_supported(struct ethtool_cmd *ep)
 		fprintf(stdout, "MII ");
 	if (mask & SUPPORTED_FIBRE)
 		fprintf(stdout, "FIBRE ");
+	if (mask & SUPPORTED_Backplane)
+		fprintf(stdout, "Backplane ");
 	fprintf(stdout, "]\n");
 
 	dump_link_caps("Supported", "Supports", mask, 0);
-- 
1.8.3.2

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH ethtool] ethtool: Report Backplane as supported port
  2014-02-20 17:48 [PATCH ethtool] ethtool: Report Backplane as supported port Ivan Vecera
@ 2014-02-21 10:40 ` Ivan Vecera
  2014-02-21 17:15   ` David Miller
  2014-02-27 22:55 ` Ben Hutchings
  1 sibling, 1 reply; 5+ messages in thread
From: Ivan Vecera @ 2014-02-21 10:40 UTC (permalink / raw)
  To: bwh, David Miller; +Cc: netdev

Hi Dave & Ben,
I see the patch is in 'not applicable' state in patchwork. The patch is 
for ethtool userspace. I though the netdev mailing list is also for 
ethtool userspace or did I used bad subject prefix?

Thanks,
Ivan

On 02/20/2014 06:48 PM, Ivan Vecera wrote:
> Signed-off-by: Ivan Vecera <ivecera@redhat.com>
> ---
>   ethtool.c | 2 ++
>   1 file changed, 2 insertions(+)
>
> diff --git a/ethtool.c b/ethtool.c
> index 315d00f..acb4397 100644
> --- a/ethtool.c
> +++ b/ethtool.c
> @@ -496,6 +496,8 @@ static void dump_supported(struct ethtool_cmd *ep)
>   		fprintf(stdout, "MII ");
>   	if (mask & SUPPORTED_FIBRE)
>   		fprintf(stdout, "FIBRE ");
> +	if (mask & SUPPORTED_Backplane)
> +		fprintf(stdout, "Backplane ");
>   	fprintf(stdout, "]\n");
>
>   	dump_link_caps("Supported", "Supports", mask, 0);
>

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH ethtool] ethtool: Report Backplane as supported port
  2014-02-21 10:40 ` Ivan Vecera
@ 2014-02-21 17:15   ` David Miller
  2014-02-24 10:34     ` Ivan Vecera
  0 siblings, 1 reply; 5+ messages in thread
From: David Miller @ 2014-02-21 17:15 UTC (permalink / raw)
  To: ivecera; +Cc: bwh, netdev

From: Ivan Vecera <ivecera@redhat.com>
Date: Fri, 21 Feb 2014 11:40:03 +0100

> I see the patch is in 'not applicable' state in patchwork. The patch
> is for ethtool userspace. I though the netdev mailing list is also for
> ethtool userspace or did I used bad subject prefix?

Ben doesn't use patchwork to manage the patch queue for the ethool
utility.

So I mark all ethtool patches that way.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH ethtool] ethtool: Report Backplane as supported port
  2014-02-21 17:15   ` David Miller
@ 2014-02-24 10:34     ` Ivan Vecera
  0 siblings, 0 replies; 5+ messages in thread
From: Ivan Vecera @ 2014-02-24 10:34 UTC (permalink / raw)
  To: David Miller; +Cc: bwh, netdev

On 02/21/2014 06:15 PM, David Miller wrote:
> From: Ivan Vecera <ivecera@redhat.com>
> Date: Fri, 21 Feb 2014 11:40:03 +0100
>
>> I see the patch is in 'not applicable' state in patchwork. The patch
>> is for ethtool userspace. I though the netdev mailing list is also for
>> ethtool userspace or did I used bad subject prefix?
>
> Ben doesn't use patchwork to manage the patch queue for the ethool
> utility.
>
> So I mark all ethtool patches that way.
>
OK, thank you Dave for the clarification.

Ivan

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH ethtool] ethtool: Report Backplane as supported port
  2014-02-20 17:48 [PATCH ethtool] ethtool: Report Backplane as supported port Ivan Vecera
  2014-02-21 10:40 ` Ivan Vecera
@ 2014-02-27 22:55 ` Ben Hutchings
  1 sibling, 0 replies; 5+ messages in thread
From: Ben Hutchings @ 2014-02-27 22:55 UTC (permalink / raw)
  To: Ivan Vecera; +Cc: netdev, bhutchings, agospoda

[-- Attachment #1: Type: text/plain, Size: 745 bytes --]

On Thu, 2014-02-20 at 18:48 +0100, Ivan Vecera wrote:
> Signed-off-by: Ivan Vecera <ivecera@redhat.com>
> ---
>  ethtool.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/ethtool.c b/ethtool.c
> index 315d00f..acb4397 100644
> --- a/ethtool.c
> +++ b/ethtool.c
> @@ -496,6 +496,8 @@ static void dump_supported(struct ethtool_cmd *ep)
>  		fprintf(stdout, "MII ");
>  	if (mask & SUPPORTED_FIBRE)
>  		fprintf(stdout, "FIBRE ");
> +	if (mask & SUPPORTED_Backplane)
> +		fprintf(stdout, "Backplane ");
>  	fprintf(stdout, "]\n");
>  
>  	dump_link_caps("Supported", "Supports", mask, 0);

Applied, thanks.

Ben.

-- 
Ben Hutchings
It is easier to change the specification to fit the program than vice versa.

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 811 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2014-02-27 22:56 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-02-20 17:48 [PATCH ethtool] ethtool: Report Backplane as supported port Ivan Vecera
2014-02-21 10:40 ` Ivan Vecera
2014-02-21 17:15   ` David Miller
2014-02-24 10:34     ` Ivan Vecera
2014-02-27 22:55 ` Ben Hutchings

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).