[PATCH iproute2-next] devlink: add 'disk' to 'fw_load_policy' string validation

[PATCH iproute2-next] devlink: add 'disk' to 'fw_load_policy' string validation

[louis.peens]

From: Louis Peens <louis.peens@netronome.com>

The 'fw_load_policy' devlink parameter supports the 'disk' value
since kernel v5.4, seems like there was some oversight in adding
this to iproute, fixed by this patch.

Signed-off-by: Louis Peens <louis.peens@netronome.com>
Reviewed-by: Simon Horman <simon.horman@netronome.com>
---
 devlink/devlink.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/devlink/devlink.c b/devlink/devlink.c
index 507972c3..a0a7770a 100644
--- a/devlink/devlink.c
+++ b/devlink/devlink.c
@@ -2349,6 +2349,11 @@ static const struct param_val_conv param_val_conv[] = {
 		.vstr = "flash",
 		.vuint = DEVLINK_PARAM_FW_LOAD_POLICY_VALUE_FLASH,
 	},
+	{
+		.name = "fw_load_policy",
+		.vstr = "disk",
+		.vuint = DEVLINK_PARAM_FW_LOAD_POLICY_VALUE_DISK,
+	},
 	{
 		.name = "reset_dev_on_drv_probe",
 		.vstr = "unknown",
-- 
2.17.1

Re: [PATCH iproute2-next] devlink: add 'disk' to 'fw_load_policy' string validation

[Stephen Hemminger]

On Fri, 19 Jun 2020 13:50:07 +0200
louis.peens@netronome.com wrote:

> From: Louis Peens <louis.peens@netronome.com>
> 
> The 'fw_load_policy' devlink parameter supports the 'disk' value
> since kernel v5.4, seems like there was some oversight in adding
> this to iproute, fixed by this patch.
> 
> Signed-off-by: Louis Peens <louis.peens@netronome.com>
> Reviewed-by: Simon Horman <simon.horman@netronome.com>

Sure applied