From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm0-f53.google.com (mail-wm0-f53.google.com [74.125.82.53]) by dpdk.org (Postfix) with ESMTP id 56742295B for ; Wed, 2 Mar 2016 23:00:00 +0100 (CET) Received: by mail-wm0-f53.google.com with SMTP id p65so7940088wmp.0 for ; Wed, 02 Mar 2016 14:00:00 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=6wind-com.20150623.gappssmtp.com; s=20150623; h=from:to:cc:subject:date:message-id:organization:user-agent :in-reply-to:references:mime-version:content-transfer-encoding; bh=2fCKUwWdE8PDVczikGQQLij+LhmI/zswwg3K5BwLZdA=; b=Lw85cDi9vzQSZmQwI2tG6ZtL/17k3CKfQUW7QYYp+t6aSXRnE48UPbfOWe6eh/04dy YqCaq9+6sJqQrjq3+vBZWiUb0LrP62jfJErgsTAVnpy0rYw+VeTtu3IXN2tjQUo7CWdZ gOnOg0x7BTw0r07S6b5H7XkuoLr+Jm/zD2J9gamDIjHZ6ABtCybPGTYpEhdKoGSZOq+V XTH7NXLsL3XPDW4+iSf8zN863WRmriuuBH1Pzb937/dbsUDb3HH2F3mXLPp+SSi0/AGL LBNHI+rrGMycVaD+LgPAc+V2IxXLeN8o9N3Mb2fOHZh0Mhpn9GNHNh6XAGX67vPQmKN3 OtGg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:organization :user-agent:in-reply-to:references:mime-version :content-transfer-encoding; bh=2fCKUwWdE8PDVczikGQQLij+LhmI/zswwg3K5BwLZdA=; b=I/RbRB5GIZM/RoJngTnn28k0qnlOiry77k28boeG3/hZzc7gXD82ACjyTSmVdBO2mN ADmVtTBSmCHkunRuHPH7Gx2KY9N38GooC+XO00zIz2DfLUVk5XTvCRMrNQie1WNaPqAr f5oLJAwieAxr2MssM4q3SLcPAYtX5SQOvmFp037XcAeHD14XtsR7/gCxS636MihsqpBY 3B4UQ+ijt9rW4tLwJbZZVetJHWR3l+DZOOky1G6UwqMbcSjmgWCwRd6EtytFcGegjivB 6A64cXnYgvRaWYlpKCcpy1JD/nGpI51D2g7y9erEZ1RGHVsQun/l9ul1frgaWtJoT1aS gSgA== X-Gm-Message-State: AD7BkJIDTeY4J2xds5CpINggzVgnBVQNc+shrYVgSg1CI5b3vQhh7IiZUVB3APGvcKgNi8GV X-Received: by 10.194.77.193 with SMTP id u1mr28276020wjw.73.1456956000215; Wed, 02 Mar 2016 14:00:00 -0800 (PST) Received: from xps13.localnet ([93.158.56.12]) by smtp.gmail.com with ESMTPSA id o128sm5998177wmb.19.2016.03.02.13.59.58 (version=TLSv1/SSLv3 cipher=OTHER); Wed, 02 Mar 2016 13:59:59 -0800 (PST) From: Thomas Monjalon To: Stephen Hurd Date: Wed, 02 Mar 2016 22:58:24 +0100 Message-ID: <2214559.7sJksEEjpg@xps13> Organization: 6WIND User-Agent: KMail/4.14.10 (Linux/4.1.6-1-ARCH; KDE/4.14.11; x86_64; ; ) In-Reply-To: <1456954614-119304-1-git-send-email-stephen.hurd@broadcom.com> References: <1456954614-119304-1-git-send-email-stephen.hurd@broadcom.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Cc: dev@dpdk.org Subject: Re: [dpdk-dev] [PATCH] drivers/net/bnxt New driver for Broadcom bnxt X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 02 Mar 2016 22:00:00 -0000 Please do not submit some disabled code and avoid #ifdef. 2016-03-02 13:36, Stephen Hurd: > +static struct eth_dev_ops bnxt_dev_ops = { > + .dev_infos_get = bnxt_dev_info_get_op, > + .dev_configure = bnxt_dev_configure_op, > + .dev_start = bnxt_dev_start_op, > + .dev_stop = bnxt_dev_stop_op, > + .dev_set_link_up = bnxt_dev_set_link_up_op, > + .dev_set_link_down = bnxt_dev_set_link_down_op, > + .dev_close = bnxt_dev_close_op, > + .stats_get = bnxt_stats_get_op, > + .stats_reset = bnxt_stats_reset_op, > + .rx_queue_setup = bnxt_rx_queue_setup_op, > + .rx_queue_release = bnxt_rx_queue_release_op, > +// .rx_queue_count = bnxt_rx_queue_count_op, > +// .rx_descriptor_done = bnxt_rx_descriptor_done_op, > + .tx_queue_setup = bnxt_tx_queue_setup_op, > + .tx_queue_release = bnxt_tx_queue_release_op, > +// .rx_queue_start = bnxt_rx_queue_start_op, > +// .rx_queue_stop = bnxt_rx_queue_stop_op, > +// .tx_queue_start = bnxt_tx_queue_start_op, > +// .tx_queue_stop = bnxt_tx_queue_stop_op, > + .reta_update = bnxt_reta_update_op, > + .reta_query = bnxt_reta_query_op, > + .rss_hash_update = bnxt_rss_hash_update_op, > + .rss_hash_conf_get = bnxt_rss_hash_conf_get_op, > + .link_update = bnxt_link_update_op, > + .promiscuous_enable = bnxt_promiscuous_enable_op, > + .promiscuous_disable = bnxt_promiscuous_disable_op, > + .allmulticast_enable = bnxt_allmulticast_enable_op, > + .allmulticast_disable = bnxt_allmulticast_disable_op, > + .mtu_set = bnxt_mtu_set_op, > + .mac_addr_add = bnxt_mac_addr_add_op, > + .mac_addr_remove = bnxt_mac_addr_remove_op, > + .vlan_filter_set = bnxt_vlan_filter_set_op, > + .vlan_strip_queue_set = bnxt_vlan_strip_queue_set_op, > + .flow_ctrl_get = bnxt_flow_ctrl_get_op, > + .flow_ctrl_set = bnxt_flow_ctrl_set_op, > +#if 0 // Phase 2/3 > + .dev_led_on = bnxt_dev_led_on_op, > + .dev_led_off = bnxt_dev_led_off_op, > + .queue_stats_mapping_set = bnxt_queue_stats_mapping_set_op, > + .vlan_tpid_set = bnxt_vlan_tpid_set_op, > + .vlan_offload_set = bnxt_vlan_offload_set_op, > + .priority_flow_ctrl_set = bnxt_priority_flow_ctrl_set_op, > + .uc_hash_table_set = bnxt_uc_hash_table_set_op, > + .uc_all_hash_table_set = bnxt_uc_all_hash_table_set_op, > + .mirror_rule_set = bnxt_mirror_rule_set_op, > + .mirror_rule_reset = bnxt_mirror_rule_reset_op, > + .set_vf_rx_mode = bnxt_set_vf_rx_mode_op, > + .set_vf_rx = bnxt_set_vf_rx_op, > + .set_vf_tx = bnxt_set_vf_tx_op, > + .set_vf_vlan_filter = bnxt_set_vf_vlan_filter_op, > + .set_queue_rate_limit = bnxt_set_queue_rate_limit_op, > + .set_vf_rate_limit = bnxt_set_vf_rate_limit_op, > + .fdir_add_signature_filter = bnxt_fdir_add_signature_filter_op, > + .fdir_update_signature_filter = bnxt_fdir_update_signature_filter_op, > + .fdir_remove_signature_filter = bnxt_fdir_remove_signature_filter_op, > + .fdir_infos_get = bnxt_fdir_info_get_op, > + .fdir_add_perfect_filter = bnxt_fdir_add_perfect_filter_op, > + .fdir_update_perfect_filter = bnxt_fdir_update_perfect_filter_op, > + .fdir_remove_perfect_filter = bnxt_fdir_remove_perfect_filter_op, > + .fdir_set_masks = bnxt_fdir_set_masks_op, > + .add_syn_filter = bnxt_add_syn_filter_op, > + .remove_syn_filter = bnxt_remove_syn_filter_op, > + .get_syn_filter = bnxt_get_syn_filter_op, > + .add_ethertype_filter = bnxt_add_ethertype_filter_op, > + .remove_ethertype_filter = bnxt_remove_ethertype_filter_op, > + .get_ethertype_filter = bnxt_get_ethertype_filter_op, > + .add_5tuple_filter = bnxt_add_5tuple_filter_op, > + .remove_5tuple_filter = bnxt_remove_5tuple_filter_op, > + .get_5tuple_filter = bnxt_get_5tuple_filter_op, > +#endif > +};