Linux kernel stable tree (mirror) https://git.radix-linux.su/kernel/linux.git/atom?h=v4.17.1 2018-03-12T16:50:42+00:00 2018-03-12T16:50:42+00:00 Mark Brown broonie@kernel.org 2018-03-12T16:50:42+00:00 urn:sha1:2889312616aecf1a1153be0fab19206a6b1f768f 2018-03-12T16:50:40+00:00 Mark Brown broonie@kernel.org 2018-03-12T16:50:40+00:00 urn:sha1:493ea0c8a6bdcdd014a5c586b91953960c7c9260 2018-03-12T16:50:35+00:00 Mark Brown broonie@kernel.org 2018-03-12T16:50:35+00:00 urn:sha1:f981c6cc14aa1c6cfd4154d2f93846bf83a4725a 2018-03-12T16:50:32+00:00 Mark Brown broonie@kernel.org 2018-03-12T16:50:32+00:00 urn:sha1:aa584bada6c2114371a415d134f09fc376c8e07b 2018-02-26T11:00:34+00:00 Charles Keepax ckeepax@opensource.cirrus.com 2018-02-22T12:59:14+00:00 urn:sha1:fb44f3cec35c6e71865012fa281ba6d4ff50a99a The handling for the first two cases in regmap_bulk_write is essentially identical. The first case is just a better implementation of the second, supporting 8 byte registers and doing the locking manually to avoid bouncing the lock for each register. Drop some redundant code by removing the second of these cases and allowing both situations to be handled by the same code. Signed-off-by: Charles Keepax <ckeepax@opensource.cirrus.com> Signed-off-by: Mark Brown <broonie@kernel.org> 2018-02-26T11:00:33+00:00 Charles Keepax ckeepax@opensource.cirrus.com 2018-02-22T12:59:13+00:00 urn:sha1:364e378b8d1679f91a29a9537a881bba17931cfb Raw writes may need to be split into small chunks if max_raw_write is set. Tidy up the code implementing this, the new code is slightly clearer, slightly shorter and slightly more efficient. Signed-off-by: Charles Keepax <ckeepax@opensource.cirrus.com> Signed-off-by: Mark Brown <broonie@kernel.org> 2018-02-26T11:00:32+00:00 Charles Keepax ckeepax@opensource.cirrus.com 2018-02-22T12:59:12+00:00 urn:sha1:7ef2c6b8689a084954cffbd102ee49c2fb72cbd4 Currently regmap_bulk_write will split a write into chunks before calling regmap_raw_write if max_raw_write is set. It is more logical for this handling to be inside regmap_raw_write itself, as this removes the need to keep re-implementing the chunking code, which would be the same for all users of regmap_raw_write. Signed-off-by: Charles Keepax <ckeepax@opensource.cirrus.com> Signed-off-by: Mark Brown <broonie@kernel.org> 2018-02-26T11:00:31+00:00 Charles Keepax ckeepax@opensource.cirrus.com 2018-02-22T12:59:11+00:00 urn:sha1:b4ecfec5ee3f282a4ac0876de332876fec9b488c Signed-off-by: Charles Keepax <ckeepax@opensource.cirrus.com> Signed-off-by: Mark Brown <broonie@kernel.org> 2018-02-26T11:00:30+00:00 Charles Keepax ckeepax@opensource.cirrus.com 2018-02-22T12:59:10+00:00 urn:sha1:0812d8ffa9955251ba0077488d4408d8987ec091 In the case were the bulk transaction is split up into smaller chunks data is passed directly to regmap_raw_write. However regmap_bulk_write uses data in host endian and regmap_raw_write expects data in device endian. As such if the host and device differ in endian the wrong data will be written to the device. Correct this issue using a similar approach to the single raw write case below it, duplicate the data into a new buffer and use parse_inplace to format the data correctly. Fixes: adaac459759d ("regmap: Introduce max_raw_read/write for regmap_bulk_read/write") Signed-off-by: Charles Keepax <ckeepax@opensource.cirrus.com> Signed-off-by: Mark Brown <broonie@kernel.org> 2018-02-26T11:00:14+00:00 Mark Brown broonie@kernel.org 2018-02-26T11:00:14+00:00 urn:sha1:2936e846c4e6df2df0dd6bcade3d269e17a707d1