Date: Fri, 22 Jan 2016 21:22:53 -0800 From: Adrian Chadd <adrian.chadd@gmail.com> To: Stanislav Galabov <sgalabov@gmail.com> Cc: Steve Kiernan <stevek@juniper.net>, "freebsd-arm@freebsd.org" <freebsd-arm@freebsd.org>, "freebsd-mips@freebsd.org" <freebsd-mips@freebsd.org> Subject: Re: SPI geom_flashmap/fdt_slicer support, FDT 'resets=' support and a move of ohci_fdt.c Message-ID: <CAJ-VmomijUZU52WQ2EJO6fSi-dzdH1XxJdUPy_cPwvKDjDde1A@mail.gmail.com> In-Reply-To: <CANiSyhR01wY65%2B2Nu%2B3Hucs%2BbDWyKhH_ejujG9CE39SaQQ2Wqg@mail.gmail.com> References: <B4B24B7D-B3EE-4F37-9E89-24FF17294C70@gmail.com> <20160122173906.147bb7c2@stevek-ubuntu.jnpr.net> <CANiSyhR01wY65%2B2Nu%2B3Hucs%2BbDWyKhH_ejujG9CE39SaQQ2Wqg@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
let me .. fix that. -a On 22 January 2016 at 21:18, Stanislav Galabov <sgalabov@gmail.com> wrote: > It looks like only part of the original patch I submitted made it into he= ad > (only the mx25l part). > We also need to change sys/conf/files as Steve pointed out, as well as > sys/geom/geom_flashmap.c: > https://bz-attachments.freebsd.org/attachment.cgi?id=3D165546 > > Best wishes, > Stanislav > > On Sat, Jan 23, 2016 at 12:39 AM, Steve Kiernan <stevek@juniper.net> wrot= e: > >> On Thu, 14 Jan 2016 11:01:10 +0200 >> Stanislav Galabov <sgalabov@gmail.com> wrote: >> >> > Hi all, >> > >> > First off, sorry for the cross-post, I wasn=E2=80=99t very sure where = this >> should go=E2=80=A6 >> > >> > I=E2=80=99ve created 3 PRs, which enable some functionality that my wo= rk on >> Ralink/Mediatek SoCs would benefit from. >> > >> > 1. https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D206227 >> > - This enables geom_flashmap and fdt_slicer support for SPI flash chip= s >> supported by the mx25l driver (sys/dev/flash/mx25l.c) >> >> I see this change was committed to head, but I don't see a corresponding >> change to sys/conf/files to ensure fdt_slicer.c and geom_flashmap.c get >> compiled. >> >> I think you also need something like the following: >> >> Index: sys/conf/files >> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D >> --- sys/conf/files (revision 294541) >> +++ sys/conf/files (working copy) >> @@ -1419,7 +1419,7 @@ >> dev/fdt/fdt_common.c optional fdt >> dev/fdt/fdt_pinctrl.c optional fdt fdt_pinctrl >> dev/fdt/fdt_pinctrl_if.m optional fdt fdt_pinctrl >> -dev/fdt/fdt_slicer.c optional fdt cfi | fdt nand >> +dev/fdt/fdt_slicer.c optional fdt cfi | fdt nand | fdt mx25l >> dev/fdt/fdt_static_dtb.S optional fdt fdt_dtb_static \ >> dependency "$S/boot/fdt/dts/${MACHINE}/${FDT_DTS_FILE}" >> dev/fdt/simplebus.c optional fdt >> @@ -3010,7 +3010,7 @@ >> geom/geom_dump.c stfgeom_dumandard >> geom/geom_event.c standard >> geom/geom_fox.c optional geom_fox >> -geom/geom_flashmap.c optional fdt cfi | fdt nand >> +geom/geom_flashmap.c optional fdt cfi | fdt nand | fdt mx25l >> geom/geom_io.c standard >> geom/geom_kern.c standard >> geom/geom_map.c optional geom_map >> > _______________________________________________ > freebsd-arm@freebsd.org mailing list > https://lists.freebsd.org/mailman/listinfo/freebsd-arm > To unsubscribe, send any mail to "freebsd-arm-unsubscribe@freebsd.org"
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAJ-VmomijUZU52WQ2EJO6fSi-dzdH1XxJdUPy_cPwvKDjDde1A>