Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/73173/?format=api
{ "id": 73173, "url": "http://patches.dpdk.org/api/patches/73173/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/20200706081222.19279-9-guinanx.sun@intel.com/", "project": { "id": 1, "url": "http://patches.dpdk.org/api/projects/1/?format=api", "name": "DPDK", "link_name": "dpdk", "list_id": "dev.dpdk.org", "list_email": "dev@dpdk.org", "web_url": "http://core.dpdk.org", "scm_url": "git://dpdk.org/dpdk", "webscm_url": "http://git.dpdk.org/dpdk", "list_archive_url": "https://inbox.dpdk.org/dev", "list_archive_url_format": "https://inbox.dpdk.org/dev/{}", "commit_url_format": "" }, "msgid": "<20200706081222.19279-9-guinanx.sun@intel.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20200706081222.19279-9-guinanx.sun@intel.com", "date": "2020-07-06T08:12:03", "name": "[v3,08/27] net/e1000/base: improve code style and fix klocwork errors", "commit_ref": null, "pull_url": null, "state": "accepted", "archived": true, "hash": "2e741553d9edd15daab96bdcbf46618b681f00c3", "submitter": { "id": 1476, "url": "http://patches.dpdk.org/api/people/1476/?format=api", "name": "Guinan Sun", "email": "guinanx.sun@intel.com" }, "delegate": { "id": 1540, "url": "http://patches.dpdk.org/api/users/1540/?format=api", "username": "qzhan15", "first_name": "Qi", "last_name": "Zhang", "email": "qi.z.zhang@intel.com" }, "mbox": "http://patches.dpdk.org/project/dpdk/patch/20200706081222.19279-9-guinanx.sun@intel.com/mbox/", "series": [ { "id": 10803, "url": "http://patches.dpdk.org/api/series/10803/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=10803", "date": "2020-07-06T08:11:55", "name": "update e1000 base code", "version": 3, "mbox": "http://patches.dpdk.org/series/10803/mbox/" } ], "comments": "http://patches.dpdk.org/api/patches/73173/comments/", "check": "warning", "checks": "http://patches.dpdk.org/api/patches/73173/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "<dev-bounces@dpdk.org>", "X-Original-To": "patchwork@inbox.dpdk.org", "Delivered-To": "patchwork@inbox.dpdk.org", "Received": [ "from dpdk.org (dpdk.org [92.243.14.124])\n\tby inbox.dpdk.org (Postfix) with ESMTP id C2D58A00C5;\n\tMon, 6 Jul 2020 10:25:13 +0200 (CEST)", "from [92.243.14.124] (localhost [127.0.0.1])\n\tby dpdk.org (Postfix) with ESMTP id B5BE21D8FB;\n\tMon, 6 Jul 2020 10:24:09 +0200 (CEST)", "from mga03.intel.com (mga03.intel.com [134.134.136.65])\n by dpdk.org (Postfix) with ESMTP id 9425D1D902\n for <dev@dpdk.org>; Mon, 6 Jul 2020 10:24:05 +0200 (CEST)", "from fmsmga001.fm.intel.com ([10.253.24.23])\n by orsmga103.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;\n 06 Jul 2020 01:24:05 -0700", "from intel.sh.intel.com ([10.239.255.48])\n by fmsmga001.fm.intel.com with ESMTP; 06 Jul 2020 01:24:01 -0700" ], "IronPort-SDR": [ "\n Htag3LklFDe5cM5GAXbShPs4Et80Q6P/8P6dz7BPPdz4IoTt33XVFCidFpK9I/GV5ZRN7u4Mfz\n H44E6wD3b++Q==", "\n cFQtadsg3Y4TF+XykUuMrh4QtNO3c4Ao1W4aFSCSX0MS3qlrwjpyYDwE8bnN1EKHNnL3+omJWA\n ctAujdOCj55w==" ], "X-IronPort-AV": [ "E=McAfee;i=\"6000,8403,9673\"; a=\"147379282\"", "E=Sophos;i=\"5.75,318,1589266800\"; d=\"scan'208\";a=\"147379282\"", "E=Sophos;i=\"5.75,318,1589266800\"; d=\"scan'208\";a=\"388128866\"" ], "X-Amp-Result": "SKIPPED(no attachment in message)", "X-Amp-File-Uploaded": "False", "X-ExtLoop1": "1", "From": "Guinan Sun <guinanx.sun@intel.com>", "To": "dev@dpdk.org", "Cc": "Jeff Guo <jia.guo@intel.com>, Zhao1 Wei <wei.zhao1@intel.com>,\n Guinan Sun <guinanx.sun@intel.com>,\n Jeff Kirsher <jeffrey.t.kirsher@intel.com>,\n Neftin Sasha <sasha.neftin@intel.com>,\n Lifshits Vitaly <vitaly.lifshits@intel.com>,\n Robert Konklewski <robertx.konklewski@intel.com>,\n Doug Dziggel <douglas.a.dziggel@intel.com>,\n Todd Fujinaka <todd.fujinaka@intel.com>,\n Jacob Keller <jacob.e.keller@intel.com>", "Date": "Mon, 6 Jul 2020 08:12:03 +0000", "Message-Id": "<20200706081222.19279-9-guinanx.sun@intel.com>", "X-Mailer": "git-send-email 2.17.1", "In-Reply-To": "<20200706081222.19279-1-guinanx.sun@intel.com>", "References": "<20200622064634.70941-1-guinanx.sun@intel.com>\n <20200706081222.19279-1-guinanx.sun@intel.com>", "Subject": "[dpdk-dev] [PATCH v3 08/27] net/e1000/base: improve code style and\n\tfix klocwork errors", "X-BeenThere": "dev@dpdk.org", "X-Mailman-Version": "2.1.15", "Precedence": "list", "List-Id": "DPDK patches and discussions <dev.dpdk.org>", "List-Unsubscribe": "<https://mails.dpdk.org/options/dev>,\n <mailto:dev-request@dpdk.org?subject=unsubscribe>", "List-Archive": "<http://mails.dpdk.org/archives/dev/>", "List-Post": "<mailto:dev@dpdk.org>", "List-Help": "<mailto:dev-request@dpdk.org?subject=help>", "List-Subscribe": "<https://mails.dpdk.org/listinfo/dev>,\n <mailto:dev-request@dpdk.org?subject=subscribe>", "Errors-To": "dev-bounces@dpdk.org", "Sender": "\"dev\" <dev-bounces@dpdk.org>" }, "content": "This patch fixes coding style and klocwork errors.\n\nFix typo in piece of code of NVM access for SPT.\nAnd cleans up the remaining instances in the shared code\nwhere it was not adhering to the Linux code standard.\nWrong description was found in the mentioned file, so fix them.\nRemove shadowing variable declarations.\n\nRelating to operands in bitwise operations having different sizes.\nUnreachable code since *clock_in_i2c_* always return success.\nDon't return unused s32 and don't check for constants.\n\nSigned-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>\nSigned-off-by: Neftin Sasha <sasha.neftin@intel.com>\nSigned-off-by: Lifshits Vitaly <vitaly.lifshits@intel.com>\nSigned-off-by: Robert Konklewski <robertx.konklewski@intel.com>\nSigned-off-by: Doug Dziggel <douglas.a.dziggel@intel.com>\nSigned-off-by: Todd Fujinaka <todd.fujinaka@intel.com>\nSigned-off-by: Jacob Keller <jacob.e.keller@intel.com>\nSigned-off-by: Guinan Sun <guinanx.sun@intel.com>\n---\n drivers/net/e1000/base/e1000_80003es2lan.c | 1 -\n drivers/net/e1000/base/e1000_82575.c | 18 ++++------\n drivers/net/e1000/base/e1000_i210.c | 10 +++---\n drivers/net/e1000/base/e1000_ich8lan.c | 40 ++++++++++++----------\n drivers/net/e1000/base/e1000_nvm.c | 2 +-\n drivers/net/e1000/base/e1000_phy.c | 5 ++-\n 6 files changed, 36 insertions(+), 40 deletions(-)", "diff": "diff --git a/drivers/net/e1000/base/e1000_80003es2lan.c b/drivers/net/e1000/base/e1000_80003es2lan.c\nindex bcfda9415..b795491c0 100644\n--- a/drivers/net/e1000/base/e1000_80003es2lan.c\n+++ b/drivers/net/e1000/base/e1000_80003es2lan.c\n@@ -1210,7 +1210,6 @@ STATIC s32 e1000_setup_copper_link_80003es2lan(struct e1000_hw *hw)\n /**\n * e1000_cfg_on_link_up_80003es2lan - es2 link configuration after link-up\n * @hw: pointer to the HW structure\n- * @duplex: current duplex setting\n *\n * Configure the KMRN interface by applying last minute quirks for\n * 10/100 operation.\ndiff --git a/drivers/net/e1000/base/e1000_82575.c b/drivers/net/e1000/base/e1000_82575.c\nindex 750f1cf01..410f2db41 100644\n--- a/drivers/net/e1000/base/e1000_82575.c\n+++ b/drivers/net/e1000/base/e1000_82575.c\n@@ -75,10 +75,10 @@ STATIC void e1000_clear_vfta_i350(struct e1000_hw *hw);\n \n STATIC void e1000_i2c_start(struct e1000_hw *hw);\n STATIC void e1000_i2c_stop(struct e1000_hw *hw);\n-STATIC s32 e1000_clock_in_i2c_byte(struct e1000_hw *hw, u8 *data);\n+STATIC void e1000_clock_in_i2c_byte(struct e1000_hw *hw, u8 *data);\n STATIC s32 e1000_clock_out_i2c_byte(struct e1000_hw *hw, u8 data);\n STATIC s32 e1000_get_i2c_ack(struct e1000_hw *hw);\n-STATIC s32 e1000_clock_in_i2c_bit(struct e1000_hw *hw, bool *data);\n+STATIC void e1000_clock_in_i2c_bit(struct e1000_hw *hw, bool *data);\n STATIC s32 e1000_clock_out_i2c_bit(struct e1000_hw *hw, bool data);\n STATIC void e1000_raise_i2c_clk(struct e1000_hw *hw, u32 *i2cctl);\n STATIC void e1000_lower_i2c_clk(struct e1000_hw *hw, u32 *i2cctl);\n@@ -1084,7 +1084,7 @@ STATIC void e1000_release_swfw_sync_82575(struct e1000_hw *hw, u16 mask)\n \t\t; /* Empty */\n \n \tswfw_sync = E1000_READ_REG(hw, E1000_SW_FW_SYNC);\n-\tswfw_sync &= ~mask;\n+\tswfw_sync &= (u32)~mask;\n \tE1000_WRITE_REG(hw, E1000_SW_FW_SYNC, swfw_sync);\n \n \te1000_put_hw_semaphore_generic(hw);\n@@ -3300,9 +3300,7 @@ s32 e1000_read_i2c_byte_generic(struct e1000_hw *hw, u8 byte_offset,\n \t\tif (status != E1000_SUCCESS)\n \t\t\tgoto fail;\n \n-\t\tstatus = e1000_clock_in_i2c_byte(hw, data);\n-\t\tif (status != E1000_SUCCESS)\n-\t\t\tgoto fail;\n+\t\te1000_clock_in_i2c_byte(hw, data);\n \n \t\tstatus = e1000_clock_out_i2c_bit(hw, nack);\n \t\tif (status != E1000_SUCCESS)\n@@ -3466,7 +3464,7 @@ STATIC void e1000_i2c_stop(struct e1000_hw *hw)\n *\n * Clocks in one byte data via I2C data/clock\n **/\n-STATIC s32 e1000_clock_in_i2c_byte(struct e1000_hw *hw, u8 *data)\n+STATIC void e1000_clock_in_i2c_byte(struct e1000_hw *hw, u8 *data)\n {\n \ts32 i;\n \tbool bit = 0;\n@@ -3478,8 +3476,6 @@ STATIC s32 e1000_clock_in_i2c_byte(struct e1000_hw *hw, u8 *data)\n \t\te1000_clock_in_i2c_bit(hw, &bit);\n \t\t*data |= bit << i;\n \t}\n-\n-\treturn E1000_SUCCESS;\n }\n \n /**\n@@ -3568,7 +3564,7 @@ STATIC s32 e1000_get_i2c_ack(struct e1000_hw *hw)\n *\n * Clocks in one bit via I2C data/clock\n **/\n-STATIC s32 e1000_clock_in_i2c_bit(struct e1000_hw *hw, bool *data)\n+STATIC void e1000_clock_in_i2c_bit(struct e1000_hw *hw, bool *data)\n {\n \tu32 i2cctl = E1000_READ_REG(hw, E1000_I2CPARAMS);\n \n@@ -3586,8 +3582,6 @@ STATIC s32 e1000_clock_in_i2c_bit(struct e1000_hw *hw, bool *data)\n \n \t/* Minimum low period of clock is 4.7 us */\n \tusec_delay(E1000_I2C_T_LOW);\n-\n-\treturn E1000_SUCCESS;\n }\n \n /**\ndiff --git a/drivers/net/e1000/base/e1000_i210.c b/drivers/net/e1000/base/e1000_i210.c\nindex 5c2fe8a3e..a6d8e3b34 100644\n--- a/drivers/net/e1000/base/e1000_i210.c\n+++ b/drivers/net/e1000/base/e1000_i210.c\n@@ -117,7 +117,7 @@ void e1000_release_swfw_sync_i210(struct e1000_hw *hw, u16 mask)\n \t\t; /* Empty */\n \n \tswfw_sync = E1000_READ_REG(hw, E1000_SW_FW_SYNC);\n-\tswfw_sync &= ~mask;\n+\tswfw_sync &= (u32)~mask;\n \tE1000_WRITE_REG(hw, E1000_SW_FW_SYNC, swfw_sync);\n \n \te1000_put_hw_semaphore_generic(hw);\n@@ -310,7 +310,7 @@ STATIC s32 e1000_write_nvm_srwr(struct e1000_hw *hw, u16 offset, u16 words,\n \t}\n \n \tfor (i = 0; i < words; i++) {\n-\t\teewr = ((offset+i) << E1000_NVM_RW_ADDR_SHIFT) |\n+\t\teewr = ((offset + i) << E1000_NVM_RW_ADDR_SHIFT) |\n \t\t\t(data[i] << E1000_NVM_RW_REG_DATA) |\n \t\t\tE1000_NVM_RW_REG_START;\n \n@@ -397,9 +397,9 @@ STATIC s32 e1000_read_invm_i210(struct e1000_hw *hw, u16 offset,\n \tswitch (offset) {\n \tcase NVM_MAC_ADDR:\n \t\tret_val = e1000_read_invm_word_i210(hw, (u8)offset, &data[0]);\n-\t\tret_val |= e1000_read_invm_word_i210(hw, (u8)offset+1,\n+\t\tret_val |= e1000_read_invm_word_i210(hw, (u8)offset + 1,\n \t\t\t\t\t\t &data[1]);\n-\t\tret_val |= e1000_read_invm_word_i210(hw, (u8)offset+2,\n+\t\tret_val |= e1000_read_invm_word_i210(hw, (u8)offset + 2,\n \t\t\t\t\t\t &data[2]);\n \t\tif (ret_val != E1000_SUCCESS)\n \t\t\tDEBUGOUT(\"MAC Addr not found in iNVM\\n\");\n@@ -776,8 +776,6 @@ void e1000_init_function_pointers_i210(struct e1000_hw *hw)\n {\n \te1000_init_function_pointers_82575(hw);\n \thw->nvm.ops.init_params = e1000_init_nvm_params_i210;\n-\n-\treturn;\n }\n \n /**\ndiff --git a/drivers/net/e1000/base/e1000_ich8lan.c b/drivers/net/e1000/base/e1000_ich8lan.c\nindex 5241cf698..823dda5f5 100644\n--- a/drivers/net/e1000/base/e1000_ich8lan.c\n+++ b/drivers/net/e1000/base/e1000_ich8lan.c\n@@ -822,7 +822,7 @@ STATIC s32 e1000_init_mac_params_ich8lan(struct e1000_hw *hw)\n /**\n * __e1000_access_emi_reg_locked - Read/write EMI register\n * @hw: pointer to the HW structure\n- * @addr: EMI address to program\n+ * @address: EMI address to program\n * @data: pointer to value to read/write from/to the EMI address\n * @read: boolean flag to indicate read or write\n *\n@@ -1546,8 +1546,6 @@ STATIC s32 e1000_check_for_copper_link_ich8lan(struct e1000_hw *hw)\n \n \n \t\tif (hw->mac.type >= e1000_pch_lpt) {\n-\t\t\tu16 phy_reg;\n-\n \t\t\thw->phy.ops.read_reg_locked(hw, I217_PLL_CLOCK_GATE_REG,\n \t\t\t\t\t\t &phy_reg);\n \t\t\tphy_reg &= ~I217_PLL_CLOCK_GATE_MASK;\n@@ -2401,7 +2399,7 @@ STATIC s32 e1000_k1_gig_workaround_hv(struct e1000_hw *hw, bool link)\n /**\n * e1000_configure_k1_ich8lan - Configure K1 power state\n * @hw: pointer to the HW structure\n- * @enable: K1 state to configure\n+ * @k1_enable: K1 state to configure\n *\n * Configure the K1 power state based on the provided parameter.\n * Assumes semaphore already acquired.\n@@ -2549,6 +2547,7 @@ STATIC s32 e1000_set_mdio_slow_mode_hv(struct e1000_hw *hw)\n /**\n * e1000_hv_phy_workarounds_ich8lan - A series of Phy workarounds to be\n * done after every PHY reset.\n+ * @hw: pointer to the HW structure\n **/\n STATIC s32 e1000_hv_phy_workarounds_ich8lan(struct e1000_hw *hw)\n {\n@@ -2875,6 +2874,7 @@ s32 e1000_lv_jumbo_workaround_ich8lan(struct e1000_hw *hw, bool enable)\n /**\n * e1000_lv_phy_workarounds_ich8lan - A series of Phy workarounds to be\n * done after every PHY reset.\n+ * @hw: pointer to the HW structure\n **/\n STATIC s32 e1000_lv_phy_workarounds_ich8lan(struct e1000_hw *hw)\n {\n@@ -3479,8 +3479,9 @@ STATIC s32 e1000_read_nvm_spt(struct e1000_hw *hw, u16 offset, u16 words,\n \n \tfor (i = 0; i < words; i += 2) {\n \t\tif (words - i == 1) {\n-\t\t\tif (dev_spec->shadow_ram[offset+i].modified) {\n-\t\t\t\tdata[i] = dev_spec->shadow_ram[offset+i].value;\n+\t\t\tif (dev_spec->shadow_ram[offset + i].modified) {\n+\t\t\t\tdata[i] =\n+\t\t\t\t dev_spec->shadow_ram[offset + i].value;\n \t\t\t} else {\n \t\t\t\toffset_to_read = act_offset + i -\n \t\t\t\t\t\t ((act_offset + i) % 2);\n@@ -3497,8 +3498,8 @@ STATIC s32 e1000_read_nvm_spt(struct e1000_hw *hw, u16 offset, u16 words,\n \t\t\t}\n \t\t} else {\n \t\t\toffset_to_read = act_offset + i;\n-\t\t\tif (!(dev_spec->shadow_ram[offset+i].modified) ||\n-\t\t\t !(dev_spec->shadow_ram[offset+i+1].modified)) {\n+\t\t\tif (!(dev_spec->shadow_ram[offset + i].modified) ||\n+\t\t\t !(dev_spec->shadow_ram[offset + i + 1].modified)) {\n \t\t\t\tret_val =\n \t\t\t\t e1000_read_flash_dword_ich8lan(hw,\n \t\t\t\t\t\t\t\t offset_to_read,\n@@ -3506,15 +3507,16 @@ STATIC s32 e1000_read_nvm_spt(struct e1000_hw *hw, u16 offset, u16 words,\n \t\t\t\tif (ret_val)\n \t\t\t\t\tbreak;\n \t\t\t}\n-\t\t\tif (dev_spec->shadow_ram[offset+i].modified)\n-\t\t\t\tdata[i] = dev_spec->shadow_ram[offset+i].value;\n+\t\t\tif (dev_spec->shadow_ram[offset + i].modified)\n+\t\t\t\tdata[i] =\n+\t\t\t\t dev_spec->shadow_ram[offset + i].value;\n \t\t\telse\n-\t\t\t\tdata[i] = (u16) (dword & 0xFFFF);\n-\t\t\tif (dev_spec->shadow_ram[offset+i].modified)\n-\t\t\t\tdata[i+1] =\n-\t\t\t\t dev_spec->shadow_ram[offset+i+1].value;\n+\t\t\t\tdata[i] = (u16)(dword & 0xFFFF);\n+\t\t\tif (dev_spec->shadow_ram[offset + i + 1].modified)\n+\t\t\t\tdata[i + 1] =\n+\t\t\t\t dev_spec->shadow_ram[offset + i + 1].value;\n \t\t\telse\n-\t\t\t\tdata[i+1] = (u16) (dword >> 16 & 0xFFFF);\n+\t\t\t\tdata[i + 1] = (u16)(dword >> 16 & 0xFFFF);\n \t\t}\n \t}\n \n@@ -3568,8 +3570,8 @@ STATIC s32 e1000_read_nvm_ich8lan(struct e1000_hw *hw, u16 offset, u16 words,\n \n \tret_val = E1000_SUCCESS;\n \tfor (i = 0; i < words; i++) {\n-\t\tif (dev_spec->shadow_ram[offset+i].modified) {\n-\t\t\tdata[i] = dev_spec->shadow_ram[offset+i].value;\n+\t\tif (dev_spec->shadow_ram[offset + i].modified) {\n+\t\t\tdata[i] = dev_spec->shadow_ram[offset + i].value;\n \t\t} else {\n \t\t\tret_val = e1000_read_flash_word_ich8lan(hw,\n \t\t\t\t\t\t\t\tact_offset + i,\n@@ -3974,8 +3976,8 @@ STATIC s32 e1000_write_nvm_ich8lan(struct e1000_hw *hw, u16 offset, u16 words,\n \tnvm->ops.acquire(hw);\n \n \tfor (i = 0; i < words; i++) {\n-\t\tdev_spec->shadow_ram[offset+i].modified = true;\n-\t\tdev_spec->shadow_ram[offset+i].value = data[i];\n+\t\tdev_spec->shadow_ram[offset + i].modified = true;\n+\t\tdev_spec->shadow_ram[offset + i].value = data[i];\n \t}\n \n \tnvm->ops.release(hw);\ndiff --git a/drivers/net/e1000/base/e1000_nvm.c b/drivers/net/e1000/base/e1000_nvm.c\nindex 19a7f77ad..ca87ec122 100644\n--- a/drivers/net/e1000/base/e1000_nvm.c\n+++ b/drivers/net/e1000/base/e1000_nvm.c\n@@ -551,7 +551,7 @@ s32 e1000_read_nvm_eerd(struct e1000_hw *hw, u16 offset, u16 words, u16 *data)\n \t}\n \n \tfor (i = 0; i < words; i++) {\n-\t\teerd = ((offset+i) << E1000_NVM_RW_ADDR_SHIFT) +\n+\t\teerd = ((offset + i) << E1000_NVM_RW_ADDR_SHIFT) +\n \t\t E1000_NVM_RW_REG_START;\n \n \t\tE1000_WRITE_REG(hw, E1000_EERD, eerd);\ndiff --git a/drivers/net/e1000/base/e1000_phy.c b/drivers/net/e1000/base/e1000_phy.c\nindex c506d2344..4995578f4 100644\n--- a/drivers/net/e1000/base/e1000_phy.c\n+++ b/drivers/net/e1000/base/e1000_phy.c\n@@ -597,7 +597,7 @@ s32 e1000_write_sfp_data_byte(struct e1000_hw *hw, u16 offset, u8 data)\n \t\t\t\t * lane and update whole word\n \t\t\t\t */\n \t\t\t\tdata_local = i2ccmd & 0xFF00;\n-\t\t\t\tdata_local |= data;\n+\t\t\t\tdata_local |= (u32)data;\n \t\t\t\ti2ccmd = ((offset <<\n \t\t\t\t\tE1000_I2CCMD_REG_ADDR_SHIFT) |\n \t\t\t\t\tE1000_I2CCMD_OPCODE_WRITE | data_local);\n@@ -3071,6 +3071,7 @@ s32 e1000_determine_phy_address(struct e1000_hw *hw)\n /**\n * e1000_get_phy_addr_for_bm_page - Retrieve PHY page address\n * @page: page to access\n+ * @reg: register to access\n *\n * Returns the phy address for the page requested.\n **/\n@@ -3508,6 +3509,7 @@ void e1000_power_down_phy_copper(struct e1000_hw *hw)\n * @offset: register offset to be read\n * @data: pointer to the read data\n * @locked: semaphore has already been acquired or not\n+ * @page_set: BM_WUC_PAGE already set and access enabled\n *\n * Acquires semaphore, if necessary, then reads the PHY register at offset\n * and stores the retrieved information in data. Release any acquired\n@@ -3618,6 +3620,7 @@ s32 e1000_read_phy_reg_page_hv(struct e1000_hw *hw, u32 offset, u16 *data)\n * @offset: register offset to write to\n * @data: data to write at register offset\n * @locked: semaphore has already been acquired or not\n+ * @page_set: BM_WUC_PAGE already set and access enabled\n *\n * Acquires semaphore, if necessary, then writes the data to PHY register\n * at the offset. Release any acquired semaphores before exiting.\n", "prefixes": [ "v3", "08/27" ] }