From nobody Sun Jun 14 23:15:24 2026 X-Original-To: dev-commits-src-all@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4gdpyB3fc6z6hdyn for ; Sun, 14 Jun 2026 23:15:30 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org [IPv6:2610:1c1:1:606c::19:3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "mxrelay.nyi.freebsd.org", Issuer "R13" (not verified)) by mx1.freebsd.org (Postfix) with ESMTPS id 4gdpy95cDQz3kgX for ; Sun, 14 Jun 2026 23:15:29 +0000 (UTC) (envelope-from git@FreeBSD.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1781478929; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=aECXi8KkbLY5+zNIqGPrAEtmgQpMj/ej1BEAftQDznU=; b=QDiZWGvB/dUecCdUMXiG2NwEM4IMvamTbOWj2oMjLe14L6ebDsIk0Me8nakyZPRlQ4woee C7ZcTT9oi9+5cqNLgjiDtDfvzfJ/KCziZMkrwQTjAEbqndl7neHxylDk2GDsJIbh02GaHB FjilwYIZeq9zNgQBEzVs1LLpYr0G8N79qsDNKwM5SJH3LHAQtaeVnR11fSOaVsv7dJTo7T urlTvwmJjpreVKtNm7c9L1p5x7oZ8ZC/vsMc2nNE+Q6viuWjOnLoCjutu5nbyXpm2kZkFD mtRgugaDIFy1ISMoiHgw1NJSLbPf0XAgJm6cbqw1DWdn9MSwqyWERFcWCkdr4A== ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1781478929; a=rsa-sha256; cv=none; b=TEO+yJoXG4N916qyRJ1TBMOIKepNrgwLxqKoHeSYrBYseaB2eEmcpf30mUDHRZOHG8vXcD 8M0dgQW3sVdVbpCW5zF4+sPo+TVCMg30us1qX6U9+i5/izpsdHJE0URajTzEf4FCmjdRlI TQ0ktwdzbx7AlltGVJskBkY5LdsyB1zcsDsKBkx3uPr6kfrO4LT19QPz7HcVlbd3hLMTQY ezwCpBs4hEIlcjKwj3HTJtC4AismjJf4YfkBjhPD6z/TB5zZLVZiasuIu7OU9D4mDx84Sj msIDzbs40VGLHnTvOBm4H8xzqnR2Fl/TwGv/RgAQwZ/EW/jfVDDG/qQb3XyXig== ARC-Authentication-Results: i=1; mx1.freebsd.org; none ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1781478929; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=aECXi8KkbLY5+zNIqGPrAEtmgQpMj/ej1BEAftQDznU=; b=yr5ty2vuL7bnAZIQV41yNqf78TGEH1AWeWKYvnGK9zgBa2bRYPHIhnsFnKK7vX/n3KbFDX 8p5QjabNPdJii/4FGeeypoNPZB1y0XtHTQiSNG7uXVkWiX68GH9jW2nz7LEXVqKDUq1A9j WHMOc8WPJz1tHlJo65cUMqUPSOvMG4juKilFYPlZ24ry7Xh4MZ/XwEV7LPxcnXjvfqcMUn rdKDRUnhtMvJK6VERIfrx381Xal/mexPC2nodIyDoNAqZYg095mGHErViTQ8JLOjpsQu4r nWCFqoPXwfaX3qHga3f6VF+n54JCWwzfB9z2Vgq1REoh4fFt1K0S3O/zmI75BQ== Received: from gitrepo.freebsd.org (gitrepo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:5]) by mxrelay.nyi.freebsd.org (Postfix) with ESMTP id 4gdpy946XxzTq5 for ; Sun, 14 Jun 2026 23:15:29 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from git (uid 1279) (envelope-from git@FreeBSD.org) id 47482 by gitrepo.freebsd.org (DragonFly Mail Agent v0.13+ on gitrepo.freebsd.org); Sun, 14 Jun 2026 23:15:24 +0000 To: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org From: Bjoern A. Zeeb Subject: git: baf8561bdc3f - main - mt76: ensure net80211 com instance before returning from driver load List-Id: Commit messages for all branches of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-all List-Help: List-Post: List-Subscribe: List-Unsubscribe: X-BeenThere: dev-commits-src-all@freebsd.org Sender: owner-dev-commits-src-all@FreeBSD.org List-Id: List-Post: List-Help: List-Subscribe: List-Unsubscribe: List-Owner: Precedence: list MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Git-Committer: bz X-Git-Repository: src X-Git-Refname: refs/heads/main X-Git-Reftype: branch X-Git-Commit: baf8561bdc3f39c542a82cd1235fbf4bf97b4310 Auto-Submitted: auto-generated Date: Sun, 14 Jun 2026 23:15:24 +0000 Message-Id: <6a2f360c.47482.371163c9@gitrepo.freebsd.org> The branch main has been updated by bz: URL: https://cgit.FreeBSD.org/src/commit/?id=baf8561bdc3f39c542a82cd1235fbf4bf97b4310 commit baf8561bdc3f39c542a82cd1235fbf4bf97b4310 Author: Bjoern A. Zeeb AuthorDate: 2026-06-08 06:50:30 +0000 Commit: Bjoern A. Zeeb CommitDate: 2026-06-14 22:55:14 +0000 mt76: ensure net80211 com instance before returning from driver load Do as we have done for iwlwifi (f808c43ad923, bee60c989745) add a completion event for device registration which calls into 802.11 and creates the wifi "device" (net80211 com instance). This is needed as otherwise the deferred work in the mt76 drivers (mt7915, mt7921, mt7925, mt7996; but not the 7615 [*]) would make driver loading return before the wifi device is there. We would then continue, e.g., during rc startup and race possibly trying to create a vap (wlan interface) with the underlying device not being registered yet and fail. [*] the 7615 does not seem to do this asynchronously so is fine. Sponsored by: The FreeBSD Foundation Tested on: 7921, others to be tested at time MFC after: 3 days --- sys/contrib/dev/mediatek/mt76/mac80211.c | 4 ++++ sys/contrib/dev/mediatek/mt76/mt76.h | 3 +++ sys/contrib/dev/mediatek/mt76/mt7915/init.c | 6 ++++++ sys/contrib/dev/mediatek/mt76/mt7921/init.c | 7 +++++++ sys/contrib/dev/mediatek/mt76/mt7925/init.c | 7 +++++++ sys/contrib/dev/mediatek/mt76/mt7996/init.c | 6 ++++++ 6 files changed, 33 insertions(+) diff --git a/sys/contrib/dev/mediatek/mt76/mac80211.c b/sys/contrib/dev/mediatek/mt76/mac80211.c index bd7aaeb4398e..1f38b7ad6697 100644 --- a/sys/contrib/dev/mediatek/mt76/mac80211.c +++ b/sys/contrib/dev/mediatek/mt76/mac80211.c @@ -814,6 +814,10 @@ int mt76_register_device(struct mt76_dev *dev, bool vht, set_bit(MT76_STATE_REGISTERED, &phy->state); sched_set_fifo_low(dev->tx_worker.task); +#if defined(__FreeBSD__) + complete(&dev->drv_start_complete); +#endif + return 0; } EXPORT_SYMBOL_GPL(mt76_register_device); diff --git a/sys/contrib/dev/mediatek/mt76/mt76.h b/sys/contrib/dev/mediatek/mt76/mt76.h index 840725b8b020..ca26a4e43b92 100644 --- a/sys/contrib/dev/mediatek/mt76/mt76.h +++ b/sys/contrib/dev/mediatek/mt76/mt76.h @@ -1026,6 +1026,9 @@ struct mt76_dev { } test_mtd; #endif struct workqueue_struct *wq; +#if defined(__FreeBSD__) + struct completion drv_start_complete; +#endif union { struct mt76_mmio mmio; diff --git a/sys/contrib/dev/mediatek/mt76/mt7915/init.c b/sys/contrib/dev/mediatek/mt76/mt7915/init.c index 4128942361ff..d87878f9a86e 100644 --- a/sys/contrib/dev/mediatek/mt76/mt7915/init.c +++ b/sys/contrib/dev/mediatek/mt76/mt7915/init.c @@ -862,6 +862,9 @@ mt7915_init_hardware(struct mt7915_dev *dev, struct mt7915_phy *phy2) mt76_wr(dev, MT_INT_SOURCE_CSR, ~0); INIT_WORK(&dev->init_work, mt7915_init_work); +#if defined(__FreeBSD__) + init_completion(&dev->mt76.drv_start_complete); +#endif ret = mt7915_dma_init(dev, phy2); if (ret) @@ -1298,6 +1301,9 @@ int mt7915_register_device(struct mt7915_dev *dev) } ieee80211_queue_work(mt76_hw(dev), &dev->init_work); +#if defined(__FreeBSD__) + wait_for_completion(&dev->mt76.drv_start_complete); +#endif dev->recovery.hw_init_done = true; diff --git a/sys/contrib/dev/mediatek/mt76/mt7921/init.c b/sys/contrib/dev/mediatek/mt76/mt7921/init.c index dbbc9e945fe2..ece84a826ea4 100644 --- a/sys/contrib/dev/mediatek/mt76/mt7921/init.c +++ b/sys/contrib/dev/mediatek/mt76/mt7921/init.c @@ -308,6 +308,9 @@ int mt7921_register_device(struct mt792x_dev *dev) INIT_WORK(&dev->reset_work, mt7921_mac_reset_work); INIT_WORK(&dev->init_work, mt7921_init_work); +#if defined(__FreeBSD__) + init_completion(&dev->mt76.drv_start_complete); +#endif INIT_WORK(&dev->phy.roc_work, mt7921_roc_work); timer_setup(&dev->phy.roc_timer, mt792x_roc_timer, 0); @@ -359,6 +362,10 @@ int mt7921_register_device(struct mt792x_dev *dev) queue_work(system_percpu_wq, &dev->init_work); +#if defined(__FreeBSD__) + wait_for_completion(&dev->mt76.drv_start_complete); +#endif + return 0; } EXPORT_SYMBOL_GPL(mt7921_register_device); diff --git a/sys/contrib/dev/mediatek/mt76/mt7925/init.c b/sys/contrib/dev/mediatek/mt76/mt7925/init.c index 31592775feb3..3b7c1668dc61 100644 --- a/sys/contrib/dev/mediatek/mt76/mt7925/init.c +++ b/sys/contrib/dev/mediatek/mt76/mt7925/init.c @@ -228,6 +228,9 @@ int mt7925_register_device(struct mt792x_dev *dev) INIT_WORK(&dev->reset_work, mt7925_mac_reset_work); INIT_WORK(&dev->init_work, mt7925_init_work); +#if defined(__FreeBSD__) + init_completion(&dev->mt76.drv_start_complete); +#endif INIT_WORK(&dev->phy.roc_work, mt7925_roc_work); timer_setup(&dev->phy.roc_timer, mt792x_roc_timer, 0); @@ -282,6 +285,10 @@ int mt7925_register_device(struct mt792x_dev *dev) queue_work(system_percpu_wq, &dev->init_work); +#if defined(__FreeBSD__) + wait_for_completion(&dev->mt76.drv_start_complete); +#endif + return 0; } EXPORT_SYMBOL_GPL(mt7925_register_device); diff --git a/sys/contrib/dev/mediatek/mt76/mt7996/init.c b/sys/contrib/dev/mediatek/mt76/mt7996/init.c index 34f1df207f19..6568f77bba37 100644 --- a/sys/contrib/dev/mediatek/mt76/mt7996/init.c +++ b/sys/contrib/dev/mediatek/mt76/mt7996/init.c @@ -1222,6 +1222,9 @@ static int mt7996_init_hardware(struct mt7996_dev *dev) } INIT_WORK(&dev->init_work, mt7996_init_work); +#if defined(__FreeBSD__) + init_completion(&dev->mt76.drv_start_complete); +#endif INIT_WORK(&dev->wed_rro.work, mt7996_wed_rro_work); INIT_LIST_HEAD(&dev->wed_rro.poll_list); spin_lock_init(&dev->wed_rro.lock); @@ -1736,6 +1739,9 @@ int mt7996_register_device(struct mt7996_dev *dev) #endif ieee80211_queue_work(mt76_hw(dev), &dev->init_work); +#if defined(__FreeBSD__) + wait_for_completion(&dev->mt76.drv_start_complete); +#endif dev->recovery.hw_init_done = true;