On Wed, 2016-01-13 at 11:37 +0530, Aneesh Kumar K.V wrote:
> Benjamin Herrenschmidt <b...@kernel.crashing.org> writes:
>
> > On Tue, 2016-01-12 at 10:42 +0300, Denis Kirjanov wrote:
> > > > +static inline unsigned long pte_io_cache_bits(void)
> > > > +{
> > > > + return _PAGE_NO_CACHE | _PAGE_GUARDED;
> > > > +}
> > > This could be just plain #define
> >
> > Or just use pgprot_noncached()
> >
> #define pgprot_noncached(prot) (__pgprot((pgprot_val(prot) &
> ~_PAGE_CACHE_CTL) | \
> _PAGE_NO_CACHE |
> _PAGE_GUARDED))
>
>
> That will return me a pgprot_t. I can fix that by using
> pgprot_val(pgprot_noncached(0)). Is that what you are suggesting ?
Shouln't ioremap just use pgprot_noncached(PAGE_KERNEL) or similar ?
Cheers,
Ben.
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev
