On 29/11/15 16:35, Álvaro Fernández Rojas wrote:
> Signed-off-by: Álvaro Fernández Rojas <noltari@xxxxxxxxx>
> ---
> drivers/leds/leds-bcm6328.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/leds/leds-bcm6328.c b/drivers/leds/leds-bcm6328.c
> index cd54264..876f0ce 100644
> --- a/drivers/leds/leds-bcm6328.c
> +++ b/drivers/leds/leds-bcm6328.c
> @@ -76,12 +76,12 @@ struct bcm6328_led {
>
> static void bcm6328_led_write(void __iomem *reg, unsigned long data)
> {
> - iowrite32be(data, reg);
> + __raw_writel(data, reg);
> }
>
> static unsigned long bcm6328_led_read(void __iomem *reg)
> {
> - return ioread32be(reg);
> + return __raw_readl(reg);
> }
>
> /**
>
If the wrapper functions don't add anything, would it not be better to
just remove them completely?
--
Simon Arlott
--
To unsubscribe from this list: send the line "unsubscribe linux-leds" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
