Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 22 Jan 2016 12:22:06 -0700
From:      Ian Lepore <ian@freebsd.org>
To:        Adrian Chadd <adrian.chadd@gmail.com>, Andrew Turner <andrew@freebsd.org>
Cc:        "src-committers@freebsd.org" <src-committers@freebsd.org>, "svn-src-all@freebsd.org" <svn-src-all@freebsd.org>, "svn-src-head@freebsd.org" <svn-src-head@freebsd.org>
Subject:   Re: svn commit: r294562 - head/sys/dev/fdt
Message-ID:  <1453490526.46848.108.camel@freebsd.org>
In-Reply-To: <CAJ-Vmo=g-89uABcSRsYWbiFVhZWbTmqZattXYvcUChDoTJS-Ug@mail.gmail.com>
References:  <201601221309.u0MD9hmn069967@repo.freebsd.org> <CAJ-Vmo=g-89uABcSRsYWbiFVhZWbTmqZattXYvcUChDoTJS-Ug@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On Fri, 2016-01-22 at 10:26 -0800, Adrian Chadd wrote:
> ... we're about to start using FDT on MIPS, so .. ?
> 
> 
> -a
> 

But mips will be using intrng from the outset, only the arm platforms
that don't yet use intrng need fdt_pic_table.

-- Ian

> 
> On 22 January 2016 at 05:09, Andrew Turner <andrew@freebsd.org>
> wrote:
> > Author: andrew
> > Date: Fri Jan 22 13:09:43 2016
> > New Revision: 294562
> > URL: https://svnweb.freebsd.org/changeset/base/294562
> > 
> > Log:
> >   Only define fdt_pic_table on arm, and when not using intrng as
> > this is
> >   the only place that uses it.
> > 
> > Modified:
> >   head/sys/dev/fdt/fdt_common.h
> > 
> > Modified: head/sys/dev/fdt/fdt_common.h
> > ===================================================================
> > ===========
> > --- head/sys/dev/fdt/fdt_common.h       Fri Jan 22 12:51:12 2016   
> >      (r294561)
> > +++ head/sys/dev/fdt/fdt_common.h       Fri Jan 22 13:09:43 2016   
> >      (r294562)
> > @@ -45,8 +45,10 @@ struct fdt_sense_level {
> >         enum intr_polarity      pol;
> >  };
> > 
> > +#if defined(__arm__) && !defined(ARM_INTRNG)
> >  typedef int (*fdt_pic_decode_t)(phandle_t, pcell_t *, int *, int
> > *, int *);
> >  extern fdt_pic_decode_t fdt_pic_table[];
> > +#endif
> > 
> >  #if defined(__arm__) || defined(__powerpc__)
> >  typedef void (*fdt_fixup_t)(phandle_t);
> > 
> 



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1453490526.46848.108.camel>