Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1363362
| From | Mark Brown <broonie@kernel.org> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: regmap: mmio: regression in pre-v4.6-rc1 |
| Date | 2016-03-23 12:40 +0100 |
| Message-ID | <rfPiO-2K3-11@gated-at.bofh.it> (permalink) |
| References | <rfMEi-Or-7@gated-at.bofh.it> <rfOmK-22n-13@gated-at.bofh.it> <rfOZs-2wg-19@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
[Multipart message — attachments visible in raw view] - view raw
On Wed, Mar 23, 2016 at 12:16:13PM +0100, Alexander Stein wrote: > On Wednesday 23 March 2016 10:34:15, Mark Brown wrote: > > Are you *sure* that this is actually big endian? Are you basing this on > > documentation or on what happened to work for you in the past. > Please refer to QorIQ LS1021A Reference Manual (REV 0) table 2.2 (CCSR block > base address map) which states that this peripheral (among _most_ but not all) > requires byte swapping. Same for DSPI. > Yeah, it sounds strange. I don't have that document. > > Have you tried tracing through the code to see what ends up happening to > > the I/O? It should come out using your architecture's big endian > > accessors. > In regmap_mmio_gen_context ctx->reg_read is set to regmap_mmio_read32le and > ctx->reg_write to regmap_mmio_write32le respectively. So how does that happen then? We set these values if the bus is default, little or native endian but if it's big endian we go into a completely different case...
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
regmap: mmio: regression in pre-v4.6-rc1 Alexander Stein <alexander.stein@systec-electronic.com> - 2016-03-23 09:50 +0100
Re: regmap: mmio: regression in pre-v4.6-rc1 Alexander Stein <alexander.stein@systec-electronic.com> - 2016-03-23 10:50 +0100
Re: regmap: mmio: regression in pre-v4.6-rc1 Mark Brown <broonie@kernel.org> - 2016-03-23 11:40 +0100
Re: regmap: mmio: regression in pre-v4.6-rc1 Alexander Stein <alexander.stein@systec-electronic.com> - 2016-03-23 12:20 +0100
Re: regmap: mmio: regression in pre-v4.6-rc1 Mark Brown <broonie@kernel.org> - 2016-03-23 12:40 +0100
Re: regmap: mmio: regression in pre-v4.6-rc1 Alexander Stein <alexander.stein@systec-electronic.com> - 2016-03-23 13:00 +0100
Re: regmap: mmio: regression in pre-v4.6-rc1 Mark Brown <broonie@kernel.org> - 2016-03-23 13:10 +0100
csiph-web