On Tue, Dec 25, 2012 at 08:35:59PM +0530, Laxman Dewangan wrote:

All TPS51632 registers are not readable/writable and
non-volatiles.

Applied both, thanks, but you might wish to consider...

+static bool is_read_reg(struct device *dev, unsigned int reg)
+{
+	switch (reg) {
+	case 0x08 ... 0x0F:
+		return false;
+	default:
+		return true;
+	}
+}

...using the new table based stuff for the access checks, at least this
one seems to map on nicely.