Re: [PATCH] staging: iio: light: isl29018: use regmap for register access

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Including Mark here for more insight/comment.

On Tuesday 17 April 2012 11:26 PM, Bryan Freed wrote:
Hey Laxman, Jonathan.

The regmap is pretty new to me.  In looking at it, I see
_regmap_write() will regcache_write() before attempting to write to
the device.
This effectively updates the cached value even when the device write
fails, which is the opposite of the current isl29018 cache policy.

It looks like we tried to change the policy back in August and got
this response:
   >  I did change one basic behavior that I think was also broken: cache
   >  the value regardless of if the transaction completed successfully or
   >  not.
   Don't do that.  That means userspace will get an invalid value if it reads
   in the meantime. If you have an error on a hardware bus - tell userspace about
   it and don't 'guess' what is in the register.
See http://driverdev.linuxdriverproject.org/pipermail/devel/2011-August/020256.html
for details.

The result is likely to be that after a failed write to one field of a
register, the next successful write to another field of that register
will update them both.

yes, the caching is done before actual transfer and it can make cache dirty in case of transfer failure. I am not sure what should be correct handling here: make cache dirty or revert the value to original one? But using the regmap will provide the common behavior across the driver and all driver will have same behavior rather than handling on differently across driver.
So if there is any other policy for this, it will be same for all driver.


--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux