Date: Fri, 12 Jan 2018 19:09:13 -0800 From: Nathan Whitehorn <nwhitehorn@freebsd.org> To: Justin Hibbits <jhibbits@freebsd.org>, src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r327907 - in head/sys: conf dev/fdt dev/ofw Message-ID: <b687e403-2e58-dcec-9a80-6c9df0271b34@freebsd.org> In-Reply-To: <201801130256.w0D2u9fn087029@repo.freebsd.org> References: <201801130256.w0D2u9fn087029@repo.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Thanks, Justin! Since the fixup stuff is now only defined for FDT_MARVELL, it would be great if it could move to sys/arm/mv now instead of being in MI code. -Nathan On 01/12/18 18:56, Justin Hibbits wrote: > Author: jhibbits > Date: Sat Jan 13 02:56:09 2018 > New Revision: 327907 > URL: https://svnweb.freebsd.org/changeset/base/327907 > > Log: > Remove fdt fixups for powerpc, they are no longer needed. > > If a fixup really is needed, it should be fixed in u-boot, not in FreeBSD. > > Suggested by: nwhitehorn > > Deleted: > head/sys/dev/fdt/fdt_powerpc.c > Modified: > head/sys/conf/files.powerpc > head/sys/dev/ofw/ofw_fdt.c > > Modified: head/sys/conf/files.powerpc > ============================================================================== > --- head/sys/conf/files.powerpc Sat Jan 13 01:36:37 2018 (r327906) > +++ head/sys/conf/files.powerpc Sat Jan 13 02:56:09 2018 (r327907) > @@ -31,7 +31,6 @@ dev/adb/adb_if.m optional adb > dev/adb/adb_buttons.c optional adb > dev/agp/agp_apple.c optional agp powermac > dev/fb/fb.c optional sc > -dev/fdt/fdt_powerpc.c optional fdt > # ofwbus depends on simplebus. > dev/fdt/simplebus.c optional aim | fdt > dev/hwpmc/hwpmc_e500.c optional hwpmc > > Modified: head/sys/dev/ofw/ofw_fdt.c > ============================================================================== > --- head/sys/dev/ofw/ofw_fdt.c Sat Jan 13 01:36:37 2018 (r327906) > +++ head/sys/dev/ofw/ofw_fdt.c Sat Jan 13 02:56:09 2018 (r327907) > @@ -430,7 +430,7 @@ ofw_fdt_package_to_path(ofw_t ofw, phandle_t package, > return (-1); > } > > -#if defined(FDT_MARVELL) || defined(__powerpc__) > +#if defined(FDT_MARVELL) > static int > ofw_fdt_fixup(ofw_t ofw) > { > @@ -477,7 +477,7 @@ ofw_fdt_fixup(ofw_t ofw) > static int > ofw_fdt_interpret(ofw_t ofw, const char *cmd, int nret, cell_t *retvals) > { > -#if defined(FDT_MARVELL) || defined(__powerpc__) > +#if defined(FDT_MARVELL) > int rv; > > /* >
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?b687e403-2e58-dcec-9a80-6c9df0271b34>