nfp: nsp: use fractional size of the buffer

NSP expresses the buffer size in MB and 4 kB blocks.  For small
buffers the kB part may make a difference, so count it in.

Signed-off-by: Jakub Kicinski <jakub.kicinski@netronome.com>
Reviewed-by: Dirk van der Merwe <dirk.vandermerwe@netronome.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

1 files changed, 7 insertions, 6 deletions

diff --git a/drivers/net/ethernet/netronome/nfp/nfpcore/nfp_nsp.c b/drivers/net/ethernet/netronome/nfp/nfpcore/nfp_nsp.c
index a9d53df0070c..22208b03ff49 100644
--- a/drivers/net/ethernet/netronome/nfp/nfpcore/nfp_nsp.c
+++ b/drivers/net/ethernet/netronome/nfp/nfpcore/nfp_nsp.c
@@ -49,6 +49,7 @@
 #define   NSP_DFLT_BUFFER_ADDRESS       GENMASK_ULL(39, 0)
 
 #define NSP_DFLT_BUFFER_CONFIG  0x20
+#define   NSP_DFLT_BUFFER_SIZE_4KB      GENMASK_ULL(15, 8)
 #define   NSP_DFLT_BUFFER_SIZE_MB       GENMASK_ULL(7, 0)
 
 #define NSP_MAGIC               0xab10
@@ -413,8 +414,8 @@ static int nfp_nsp_command(struct nfp_nsp *state, u16 code)
 static int
 nfp_nsp_command_buf(struct nfp_nsp *nsp, struct nfp_nsp_command_buf_arg *arg)
 {
+        unsigned int def_size, max_size;
         struct nfp_cpp *cpp = nsp->cpp;
-        unsigned int max_size;
         u64 reg, cpp_buf;
         int ret, err;
         u32 cpp_id;
@@ -433,11 +434,11 @@ nfp_nsp_command_buf(struct nfp_nsp *nsp, struct nfp_nsp_command_buf_arg *arg)
                 return err;
 
         max_size = max(arg->in_size, arg->out_size);
-        if (FIELD_GET(NSP_DFLT_BUFFER_SIZE_MB, reg) * SZ_1M < max_size) {
-                nfp_err(cpp, "NSP: default buffer too small for command 0x%04x (%llu < %u)\n",
-                        arg->arg.code,
-                        FIELD_GET(NSP_DFLT_BUFFER_SIZE_MB, reg) * SZ_1M,
-                        max_size);
+        def_size = FIELD_GET(NSP_DFLT_BUFFER_SIZE_MB, reg) * SZ_1M +
+                   FIELD_GET(NSP_DFLT_BUFFER_SIZE_4KB, reg) * SZ_4K;
+        if (def_size < max_size) {
+                nfp_err(cpp, "NSP: default buffer too small for command 0x%04x (%u < %u)\n",
+                        arg->arg.code, def_size, max_size);
                 return -EINVAL;
         }