Review-me: in x8h7_can module, don't consider priv->tx_len > 0 as busy state

MaxPayne86 · MaxPayne86 · commit dabb3118f9c0 · 2023-09-19T16:00:59.000+02:00
diff --git a/recipes-kernel/kernel-modules/x8h7/x8h7_can.c b/recipes-kernel/kernel-modules/x8h7/x8h7_can.c
@@ -685,7 +685,8 @@ static netdev_tx_t x8h7_can_start_xmit(struct sk_buff *skb,
 
   DBG_PRINT("\n");
 
-  if (priv->tx_skb || priv->tx_len) {
+  //if (priv->tx_skb || priv->tx_len) { // @TODO: original impl.
+  if (priv->tx_skb) {
     DBG_ERROR("hard_xmit called while tx busy\n");
     return NETDEV_TX_BUSY;
   }

Original file line number	Diff line number	Diff line change
`@@ -685,7 +685,8 @@ static netdev_tx_t x8h7_can_start_xmit(struct sk_buff *skb,`
`685`	`685`
`686`	`686`	`DBG_PRINT("\n");`
`687`	`687`
`688`		`- if (priv->tx_skb \|\| priv->tx_len) {`
	`688`	`+ //if (priv->tx_skb \|\| priv->tx_len) { // @TODO: original impl.`
	`689`	`+ if (priv->tx_skb) {`
`689`	`690`	`DBG_ERROR("hard_xmit called while tx busy\n");`
`690`	`691`	`return NETDEV_TX_BUSY;`
`691`	`692`	`}`