On Wed, 27 Feb 2019, at 08:26, Patrick Venture wrote: > On Tue, Feb 26, 2019 at 1:42 PM Patrick Venture <vent...@google.com> wrote: > > > > On Sun, Feb 24, 2019 at 5:26 PM Andrew Jeffery <and...@aj.id.au> wrote: > > > > > > > > > > > > + > > > > + /* Access to these needs to be locked, held via probe, mapping > > > > ioctl, > > > > + * and release, remove. > > > > + */ > > > > + struct mutex tracking; > > > > + u32 readers; > > > > + u32 readerwriters[P2A_REGION_COUNT]; > > > > > > Might be better to use refcount_t here instead of u32? > > > > Ack > > refcount requires atomic, is that something we have with the ast2400, 2500? Ah, we do on the 2500 (armv6) but not the 2400 (armv5)
- Re: [PATCH 2/2] drivers/misc: Add Aspeed P2A control driv... Patrick Venture
- Re: [PATCH 2/2] drivers/misc: Add Aspeed P2A control... Patrick Venture
- Re: [PATCH 2/2] drivers/misc: Add Aspeed P2A con... Andrew Jeffery
- Re: [PATCH 2/2] drivers/misc: Add Aspeed P2A... Patrick Venture
- Re: [PATCH 2/2] drivers/misc: Add Aspeed P2A control... Andrew Jeffery
- Re: [PATCH 2/2] drivers/misc: Add Aspeed P2A con... Patrick Venture
- Re: [PATCH 2/2] drivers/misc: Add Aspeed P2A... Andrew Jeffery
- Re: [PATCH 2/2] drivers/misc: Add Aspeed... Patrick Venture
- Re: [PATCH 2/2] drivers/misc: Add Aspeed P2A control... Florian Fainelli
- Re: [PATCH 2/2] drivers/misc: Add Aspeed P2A con... Patrick Venture
- Re: [PATCH 2/2] drivers/misc: Add Aspeed P2A con... Andrew Jeffery