Date: Wed, 4 May 2022 16:32:59 +0200 From: Piotr Kubaj <pkubaj@anongoth.pl> To: Emmanuel Vadot <manu@bidouilliste.com> Cc: ports-committers@freebsd.org, dev-commits-ports-all@freebsd.org, dev-commits-ports-main@freebsd.org Subject: Re: git: ecf09fe34600 - main - graphics/gpu-firmware*-kmod: disable amdgpu on powerpc64 Message-ID: <YnKOm%2BJMm4E1AHY%2B@KGPE-D16> In-Reply-To: <20220504152200.5881461e1e927b155884fe4c@bidouilliste.com> References: <202205041257.244CvQYJ029723@gitrepo.freebsd.org> <20220504152200.5881461e1e927b155884fe4c@bidouilliste.com>
next in thread | previous in thread | raw e-mail | index | archive | help
--SEf06/jLY3kNR1jt Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On 22-05-04 15:22:00, Emmanuel Vadot wrote: > On Wed, 4 May 2022 12:57:26 GMT > Piotr Kubaj <pkubaj@FreeBSD.org> wrote: >=20 > > The branch main has been updated by pkubaj: > >=20 > > URL: https://cgit.FreeBSD.org/ports/commit/?id=3Decf09fe3460067b442ba41= a6adb85897c99442b5 > >=20 > > commit ecf09fe3460067b442ba41a6adb85897c99442b5 > > Author: Piotr Kubaj <pkubaj@FreeBSD.org> > > AuthorDate: 2022-05-04 12:54:19 +0000 > > Commit: Piotr Kubaj <pkubaj@FreeBSD.org> > > CommitDate: 2022-05-04 12:54:19 +0000 > >=20 > > graphics/gpu-firmware*-kmod: disable amdgpu on powerpc64 > > =20 > > amdgpu has known endianness issues and doesn't work on powerpc64. > > radeonkms works, while amdgpu works on powerpc64le. > > --- > > graphics/gpu-firmware-amd-kmod/Makefile | 4 +- > > graphics/gpu-firmware-kmod/Makefile | 67 +++++++++++++++++--------= -------- > > 2 files changed, 37 insertions(+), 34 deletions(-) > >=20 >=20 > Can you respect MAINTAINER and open a review next time ? I believe this falls under tier 2 blanket. Correct me if I'm mistaken. > Also why doing that at all ? Because if someone wants to work on > amdgpu on powerpc64 they now need to revert your commit to have the > firmware build. I really don't see a point of doing that. To avoid giving a false impression that this is supported. If anyone works on it, they'll probably use Linux, which has the same issue but you can use upstream DRM code on it. But with big-endian usage fading, and modern POWER supporting little-endian mode, I doubt it will happen, especially considering the size of amdgpu dri= vers. >=20 > --=20 > Emmanuel Vadot <manu@bidouilliste.com> <manu@FreeBSD.org> --SEf06/jLY3kNR1jt Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCgAdFiEEycyIeNkkgohzsoorelmbhSCDnJ0FAmJyjpsACgkQelmbhSCD nJ27mA/9FTqJUsye8hIfoCX/8qgPqLMQsN0HKdTNeOkFgzH7K5QT4FECuKf7DZXQ S+FlJ4cIqaH7bw+k/O2ccZTzyGIVFlD2mOrwHcAuin9zcJ7/dumhSmg3spf8nUGX YHaaqtR8IWZi1aDjjqcFAqdiQCGPn7amRzq1Hp226oOfki2DHdt1hPN5yzWjyped fgAVeLl4kCZ7wuVCd3smSUSGgUEY1YFImd6r2H3dzN1TlM7/RZgggynpCoejGdr1 4rlouzhUyZdGHfLRF+DYJUZoE4JGVZmKtOrHp0aPy8B3+Sk9d8AgmCSBDlgS/hAz FS+f01ETDg0NnFst5AXx+jhAy2jwwu0YczSAEVM+78MSBwKwHHdULC8uJWWM7Ivi VDBojaJaJJG3XRwNZPF+FDQZtqz9aSQZbaO2ha4F0sY2ADXxQTbUfHncQuGSBnaE Gz6+G4LfJ92CnOI1NEAbk7ctkAsFyw5KfDzQSee0KBG0BvqiaqNhH7SNa34rEY6d VxcB3LjxlPR+R6MWtDm8oBONTNBy0Mj+b8OYFk2O6JFhvvtcXVpgrMYzg/6h4vpZ uihL6Gh597zFx0oVEf7U0WGTXB/3EaziLfGVdv5UT3Fwk+5uTEAc7V303GjMk3n8 cnxz9CN1BeTNjmuyW79XX+Z/x327op7eswG3KeeYpsgR1BjSH+I= =9lVZ -----END PGP SIGNATURE----- --SEf06/jLY3kNR1jt--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?YnKOm%2BJMm4E1AHY%2B>