* [PATCH net-next 0/2] qed: Support for NVM config attributes. @ 2019-07-24 4:51 Sudarsana Reddy Kalluru 2019-07-24 4:51 ` [PATCH net-next 1/2] qed: Add APIs " Sudarsana Reddy Kalluru 2019-07-24 4:51 ` [PATCH net-next 2/2] qed: Add API for flashing the nvm attributes Sudarsana Reddy Kalluru 0 siblings, 2 replies; 6+ messages in thread From: Sudarsana Reddy Kalluru @ 2019-07-24 4:51 UTC (permalink / raw) To: davem; +Cc: netdev, mkalderon, aelior The patch series add support for managing the NVM config attributes. Patch (1) adds interfaces for read/write config attributes from MFW. Patch (2) adds driver interface for updating the config attributes. Sudarsana Reddy Kalluru (2): qed: Add APIs for NVM config attributes. qed: Add API for flashing the nvm attributes. drivers/net/ethernet/qlogic/qed/qed_hsi.h | 17 ++++++++ drivers/net/ethernet/qlogic/qed/qed_main.c | 65 ++++++++++++++++++++++++++++++ drivers/net/ethernet/qlogic/qed/qed_mcp.c | 64 +++++++++++++++++++++++++++++ drivers/net/ethernet/qlogic/qed/qed_mcp.h | 14 +++++++ include/linux/qed/qed_if.h | 1 + 5 files changed, 161 insertions(+) -- 1.8.3.1 ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH net-next 1/2] qed: Add APIs for NVM config attributes. 2019-07-24 4:51 [PATCH net-next 0/2] qed: Support for NVM config attributes Sudarsana Reddy Kalluru @ 2019-07-24 4:51 ` Sudarsana Reddy Kalluru 2019-07-24 4:51 ` [PATCH net-next 2/2] qed: Add API for flashing the nvm attributes Sudarsana Reddy Kalluru 1 sibling, 0 replies; 6+ messages in thread From: Sudarsana Reddy Kalluru @ 2019-07-24 4:51 UTC (permalink / raw) To: davem; +Cc: netdev, mkalderon, aelior The patch adds APIs for reading/configuring the NVM config attributes using Management FW interfaces. Signed-off-by: Sudarsana Reddy Kalluru <skalluru@marvell.com> Signed-off-by: Ariel Elior <aelior@marvell.com> --- drivers/net/ethernet/qlogic/qed/qed_hsi.h | 17 ++++++++ drivers/net/ethernet/qlogic/qed/qed_mcp.c | 64 +++++++++++++++++++++++++++++++ drivers/net/ethernet/qlogic/qed/qed_mcp.h | 14 +++++++ 3 files changed, 95 insertions(+) diff --git a/drivers/net/ethernet/qlogic/qed/qed_hsi.h b/drivers/net/ethernet/qlogic/qed/qed_hsi.h index e054f6c..557a12e 100644 --- a/drivers/net/ethernet/qlogic/qed/qed_hsi.h +++ b/drivers/net/ethernet/qlogic/qed/qed_hsi.h @@ -12580,6 +12580,8 @@ struct public_drv_mb { #define DRV_MSG_CODE_BW_UPDATE_ACK 0x32000000 #define DRV_MSG_CODE_NIG_DRAIN 0x30000000 #define DRV_MSG_CODE_S_TAG_UPDATE_ACK 0x3b000000 +#define DRV_MSG_CODE_GET_NVM_CFG_OPTION 0x003e0000 +#define DRV_MSG_CODE_SET_NVM_CFG_OPTION 0x003f0000 #define DRV_MSG_CODE_INITIATE_PF_FLR 0x02010000 #define DRV_MSG_CODE_VF_DISABLED_DONE 0xc0000000 #define DRV_MSG_CODE_CFG_VF_MSIX 0xc0010000 @@ -12748,6 +12750,21 @@ struct public_drv_mb { #define DRV_MB_PARAM_FEATURE_SUPPORT_PORT_EEE 0x00000002 #define DRV_MB_PARAM_FEATURE_SUPPORT_FUNC_VLINK 0x00010000 +#define DRV_MB_PARAM_NVM_CFG_OPTION_ID_SHIFT 0 +#define DRV_MB_PARAM_NVM_CFG_OPTION_ID_MASK 0x0000FFFF +#define DRV_MB_PARAM_NVM_CFG_OPTION_ALL_SHIFT 16 +#define DRV_MB_PARAM_NVM_CFG_OPTION_ALL_MASK 0x00010000 +#define DRV_MB_PARAM_NVM_CFG_OPTION_INIT_SHIFT 17 +#define DRV_MB_PARAM_NVM_CFG_OPTION_INIT_MASK 0x00020000 +#define DRV_MB_PARAM_NVM_CFG_OPTION_COMMIT_SHIFT 18 +#define DRV_MB_PARAM_NVM_CFG_OPTION_COMMIT_MASK 0x00040000 +#define DRV_MB_PARAM_NVM_CFG_OPTION_FREE_SHIFT 19 +#define DRV_MB_PARAM_NVM_CFG_OPTION_FREE_MASK 0x00080000 +#define DRV_MB_PARAM_NVM_CFG_OPTION_ENTITY_SEL_SHIFT 20 +#define DRV_MB_PARAM_NVM_CFG_OPTION_ENTITY_SEL_MASK 0x00100000 +#define DRV_MB_PARAM_NVM_CFG_OPTION_ENTITY_ID_SHIFT 24 +#define DRV_MB_PARAM_NVM_CFG_OPTION_ENTITY_ID_MASK 0x0f000000 + u32 fw_mb_header; #define FW_MSG_CODE_MASK 0xffff0000 #define FW_MSG_CODE_UNSUPPORTED 0x00000000 diff --git a/drivers/net/ethernet/qlogic/qed/qed_mcp.c b/drivers/net/ethernet/qlogic/qed/qed_mcp.c index 758702c..573911a 100644 --- a/drivers/net/ethernet/qlogic/qed/qed_mcp.c +++ b/drivers/net/ethernet/qlogic/qed/qed_mcp.c @@ -3750,3 +3750,67 @@ int qed_mcp_get_ppfid_bitmap(struct qed_hwfn *p_hwfn, struct qed_ptt *p_ptt) return 0; } + +int qed_mcp_nvm_get_cfg(struct qed_hwfn *p_hwfn, struct qed_ptt *p_ptt, + u16 option_id, u8 entity_id, u16 flags, u8 *p_buf, + u32 *p_len) +{ + u32 mb_param = 0, resp, param; + int rc; + + QED_MFW_SET_FIELD(mb_param, DRV_MB_PARAM_NVM_CFG_OPTION_ID, option_id); + if (flags & QED_NVM_CFG_OPTION_INIT) + QED_MFW_SET_FIELD(mb_param, + DRV_MB_PARAM_NVM_CFG_OPTION_INIT, 1); + if (flags & QED_NVM_CFG_OPTION_FREE) + QED_MFW_SET_FIELD(mb_param, + DRV_MB_PARAM_NVM_CFG_OPTION_FREE, 1); + if (flags & QED_NVM_CFG_OPTION_ENTITY_SEL) { + QED_MFW_SET_FIELD(mb_param, + DRV_MB_PARAM_NVM_CFG_OPTION_ENTITY_SEL, 1); + QED_MFW_SET_FIELD(mb_param, + DRV_MB_PARAM_NVM_CFG_OPTION_ENTITY_ID, + entity_id); + } + + rc = qed_mcp_nvm_rd_cmd(p_hwfn, p_ptt, + DRV_MSG_CODE_GET_NVM_CFG_OPTION, + mb_param, &resp, ¶m, p_len, (u32 *)p_buf); + + return rc; +} + +int qed_mcp_nvm_set_cfg(struct qed_hwfn *p_hwfn, struct qed_ptt *p_ptt, + u16 option_id, u8 entity_id, u16 flags, u8 *p_buf, + u32 len) +{ + u32 mb_param = 0, resp, param; + int rc; + + QED_MFW_SET_FIELD(mb_param, DRV_MB_PARAM_NVM_CFG_OPTION_ID, option_id); + if (flags & QED_NVM_CFG_OPTION_ALL) + QED_MFW_SET_FIELD(mb_param, + DRV_MB_PARAM_NVM_CFG_OPTION_ALL, 1); + if (flags & QED_NVM_CFG_OPTION_INIT) + QED_MFW_SET_FIELD(mb_param, + DRV_MB_PARAM_NVM_CFG_OPTION_INIT, 1); + if (flags & QED_NVM_CFG_OPTION_COMMIT) + QED_MFW_SET_FIELD(mb_param, + DRV_MB_PARAM_NVM_CFG_OPTION_COMMIT, 1); + if (flags & QED_NVM_CFG_OPTION_FREE) + QED_MFW_SET_FIELD(mb_param, + DRV_MB_PARAM_NVM_CFG_OPTION_FREE, 1); + if (flags & QED_NVM_CFG_OPTION_ENTITY_SEL) { + QED_MFW_SET_FIELD(mb_param, + DRV_MB_PARAM_NVM_CFG_OPTION_ENTITY_SEL, 1); + QED_MFW_SET_FIELD(mb_param, + DRV_MB_PARAM_NVM_CFG_OPTION_ENTITY_ID, + entity_id); + } + + rc = qed_mcp_nvm_wr_cmd(p_hwfn, p_ptt, + DRV_MSG_CODE_SET_NVM_CFG_OPTION, + mb_param, &resp, ¶m, len, (u32 *)p_buf); + + return rc; +} diff --git a/drivers/net/ethernet/qlogic/qed/qed_mcp.h b/drivers/net/ethernet/qlogic/qed/qed_mcp.h index e4f8fe4..550b4dd 100644 --- a/drivers/net/ethernet/qlogic/qed/qed_mcp.h +++ b/drivers/net/ethernet/qlogic/qed/qed_mcp.h @@ -251,6 +251,12 @@ struct qed_mfw_tlv_generic { struct qed_mfw_tlv_iscsi iscsi; }; +#define QED_NVM_CFG_OPTION_ALL BIT(0) +#define QED_NVM_CFG_OPTION_INIT BIT(1) +#define QED_NVM_CFG_OPTION_COMMIT BIT(2) +#define QED_NVM_CFG_OPTION_FREE BIT(3) +#define QED_NVM_CFG_OPTION_ENTITY_SEL BIT(4) + /** * @brief - returns the link params of the hw function * @@ -1202,4 +1208,12 @@ void qed_mcp_resc_lock_default_init(struct qed_resc_lock_params *p_lock, */ int qed_mcp_get_ppfid_bitmap(struct qed_hwfn *p_hwfn, struct qed_ptt *p_ptt); +int qed_mcp_nvm_get_cfg(struct qed_hwfn *p_hwfn, struct qed_ptt *p_ptt, + u16 option_id, u8 entity_id, u16 flags, u8 *p_buf, + u32 *p_len); + +int qed_mcp_nvm_set_cfg(struct qed_hwfn *p_hwfn, struct qed_ptt *p_ptt, + u16 option_id, u8 entity_id, u16 flags, u8 *p_buf, + u32 len); + #endif -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH net-next 2/2] qed: Add API for flashing the nvm attributes. 2019-07-24 4:51 [PATCH net-next 0/2] qed: Support for NVM config attributes Sudarsana Reddy Kalluru 2019-07-24 4:51 ` [PATCH net-next 1/2] qed: Add APIs " Sudarsana Reddy Kalluru @ 2019-07-24 4:51 ` Sudarsana Reddy Kalluru 2019-07-24 19:42 ` Saeed Mahameed 1 sibling, 1 reply; 6+ messages in thread From: Sudarsana Reddy Kalluru @ 2019-07-24 4:51 UTC (permalink / raw) To: davem; +Cc: netdev, mkalderon, aelior The patch adds driver interface for reading the NVM config request and update the attributes on nvm config flash partition. Signed-off-by: Sudarsana Reddy Kalluru <skalluru@marvell.com> Signed-off-by: Ariel Elior <aelior@marvell.com> --- drivers/net/ethernet/qlogic/qed/qed_main.c | 65 ++++++++++++++++++++++++++++++ include/linux/qed/qed_if.h | 1 + 2 files changed, 66 insertions(+) diff --git a/drivers/net/ethernet/qlogic/qed/qed_main.c b/drivers/net/ethernet/qlogic/qed/qed_main.c index 829dd60..54f00d2 100644 --- a/drivers/net/ethernet/qlogic/qed/qed_main.c +++ b/drivers/net/ethernet/qlogic/qed/qed_main.c @@ -67,6 +67,8 @@ #define QED_ROCE_QPS (8192) #define QED_ROCE_DPIS (8) #define QED_RDMA_SRQS QED_ROCE_QPS +#define QED_NVM_CFG_SET_FLAGS 0xE +#define QED_NVM_CFG_SET_PF_FLAGS 0x1E static char version[] = "QLogic FastLinQ 4xxxx Core Module qed " DRV_MODULE_VERSION "\n"; @@ -2227,6 +2229,66 @@ static int qed_nvm_flash_image_validate(struct qed_dev *cdev, return 0; } +/* Binary file format - + * /----------------------------------------------------------------------\ + * 0B | 0x5 [command index] | + * 4B | Entity ID | Reserved | Number of config attributes | + * 8B | Config ID | Length | Value | + * | | + * \----------------------------------------------------------------------/ + * There can be several Cfg_id-Length-Value sets as specified by 'Number of...'. + * Entity ID - A non zero entity value for which the config need to be updated. + */ +static int qed_nvm_flash_cfg_write(struct qed_dev *cdev, const u8 **data) +{ + struct qed_hwfn *hwfn = QED_LEADING_HWFN(cdev); + u8 entity_id, len, buf[32]; + struct qed_ptt *ptt; + u16 cfg_id, count; + int rc = 0, i; + u32 flags; + + ptt = qed_ptt_acquire(hwfn); + if (!ptt) + return -EAGAIN; + + /* NVM CFG ID attribute header */ + *data += 4; + entity_id = **data; + *data += 2; + count = *((u16 *)*data); + *data += 2; + + DP_VERBOSE(cdev, NETIF_MSG_DRV, + "Read config ids: entity id %02x num _attrs = %0d\n", + entity_id, count); + /* NVM CFG ID attributes */ + for (i = 0; i < count; i++) { + cfg_id = *((u16 *)*data); + *data += 2; + len = **data; + (*data)++; + memcpy(buf, *data, len); + *data += len; + + flags = entity_id ? QED_NVM_CFG_SET_PF_FLAGS : + QED_NVM_CFG_SET_FLAGS; + + DP_VERBOSE(cdev, NETIF_MSG_DRV, + "cfg_id = %d len = %d\n", cfg_id, len); + rc = qed_mcp_nvm_set_cfg(hwfn, ptt, cfg_id, entity_id, flags, + buf, len); + if (rc) { + DP_ERR(cdev, "Error %d configuring %d\n", rc, cfg_id); + break; + } + } + + qed_ptt_release(hwfn, ptt); + + return rc; +} + static int qed_nvm_flash(struct qed_dev *cdev, const char *name) { const struct firmware *image; @@ -2268,6 +2330,9 @@ static int qed_nvm_flash(struct qed_dev *cdev, const char *name) rc = qed_nvm_flash_image_access(cdev, &data, &check_resp); break; + case QED_NVM_FLASH_CMD_NVM_CFG_ID: + rc = qed_nvm_flash_cfg_write(cdev, &data); + break; default: DP_ERR(cdev, "Unknown command %08x\n", cmd_type); rc = -EINVAL; diff --git a/include/linux/qed/qed_if.h b/include/linux/qed/qed_if.h index eef02e6..23805ea 100644 --- a/include/linux/qed/qed_if.h +++ b/include/linux/qed/qed_if.h @@ -804,6 +804,7 @@ enum qed_nvm_flash_cmd { QED_NVM_FLASH_CMD_FILE_DATA = 0x2, QED_NVM_FLASH_CMD_FILE_START = 0x3, QED_NVM_FLASH_CMD_NVM_CHANGE = 0x4, + QED_NVM_FLASH_CMD_NVM_CFG_ID = 0x5, QED_NVM_FLASH_CMD_NVM_MAX, }; -- 1.8.3.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH net-next 2/2] qed: Add API for flashing the nvm attributes. 2019-07-24 4:51 ` [PATCH net-next 2/2] qed: Add API for flashing the nvm attributes Sudarsana Reddy Kalluru @ 2019-07-24 19:42 ` Saeed Mahameed 2019-07-25 0:48 ` Sudarsana Reddy Kalluru 0 siblings, 1 reply; 6+ messages in thread From: Saeed Mahameed @ 2019-07-24 19:42 UTC (permalink / raw) To: skalluru@marvell.com, davem@davemloft.net Cc: aelior@marvell.com, mkalderon@marvell.com, netdev@vger.kernel.org On Tue, 2019-07-23 at 21:51 -0700, Sudarsana Reddy Kalluru wrote: > The patch adds driver interface for reading the NVM config request > and > update the attributes on nvm config flash partition. > You didn't not use the get_cfg API you added in previous patch. Also can you please clarify how the user reads/write from/to NVM config ? i mean what UAPIs and tools are being used ? > Signed-off-by: Sudarsana Reddy Kalluru <skalluru@marvell.com> > Signed-off-by: Ariel Elior <aelior@marvell.com> > --- > drivers/net/ethernet/qlogic/qed/qed_main.c | 65 > ++++++++++++++++++++++++++++++ > include/linux/qed/qed_if.h | 1 + > 2 files changed, 66 insertions(+) > > diff --git a/drivers/net/ethernet/qlogic/qed/qed_main.c > b/drivers/net/ethernet/qlogic/qed/qed_main.c > index 829dd60..54f00d2 100644 > --- a/drivers/net/ethernet/qlogic/qed/qed_main.c > +++ b/drivers/net/ethernet/qlogic/qed/qed_main.c > @@ -67,6 +67,8 @@ > #define QED_ROCE_QPS (8192) > #define QED_ROCE_DPIS (8) > #define QED_RDMA_SRQS QED_ROCE_QPS > +#define QED_NVM_CFG_SET_FLAGS 0xE > +#define QED_NVM_CFG_SET_PF_FLAGS 0x1E > > static char version[] = > "QLogic FastLinQ 4xxxx Core Module qed " DRV_MODULE_VERSION > "\n"; > @@ -2227,6 +2229,66 @@ static int qed_nvm_flash_image_validate(struct > qed_dev *cdev, > return 0; > } > > +/* Binary file format - > + * /---------------------------------------------------------- > ------------\ > + * 0B | 0x5 [command > index] | > + * 4B | Entity ID | Reserved | Number of config > attributes | > + * 8B | Config ID | Length | > Value | > + > * | > | > + * \---------------------------------------------------------- > ------------/ > + * There can be several Cfg_id-Length-Value sets as specified by > 'Number of...'. > + * Entity ID - A non zero entity value for which the config need to > be updated. > + */ > +static int qed_nvm_flash_cfg_write(struct qed_dev *cdev, const u8 > **data) > +{ > + struct qed_hwfn *hwfn = QED_LEADING_HWFN(cdev); > + u8 entity_id, len, buf[32]; > + struct qed_ptt *ptt; > + u16 cfg_id, count; > + int rc = 0, i; > + u32 flags; > + > + ptt = qed_ptt_acquire(hwfn); > + if (!ptt) > + return -EAGAIN; > + > + /* NVM CFG ID attribute header */ > + *data += 4; > + entity_id = **data; > + *data += 2; > + count = *((u16 *)*data); > + *data += 2; > + > + DP_VERBOSE(cdev, NETIF_MSG_DRV, > + "Read config ids: entity id %02x num _attrs = > %0d\n", > + entity_id, count); > + /* NVM CFG ID attributes */ > + for (i = 0; i < count; i++) { > + cfg_id = *((u16 *)*data); > + *data += 2; > + len = **data; > + (*data)++; > + memcpy(buf, *data, len); > + *data += len; > + > + flags = entity_id ? QED_NVM_CFG_SET_PF_FLAGS : > + QED_NVM_CFG_SET_FLAGS; > + > + DP_VERBOSE(cdev, NETIF_MSG_DRV, > + "cfg_id = %d len = %d\n", cfg_id, len); > + rc = qed_mcp_nvm_set_cfg(hwfn, ptt, cfg_id, entity_id, > flags, > + buf, len); > + if (rc) { > + DP_ERR(cdev, "Error %d configuring %d\n", rc, > cfg_id); > + break; > + } > + } > + > + qed_ptt_release(hwfn, ptt); > + > + return rc; > +} > + > static int qed_nvm_flash(struct qed_dev *cdev, const char *name) > { > const struct firmware *image; > @@ -2268,6 +2330,9 @@ static int qed_nvm_flash(struct qed_dev *cdev, > const char *name) > rc = qed_nvm_flash_image_access(cdev, &data, > &check_resp); > break; > + case QED_NVM_FLASH_CMD_NVM_CFG_ID: > + rc = qed_nvm_flash_cfg_write(cdev, &data); > + break; > default: > DP_ERR(cdev, "Unknown command %08x\n", > cmd_type); > rc = -EINVAL; > diff --git a/include/linux/qed/qed_if.h b/include/linux/qed/qed_if.h > index eef02e6..23805ea 100644 > --- a/include/linux/qed/qed_if.h > +++ b/include/linux/qed/qed_if.h > @@ -804,6 +804,7 @@ enum qed_nvm_flash_cmd { > QED_NVM_FLASH_CMD_FILE_DATA = 0x2, > QED_NVM_FLASH_CMD_FILE_START = 0x3, > QED_NVM_FLASH_CMD_NVM_CHANGE = 0x4, > + QED_NVM_FLASH_CMD_NVM_CFG_ID = 0x5, > QED_NVM_FLASH_CMD_NVM_MAX, > }; > ^ permalink raw reply [flat|nested] 6+ messages in thread
* RE: [PATCH net-next 2/2] qed: Add API for flashing the nvm attributes. 2019-07-24 19:42 ` Saeed Mahameed @ 2019-07-25 0:48 ` Sudarsana Reddy Kalluru 2019-07-25 20:56 ` Saeed Mahameed 0 siblings, 1 reply; 6+ messages in thread From: Sudarsana Reddy Kalluru @ 2019-07-25 0:48 UTC (permalink / raw) To: Saeed Mahameed, davem@davemloft.net Cc: Ariel Elior, Michal Kalderon, netdev@vger.kernel.org > -----Original Message----- > From: Saeed Mahameed <saeedm@mellanox.com> > Sent: Thursday, July 25, 2019 1:13 AM > To: Sudarsana Reddy Kalluru <skalluru@marvell.com>; > davem@davemloft.net > Cc: Ariel Elior <aelior@marvell.com>; Michal Kalderon > <mkalderon@marvell.com>; netdev@vger.kernel.org > Subject: [EXT] Re: [PATCH net-next 2/2] qed: Add API for flashing the nvm > attributes. > > External Email > > ---------------------------------------------------------------------- > On Tue, 2019-07-23 at 21:51 -0700, Sudarsana Reddy Kalluru wrote: > > The patch adds driver interface for reading the NVM config request and > > update the attributes on nvm config flash partition. > > > > You didn't not use the get_cfg API you added in previous patch. Thanks for your review. Will move this API to the next patch series which will plan to send shortly. > > Also can you please clarify how the user reads/write from/to NVM config > ? i mean what UAPIs and tools are being used ? NVM config/partition will be updated using ethtool flash update command (i.e., ethtool -f) just like the update of other flash partitions of qed device. Example code path, ethool-flash_device --> qede_flash_device() --> qed_nvm_flash() --> qed_nvm_flash_cfg_write() > > > Signed-off-by: Sudarsana Reddy Kalluru <skalluru@marvell.com> > > Signed-off-by: Ariel Elior <aelior@marvell.com> > > --- > > drivers/net/ethernet/qlogic/qed/qed_main.c | 65 > > ++++++++++++++++++++++++++++++ > > include/linux/qed/qed_if.h | 1 + > > 2 files changed, 66 insertions(+) > > > > diff --git a/drivers/net/ethernet/qlogic/qed/qed_main.c > > b/drivers/net/ethernet/qlogic/qed/qed_main.c > > index 829dd60..54f00d2 100644 > > --- a/drivers/net/ethernet/qlogic/qed/qed_main.c > > +++ b/drivers/net/ethernet/qlogic/qed/qed_main.c > > @@ -67,6 +67,8 @@ > > #define QED_ROCE_QPS (8192) > > #define QED_ROCE_DPIS (8) > > #define QED_RDMA_SRQS QED_ROCE_QPS > > +#define QED_NVM_CFG_SET_FLAGS 0xE > > +#define QED_NVM_CFG_SET_PF_FLAGS 0x1E > > > > static char version[] = > > "QLogic FastLinQ 4xxxx Core Module qed " DRV_MODULE_VERSION > > "\n"; > > @@ -2227,6 +2229,66 @@ static int qed_nvm_flash_image_validate(struct > > qed_dev *cdev, > > return 0; > > } > > > > +/* Binary file format - > > + * /---------------------------------------------------------- > > ------------\ > > + * 0B | 0x5 [command > > index] | > > + * 4B | Entity ID | Reserved | Number of config > > attributes | > > + * 8B | Config ID | Length | > > Value | > > + > > * | > > | > > + * \---------------------------------------------------------- > > ------------/ > > + * There can be several Cfg_id-Length-Value sets as specified by > > 'Number of...'. > > + * Entity ID - A non zero entity value for which the config need to > > be updated. > > + */ > > +static int qed_nvm_flash_cfg_write(struct qed_dev *cdev, const u8 > > **data) > > +{ > > + struct qed_hwfn *hwfn = QED_LEADING_HWFN(cdev); > > + u8 entity_id, len, buf[32]; > > + struct qed_ptt *ptt; > > + u16 cfg_id, count; > > + int rc = 0, i; > > + u32 flags; > > + > > + ptt = qed_ptt_acquire(hwfn); > > + if (!ptt) > > + return -EAGAIN; > > + > > + /* NVM CFG ID attribute header */ > > + *data += 4; > > + entity_id = **data; > > + *data += 2; > > + count = *((u16 *)*data); > > + *data += 2; > > + > > + DP_VERBOSE(cdev, NETIF_MSG_DRV, > > + "Read config ids: entity id %02x num _attrs = > > %0d\n", > > + entity_id, count); > > + /* NVM CFG ID attributes */ > > + for (i = 0; i < count; i++) { > > + cfg_id = *((u16 *)*data); > > + *data += 2; > > + len = **data; > > + (*data)++; > > + memcpy(buf, *data, len); > > + *data += len; > > + > > + flags = entity_id ? QED_NVM_CFG_SET_PF_FLAGS : > > + QED_NVM_CFG_SET_FLAGS; > > + > > + DP_VERBOSE(cdev, NETIF_MSG_DRV, > > + "cfg_id = %d len = %d\n", cfg_id, len); > > + rc = qed_mcp_nvm_set_cfg(hwfn, ptt, cfg_id, entity_id, > > flags, > > + buf, len); > > + if (rc) { > > + DP_ERR(cdev, "Error %d configuring %d\n", rc, > > cfg_id); > > + break; > > + } > > + } > > + > > + qed_ptt_release(hwfn, ptt); > > + > > + return rc; > > +} > > + > > static int qed_nvm_flash(struct qed_dev *cdev, const char *name) > > { > > const struct firmware *image; > > @@ -2268,6 +2330,9 @@ static int qed_nvm_flash(struct qed_dev *cdev, > > const char *name) > > rc = qed_nvm_flash_image_access(cdev, &data, > > &check_resp); > > break; > > + case QED_NVM_FLASH_CMD_NVM_CFG_ID: > > + rc = qed_nvm_flash_cfg_write(cdev, &data); > > + break; > > default: > > DP_ERR(cdev, "Unknown command %08x\n", > > cmd_type); > > rc = -EINVAL; > > diff --git a/include/linux/qed/qed_if.h b/include/linux/qed/qed_if.h > > index eef02e6..23805ea 100644 > > --- a/include/linux/qed/qed_if.h > > +++ b/include/linux/qed/qed_if.h > > @@ -804,6 +804,7 @@ enum qed_nvm_flash_cmd { > > QED_NVM_FLASH_CMD_FILE_DATA = 0x2, > > QED_NVM_FLASH_CMD_FILE_START = 0x3, > > QED_NVM_FLASH_CMD_NVM_CHANGE = 0x4, > > + QED_NVM_FLASH_CMD_NVM_CFG_ID = 0x5, > > QED_NVM_FLASH_CMD_NVM_MAX, > > }; > > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH net-next 2/2] qed: Add API for flashing the nvm attributes. 2019-07-25 0:48 ` Sudarsana Reddy Kalluru @ 2019-07-25 20:56 ` Saeed Mahameed 0 siblings, 0 replies; 6+ messages in thread From: Saeed Mahameed @ 2019-07-25 20:56 UTC (permalink / raw) To: skalluru@marvell.com, davem@davemloft.net Cc: aelior@marvell.com, mkalderon@marvell.com, netdev@vger.kernel.org On Thu, 2019-07-25 at 00:48 +0000, Sudarsana Reddy Kalluru wrote: > > > > > -----Original Message----- > > From: Saeed Mahameed <saeedm@mellanox.com> > > Sent: Thursday, July 25, 2019 1:13 AM > > To: Sudarsana Reddy Kalluru <skalluru@marvell.com>; > > davem@davemloft.net > > Cc: Ariel Elior <aelior@marvell.com>; Michal Kalderon > > <mkalderon@marvell.com>; netdev@vger.kernel.org > > Subject: [EXT] Re: [PATCH net-next 2/2] qed: Add API for flashing > > the nvm > > attributes. > > > > External Email > > > > ----------------------------------------------------------------- > > ----- > > On Tue, 2019-07-23 at 21:51 -0700, Sudarsana Reddy Kalluru wrote: > > > The patch adds driver interface for reading the NVM config > > > request and > > > update the attributes on nvm config flash partition. > > > > > > > You didn't not use the get_cfg API you added in previous patch. > Thanks for your review. Will move this API to the next patch series > which will plan to send shortly. > > > Also can you please clarify how the user reads/write from/to NVM > > config > > ? i mean what UAPIs and tools are being used ? > NVM config/partition will be updated using ethtool flash update > command (i.e., ethtool -f) just like the update of > other flash partitions of qed device. Example code path, > ethool-flash_device --> qede_flash_device() --> qed_nvm_flash() --> > qed_nvm_flash_cfg_write() > I see, thanks for the clarification. ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2019-07-25 20:56 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2019-07-24 4:51 [PATCH net-next 0/2] qed: Support for NVM config attributes Sudarsana Reddy Kalluru 2019-07-24 4:51 ` [PATCH net-next 1/2] qed: Add APIs " Sudarsana Reddy Kalluru 2019-07-24 4:51 ` [PATCH net-next 2/2] qed: Add API for flashing the nvm attributes Sudarsana Reddy Kalluru 2019-07-24 19:42 ` Saeed Mahameed 2019-07-25 0:48 ` Sudarsana Reddy Kalluru 2019-07-25 20:56 ` Saeed Mahameed
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox