Date: Thu, 25 Apr 2024 16:11:38 -0600 From: Warner Losh <imp@bsdimp.com> To: Kyle Evans <kevans@freebsd.org> Cc: =?UTF-8?Q?Klaus_K=C3=BCchemann?= <maciphone2@googlemail.com>, Colin Percival <cperciva@tarsnap.com>, src-committers <src-committers@freebsd.org>, "<dev-commits-src-all@freebsd.org>" <dev-commits-src-all@freebsd.org>, "<dev-commits-src-main@freebsd.org>" <dev-commits-src-main@freebsd.org> Subject: Re: git: 1bd4f769caf8 - main - add dtb support for RPI CM4, CM4s, CM4_ioBoard Message-ID: <CANCZdfr9_AATSeju-h3h2kMgUvcreTNVfFmMm870bqXGtuFULw@mail.gmail.com> In-Reply-To: <142362e6-8a88-43f5-bff6-6ff90a12802e@FreeBSD.org> References: <202404192311.43JNBiVS059729@gitrepo.freebsd.org> <0100018f169ec80a-47d419c2-2cd6-4ee1-89da-3f4782429089-000000@email.amazonses.com> <CANCZdfp0S4bqAMjj1HQt3gHwr56T%2B_tyo7w1xi5GG1pXPWAOzA@mail.gmail.com> <0100018f16c8f23a-eacbe781-3f6e-4f02-9b4d-d26468364b1f-000000@email.amazonses.com> <7473b5ed-433b-4d51-8a23-cf6464eb4416@FreeBSD.org> <7FA67865-F391-4773-86BA-9998B3EC855E@googlemail.com> <142362e6-8a88-43f5-bff6-6ff90a12802e@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--000000000000fb51510616f312a6 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Thu, Apr 25, 2024, 4:09=E2=80=AFPM Kyle Evans <kevans@freebsd.org> wrote= : > On 4/25/24 16:44, Klaus K=C3=BCchemann wrote: > > > > > >> Am 25.04.2024 um 22:01 schrieb Kyle Evans <kevans@FreeBSD.org>: > >> > >> On 4/25/24 14:43, Colin Percival wrote: > >>> On 4/25/24 12:24, Warner Losh wrote: > >>>> On Thu, Apr 25, 2024, 12:57=E2=80=AFPM Colin Percival <cperciva@tars= nap.com > >>>> <mailto:cperciva@tarsnap.com>> wrote: > >>>> On 4/19/24 16:11, Warner Losh wrote: > >>>> > add dtb support for RPI CM4, CM4s, CM4_ioBoard > >>>> > > >>>> > Signed-off-by: Klaus K=C3=BCchemann <maciphone2@googlema= il.com > >>>> <mailto:maciphone2@googlemail.com>> > >>>> > Reviewed by: imp > >>>> > Pull Request: > >>>> https://github.com/freebsd/freebsd-src/pull/1182 > >>>> <https://github.com/freebsd/freebsd-src/pull/1182>; > >>>> > --- > >>>> > --- a/release/arm64/RPI.conf > >>>> > +++ b/release/arm64/RPI.conf > >>>> > -DTB=3D"bcm2710-rpi-2-b.dtb bcm2710-rpi-3-b.dtb > >>>> bcm2710-rpi-3-b-plus.dtb > >>>> bcm2710-rpi-cm3.dtb bcm2711-rpi-4-b.dtb" > >>>> > +DTB=3D"bcm2710-rpi-2-b.dtb bcm2710-rpi-3-b.dtb > >>>> bcm2710-rpi-3-b-plus.dtb > >>>> bcm2710-rpi-cm3.dtb \ > >>>> > + bcm2711-rpi-4-b.dtb bcm2711-rpi-cm4-io.dtb > >>>> bcm2711-rpi-cm4.dtb > >>>> bcm2711-rpi-cm4s.dtb" > >>>> > >>>> This broke the RPI release build, since bcm2711-rpi-cm4s.dtb > >>>> doesn't exist. > >>>> Is it provided by a port which we should add to the RPI build? > >>>> > >>>> I thought it was in tree. I'll look into this and report back. > >>> Entirely possible. The release-building code is trying to copy it fr= om > >>> /usr/local/share/rpi-firmware/bcm2711-rpi-cm4s.dtb > >>> so if it's somewhere else we need to adjust that code. > >> > >> The rpi-firmware port is supposed to provide all .dtb used here, but > >> we've never installed that one. It really shouldn't have been > >> included in the list to begin with, but I guess if there's demand we > >> should fix the port to include it. > >> > >> Thanks, > >> > >> Kyle Evans > > > > we only have tested/fixed the CM4-model, not forceably need the > > cm4s.dtb , so no objections to remove cm4s.dtb from the dtb-list at > > least temporarily. > > > > But please hold cm4.dtb and the cm4_io_.dtb, which will then hopefully > > not break the rpi-release anymore. > > > > I'm going to go ahead and remove just the rpi-cm4s.dtb, feel free to > update the port (sysutils/rpi-firmware) to grab it then try again once > new packages are available with it so that we don't break the snapshot > again from the same file. > Thanks. Warner Thanks, > > Kyle Evans > --000000000000fb51510616f312a6 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable <div dir=3D"auto"><div><br><br><div class=3D"gmail_quote"><div dir=3D"ltr" = class=3D"gmail_attr">On Thu, Apr 25, 2024, 4:09=E2=80=AFPM Kyle Evans <<= a href=3D"mailto:kevans@freebsd.org">kevans@freebsd.org</a>> wrote:<br><= /div><blockquote class=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;border-le= ft:1px #ccc solid;padding-left:1ex">On 4/25/24 16:44, Klaus K=C3=BCchemann = wrote:<br> > <br> > <br> >> Am 25.04.2024 um 22:01 schrieb Kyle Evans <kevans@FreeBSD.org&g= t;:<br> >><br> >> On 4/25/24 14:43, Colin Percival wrote:<br> >>> On 4/25/24 12:24, Warner Losh wrote:<br> >>>> On Thu, Apr 25, 2024, 12:57=E2=80=AFPM Colin Percival <= <a href=3D"mailto:cperciva@tarsnap.com" target=3D"_blank" rel=3D"noreferrer= ">cperciva@tarsnap.com</a> <br> >>>> <mailto:<a href=3D"mailto:cperciva@tarsnap.com" target= =3D"_blank" rel=3D"noreferrer">cperciva@tarsnap.com</a>>> wrote:<br> >>>> =C2=A0=C2=A0=C2=A0 On 4/19/24 16:11, Warner Losh wrote:<br= > >>>> =C2=A0=C2=A0=C2=A0=C2=A0 >=C2=A0 =C2=A0 =C2=A0 add dtb = support for RPI CM4, CM4s, CM4_ioBoard<br> >>>> =C2=A0=C2=A0=C2=A0=C2=A0 ><br> >>>> =C2=A0=C2=A0=C2=A0=C2=A0 >=C2=A0 =C2=A0 =C2=A0 Signed-o= ff-by: Klaus K=C3=BCchemann <<a href=3D"mailto:maciphone2@googlemail.com= " target=3D"_blank" rel=3D"noreferrer">maciphone2@googlemail.com</a><br> >>>> =C2=A0=C2=A0=C2=A0 <mailto:<a href=3D"mailto:maciphone2= @googlemail.com" target=3D"_blank" rel=3D"noreferrer">maciphone2@googlemail= .com</a>>><br> >>>> =C2=A0=C2=A0=C2=A0=C2=A0 >=C2=A0 =C2=A0 =C2=A0 Reviewed= by: imp<br> >>>> =C2=A0=C2=A0=C2=A0=C2=A0 >=C2=A0 =C2=A0 =C2=A0 Pull Req= uest: <br> >>>> <a href=3D"https://github.com/freebsd/freebsd-src/pull/118= 2" rel=3D"noreferrer noreferrer" target=3D"_blank">https://github.com/freeb= sd/freebsd-src/pull/1182</a><br> >>>> =C2=A0=C2=A0=C2=A0 <<a href=3D"https://github.com/freeb= sd/freebsd-src/pull/1182" rel=3D"noreferrer noreferrer" target=3D"_blank">h= ttps://github.com/freebsd/freebsd-src/pull/1182</a>><br> >>>> =C2=A0=C2=A0=C2=A0=C2=A0 > ---<br> >>>> =C2=A0=C2=A0=C2=A0=C2=A0 > --- a/release/arm64/RPI.conf= <br> >>>> =C2=A0=C2=A0=C2=A0=C2=A0 > +++ b/release/arm64/RPI.conf= <br> >>>> =C2=A0=C2=A0=C2=A0=C2=A0 > -DTB=3D"bcm2710-rpi-2-b= .dtb bcm2710-rpi-3-b.dtb <br> >>>> bcm2710-rpi-3-b-plus.dtb<br> >>>> =C2=A0=C2=A0=C2=A0 bcm2710-rpi-cm3.dtb bcm2711-rpi-4-b.dtb= "<br> >>>> =C2=A0=C2=A0=C2=A0=C2=A0 > +DTB=3D"bcm2710-rpi-2-b= .dtb bcm2710-rpi-3-b.dtb <br> >>>> bcm2710-rpi-3-b-plus.dtb<br> >>>> =C2=A0=C2=A0=C2=A0 bcm2710-rpi-cm3.dtb \<br> >>>> =C2=A0=C2=A0=C2=A0=C2=A0 > +=C2=A0 =C2=A0 =C2=A0bcm2711= -rpi-4-b.dtb bcm2711-rpi-cm4-io.dtb <br> >>>> bcm2711-rpi-cm4.dtb<br> >>>> =C2=A0=C2=A0=C2=A0 bcm2711-rpi-cm4s.dtb"<br> >>>><br> >>>> =C2=A0=C2=A0=C2=A0 This broke the RPI release build, since= bcm2711-rpi-cm4s.dtb <br> >>>> doesn't exist.<br> >>>> =C2=A0=C2=A0=C2=A0 Is it provided by a port which we shoul= d add to the RPI build?<br> >>>><br> >>>> I thought it was in tree. I'll look into this and repo= rt back.<br> >>> Entirely possible.=C2=A0 The release-building code is trying t= o copy it from<br> >>> /usr/local/share/rpi-firmware/bcm2711-rpi-cm4s.dtb<br> >>> so if it's somewhere else we need to adjust that code.<br> >><br> >> The rpi-firmware port is supposed to provide all .dtb used here, b= ut <br> >> we've never installed that one.=C2=A0 It really shouldn't = have been <br> >> included in the list to begin with, but I guess if there's dem= and we <br> >> should fix the port to include it.<br> >><br> >> Thanks,<br> >><br> >> Kyle Evans<br> > <br> > we only have tested/fixed the CM4-model, =C2=A0not forceably need the = <br> > cm4s.dtb , so no objections to remove cm4s.dtb from the dtb-list at <b= r> > least temporarily.<br> > <br> > But please hold cm4.dtb and the cm4_io_.dtb, which will then hopefully= <br> > not break the rpi-release anymore.<br> > <br> <br> I'm going to go ahead and remove just the rpi-cm4s.dtb, feel free to <b= r> update the port (sysutils/rpi-firmware) to grab it then try again once <br> new packages are available with it so that we don't break the snapshot = <br> again from the same file.<br></blockquote></div></div><div dir=3D"auto"><br= ></div><div dir=3D"auto">Thanks.</div><div dir=3D"auto"><br></div><div dir= =3D"auto">Warner=C2=A0</div><div dir=3D"auto"><br></div><div dir=3D"auto"><= div class=3D"gmail_quote"><blockquote class=3D"gmail_quote" style=3D"margin= :0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> Thanks,<br> <br> Kyle Evans<br> </blockquote></div></div></div> --000000000000fb51510616f312a6--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfr9_AATSeju-h3h2kMgUvcreTNVfFmMm870bqXGtuFULw>