RE: [net-next v5 06/15] iecm: Implement mailbox functionality

["Brady, Alan" <alan.brady@intel.com>]

> -----Original Message-----
> From: Tony Nguyen <anthony.l.nguyen@intel.com>
> Sent: Monday, August 24, 2020 10:33 AM
> To: davem@davemloft.net
> Cc: Michael, Alice <alice.michael@intel.com>; netdev@vger.kernel.org;
> nhorman@redhat.com; sassmann@redhat.com; Kirsher, Jeffrey T
> <jeffrey.t.kirsher@intel.com>; Nguyen, Anthony L
> <anthony.l.nguyen@intel.com>; Brady, Alan <alan.brady@intel.com>; Burra,
> Phani R <phani.r.burra@intel.com>; Hay, Joshua A <joshua.a.hay@intel.com>;
> Chittim, Madhu <madhu.chittim@intel.com>; Linga, Pavan Kumar
> <pavan.kumar.linga@intel.com>; Skidmore, Donald C
> <donald.c.skidmore@intel.com>; Brandeburg, Jesse
> <jesse.brandeburg@intel.com>; Samudrala, Sridhar
> <sridhar.samudrala@intel.com>
> Subject: [net-next v5 06/15] iecm: Implement mailbox functionality
> 
> From: Alice Michael <alice.michael@intel.com>
> 
> Implement mailbox setup, take down, and commands.
> 
> Signed-off-by: Alice Michael <alice.michael@intel.com>
> Signed-off-by: Alan Brady <alan.brady@intel.com>
> Signed-off-by: Phani Burra <phani.r.burra@intel.com>
> Signed-off-by: Joshua Hay <joshua.a.hay@intel.com>
> Signed-off-by: Madhu Chittim <madhu.chittim@intel.com>
> Signed-off-by: Pavan Kumar Linga <pavan.kumar.linga@intel.com>
> Reviewed-by: Donald Skidmore <donald.c.skidmore@intel.com>
> Reviewed-by: Jesse Brandeburg <jesse.brandeburg@intel.com>
> Reviewed-by: Sridhar Samudrala <sridhar.samudrala@intel.com>
> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
> Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com>
> ---
>  .../net/ethernet/intel/iecm/iecm_controlq.c   | 498 +++++++++++++++++-
>  .../ethernet/intel/iecm/iecm_controlq_setup.c | 105 +++-
>  drivers/net/ethernet/intel/iecm/iecm_lib.c    |  94 +++-
>  .../net/ethernet/intel/iecm/iecm_virtchnl.c   | 414 ++++++++++++++-
>  4 files changed, 1082 insertions(+), 29 deletions(-)
> 
>  /**
> @@ -28,7 +36,32 @@ static enum iecm_status iecm_ctlq_init_regs(struct
> iecm_hw *hw,
>  					    struct iecm_ctlq_info *cq,

I believe we have made a grave error and sent the wrong version.  iecm_status enum should be completely removed.  Sincere apologies for the thrash.  Will be sending another version up soon.

-Alan