Commit 34e653ef authored by Lee Jones's avatar Lee Jones Committed by Jakub Kicinski

net: usb: r8152: Provide missing documentation for some struct members

Fixes the following W=1 kernel build warning(s):

 drivers/net/usb/r8152.c:934: warning: Function parameter or member 'blk_hdr' not described in 'fw_mac'
 drivers/net/usb/r8152.c:934: warning: Function parameter or member 'reserved' not described in 'fw_mac'
 drivers/net/usb/r8152.c:947: warning: Function parameter or member 'blk_hdr' not described in 'fw_phy_patch_key'
 drivers/net/usb/r8152.c:947: warning: Function parameter or member 'reserved' not described in 'fw_phy_patch_key'
 drivers/net/usb/r8152.c:986: warning: Function parameter or member 'blk_hdr' not described in 'fw_phy_nc'
 drivers/net/usb/r8152.c:986: warning: Function parameter or member 'mode_pre' not described in 'fw_phy_nc'
 drivers/net/usb/r8152.c:986: warning: Function parameter or member 'mode_post' not described in 'fw_phy_nc'
 drivers/net/usb/r8152.c:986: warning: Function parameter or member 'reserved' not described in 'fw_phy_nc'
Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
Acked-by: default avatarHayes Wang <hayeswang@realtek.com>
Link: https://lore.kernel.org/r/20201102114512.1062724-23-lee.jones@linaro.orgSigned-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent c8f638b7
...@@ -898,6 +898,7 @@ struct fw_header { ...@@ -898,6 +898,7 @@ struct fw_header {
* struct fw_mac - a firmware block used by RTL_FW_PLA and RTL_FW_USB. * struct fw_mac - a firmware block used by RTL_FW_PLA and RTL_FW_USB.
* The layout of the firmware block is: * The layout of the firmware block is:
* <struct fw_mac> + <info> + <firmware data>. * <struct fw_mac> + <info> + <firmware data>.
* @blk_hdr: firmware descriptor (type, length)
* @fw_offset: offset of the firmware binary data. The start address of * @fw_offset: offset of the firmware binary data. The start address of
* the data would be the address of struct fw_mac + @fw_offset. * the data would be the address of struct fw_mac + @fw_offset.
* @fw_reg: the register to load the firmware. Depends on chip. * @fw_reg: the register to load the firmware. Depends on chip.
...@@ -911,6 +912,7 @@ struct fw_header { ...@@ -911,6 +912,7 @@ struct fw_header {
* @bp_num: the break point number which needs to be set for this firmware. * @bp_num: the break point number which needs to be set for this firmware.
* Depends on the firmware. * Depends on the firmware.
* @bp: break points. Depends on firmware. * @bp: break points. Depends on firmware.
* @reserved: reserved space (unused)
* @fw_ver_reg: the register to store the fw version. * @fw_ver_reg: the register to store the fw version.
* @fw_ver_data: the firmware version of the current type. * @fw_ver_data: the firmware version of the current type.
* @info: additional information for debugging, and is followed by the * @info: additional information for debugging, and is followed by the
...@@ -936,8 +938,10 @@ struct fw_mac { ...@@ -936,8 +938,10 @@ struct fw_mac {
/** /**
* struct fw_phy_patch_key - a firmware block used by RTL_FW_PHY_START. * struct fw_phy_patch_key - a firmware block used by RTL_FW_PHY_START.
* This is used to set patch key when loading the firmware of PHY. * This is used to set patch key when loading the firmware of PHY.
* @blk_hdr: firmware descriptor (type, length)
* @key_reg: the register to write the patch key. * @key_reg: the register to write the patch key.
* @key_data: patch key. * @key_data: patch key.
* @reserved: reserved space (unused)
*/ */
struct fw_phy_patch_key { struct fw_phy_patch_key {
struct fw_block blk_hdr; struct fw_block blk_hdr;
...@@ -950,6 +954,7 @@ struct fw_phy_patch_key { ...@@ -950,6 +954,7 @@ struct fw_phy_patch_key {
* struct fw_phy_nc - a firmware block used by RTL_FW_PHY_NC. * struct fw_phy_nc - a firmware block used by RTL_FW_PHY_NC.
* The layout of the firmware block is: * The layout of the firmware block is:
* <struct fw_phy_nc> + <info> + <firmware data>. * <struct fw_phy_nc> + <info> + <firmware data>.
* @blk_hdr: firmware descriptor (type, length)
* @fw_offset: offset of the firmware binary data. The start address of * @fw_offset: offset of the firmware binary data. The start address of
* the data would be the address of struct fw_phy_nc + @fw_offset. * the data would be the address of struct fw_phy_nc + @fw_offset.
* @fw_reg: the register to load the firmware. Depends on chip. * @fw_reg: the register to load the firmware. Depends on chip.
...@@ -960,6 +965,7 @@ struct fw_phy_patch_key { ...@@ -960,6 +965,7 @@ struct fw_phy_patch_key {
* @mode_reg: the regitster of switching the mode. * @mode_reg: the regitster of switching the mode.
* @mod_pre: the mode needing to be set before loading the firmware. * @mod_pre: the mode needing to be set before loading the firmware.
* @mod_post: the mode to be set when finishing to load the firmware. * @mod_post: the mode to be set when finishing to load the firmware.
* @reserved: reserved space (unused)
* @bp_start: the start register of break points. Depends on chip. * @bp_start: the start register of break points. Depends on chip.
* @bp_num: the break point number which needs to be set for this firmware. * @bp_num: the break point number which needs to be set for this firmware.
* Depends on the firmware. * Depends on the firmware.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment