Re: [PATCH] regmap: don't corrupt work buffer in _regmap_raw_write()

From: Mark Brown
Date: Mon Apr 08 2013 - 08:01:31 EST

On Mon, Apr 08, 2013 at 12:24:22PM +0200, Paul Bolle wrote:

> 0) This patch ended up as mainline commit
> bc8ce4afd7ee7e1421c935d24b1f879f82afdd4e, which is part of v3.9-rc6.

Numbering your paragraphs and using full commit IDs like this doesn't
help with legibility...

> 2) The following (draft) patch silences this warning. I'm a bit
> uncertain what the regmap_parse_*() functions are meant to do. So I'd
> like to first ask whether something along these lines is acceptable.

Nope, this breaks v3.9 - in that version they both modify in place and
return the parsed value. In v3.10 it already does what you have because
the in place and return value versions have been split.

We need to either revert the commit from v3.9 on the basis that nobody
noticed the issue until some other work so it can't be that bad or come
up with a more invasive fix, at this point in the cycle I'm more
inclined to do the latter.

Attachment: signature.asc
Description: Digital signature