diff options
author | Viresh Kumar <viresh.kumar@linaro.org> | 2016-08-11 13:27:25 -0700 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@google.com> | 2016-08-12 17:38:19 +0200 |
commit | ac96a609dca8b1909f93d13277dd5368e2de5a2a (patch) | |
tree | 2d578b9e7c8f22b12c6106044156b2d817c09553 /drivers/staging/greybus/firmware.h | |
parent | 0675363ac7ffe27a25bf237ce2bf1e084637a3a0 (diff) | |
download | op-kernel-dev-ac96a609dca8b1909f93d13277dd5368e2de5a2a.zip op-kernel-dev-ac96a609dca8b1909f93d13277dd5368e2de5a2a.tar.gz |
greybus: firmware: add comment to show size calculations
The firmware file name's maximum length is set to 56 right now, but it
isn't very much readable how we got to that value (unless someone reads
that line).
Update the comment to show calculations.
Reported-by: Alex Elder <alex.elder@linaro.org>
Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>
Diffstat (limited to 'drivers/staging/greybus/firmware.h')
-rw-r--r-- | drivers/staging/greybus/firmware.h | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/staging/greybus/firmware.h b/drivers/staging/greybus/firmware.h index ec6bfb7..f4f0db1 100644 --- a/drivers/staging/greybus/firmware.h +++ b/drivers/staging/greybus/firmware.h @@ -14,7 +14,10 @@ #define FW_NAME_PREFIX "gmp_" -/* Length of the string in format: "FW_NAME_PREFIX""%08x_%08x_%08x_%08x_%s.tftf" */ +/* + * Length of the string in format: "FW_NAME_PREFIX""%08x_%08x_%08x_%08x_%s.tftf" + * (3 + 1 + 4 * (8 + 1) + 10 + 1 + 4 + 1) + */ #define FW_NAME_SIZE 56 /* Firmware Management Protocol specific functions */ |