diff options
| author | Maxime Ripard <maxime@cerno.tech> | 2023-01-02 12:52:55 +0100 |
|---|---|---|
| committer | Florian Fainelli <f.fainelli@gmail.com> | 2023-01-10 13:44:04 -0800 |
| commit | a1904fa4dfc929e5ea36bd409cf0d800df56fb8b (patch) | |
| tree | 1417bc9e1f12fe3606ba38e2e87619a7c8d5bd7f /include/soc/bcm2835 | |
| parent | 1b929c02afd37871d5afb9d498426f83432e71c2 (diff) | |
| download | linux-a1904fa4dfc929e5ea36bd409cf0d800df56fb8b.tar.gz linux-a1904fa4dfc929e5ea36bd409cf0d800df56fb8b.tar.bz2 linux-a1904fa4dfc929e5ea36bd409cf0d800df56fb8b.zip | |
firmware: raspberrypi: Fix type assignment
We silently cast an unsigned int into a __le32 which makes sparse
complain. Moreover, we never actually convert endianness between the
CPU's and the expected little-endian value. Fix both at once by calling
cpu_to_le32().
Fixes: 40c31955e4e9 ("firmware: raspberrypi: Provide a helper to query a clock max rate")
Reported-by: kernel test robot <lkp@intel.com>
Link: https://lore.kernel.org/r/20221116091712.1309651-3-maxime@cerno.tech
Signed-off-by: Maxime Ripard <maxime@cerno.tech>
Link: https://lore.kernel.org/r/20230102115255.17802-1-maxime@cerno.tech
Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
Diffstat (limited to 'include/soc/bcm2835')
| -rw-r--r-- | include/soc/bcm2835/raspberrypi-firmware.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/soc/bcm2835/raspberrypi-firmware.h b/include/soc/bcm2835/raspberrypi-firmware.h index ab955591cb72..73cac8d0287e 100644 --- a/include/soc/bcm2835/raspberrypi-firmware.h +++ b/include/soc/bcm2835/raspberrypi-firmware.h @@ -170,7 +170,7 @@ struct rpi_firmware_clk_rate_request { #define RPI_FIRMWARE_CLK_RATE_REQUEST(_id) \ { \ - .id = _id, \ + .id = cpu_to_le32(_id), \ } #if IS_ENABLED(CONFIG_RASPBERRYPI_FIRMWARE) |