On 6/12/2020 4:23 AM, Guinan Sun wrote:
> For the "Apply Update" command the firmware does not
> given an response. For this command, success should
> be return. Based on information at EAS.
>
> Signed-off-by: Mateusz Kowalski <[email protected]>
> Signed-off-by: Guinan Sun <[email protected]>
<...>
> diff --git a/drivers/net/ixgbe/base/ixgbe_type.h
> b/drivers/net/ixgbe/base/ixgbe_type.h
> index 0470b1dfc..33ca659cd 100644
> --- a/drivers/net/ixgbe/base/ixgbe_type.h
> +++ b/drivers/net/ixgbe/base/ixgbe_type.h
> @@ -4366,4 +4366,16 @@ struct ixgbe_hw {
> #define IXGBE_NW_MNG_IF_SEL_MDIO_PHY_ADD \
> (0x1F << IXGBE_NW_MNG_IF_SEL_MDIO_PHY_ADD_SHIFT)
>
> +/* Code Command (Flash I/F Interface) */
> +#define IXGBE_HOST_INTERFACE_FLASH_READ_CMD 0x30
> +#define IXGBE_HOST_INTERFACE_SHADOW_RAM_READ_CMD 0x31
> +#define IXGBE_HOST_INTERFACE_FLASH_WRITE_CMD 0x32
> +#define IXGBE_HOST_INTERFACE_SHADOW_RAM_WRITE_CMD 0x33
> +#define IXGBE_HOST_INTERFACE_FLASH_MODULE_UPDATE_CMD 0x34
> +#define IXGBE_HOST_INTERFACE_FLASH_BLOCK_EREASE_CMD 0x35
> +#define IXGBE_HOST_INTERFACE_SHADOW_RAM_DUMP_CMD 0x36
> +#define IXGBE_HOST_INTERFACE_FLASH_INFO_CMD 0x37
> +#define IXGBE_HOST_INTERFACE_APPLY_UPDATE_CMD 0x38
> +#define IXGBE_HOST_INTERFACE_MASK_CMD
> 0x000000FF
> +
> #endif /* _IXGBE_TYPE_H_ */
>
Previous patch uses these commands in a hardcoded way, since we are adding
defines, why not them in the previous patch so we can use the macros in previous
patch?