On Mon, Aug 06, 2018 at 03:04:44PM +0300, Stefan Popa wrote: > + if (!regmap_volatile(map, reg) || !regmap_readable(map, reg)) { > + ret = -EINVAL; > + goto out_unlock; > + } I suggested having an explicit operation to check if a register supports this mode of operation when I reviewed an earlier version, I didn't notice a reply on that?