Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1368469 > unrolled thread

Applied "regmap: mmio: Explicitly say little endian is the defualt in the bus config" to the regmap tree

Back to article view | Back to linux.kernel

Contents

  Applied "regmap: mmio: Explicitly say little endian is the defualt in the bus config" to the regmap tree Mark Brown <broonie@kernel.org> - 2016-03-31 19:30 +0200
    Re: Applied  "regmap: mmio: Explicitly say little endian is the defualt in the bus config"  to the regmap tree Krzysztof Kozlowski <k.kozlowski@samsung.com> - 2016-04-01 03:00 +0200

#1368469 — Applied "regmap: mmio: Explicitly say little endian is the defualt in the bus config" to the regmap tree

The patch

   regmap: mmio: Explicitly say little endian is the defualt in the bus config

has been applied to the regmap tree at

   git://git.kernel.org/pub/scm/linux/kernel/git/broonie/regmap.git 

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.  

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark

From d055d9a27cf5b0026c14197a0ac43ecf9862afae Mon Sep 17 00:00:00 2001
From: Mark Brown <broonie@kernel.org>
Date: Thu, 31 Mar 2016 10:18:09 -0700
Subject: [PATCH] regmap: mmio: Explicitly say little endian is the defualt in
 the bus config

Otherwise the DT parsing will default to big endian if nothing is
specified.

Reported-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
---
 drivers/base/regmap/regmap-mmio.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/base/regmap/regmap-mmio.c b/drivers/base/regmap/regmap-mmio.c
index 7526906ca080..2fd654a38066 100644
--- a/drivers/base/regmap/regmap-mmio.c
+++ b/drivers/base/regmap/regmap-mmio.c
@@ -212,6 +212,7 @@ static const struct regmap_bus regmap_mmio = {
 	.reg_write = regmap_mmio_write,
 	.reg_read = regmap_mmio_read,
 	.free_context = regmap_mmio_free_context,
+	.val_endian = REGMAP_ENDIAN_LITTLE,
 };
 
 static struct regmap_mmio_context *regmap_mmio_gen_context(struct device *dev,
-- 
2.8.0.rc3

[toc] | [next] | [standalone]

#1368918 — Re: Applied "regmap: mmio: Explicitly say little endian is the defualt in the bus config" to the regmap tree

On 01.04.2016 02:20, Mark Brown wrote:
> The patch
> 
>    regmap: mmio: Explicitly say little endian is the defualt in the bus config
> 
> has been applied to the regmap tree at
> 
>    git://git.kernel.org/pub/scm/linux/kernel/git/broonie/regmap.git 
> 
> All being well this means that it will be integrated into the linux-next
> tree (usually sometime in the next 24 hours) and sent to Linus during
> the next merge window (or sooner if it is a bug fix), however if
> problems are discovered then the patch may be dropped or reverted.  
> 
> You may get further e-mails resulting from automated or manual testing
> and review of the tree, please engage with people reporting problems and
> send followup patches addressing any issues that are reported if needed.
> 
> If any updates are required or you are submitting further changes they
> should be sent as incremental updates against current git, existing
> patches will not be replaced.
> 
> Please add any relevant lists and maintainers to the CCs when replying
> to this mail.
> 
> Thanks,
> Mark
> 
>>From d055d9a27cf5b0026c14197a0ac43ecf9862afae Mon Sep 17 00:00:00 2001
> From: Mark Brown <broonie@kernel.org>
> Date: Thu, 31 Mar 2016 10:18:09 -0700
> Subject: [PATCH] regmap: mmio: Explicitly say little endian is the defualt in
>  the bus config
> 
> Otherwise the DT parsing will default to big endian if nothing is
> specified.
> 
> Reported-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
> Signed-off-by: Mark Brown <broonie@kernel.org>

I tested your for-next branch:
====
commit 1688fc1d57333f8808f8cf04b1a5fdeb10c307c0
Merge: 835df66b60dc 267c85860308 6e4f28780f38
Author: Mark Brown <broonie@kernel.org>
Date:   Thu Mar 31 13:42:36 2016 -0700

    Merge remote-tracking branches 'regmap/topic/doc' and
'regmap/topic/flat' into regmap-next
====

... and on my board (arch/arm/boot/dts/exynos4412-trats2.dts) the
endianness for syscon reboot is fixed:

Tested-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>


Best regards,
Krzysztof

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web