Skip to content

Commit

Permalink
liquidio VF: fix incorrect struct being used
Browse files Browse the repository at this point in the history
The VF driver is using the wrong struct when sending commands to the NIC
firmware, sometimes causing adverse effects in the firmware.  The right
struct is the one that the PF is using, so make the VF use that as well.

Signed-off-by: Prasad Kanneganti <prasad.kanneganti@cavium.com>
Signed-off-by: Felix Manlunas <felix.manlunas@cavium.com>
Signed-off-by: Derek Chickles <derek.chickles@cavium.com>
Signed-off-by: Satanand Burla <satananda.burla@cavium.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
  • Loading branch information
Prasad Kanneganti authored and davem330 committed Jan 6, 2017
1 parent c1878f7 commit 90028b0
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions drivers/net/ethernet/cavium/liquidio/octeon_nic.c
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ octeon_alloc_soft_command_resp(struct octeon_device *oct,
/* Add in the response related fields. Opcode and Param are already
* there.
*/
if (OCTEON_CN23XX_PF(oct)) {
if (OCTEON_CN23XX_PF(oct) || OCTEON_CN23XX_VF(oct)) {
ih3 = (struct octeon_instr_ih3 *)&sc->cmd.cmd3.ih3;
rdp = (struct octeon_instr_rdp *)&sc->cmd.cmd3.rdp;
irh = (struct octeon_instr_irh *)&sc->cmd.cmd3.irh;
Expand All @@ -70,7 +70,7 @@ octeon_alloc_soft_command_resp(struct octeon_device *oct,

*sc->status_word = COMPLETION_WORD_INIT;

if (OCTEON_CN23XX_PF(oct))
if (OCTEON_CN23XX_PF(oct) || OCTEON_CN23XX_VF(oct))
sc->cmd.cmd3.rptr = sc->dmarptr;
else
sc->cmd.cmd2.rptr = sc->dmarptr;
Expand Down

0 comments on commit 90028b0

Please sign in to comment.