Re: [PATCH v3 6/7] arm64: Advertise ARM64_HAS_DCPODP cpu feature

[Dave Martin <Dave dot Martin at arm dot com>]

On Wed, Apr 03, 2019 at 10:23:42AM +0100, Andrew Murray wrote:
> On Tue, Apr 02, 2019 at 03:59:58PM +0100, Dave Martin wrote:
> > On Mon, Apr 01, 2019 at 11:45:14AM +0100, Andrew Murray wrote:
> > > Advertise ARM64_HAS_DCPOP when both DC CVAP and DC CVADP are supported.
> > 
> > Do you mean ARM64_HAS_DCPODP?
> 
> Yes I did - good catch.
> 
> > 
> > And do we need this?  This capability flag doesn't currently appear to
> > be used for anything (which makes me wonder whether it _should_ be wired
> > up to something in the kernel).
> > 
> > Do we expect the kernel to do something special with this in the future?
> > 
> > OTOH, we get a nice printk when the feature is detected, and the code
> > size cost is insignificant.  So, if there's a reasonable expectation that
> > we will use it someday, I don't see a big problem with having it.
> 
> Yes it's not currently used, so I'm happy to drop this patch if people prefer
> that - however it does complement the existing DCPOP cap which is present.

OK, sounds fine.

Maybe add a note in the commit message that this is here for consistency
with DCPOP, and we anticipate it being used in the future.

Cheers
---Dave