Date: Thu, 25 Apr 2024 23:44:45 +0200 From: =?utf-8?Q?Klaus_K=C3=BCchemann?= <maciphone2@googlemail.com> To: Kyle Evans <kevans@FreeBSD.org>, cperciva@tarsnap.com, Warner Losh <imp@bsdimp.com>, src-committers@freebsd.org, dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org Subject: Re: git: 1bd4f769caf8 - main - add dtb support for RPI CM4, CM4s, CM4_ioBoard Message-ID: <7FA67865-F391-4773-86BA-9998B3EC855E@googlemail.com> In-Reply-To: <7473b5ed-433b-4d51-8a23-cf6464eb4416@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>
next in thread | previous in thread | raw e-mail | index | archive | help
--Apple-Mail=_022483E1-1624-4E13-9994-F726AF53F022 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=utf-8 > Am 25.04.2024 um 22:01 schrieb Kyle Evans <kevans@FreeBSD.org>: >=20 > 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@tarsnap.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@googlemail.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" >>>=20 >>> 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? >>>=20 >>> 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 = from >> /usr/local/share/rpi-firmware/bcm2711-rpi-cm4s.dtb >> so if it's somewhere else we need to adjust that code. >=20 > 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. >=20 > Thanks, >=20 > 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. =20 But please hold cm4.dtb and the cm4_io_.dtb, which will then hopefully = not break the rpi-release anymore.=20 Of course would be better to upgrade the port. thank you Regards Klaus --Apple-Mail=_022483E1-1624-4E13-9994-F726AF53F022 Content-Transfer-Encoding: quoted-printable Content-Type: text/html; charset=utf-8 <html><head><meta http-equiv=3D"content-type" content=3D"text/html; = charset=3Dutf-8"></head><body style=3D"overflow-wrap: break-word; = -webkit-nbsp-mode: space; line-break: after-white-space;"><br = id=3D"lineBreakAtBeginningOfMessage"><div><br><blockquote = type=3D"cite"><div>Am 25.04.2024 um 22:01 schrieb Kyle Evans = <kevans@FreeBSD.org>:</div><br = class=3D"Apple-interchange-newline"><div><div>On 4/25/24 14:43, Colin = Percival wrote:<br><blockquote type=3D"cite">On 4/25/24 12:24, Warner = Losh wrote:<br><blockquote type=3D"cite">On Thu, Apr 25, 2024, = 12:57=E2=80=AFPM Colin Percival <cperciva@tarsnap.com = <mailto:cperciva@tarsnap.com>> wrote:<br> On = 4/19/24 16:11, Warner Losh wrote:<br> > = add dtb support for RPI CM4, CM4s, = CM4_ioBoard<br> ><br> = > Signed-off-by: Klaus K=C3=BCchemann = <maciphone2@googlemail.com<br> = <mailto:maciphone2@googlemail.com>><br> = > Reviewed by: imp<br> = > Pull Request: = https://github.com/freebsd/freebsd-src/pull/1182<br> = <https://github.com/freebsd/freebsd-src/pull/1182><br> &n= bsp; > ---<br> > --- = a/release/arm64/RPI.conf<br> > +++ = b/release/arm64/RPI.conf<br> > = -DTB=3D"bcm2710-rpi-2-b.dtb bcm2710-rpi-3-b.dtb = bcm2710-rpi-3-b-plus.dtb<br> bcm2710-rpi-cm3.dtb = bcm2711-rpi-4-b.dtb"<br> > = +DTB=3D"bcm2710-rpi-2-b.dtb bcm2710-rpi-3-b.dtb = bcm2710-rpi-3-b-plus.dtb<br> bcm2710-rpi-cm3.dtb = \<br> > + = bcm2711-rpi-4-b.dtb bcm2711-rpi-cm4-io.dtb = bcm2711-rpi-cm4.dtb<br> = bcm2711-rpi-cm4s.dtb"<br><br> This broke the RPI = release build, since bcm2711-rpi-cm4s.dtb doesn't = exist.<br> Is it provided by a port which we should = add to the RPI build?<br><br>I thought it was in tree. I'll look into = this and report back.<br></blockquote>Entirely possible. The = release-building code is trying to 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></blockquote><br>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.<br><br>Thanks,<br><br>Kyle = Evans<br></div></div></blockquote><br></div><div><p style=3D"margin: = 0px; font-style: normal; font-variant-caps: normal; font-stretch: = normal; font-size: 13px; line-height: normal; font-family: = "Helvetica Neue"; font-size-adjust: none; font-kerning: auto; = font-variant-alternates: normal; font-variant-ligatures: normal; = font-variant-numeric: normal; font-variant-east-asian: normal; = font-variant-position: normal; font-variant-emoji: normal; = font-feature-settings: normal; font-optical-sizing: auto; = font-variation-settings: normal;">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. </p><p = style=3D"margin: 0px; font-style: normal; font-variant-caps: normal; = font-stretch: normal; font-size: 13px; line-height: normal; font-family: = "Helvetica Neue"; font-size-adjust: none; font-kerning: auto; = font-variant-alternates: normal; font-variant-ligatures: normal; = font-variant-numeric: normal; font-variant-east-asian: normal; = font-variant-position: normal; font-variant-emoji: normal; = font-feature-settings: normal; font-optical-sizing: auto; = font-variation-settings: normal;">But please hold cm4.dtb and the = cm4_io_.dtb, which will then hopefully not break the rpi-release = anymore. </p><p style=3D"margin: 0px; font-style: normal; = font-variant-caps: normal; font-stretch: normal; font-size: 13px; = line-height: normal; font-family: "Helvetica Neue"; = font-size-adjust: none; font-kerning: auto; font-variant-alternates: = normal; font-variant-ligatures: normal; font-variant-numeric: normal; = font-variant-east-asian: normal; font-variant-position: normal; = font-variant-emoji: normal; font-feature-settings: normal; = font-optical-sizing: auto; font-variation-settings: normal;">Of course = would be better to upgrade the port.</p> <p style=3D"margin: 0px; font-style: normal; font-variant-caps: normal; = font-stretch: normal; font-size: 13px; line-height: normal; font-family: = "Helvetica Neue"; font-size-adjust: none; font-kerning: auto; = font-variant-alternates: normal; font-variant-ligatures: normal; = font-variant-numeric: normal; font-variant-east-asian: normal; = font-variant-position: normal; font-variant-emoji: normal; = font-feature-settings: normal; font-optical-sizing: auto; = font-variation-settings: normal;">thank you</p> <p style=3D"margin: 0px; font-style: normal; font-variant-caps: normal; = font-stretch: normal; font-size: 13px; line-height: normal; font-family: = "Helvetica Neue"; font-size-adjust: none; font-kerning: auto; = font-variant-alternates: normal; font-variant-ligatures: normal; = font-variant-numeric: normal; font-variant-east-asian: normal; = font-variant-position: normal; font-variant-emoji: normal; = font-feature-settings: normal; font-optical-sizing: auto; = font-variation-settings: normal;">Regards</p><p style=3D"margin: 0px; = font-style: normal; font-variant-caps: normal; font-stretch: normal; = font-size: 13px; line-height: normal; font-family: "Helvetica = Neue"; font-size-adjust: none; font-kerning: auto; = font-variant-alternates: normal; font-variant-ligatures: normal; = font-variant-numeric: normal; font-variant-east-asian: normal; = font-variant-position: normal; font-variant-emoji: normal; = font-feature-settings: normal; font-optical-sizing: auto; = font-variation-settings: normal;">Klaus</p></div><br></body></html>= --Apple-Mail=_022483E1-1624-4E13-9994-F726AF53F022--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?7FA67865-F391-4773-86BA-9998B3EC855E>