lkml.org 
[lkml]   [2018]   [Jan]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 3/3] regcache: flat: Add valid bit to this cache type
    On Mon, Jan 08, 2018 at 10:25:28AM -0600, Andrew F. Davis wrote:

    > I can understand the need for a fast cache type, but without this change
    > the implementation is simply wrong IMHO. Reading from a register that
    > has not been read/written to should not just assume the default value is
    > 0, it should first read and load the real initial value into the cache,
    > just like other cache types do.

    Users are supposed to ensure that the cache is fully initialized either
    by supplying defaults or writing to all the registers. Adding reads is
    problematic since we'd suddenly start reading from hardware which might
    not like it.

    > Also, it looks like this cache type is used mostly in I2C devices
    > (outside of sound/soc/, where in there, like you said MMIO devices are
    > more common).

    Sounds like a good thing to do an audit and contribute fixes for...
    [unhandled content-type:application/pgp-signature]
    \
     
     \ /
      Last update: 2018-01-14 23:17    [W:4.537 / U:0.128 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site