Allows release of the network buffer if gmac_dev_write() fails and the ero-copy driver is being used.

Emil Popov · Emil Popov · commit cb12473a3f5d · 2023-10-18T11:45:39.000-04:00
Thanks @htibosch
diff --git a/source/portable/NetworkInterface/DriverSAM/NetworkInterface.c b/source/portable/NetworkInterface/DriverSAM/NetworkInterface.c
@@ -661,8 +661,12 @@ static BaseType_t prvSAM_NetworkInterfaceOutput( NetworkInterface_t * pxInterfac
 
         #if ( ipconfigZERO_COPY_TX_DRIVER != 0 )
         {
-            /* Confirm that the pxDescriptor may be kept by the driver. */
-            bReleaseAfterSend = pdFALSE;
+            if( ulResult == GMAC_OK )
+            {
+                /* The message was send in a zero-copy way.
+                 * It will be released after a succeful transmission. */
+                bReleaseAfterSend = pdFALSE;
+            }
         }
         #endif /* ipconfigZERO_COPY_TX_DRIVER */
         /* Not interested in a call-back after TX. */

Original file line number	Diff line number	Diff line change
`@@ -661,8 +661,12 @@ static BaseType_t prvSAM_NetworkInterfaceOutput( NetworkInterface_t * pxInterfac`
`661`	`661`
`662`	`662`	`#if ( ipconfigZERO_COPY_TX_DRIVER != 0 )`
`663`	`663`	`{`
`664`		`- /* Confirm that the pxDescriptor may be kept by the driver. */`
`665`		`- bReleaseAfterSend = pdFALSE;`
	`664`	`+ if( ulResult == GMAC_OK )`
	`665`	`+ {`
	`666`	`+ /* The message was send in a zero-copy way.`
	`667`	`+ * It will be released after a succeful transmission. */`
	`668`	`+ bReleaseAfterSend = pdFALSE;`
	`669`	`+ }`
`666`	`670`	`}`
`667`	`671`	`#endif /* ipconfigZERO_COPY_TX_DRIVER */`
`668`	`672`	`/* Not interested in a call-back after TX. */`