From: "Michael S. Tsirkin" <mst@redhat.com>
To: Jason Baron <jbaron@akamai.com>
Cc: jasowang@redhat.com, qemu-devel@nongnu.org,
Jiri Pirko <jiri@resnulli.us>,
virtio-dev@lists.oasis-open.org
Subject: [virtio-dev] Re: [PATCH 2/4] rocker: drop local duplex definitions
Date: Fri, 2 Mar 2018 19:57:25 +0200 [thread overview]
Message-ID: <20180302195625-mutt-send-email-mst@kernel.org> (raw)
In-Reply-To: <ee94e55496562390786ad924f750de1cf3472644.1519961667.git.jbaron@akamai.com>
On Thu, Mar 01, 2018 at 10:46:34PM -0500, Jason Baron wrote:
> Make use of duplex definitions from net/eth.h.
>
> Signed-off-by: Jason Baron <jbaron@akamai.com>
> Cc: "Michael S. Tsirkin" <mst@redhat.com>
> Cc: Jason Wang <jasowang@redhat.com>
> Cc: Jiri Pirko <jiri@resnulli.us>
> Cc: virtio-dev@lists.oasis-open.org
> ---
> hw/net/rocker/rocker_fp.c | 6 +-----
> 1 file changed, 1 insertion(+), 5 deletions(-)
>
> diff --git a/hw/net/rocker/rocker_fp.c b/hw/net/rocker/rocker_fp.c
> index 4b3c984..13a14a0 100644
> --- a/hw/net/rocker/rocker_fp.c
> +++ b/hw/net/rocker/rocker_fp.c
> @@ -16,17 +16,13 @@
>
> #include "qemu/osdep.h"
> #include "net/clients.h"
> +#include "net/eth.h"
>
> #include "rocker.h"
> #include "rocker_hw.h"
> #include "rocker_fp.h"
> #include "rocker_world.h"
>
> -enum duplex {
> - DUPLEX_HALF = 0,
> - DUPLEX_FULL
> -};
> -
This is hardware emulation, so I actually suspect it
should be renamed ROCKER_FP_HALF/ROCKER_FP_FULL.
> struct fp_port {
> Rocker *r;
> World *world;
> --
> 2.7.4
---------------------------------------------------------------------
To unsubscribe, e-mail: virtio-dev-unsubscribe@lists.oasis-open.org
For additional commands, e-mail: virtio-dev-help@lists.oasis-open.org
next prev parent reply other threads:[~2018-03-02 17:57 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <cover.1519961667.git.jbaron@akamai.com>
2018-03-02 3:46 ` [virtio-dev] [PATCH 1/4] eth: add speed and duplex definitions Jason Baron
2018-03-02 17:54 ` [virtio-dev] " Michael S. Tsirkin
2018-03-06 17:53 ` Jason Baron
2018-03-06 18:15 ` Michael S. Tsirkin
2018-03-08 3:46 ` Jason Baron
2018-03-02 3:46 ` [virtio-dev] [PATCH 2/4] rocker: drop local " Jason Baron
2018-03-02 17:57 ` Michael S. Tsirkin [this message]
2018-03-02 3:46 ` [virtio-dev] [PATCH 3/4] virtio-net: use 64-bit values for feature flags Jason Baron
2018-03-02 3:46 ` [virtio-dev] [PATCH 4/4] virtio-net: add linkspeed and duplex settings to virtio-net Jason Baron
2018-03-02 7:14 ` [virtio-dev] " Jason Wang
2018-03-02 16:59 ` Jason Baron
2018-03-02 20:22 ` Michael S. Tsirkin
2018-03-06 17:57 ` Jason Baron
2018-03-02 20:19 ` Michael S. Tsirkin
2018-03-04 13:05 ` [virtio-dev] " Yan Vugenfirer
2018-03-06 18:02 ` Jason Baron
2018-03-06 18:13 ` Michael S. Tsirkin
2018-03-08 12:48 ` Yan Vugenfirer
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=20180302195625-mutt-send-email-mst@kernel.org \
--to=mst@redhat.com \
--cc=jasowang@redhat.com \
--cc=jbaron@akamai.com \
--cc=jiri@resnulli.us \
--cc=qemu-devel@nongnu.org \
--cc=virtio-dev@lists.oasis-open.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox