diff options
| author |   Jakub Kicinski <jakub.kicinski@netronome.com> | 2018-01-17 18:50:55 -0800 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2018-01-19 15:44:18 -0500 |
| commit | aa3f4b69a794f5ac1bdc501eaf964ce48901836e (patch) | |
| tree | c759a3ca0c11efe7ed3949ffba1b891d96376796 /drivers/net/ethernet/netronome/nfp/nfpcore/nfp_rtsym.c | |
| parent | Merge branch 'tcp-min-rtt' (diff) | |
| download | linux-dev-aa3f4b69a794f5ac1bdc501eaf964ce48901836e.tar.xz linux-dev-aa3f4b69a794f5ac1bdc501eaf964ce48901836e.zip | |
nfp: core: make scalar CPP helpers fail on short accesses
Currently the helpers for accessing 4 or 8 byte values over
the CPP bus return the length of IO on success. If the IO
was short caller has to deal with error handling. The short
IO for 4/8B values is completely impractical. Make the
helpers return an error if full access was not possible.
Fix the few places which are actually dealing with errors
correctly, most call sites already only deal with negative
return codes.
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>
Diffstat (limited to '')
| -rw-r--r-- | drivers/net/ethernet/netronome/nfp/nfpcore/nfp_rtsym.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/drivers/net/ethernet/netronome/nfp/nfpcore/nfp_rtsym.c b/drivers/net/ethernet/netronome/nfp/nfpcore/nfp_rtsym.c index ecda474ac7c3..46107aefad1c 100644 --- a/drivers/net/ethernet/netronome/nfp/nfpcore/nfp_rtsym.c +++ b/drivers/net/ethernet/netronome/nfp/nfpcore/nfp_rtsym.c @@ -277,10 +277,6 @@ u64 nfp_rtsym_read_le(struct nfp_rtsym_table *rtbl, const char *name, break; } - if (err == sym->size) - err = 0; - else if (err >= 0) - err = -EIO; exit: if (error) *error = err; |