Hi, Dave,

Dave Jiang <dave.ji...@intel.com> writes:

> Adding firmware output of firmware information when ndctl list -D -F is used.
> Components displayed are current firmware version, updated firmware version,
> and if a coldboot is required (firmware updated).
>
> Signed-off-by: Dave Jiang <dave.ji...@intel.com>
> ---
>  Documentation/ndctl/ndctl-list.txt |   13 +++++++++++++
>  1 file changed, 13 insertions(+)
>
> @@ -420,6 +422,25 @@ int cmd_list(int argc, const char **argv, void *ctx)
>                               }
>                       }
>  
> +                     if (list.firmware) {
> +                             struct json_object *jfirmware;
> +
> +                             jfirmware = util_dimm_firmware_to_json(dimm);
> +                             if (jfirmware)
> +                                     json_object_object_add(jdimm,
> +                                                     "firmware",
> +                                                     jfirmware);
> +                             else if (ndctl_dimm_is_cmd_supported(dimm,
> +                                                     ND_CMD_CALL)) {
> +                                     /*
> +                                      * Failed to retrieve firmware
> +                                      * version.
> +                                      */
> +                                     fail("\n");

I'm hitting this failure.  Also, you might consider adding some text,
there.  :)

-Jeff
_______________________________________________
Linux-nvdimm mailing list
Linux-nvdimm@lists.01.org
https://lists.01.org/mailman/listinfo/linux-nvdimm

Reply via email to