Commit 76536caa authored by Sam Protsenko's avatar Sam Protsenko Committed by Herbert Xu

hwrng: exynos - Improve coding style

Fix obvious style issues. Some of those were found with checkpatch, and
some just contradict the kernel coding style guide.

No functional change.
Signed-off-by: default avatarSam Protsenko <semen.protsenko@linaro.org>
Reviewed-by: default avatarKrzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Acked-by: default avatarŁukasz Stelmach <l.stelmach@samsung.com>
Signed-off-by: default avatarHerbert Xu <herbert@gondor.apana.org.au>
parent 70003f51
...@@ -23,27 +23,26 @@ ...@@ -23,27 +23,26 @@
#include <linux/platform_device.h> #include <linux/platform_device.h>
#include <linux/pm_runtime.h> #include <linux/pm_runtime.h>
#define EXYNOS_TRNG_CLKDIV (0x0) #define EXYNOS_TRNG_CLKDIV 0x0
#define EXYNOS_TRNG_CTRL (0x20) #define EXYNOS_TRNG_CTRL 0x20
#define EXYNOS_TRNG_CTRL_RNGEN BIT(31) #define EXYNOS_TRNG_CTRL_RNGEN BIT(31)
#define EXYNOS_TRNG_POST_CTRL (0x30) #define EXYNOS_TRNG_POST_CTRL 0x30
#define EXYNOS_TRNG_ONLINE_CTRL (0x40) #define EXYNOS_TRNG_ONLINE_CTRL 0x40
#define EXYNOS_TRNG_ONLINE_STAT (0x44) #define EXYNOS_TRNG_ONLINE_STAT 0x44
#define EXYNOS_TRNG_ONLINE_MAXCHI2 (0x48) #define EXYNOS_TRNG_ONLINE_MAXCHI2 0x48
#define EXYNOS_TRNG_FIFO_CTRL (0x50) #define EXYNOS_TRNG_FIFO_CTRL 0x50
#define EXYNOS_TRNG_FIFO_0 (0x80) #define EXYNOS_TRNG_FIFO_0 0x80
#define EXYNOS_TRNG_FIFO_1 (0x84) #define EXYNOS_TRNG_FIFO_1 0x84
#define EXYNOS_TRNG_FIFO_2 (0x88) #define EXYNOS_TRNG_FIFO_2 0x88
#define EXYNOS_TRNG_FIFO_3 (0x8c) #define EXYNOS_TRNG_FIFO_3 0x8c
#define EXYNOS_TRNG_FIFO_4 (0x90) #define EXYNOS_TRNG_FIFO_4 0x90
#define EXYNOS_TRNG_FIFO_5 (0x94) #define EXYNOS_TRNG_FIFO_5 0x94
#define EXYNOS_TRNG_FIFO_6 (0x98) #define EXYNOS_TRNG_FIFO_6 0x98
#define EXYNOS_TRNG_FIFO_7 (0x9c) #define EXYNOS_TRNG_FIFO_7 0x9c
#define EXYNOS_TRNG_FIFO_LEN (8) #define EXYNOS_TRNG_FIFO_LEN 8
#define EXYNOS_TRNG_CLOCK_RATE (500000) #define EXYNOS_TRNG_CLOCK_RATE 500000
struct exynos_trng_dev { struct exynos_trng_dev {
struct device *dev; struct device *dev;
...@@ -55,13 +54,10 @@ struct exynos_trng_dev { ...@@ -55,13 +54,10 @@ struct exynos_trng_dev {
static int exynos_trng_do_read(struct hwrng *rng, void *data, size_t max, static int exynos_trng_do_read(struct hwrng *rng, void *data, size_t max,
bool wait) bool wait)
{ {
struct exynos_trng_dev *trng; struct exynos_trng_dev *trng = (struct exynos_trng_dev *)rng->priv;
int val; int val;
max = min_t(size_t, max, (EXYNOS_TRNG_FIFO_LEN * 4)); max = min_t(size_t, max, (EXYNOS_TRNG_FIFO_LEN * 4));
trng = (struct exynos_trng_dev *)rng->priv;
writel_relaxed(max * 8, trng->mem + EXYNOS_TRNG_FIFO_CTRL); writel_relaxed(max * 8, trng->mem + EXYNOS_TRNG_FIFO_CTRL);
val = readl_poll_timeout(trng->mem + EXYNOS_TRNG_FIFO_CTRL, val, val = readl_poll_timeout(trng->mem + EXYNOS_TRNG_FIFO_CTRL, val,
val == 0, 200, 1000000); val == 0, 200, 1000000);
...@@ -87,7 +83,7 @@ static int exynos_trng_init(struct hwrng *rng) ...@@ -87,7 +83,7 @@ static int exynos_trng_init(struct hwrng *rng)
*/ */
val = sss_rate / (EXYNOS_TRNG_CLOCK_RATE * 2); val = sss_rate / (EXYNOS_TRNG_CLOCK_RATE * 2);
if (val > 0x7fff) { if (val > 0x7fff) {
dev_err(trng->dev, "clock divider too large: %d", val); dev_err(trng->dev, "clock divider too large: %d\n", val);
return -ERANGE; return -ERANGE;
} }
val = val << 1; val = val << 1;
...@@ -122,7 +118,7 @@ static int exynos_trng_probe(struct platform_device *pdev) ...@@ -122,7 +118,7 @@ static int exynos_trng_probe(struct platform_device *pdev)
trng->rng.init = exynos_trng_init; trng->rng.init = exynos_trng_init;
trng->rng.read = exynos_trng_do_read; trng->rng.read = exynos_trng_do_read;
trng->rng.priv = (unsigned long) trng; trng->rng.priv = (unsigned long)trng;
platform_set_drvdata(pdev, trng); platform_set_drvdata(pdev, trng);
trng->dev = &pdev->dev; trng->dev = &pdev->dev;
...@@ -225,6 +221,7 @@ static struct platform_driver exynos_trng_driver = { ...@@ -225,6 +221,7 @@ static struct platform_driver exynos_trng_driver = {
}; };
module_platform_driver(exynos_trng_driver); module_platform_driver(exynos_trng_driver);
MODULE_AUTHOR("Łukasz Stelmach"); MODULE_AUTHOR("Łukasz Stelmach");
MODULE_DESCRIPTION("H/W TRNG driver for Exynos chips"); MODULE_DESCRIPTION("H/W TRNG driver for Exynos chips");
MODULE_LICENSE("GPL v2"); MODULE_LICENSE("GPL v2");
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment