From nobody Sat Jun 20 19:10:43 2026 X-Original-To: dev-commits-src-main@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 4gjPDz5LKtz6hNrx for ; Sat, 20 Jun 2026 19:10:43 +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 "YR1" (not verified)) by mx1.freebsd.org (Postfix) with ESMTPS id 4gjPDz2pPkz3vCx for ; Sat, 20 Jun 2026 19:10:43 +0000 (UTC) (envelope-from git@FreeBSD.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1781982643; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=lv19Tm26iduCmbXllAazpI8lbMyUY7l6lPLlwPmSY9w=; b=K7Tquw8csLJLEV8oT3eEuymWbzFPQy/mbqkBr74xkDyBVLkrcGyuy14F12r7xtNiNncvTY LXe9f9dW4GBkmHRVzADs2mUKXr+n4HNF03l7fMKbvzBqd8U+0YS9tZmEXL+HZk+IIsBViX SIahHCmtwkA8KrbD1hfRfIrYInDONPxWH5gkvdB26r/NGaxJ0f3juXgErRjpGBbqnV+knB 3nIWf3ODVELvxq7EqiIloimmf6+1uJMpurKSz+nIUlSNo856qjKJf49syFkxzPwMzuyypV i1Ii1NcjJv8wC9jKPxbNd2LpdmHjFXlT7GkgjnNzFMhbE9bI1dwlosU52iwm6A== ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1781982643; a=rsa-sha256; cv=none; b=QhJILqkjucAfGiZvQfsl6TcYneLzPeZcI9CS9lBd9gGi3en//HBO0VsBH9JUlPt2A1wesE Z9qz8JQubS4tByRBoDefGmujMFJPxpd5LRkRa6k1Y+S395jrA8z/cjpFPl12v33pRxkkAj 7aYn//X8KP4PYN/+XSYCr9wvGxxLOGMSO3H+dO7WlgpKnWpk4TBcUOHHrwzEbajLj93yLD cs6CQFQHAOdMaGqZgFVvKHiWshj/1qnVAfy1rP33CuYGMmRtGY333d7l9A9SOZY7VT5QzB Y2WzzB6qKhT2QmPUej5+3rMMgCMX47dingenC42qaeZ0DOG5W4s9XeKXevMQZA== 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=1781982643; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=lv19Tm26iduCmbXllAazpI8lbMyUY7l6lPLlwPmSY9w=; b=dk3BV/AmLty90Wix6A6GsdIIQsUNXKKwphFadP4gQ0wzPJBPeslH0xXcrAMnUO8lOxnNf7 dIYvERYQ0plvOcvxhYcodyquWOQgkwD74tOshi9mEaakZXvF+66IQWJLH+LZwDSsAa1Ly3 D+0qzbAtYW67or8VrNILowsUHRD2lLdIgmx2OQtDoH/MsMHF5AlVry75uu/4vDOMo1HZJ7 ztTgfJKAuCUVTVuHMbRDr+rJZBU53Pk1P3Iy/Clt2W7OsfQL0JnLThmzY8McE3oMopSOiZ 7fFX7jRoUG5oY0YSzPAoO0ZiVYm9SZkIPIssmaL54sryV8AKfHD/VRNk6Hzhqg== Received: from gitrepo.freebsd.org (gitrepo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:5]) by mxrelay.nyi.freebsd.org (Postfix) with ESMTP id 4gjPDz1syKzk6Z for ; Sat, 20 Jun 2026 19:10:43 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from git (uid 1279) (envelope-from git@FreeBSD.org) id 333cd by gitrepo.freebsd.org (DragonFly Mail Agent v0.13+ on gitrepo.freebsd.org); Sat, 20 Jun 2026 19:10:43 +0000 To: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org Cc: Nick Price From: Adrian Chadd Subject: git: e44579e23430 - main - aq(4): naming and exposure List-Id: Commit messages for the main branch of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-main List-Help: List-Post: List-Subscribe: List-Unsubscribe: X-BeenThere: dev-commits-src-main@freebsd.org Sender: owner-dev-commits-src-main@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: adrian X-Git-Repository: src X-Git-Refname: refs/heads/main X-Git-Reftype: branch X-Git-Commit: e44579e23430ff2084b03c9b3d486f05e617e04f Auto-Submitted: auto-generated Date: Sat, 20 Jun 2026 19:10:43 +0000 Message-Id: <6a36e5b3.333cd.2e8b4bff@gitrepo.freebsd.org> The branch main has been updated by adrian: URL: https://cgit.FreeBSD.org/src/commit/?id=e44579e23430ff2084b03c9b3d486f05e617e04f commit e44579e23430ff2084b03c9b3d486f05e617e04f Author: Nick Price AuthorDate: 2026-06-20 19:03:37 +0000 Commit: Adrian Chadd CommitDate: 2026-06-20 19:10:16 +0000 aq(4): naming and exposure Reviewed by: adrian Differential Revision: https://reviews.freebsd.org/D57656 --- sys/dev/aq/aq_fw.c | 40 ++++++++++++++++++---------------------- sys/dev/aq/aq_fw.h | 2 ++ sys/dev/aq/aq_fw1x.c | 32 ++++++++++++++++---------------- sys/dev/aq/aq_fw2x.c | 52 ++++++++++++++++++++++++++-------------------------- sys/dev/aq/aq_main.c | 2 +- 5 files changed, 63 insertions(+), 65 deletions(-) diff --git a/sys/dev/aq/aq_fw.c b/sys/dev/aq/aq_fw.c index 7f9496add875..72751c81b773 100644 --- a/sys/dev/aq/aq_fw.c +++ b/sys/dev/aq/aq_fw.c @@ -85,14 +85,10 @@ const uint32_t RBL_STATUS_HOST_BOOT = 0xf1a7; const uint32_t SCRATCHPAD_FW_LOADER_STATUS = (0x40 / sizeof(uint32_t)); -extern const struct aq_firmware_ops aq_fw1x_ops; -extern const struct aq_firmware_ops aq_fw2x_ops; - - -int mac_soft_reset_(struct aq_hw* hw, enum aq_fw_bootloader_mode* mode); -int mac_soft_reset_flb_(struct aq_hw* hw); -int mac_soft_reset_rbl_(struct aq_hw* hw, enum aq_fw_bootloader_mode* mode); -int wait_init_mac_firmware_(struct aq_hw* hw); +static int mac_soft_reset(struct aq_hw* hw, enum aq_fw_bootloader_mode* mode); +static int mac_soft_reset_flb(struct aq_hw* hw); +static int mac_soft_reset_rbl(struct aq_hw* hw, enum aq_fw_bootloader_mode* mode); +static int wait_init_mac_firmware(struct aq_hw* hw); int @@ -124,14 +120,14 @@ aq_fw_reset(struct aq_hw* hw) * 2) Driver may skip reset sequence and save time. */ if (hw->fast_start_enabled && !ver) { - int err = wait_init_mac_firmware_(hw); + int err = wait_init_mac_firmware(hw); /* Skip reset as it just completed */ if (!err) return (0); } enum aq_fw_bootloader_mode mode = boot_mode_unknown; - int err = mac_soft_reset_(hw, &mode); + int err = mac_soft_reset(hw, &mode); if (err != 0) { device_printf(hw->dev, "MAC reset failed: %d\n", err); return (err); @@ -141,12 +137,12 @@ aq_fw_reset(struct aq_hw* hw) case boot_mode_flb: aq_log("FLB> F/W successfully loaded from flash."); hw->flash_present = true; - return wait_init_mac_firmware_(hw); + return wait_init_mac_firmware(hw); case boot_mode_rbl_flash: aq_log("RBL> F/W loaded from flash. Host Bootload disabled."); hw->flash_present = true; - return wait_init_mac_firmware_(hw); + return wait_init_mac_firmware(hw); case boot_mode_unknown: device_printf(hw->dev, "F/W bootload error: unknown bootloader type\n"); @@ -196,21 +192,21 @@ aq_fw_ops_init(struct aq_hw* hw) } -int -mac_soft_reset_(struct aq_hw* hw, enum aq_fw_bootloader_mode* mode /*= nullptr*/) +static int +mac_soft_reset(struct aq_hw* hw, enum aq_fw_bootloader_mode* mode /*= nullptr*/) { if (hw->rbl_enabled) { - return mac_soft_reset_rbl_(hw, mode); + return mac_soft_reset_rbl(hw, mode); } else { if (mode) *mode = boot_mode_flb; - return mac_soft_reset_flb_(hw); + return mac_soft_reset_flb(hw); } } -int -mac_soft_reset_flb_(struct aq_hw* hw) +static int +mac_soft_reset_flb(struct aq_hw* hw) { int k; @@ -296,8 +292,8 @@ mac_soft_reset_flb_(struct aq_hw* hw) return (0); } -int -mac_soft_reset_rbl_(struct aq_hw* hw, enum aq_fw_bootloader_mode* mode) +static int +mac_soft_reset_rbl(struct aq_hw* hw, enum aq_fw_bootloader_mode* mode) { trace(dbg_init, "RBL> MAC reset STARTED!"); @@ -349,8 +345,8 @@ mac_soft_reset_rbl_(struct aq_hw* hw, enum aq_fw_bootloader_mode* mode) return (0); } -int -wait_init_mac_firmware_(struct aq_hw* hw) +static int +wait_init_mac_firmware(struct aq_hw* hw) { for (int i = 0; i < MAC_FW_START_TIMEOUT_MS; ++i) { if ((hw->fw_version.raw = AQ_READ_REG(hw, 0x18)) != 0) diff --git a/sys/dev/aq/aq_fw.h b/sys/dev/aq/aq_fw.h index 9195c91d6b5f..be6661db7d57 100644 --- a/sys/dev/aq/aq_fw.h +++ b/sys/dev/aq/aq_fw.h @@ -70,6 +70,8 @@ struct aq_firmware_ops int (*led_control)(struct aq_hw* hw, uint32_t mode); }; +extern const struct aq_firmware_ops aq_fw1x_ops; +extern const struct aq_firmware_ops aq_fw2x_ops; int aq_fw_reset(struct aq_hw* hw); int aq_fw_ops_init(struct aq_hw* hw); diff --git a/sys/dev/aq/aq_fw1x.c b/sys/dev/aq/aq_fw1x.c index 731370e94f3e..72b7f0456ddb 100644 --- a/sys/dev/aq/aq_fw1x.c +++ b/sys/dev/aq/aq_fw1x.c @@ -79,18 +79,18 @@ union fw1x_state_reg { }; }; -int fw1x_reset(struct aq_hw* hw); +static int fw1x_reset(struct aq_hw* hw); -int fw1x_set_mode(struct aq_hw* hw, enum aq_hw_fw_mpi_state mode, +static int fw1x_set_mode(struct aq_hw* hw, enum aq_hw_fw_mpi_state mode, enum aq_fw_link_speed speed); -int fw1x_get_mode(struct aq_hw* hw, enum aq_hw_fw_mpi_state* mode, +static int fw1x_get_mode(struct aq_hw* hw, enum aq_hw_fw_mpi_state* mode, enum aq_fw_link_speed* speed, enum aq_fw_link_fc* fc); -int fw1x_get_mac_addr(struct aq_hw* hw, uint8_t* mac_addr); -int fw1x_get_stats(struct aq_hw* hw, struct aq_hw_stats* stats); +static int fw1x_get_mac_addr(struct aq_hw* hw, uint8_t* mac_addr); +static int fw1x_get_stats(struct aq_hw* hw, struct aq_hw_stats* stats); static enum fw1x_mode -mpi_mode_to_fw1x_(enum aq_hw_fw_mpi_state mode) +mpi_mode_to_fw1x(enum aq_hw_fw_mpi_state mode) { switch (mode) { case MPI_DEINIT: @@ -114,7 +114,7 @@ mpi_mode_to_fw1x_(enum aq_hw_fw_mpi_state mode) } static enum aq_fw1x_rate -link_speed_mask_to_fw1x_(uint32_t /*aq_fw_link_speed*/ speed) +link_speed_mask_to_fw1x(uint32_t /*aq_fw_link_speed*/ speed) { uint32_t rate = 0; if (speed & aq_fw_10G) @@ -138,7 +138,7 @@ link_speed_mask_to_fw1x_(uint32_t /*aq_fw_link_speed*/ speed) } static enum aq_fw_link_speed -fw1x_rate_to_link_speed_(enum aq_fw1x_rate rate) +fw1x_rate_to_link_speed(enum aq_fw1x_rate rate) { switch (rate) { case FW1X_RATE_10G: @@ -163,7 +163,7 @@ fw1x_rate_to_link_speed_(enum aq_fw1x_rate rate) return (aq_fw_none); } -int +static int fw1x_reset(struct aq_hw* hw) { uint32_t tid0 = ~0u; /*< Initial value of MBOX transactionId. */ @@ -200,13 +200,13 @@ fw1x_reset(struct aq_hw* hw) return (EBUSY); } -int +static int fw1x_set_mode(struct aq_hw* hw, enum aq_hw_fw_mpi_state mode, enum aq_fw_link_speed speed) { union fw1x_state_reg state = {0}; - state.mode = mpi_mode_to_fw1x_(mode); - state.speed = link_speed_mask_to_fw1x_(speed); + state.mode = mpi_mode_to_fw1x(mode); + state.speed = link_speed_mask_to_fw1x(speed); trace(dbg_init, "fw1x> set mode %d, rate mask = %#x; raw = %#x", state.mode, state.speed, state.val); @@ -216,7 +216,7 @@ fw1x_set_mode(struct aq_hw* hw, enum aq_hw_fw_mpi_state mode, return (0); } -int +static int fw1x_get_mode(struct aq_hw* hw, enum aq_hw_fw_mpi_state* mode, enum aq_fw_link_speed* speed, enum aq_fw_link_fc* fc) { @@ -246,7 +246,7 @@ fw1x_get_mode(struct aq_hw* hw, enum aq_hw_fw_mpi_state* mode, *mode = md; if (speed) - *speed = fw1x_rate_to_link_speed_(state.speed); + *speed = fw1x_rate_to_link_speed(state.speed); *fc = aq_fw_fc_none; @@ -255,7 +255,7 @@ fw1x_get_mode(struct aq_hw* hw, enum aq_hw_fw_mpi_state* mode, } -int +static int fw1x_get_mac_addr(struct aq_hw* hw, uint8_t* mac) { int err = EFAULT; @@ -291,7 +291,7 @@ fw1x_get_mac_addr(struct aq_hw* hw, uint8_t* mac) return (0); } -int +static int fw1x_get_stats(struct aq_hw* hw, struct aq_hw_stats* stats) { int err = 0; diff --git a/sys/dev/aq/aq_fw2x.c b/sys/dev/aq/aq_fw2x.c index 339644d0c8e1..bf18f43fd548 100644 --- a/sys/dev/aq/aq_fw2x.c +++ b/sys/dev/aq/aq_fw2x.c @@ -193,19 +193,19 @@ struct fw2x_mailbox // struct fwHostInterface #define FW2X_LED_DEFAULT 0x0U // Firmware v2-3.x specific functions. -int fw2x_reset(struct aq_hw* hw); +static int fw2x_reset(struct aq_hw* hw); -int fw2x_set_mode(struct aq_hw* hw, enum aq_hw_fw_mpi_state mode, +static int fw2x_set_mode(struct aq_hw* hw, enum aq_hw_fw_mpi_state mode, enum aq_fw_link_speed speed); -int fw2x_get_mode(struct aq_hw* hw, enum aq_hw_fw_mpi_state* mode, +static int fw2x_get_mode(struct aq_hw* hw, enum aq_hw_fw_mpi_state* mode, enum aq_fw_link_speed* speed, enum aq_fw_link_fc* fc); -int fw2x_get_mac_addr(struct aq_hw* hw, uint8_t* mac); -int fw2x_get_stats(struct aq_hw* hw, struct aq_hw_stats* stats); +static int fw2x_get_mac_addr(struct aq_hw* hw, uint8_t* mac); +static int fw2x_get_stats(struct aq_hw* hw, struct aq_hw_stats* stats); static uint64_t -read64_(struct aq_hw* hw, uint32_t addr) +read64(struct aq_hw* hw, uint32_t addr) { uint64_t lo, hi, hi2; @@ -220,26 +220,26 @@ read64_(struct aq_hw* hw, uint32_t addr) } static uint64_t -get_mpi_ctrl_(struct aq_hw* hw) +get_mpi_ctrl(struct aq_hw* hw) { - return read64_(hw, FW2X_MPI_CONTROL_ADDR); + return read64(hw, FW2X_MPI_CONTROL_ADDR); } static uint64_t -get_mpi_state_(struct aq_hw* hw) +get_mpi_state(struct aq_hw* hw) { - return read64_(hw, FW2X_MPI_STATE_ADDR); + return read64(hw, FW2X_MPI_STATE_ADDR); } static void -set_mpi_ctrl_(struct aq_hw* hw, uint64_t value) +set_mpi_ctrl(struct aq_hw* hw, uint64_t value) { AQ_WRITE_REG(hw, FW2X_MPI_CONTROL_ADDR, (uint32_t)value); AQ_WRITE_REG(hw, FW2X_MPI_CONTROL_ADDR + 4, (uint32_t)(value >> 32)); } -int +static int fw2x_reset(struct aq_hw* hw) { struct fw2x_capabilities caps = {0}; @@ -263,7 +263,7 @@ fw2x_reset(struct aq_hw* hw) static enum aq_fw2x_rate -link_speed_mask_to_fw2x_(uint32_t speed) +link_speed_mask_to_fw2x(uint32_t speed) { uint32_t rate = 0; @@ -288,17 +288,17 @@ link_speed_mask_to_fw2x_(uint32_t speed) } -int +static int fw2x_set_mode(struct aq_hw* hw, enum aq_hw_fw_mpi_state mode, enum aq_fw_link_speed speed) { - uint64_t mpi_ctrl = get_mpi_ctrl_(hw); + uint64_t mpi_ctrl = get_mpi_ctrl(hw); AQ_DBG_ENTERA("speed=%d", speed); switch (mode) { case MPI_INIT: mpi_ctrl &= ~FW2X_RATE_MASK; - mpi_ctrl |= link_speed_mask_to_fw2x_(speed); + mpi_ctrl |= link_speed_mask_to_fw2x(speed); mpi_ctrl &= ~FW2X_CAP_LINK_DROP; #if 0 // #todo #flowcontrol #pause #eee if (pHal->pCfg->eee) @@ -320,22 +320,22 @@ fw2x_set_mode(struct aq_hw* hw, enum aq_hw_fw_mpi_state mode, return (EINVAL); } - set_mpi_ctrl_(hw, mpi_ctrl); + set_mpi_ctrl(hw, mpi_ctrl); AQ_DBG_EXIT(0); return (0); } -int +static int fw2x_get_mode(struct aq_hw* hw, enum aq_hw_fw_mpi_state* mode, enum aq_fw_link_speed* link_speed, enum aq_fw_link_fc* fc) { - uint64_t mpi_state = get_mpi_state_(hw); + uint64_t mpi_state = get_mpi_state(hw); uint32_t rates = mpi_state & FW2X_RATE_MASK; // AQ_DBG_ENTER(); if (mode) { - uint64_t mpi_ctrl = get_mpi_ctrl_(hw); + uint64_t mpi_ctrl = get_mpi_ctrl(hw); if (mpi_ctrl & FW2X_RATE_MASK) *mode = MPI_INIT; else @@ -366,7 +366,7 @@ fw2x_get_mode(struct aq_hw* hw, enum aq_hw_fw_mpi_state* mode, } -int +static int fw2x_get_mac_addr(struct aq_hw* hw, uint8_t* mac) { int err = EFAULT; @@ -400,7 +400,7 @@ fw2x_get_mac_addr(struct aq_hw* hw, uint8_t* mac) } static inline void -fw2x_stats_to_fw_stats_(struct aq_hw_stats* dst, +fw2x_stats_to_fw_stats(struct aq_hw_stats* dst, const struct fw2x_msm_statistics* src) { dst->uprc = src->uprc; @@ -422,7 +422,7 @@ fw2x_stats_to_fw_stats_(struct aq_hw_stats* dst, } -int +static int fw2x_get_stats(struct aq_hw* hw, struct aq_hw_stats* stats) { struct fw2x_msm_statistics fw2x_stats = {0}; @@ -439,15 +439,15 @@ fw2x_get_stats(struct aq_hw* hw, struct aq_hw_stats* stats) hw->mbox_addr + offsetof(struct fw2x_mailbox, msm), (uint32_t*)&fw2x_stats, sizeof fw2x_stats/sizeof(uint32_t)); - fw2x_stats_to_fw_stats_(stats, &fw2x_stats); + fw2x_stats_to_fw_stats(stats, &fw2x_stats); if (err != 0) trace_error(dbg_fw, "fw2x> download statistics data FAILED, error %d", err); - mpi_ctrl = get_mpi_ctrl_(hw); + mpi_ctrl = get_mpi_ctrl(hw); mpi_ctrl ^= FW2X_CAP_STATISTICS; - set_mpi_ctrl_(hw, mpi_ctrl); + set_mpi_ctrl(hw, mpi_ctrl); return (err); } diff --git a/sys/dev/aq/aq_main.c b/sys/dev/aq/aq_main.c index e1b294012b27..d4f687fee8f1 100644 --- a/sys/dev/aq/aq_main.c +++ b/sys/dev/aq/aq_main.c @@ -75,7 +75,7 @@ __FBSDID("$FreeBSD$"); MALLOC_DEFINE(M_AQ, "aq", "Aquantia"); -char aq_driver_version[] = AQ_VER; +static const char aq_driver_version[] = AQ_VER; #define AQUANTIA_VENDOR_ID 0x1D6A