Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 10 Nov 2017 22:14:57 -0800
From:      Nathan Whitehorn <nwhitehorn@freebsd.org>
To:        Warner Losh <imp@freebsd.org>, src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   Re: svn commit: r325688 - head/sys/boot/powerpc/ps3
Message-ID:  <35853e0e-a315-6f76-5431-b922e0f5700d@freebsd.org>
In-Reply-To: <201711102354.vAANsJDk074902@repo.freebsd.org>
References:  <201711102354.vAANsJDk074902@repo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
The PS3 port *does* support FDT, though not through this loader. On a 
related note, I'm planning to just svn rm loader.ps3 in the next few weeks.
-Nathan

On 11/10/17 15:54, Warner Losh wrote:
> Author: imp
> Date: Fri Nov 10 23:54:18 2017
> New Revision: 325688
> URL: https://svnweb.freebsd.org/changeset/base/325688
>
> Log:
>    FDT support doesn't make sense for ps3. There's no support in the ps3
>    port for FDT, and it's unlikely to grow support for that anytime soon.
>    When it does, support can be added back easily enough.
>    
>    Supported by: Netflix
>
> Modified:
>    head/sys/boot/powerpc/ps3/Makefile
>
> Modified: head/sys/boot/powerpc/ps3/Makefile
> ==============================================================================
> --- head/sys/boot/powerpc/ps3/Makefile	Fri Nov 10 23:54:13 2017	(r325687)
> +++ head/sys/boot/powerpc/ps3/Makefile	Fri Nov 10 23:54:18 2017	(r325688)
> @@ -7,7 +7,6 @@ LOADER_NET_SUPPORT?=	yes
>   LOADER_NFS_SUPPORT?=	yes
>   LOADER_TFTP_SUPPORT?=	no
>   LOADER_GZIP_SUPPORT?=	yes
> -LOADER_FDT_SUPPORT?=	no
>   LOADER_BZIP2_SUPPORT?=	no
>   
>   .include <bsd.init.mk>
> @@ -24,12 +23,6 @@ SRCS+=		lv1call.S ps3cons.c font.h ps3mmu.c ps3net.c p
>   		ps3stor.c ps3disk.c ps3cdrom.c
>   SRCS+=		ucmpdi2.c
>   
> -.if ${LOADER_FDT_SUPPORT} == "yes"
> -CFLAGS+=	-I${FDTSRC}
> -CFLAGS+=	-DLOADER_FDT_SUPPORT
> -LIBFDT=		${BOOTOBJ}/fdt/libfdt.a
> -.endif
> -
>   CFLAGS+=	-mcpu=powerpc64
>   
>   # Always add MI sources
> @@ -55,7 +48,7 @@ SC_DFLT_FONT=cp437
>   font.h:
>   	uudecode < /usr/share/syscons/fonts/${SC_DFLT_FONT}-8x16.fnt && file2c 'u_char dflt_font_16[16*256] = {' '};' < ${SC_DFLT_FONT}-8x16 > font.h && uudecode < /usr/share/syscons/fonts/${SC_DFLT_FONT}-8x14.fnt && file2c 'u_char dflt_font_14[14*256] = {' '};' < ${SC_DFLT_FONT}-8x14 >> font.h && uudecode < /usr/share/syscons/fonts/${SC_DFLT_FONT}-8x8.fnt && file2c 'u_char dflt_font_8[8*256] = {' '};' < ${SC_DFLT_FONT}-8x8 >> font.h
>   
> -loader.help: help.common ${FDTSRC}/help.fdt
> +loader.help: help.common
>   	cat ${.ALLSRC} | \
>   	    awk -f ${LDRSRC}/merge_help.awk > ${.TARGET}
>   
>




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?35853e0e-a315-6f76-5431-b922e0f5700d>