wifi: esp_at: notify receivers on async close

Jordan Yates · carlescufi · commit a2c0bd8dd1f2 · 2022-05-13T14:53:48.000+02:00
`esp_close_work` can be queued from the `on_cmd_closed`, which clears
`ESP_SOCK_CONNECTED` and sets `ESP_SOCK_CLOSE_PENDING`, but does no
further work. The receive callback should still be run with no data when
the socket is closed through this mechanism.

Signed-off-by: Jordan Yates &lt;jordan.yates@data61.csiro.au&gt;
diff --git a/drivers/wifi/esp_at/esp_offload.c b/drivers/wifi/esp_at/esp_offload.c
@@ -517,7 +517,7 @@ void esp_close_work(struct k_work *work)
 	}
 
 	/* Should we notify that the socket has been closed? */
-	if (old_flags & ESP_SOCK_CONNECTED) {
+	if (old_flags & ESP_SOCK_CLOSE_PENDING) {
 		k_mutex_lock(&sock->lock, K_FOREVER);
 		if (sock->recv_cb) {
 			sock->recv_cb(sock->context, NULL, NULL, NULL, 0,

Original file line number	Diff line number	Diff line change
`@@ -517,7 +517,7 @@ void esp_close_work(struct k_work *work)`
`517`	`517`	`}`
`518`	`518`
`519`	`519`	`/* Should we notify that the socket has been closed? */`
`520`		`- if (old_flags & ESP_SOCK_CONNECTED) {`
	`520`	`+ if (old_flags & ESP_SOCK_CLOSE_PENDING) {`
`521`	`521`	`k_mutex_lock(&sock->lock, K_FOREVER);`
`522`	`522`	`if (sock->recv_cb) {`
`523`	`523`	`sock->recv_cb(sock->context, NULL, NULL, NULL, 0,`