Date: Sat, 11 Aug 2018 18:18:37 -0600 From: Warner Losh <imp@bsdimp.com> To: Ian Lepore <ian@freebsd.org> Cc: "freebsd-arm@freebsd.org" <freebsd-arm@freebsd.org> Subject: Re: Kill old, non-INTRNG code on ARM Message-ID: <CANCZdfq783avYR1TUe51y_SkLaxTRha8ETqrPgfKaJxwAVTiiQ@mail.gmail.com> In-Reply-To: <CANCZdfqAWp7%2BpaPSq-fkDuo=cd-kx_Dtfzd_6WjP=2gDjUtZUA@mail.gmail.com> References: <CANCZdfo3D1P7eOkZGuLNTPdQ6V00W1-gieVYTE7R=-qaYnehbA@mail.gmail.com> <1534007513.31375.9.camel@freebsd.org> <CANCZdfqAWp7%2BpaPSq-fkDuo=cd-kx_Dtfzd_6WjP=2gDjUtZUA@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Sat, Aug 11, 2018 at 11:40 AM, Warner Losh <imp@bsdimp.com> wrote: > > > On Sat, Aug 11, 2018, 11:11 AM Ian Lepore <ian@freebsd.org> wrote: > >> On Sat, 2018-08-11 at 08:52 -0600, Warner Losh wrote: >> > One last thing, even though we're in a slush, is to kill the pre- >> > INTRNG >> > code. >> > >> > It's required for armv6 and v7 for some time (only the Amlogic 8726 >> > boards >> > remain broken there). And we're down to two families of chips which >> > support >> > armv5: rt1310 and the Marvel gear. Both of them support INTRNG. >> > >> > I propose we eliminate the INTRNG option and unifdef the code to >> > remove the >> > undefined case. >> > >> > Comments? >> > >> >> Wait a sec, the INTRNG option can't go away, it's used by multiple >> arches these days. What we can do is make it on-by-default for arm* >> arches, and eliminate any #if[n]def INTRNG in arm-specific code. >> > > Yes. That's a better way of saying what I had in mind. GC the code that > will never compile on arm so we don't have to support it on 12. Or rather > trip over it... > OK. I misread things. Marvell still isn't converted to INTRNG. I have a more modest patch that moves things around a little in the config files to help... Warner
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfq783avYR1TUe51y_SkLaxTRha8ETqrPgfKaJxwAVTiiQ>