Message ID | 1588601168-27576-10-git-send-email-sai.pavan.boddu@xilinx.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | Cadence GEM Fixes | expand |
On Mon, May 04, 2020 at 07:36:07PM +0530, Sai Pavan Boddu wrote: > TX_LAST bit should not be set by hardware, its set by guest to inform > the last bd of the frame. Reviewed-by: Edgar E. Iglesias <edgar.iglesias@xilinx.com> > > Signed-off-by: Sai Pavan Boddu <sai.pavan.boddu@xilinx.com> > Signed-off-by: Edgar E. Iglesias <edgar.iglesias@xilinx.com> > --- > hw/net/cadence_gem.c | 6 ------ > 1 file changed, 6 deletions(-) > > diff --git a/hw/net/cadence_gem.c b/hw/net/cadence_gem.c > index ac3a553..f0bf2e7 100644 > --- a/hw/net/cadence_gem.c > +++ b/hw/net/cadence_gem.c > @@ -345,11 +345,6 @@ static inline unsigned tx_desc_get_last(uint32_t *desc) > return (desc[1] & DESC_1_TX_LAST) ? 1 : 0; > } > > -static inline void tx_desc_set_last(uint32_t *desc) > -{ > - desc[1] |= DESC_1_TX_LAST; > -} > - > static inline unsigned tx_desc_get_length(uint32_t *desc) > { > return desc[1] & DESC_1_LENGTH; > @@ -1270,7 +1265,6 @@ static void gem_transmit(CadenceGEMState *s) > > /* read next descriptor */ > if (tx_desc_get_wrap(desc)) { > - tx_desc_set_last(desc); > > if (s->regs[GEM_DMACFG] & GEM_DMACFG_ADDR_64B) { > packet_desc_addr = s->regs[GEM_TBQPH]; > -- > 2.7.4 >
diff --git a/hw/net/cadence_gem.c b/hw/net/cadence_gem.c index ac3a553..f0bf2e7 100644 --- a/hw/net/cadence_gem.c +++ b/hw/net/cadence_gem.c @@ -345,11 +345,6 @@ static inline unsigned tx_desc_get_last(uint32_t *desc) return (desc[1] & DESC_1_TX_LAST) ? 1 : 0; } -static inline void tx_desc_set_last(uint32_t *desc) -{ - desc[1] |= DESC_1_TX_LAST; -} - static inline unsigned tx_desc_get_length(uint32_t *desc) { return desc[1] & DESC_1_LENGTH; @@ -1270,7 +1265,6 @@ static void gem_transmit(CadenceGEMState *s) /* read next descriptor */ if (tx_desc_get_wrap(desc)) { - tx_desc_set_last(desc); if (s->regs[GEM_DMACFG] & GEM_DMACFG_ADDR_64B) { packet_desc_addr = s->regs[GEM_TBQPH];