diff options
| author | Xiubo Li <Li.Xiubo@freescale.com> | 2014-09-26 03:59:13 (GMT) |
|---|---|---|
| committer | Matthew Weigel <Matthew.Weigel@freescale.com> | 2014-12-11 18:36:33 (GMT) |
| commit | cf08d43a077f6f7e975885610bde40e2479e093b (patch) | |
| tree | 2316642617b97e7361601bb9c687ae131cf7aefc /Documentation | |
| parent | 9cc091a16151c080f0f45ec3199c1bc5b9b51d49 (diff) | |
| download | linux-fsl-qoriq-cf08d43a077f6f7e975885610bde40e2479e093b.tar.xz | |
Split regmap_get_endian() in two functions
Split regmap_get_endian() in two functions, regmap_get_reg_endian() and
regmap_get_val_endian().
This allows to:
- Get rid of the three switch()es on "type", incl. error handling in
three "default" cases,
- Get rid of the regmap_endian_type enum,
- Get rid of the non-NULL check of "config" (regmap_init() already
checks for that),
- Get rid of the "endian" output parameters, and just return the
regmap_endian enum value, as the functions can no longer fail.
This saves 21 lines of code (despite the still-present
one-comment-per-line over-documentation), and 30 bytes of code on ARM
V7.
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Reviewed-by: Stephen Warren <swarren@nvidia.com>
Signed-off-by: Mark Brown <broonie@linaro.org>
---
This patch is pulled back from upstream:
commit cf673fbc6342b1c2310cdfdc4ed99f18f866b8e4
Change-Id: Ifea4f2c83977bb81b50853de097f52bb586fc6ce
Reviewed-on: http://git.am.freescale.net:8181/19859
Tested-by: Review Code-CDREVIEW <CDREVIEW@freescale.com>
Reviewed-by: Zhengxiong Jin <Jason.Jin@freescale.com>
Diffstat (limited to 'Documentation')
0 files changed, 0 insertions, 0 deletions