Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 28 Sep 2018 10:36:27 -0700
From:      Steve Kargl <sgk@troutmask.apl.washington.edu>
To:        Johannes Lundberg <johalun0@gmail.com>
Cc:        Niclas Zeising <zeising+freebsd@daemonic.se>, x11-list freebsd <freebsd-x11@freebsd.org>
Subject:   Re: fence_wait returned with error -512
Message-ID:  <20180928173627.GA1082@troutmask.apl.washington.edu>
In-Reply-To: <CAECmPwugK_npMO4gOZKvZ6=J=Kq0m4EebupsACjvB%2B39znWxMQ@mail.gmail.com>
References:  <20180927014128.GA14820@troutmask.apl.washington.edu> <daf96fc7-3902-6461-a1b6-554ed24f1518@daemonic.se> <20180927190409.GA27756@troutmask.apl.washington.edu> <b8e08aee-c532-b92c-26d8-0f82f586cb2e@daemonic.se> <20180927201928.GA86166@troutmask.apl.washington.edu> <ed292bb9-5e5a-7de8-1440-7da718e3f6c8@daemonic.se> <20180927211815.GB89730@troutmask.apl.washington.edu> <CAECmPwtt5RGf7uuZcWgD4t_07QqPTb0WZEf6mQJ8aK3XTOFBRw@mail.gmail.com> <20180927233955.GA918@troutmask.apl.washington.edu> <CAECmPwugK_npMO4gOZKvZ6=J=Kq0m4EebupsACjvB%2B39znWxMQ@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, Sep 27, 2018 at 05:48:35PM -0700, Johannes Lundberg wrote:
>
> I pushed a fix for firmware names. Please run git pull and try again.
> 

The semi-obvious diff to git code (see patch following .sig)
allows at least the loading of one module.

% kldstat | grep radeon
11    1 0xffffffff81622000    db810 radeonkms.ko
14    1 0xffffffff81748000     12f7 radeon_CAICOS_pfp_bin.ko

Upon trying to start Xorg, I now see

info: [drm] Loading CAICOS Microcode
error: [drm:pid919:ni_init_microcode] *ERROR* ni_cp: Failed to load firmware "radeon_CAICOS_pfp_bin"
error: [drm:pid919:evergreen_startup] *ERROR* Failed to load firmware!
drmn0: error: disabling GPU acceleration
drmn0: warning: 0xfffff800029ec400 unpin not necessary
drmn0: warning: 0xfffff800029ec400 unpin not necessary
error: [drm:pid919:evergreen_init] *ERROR* radeon: MC ucode required for NI+.
drmn0: error: Fatal error during GPU init
info: [drm] radeon: finishing device.
info: [drm] radeon: ttm finalized
device_attach: drmn0 attach returned 22

-- 
Steve


diff --git a/src/dev/drm2/radeon/ni.c b/src/dev/drm2/radeon/ni.c
index e254c83..09b63e7 100644
--- a/src/dev/drm2/radeon/ni.c
+++ b/src/dev/drm2/radeon/ni.c
@@ -63,23 +63,23 @@ extern int si_rlc_init(struct radeon_device *rdev);
 
 #ifdef __linux__
 /* Firmware Names */
-MODULE_FIRMWARE("radeon/BARTS_pfp.bin");
-MODULE_FIRMWARE("radeon/BARTS_me.bin");
-MODULE_FIRMWARE("radeon/BARTS_mc.bin");
-MODULE_FIRMWARE("radeon/BTC_rlc.bin");
-MODULE_FIRMWARE("radeon/TURKS_pfp.bin");
-MODULE_FIRMWARE("radeon/TURKS_me.bin");
-MODULE_FIRMWARE("radeon/TURKS_mc.bin");
-MODULE_FIRMWARE("radeon/CAICOS_pfp.bin");
-MODULE_FIRMWARE("radeon/CAICOS_me.bin");
-MODULE_FIRMWARE("radeon/CAICOS_mc.bin");
-MODULE_FIRMWARE("radeon/CAYMAN_pfp.bin");
-MODULE_FIRMWARE("radeon/CAYMAN_me.bin");
-MODULE_FIRMWARE("radeon/CAYMAN_mc.bin");
-MODULE_FIRMWARE("radeon/CAYMAN_rlc.bin");
-MODULE_FIRMWARE("radeon/ARUBA_pfp.bin");
-MODULE_FIRMWARE("radeon/ARUBA_me.bin");
-MODULE_FIRMWARE("radeon/ARUBA_rlc.bin");
+MODULE_FIRMWARE("radeon_BARTS_pfp_bin");
+MODULE_FIRMWARE("radeon_BARTS_me_bin");
+MODULE_FIRMWARE("radeon_BARTS_mc_bin");
+MODULE_FIRMWARE("radeon_BTC_rlc_bin");
+MODULE_FIRMWARE("radeon_TURKS_pfp_bin");
+MODULE_FIRMWARE("radeon_TURKS_me_bin");
+MODULE_FIRMWARE("radeon_TURKS_mc_bin");
+MODULE_FIRMWARE("radeon_CAICOS_pfp_bin");
+MODULE_FIRMWARE("radeon_CAICOS_me_bin");
+MODULE_FIRMWARE("radeon_CAICOS_mc_bin");
+MODULE_FIRMWARE("radeon_CAYMAN_pfp_bin");
+MODULE_FIRMWARE("radeon_CAYMAN_me_bin");
+MODULE_FIRMWARE("radeon_CAYMAN_mc_bin");
+MODULE_FIRMWARE("radeon_CAYMAN_rlc_bin");
+MODULE_FIRMWARE("radeon_ARUBA_pfp_bin");
+MODULE_FIRMWARE("radeon_ARUBA_me_bin");
+MODULE_FIRMWARE("radeon_ARUBA_rlc_bin");
 #endif
 
 #define BTC_IO_MC_REGS_SIZE 29
@@ -346,7 +346,7 @@ int ni_init_microcode(struct radeon_device *rdev)
 	DRM_INFO("Loading %s Microcode\n", chip_name);
 	err = 0;
 
-	snprintf(fw_name, sizeof(fw_name), "radeon/%s_pfp.bin", chip_name);
+	snprintf(fw_name, sizeof(fw_name), "radeon_%s_pfp_bin", chip_name);
 	rdev->pfp_fw = firmware_get(fw_name);
 	if (rdev->pfp_fw == NULL) {
 		err = -ENOENT;
@@ -360,7 +360,7 @@ int ni_init_microcode(struct radeon_device *rdev)
 		goto out;
 	}
 
-	snprintf(fw_name, sizeof(fw_name), "radeon/%s_me.bin", chip_name);
+	snprintf(fw_name, sizeof(fw_name), "radeon_%s_me_bin", chip_name);
 	rdev->me_fw = firmware_get(fw_name);
 	if (rdev->me_fw == NULL) {
 		err = -ENOENT;
@@ -373,7 +373,7 @@ int ni_init_microcode(struct radeon_device *rdev)
 		err = -EINVAL;
 	}
 
-	snprintf(fw_name, sizeof(fw_name), "radeon/%s_rlc.bin", rlc_chip_name);
+	snprintf(fw_name, sizeof(fw_name), "radeon_%s_rlc_bin", rlc_chip_name);
 	rdev->rlc_fw = firmware_get(fw_name);
 	if (rdev->rlc_fw == NULL) {
 		err = -ENOENT;
@@ -388,7 +388,7 @@ int ni_init_microcode(struct radeon_device *rdev)
 
 	/* no MC ucode on TN */
 	if (!(rdev->flags & RADEON_IS_IGP)) {
-		snprintf(fw_name, sizeof(fw_name), "radeon/%s_mc.bin", chip_name);
+		snprintf(fw_name, sizeof(fw_name), "radeon_%s_mc_bin", chip_name);
 		rdev->mc_fw = firmware_get(fw_name);
 		if (rdev->mc_fw == NULL) {
 			err = -ENOENT;
diff --git a/src/dev/drm2/radeon/r600.c b/src/dev/drm2/radeon/r600.c
index d9b0322..30a7aca 100644
--- a/src/dev/drm2/radeon/r600.c
+++ b/src/dev/drm2/radeon/r600.c
@@ -52,47 +52,47 @@ __FBSDID("$FreeBSD$");
 
 #ifdef __linux__
 /* Firmware Names */
-MODULE_FIRMWARE("radeon/R600_pfp.bin");
-MODULE_FIRMWARE("radeon/R600_me.bin");
-MODULE_FIRMWARE("radeon/RV610_pfp.bin");
-MODULE_FIRMWARE("radeon/RV610_me.bin");
-MODULE_FIRMWARE("radeon/RV630_pfp.bin");
-MODULE_FIRMWARE("radeon/RV630_me.bin");
-MODULE_FIRMWARE("radeon/RV620_pfp.bin");
-MODULE_FIRMWARE("radeon/RV620_me.bin");
-MODULE_FIRMWARE("radeon/RV635_pfp.bin");
-MODULE_FIRMWARE("radeon/RV635_me.bin");
-MODULE_FIRMWARE("radeon/RV670_pfp.bin");
-MODULE_FIRMWARE("radeon/RV670_me.bin");
-MODULE_FIRMWARE("radeon/RS780_pfp.bin");
-MODULE_FIRMWARE("radeon/RS780_me.bin");
-MODULE_FIRMWARE("radeon/RV770_pfp.bin");
-MODULE_FIRMWARE("radeon/RV770_me.bin");
-MODULE_FIRMWARE("radeon/RV730_pfp.bin");
-MODULE_FIRMWARE("radeon/RV730_me.bin");
-MODULE_FIRMWARE("radeon/RV710_pfp.bin");
-MODULE_FIRMWARE("radeon/RV710_me.bin");
-MODULE_FIRMWARE("radeon/R600_rlc.bin");
-MODULE_FIRMWARE("radeon/R700_rlc.bin");
-MODULE_FIRMWARE("radeon/CEDAR_pfp.bin");
-MODULE_FIRMWARE("radeon/CEDAR_me.bin");
-MODULE_FIRMWARE("radeon/CEDAR_rlc.bin");
-MODULE_FIRMWARE("radeon/REDWOOD_pfp.bin");
-MODULE_FIRMWARE("radeon/REDWOOD_me.bin");
-MODULE_FIRMWARE("radeon/REDWOOD_rlc.bin");
-MODULE_FIRMWARE("radeon/JUNIPER_pfp.bin");
-MODULE_FIRMWARE("radeon/JUNIPER_me.bin");
-MODULE_FIRMWARE("radeon/JUNIPER_rlc.bin");
-MODULE_FIRMWARE("radeon/CYPRESS_pfp.bin");
-MODULE_FIRMWARE("radeon/CYPRESS_me.bin");
-MODULE_FIRMWARE("radeon/CYPRESS_rlc.bin");
-MODULE_FIRMWARE("radeon/PALM_pfp.bin");
-MODULE_FIRMWARE("radeon/PALM_me.bin");
-MODULE_FIRMWARE("radeon/SUMO_rlc.bin");
-MODULE_FIRMWARE("radeon/SUMO_pfp.bin");
-MODULE_FIRMWARE("radeon/SUMO_me.bin");
-MODULE_FIRMWARE("radeon/SUMO2_pfp.bin");
-MODULE_FIRMWARE("radeon/SUMO2_me.bin");
+MODULE_FIRMWARE("radeon_R600_pfp_bin");
+MODULE_FIRMWARE("radeon_R600_me_bin");
+MODULE_FIRMWARE("radeon_RV610_pfp_bin");
+MODULE_FIRMWARE("radeon_RV610_me_bin");
+MODULE_FIRMWARE("radeon_RV630_pfp_bin");
+MODULE_FIRMWARE("radeon_RV630_me_bin");
+MODULE_FIRMWARE("radeon_RV620_pfp_bin");
+MODULE_FIRMWARE("radeon_RV620_me_bin");
+MODULE_FIRMWARE("radeon_RV635_pfp_bin");
+MODULE_FIRMWARE("radeon_RV635_me_bin");
+MODULE_FIRMWARE("radeon_RV670_pfp_bin");
+MODULE_FIRMWARE("radeon_RV670_me_bin");
+MODULE_FIRMWARE("radeon_RS780_pfp_bin");
+MODULE_FIRMWARE("radeon_RS780_me_bin");
+MODULE_FIRMWARE("radeon_RV770_pfp_bin");
+MODULE_FIRMWARE("radeon_RV770_me_bin");
+MODULE_FIRMWARE("radeon_RV730_pfp_bin");
+MODULE_FIRMWARE("radeon_RV730_me_bin");
+MODULE_FIRMWARE("radeon_RV710_pfp_bin");
+MODULE_FIRMWARE("radeon_RV710_me_bin");
+MODULE_FIRMWARE("radeon_R600_rlc_bin");
+MODULE_FIRMWARE("radeon_R700_rlc_bin");
+MODULE_FIRMWARE("radeon_CEDAR_pfp_bin");
+MODULE_FIRMWARE("radeon_CEDAR_me_bin");
+MODULE_FIRMWARE("radeon_CEDAR_rlc_bin");
+MODULE_FIRMWARE("radeon_REDWOOD_pfp_bin");
+MODULE_FIRMWARE("radeon_REDWOOD_me_bin");
+MODULE_FIRMWARE("radeon_REDWOOD_rlc_bin");
+MODULE_FIRMWARE("radeon_JUNIPER_pfp_bin");
+MODULE_FIRMWARE("radeon_JUNIPER_me_bin");
+MODULE_FIRMWARE("radeon_JUNIPER_rlc_bin");
+MODULE_FIRMWARE("radeon_CYPRESS_pfp_bin");
+MODULE_FIRMWARE("radeon_CYPRESS_me_bin");
+MODULE_FIRMWARE("radeon_CYPRESS_rlc_bin");
+MODULE_FIRMWARE("radeon_PALM_pfp_bin");
+MODULE_FIRMWARE("radeon_PALM_me_bin");
+MODULE_FIRMWARE("radeon_SUMO_rlc_bin");
+MODULE_FIRMWARE("radeon_SUMO_pfp_bin");
+MODULE_FIRMWARE("radeon_SUMO_me_bin");
+MODULE_FIRMWARE("radeon_SUMO2_pfp_bin");
+MODULE_FIRMWARE("radeon_SUMO2_me_bin");
 #endif
 
 int r600_debugfs_mc_info_init(struct radeon_device *rdev);
@@ -2066,7 +2066,7 @@ int r600_init_microcode(struct radeon_device *rdev)
 	DRM_INFO("Loading %s Microcode\n", chip_name);
 	err = 0;
 
-	snprintf(fw_name, sizeof(fw_name), "radeon/%s_pfp.bin", chip_name);
+	snprintf(fw_name, sizeof(fw_name), "radeon_%s_pfp_bin", chip_name);
 	rdev->pfp_fw = firmware_get(fw_name);
 	if (rdev->pfp_fw == NULL) {
 		err = -ENOENT;
@@ -2080,7 +2080,7 @@ int r600_init_microcode(struct radeon_device *rdev)
 		goto out;
 	}
 
-	snprintf(fw_name, sizeof(fw_name), "radeon/%s_me.bin", chip_name);
+	snprintf(fw_name, sizeof(fw_name), "radeon_%s_me_bin", chip_name);
 	rdev->me_fw = firmware_get(fw_name);
 	if (rdev->me_fw == NULL) {
 		err = -ENOENT;
@@ -2093,7 +2093,7 @@ int r600_init_microcode(struct radeon_device *rdev)
 		err = -EINVAL;
 	}
 
-	snprintf(fw_name, sizeof(fw_name), "radeon/%s_rlc.bin", rlc_chip_name);
+	snprintf(fw_name, sizeof(fw_name), "radeon_%s_rlc_bin", rlc_chip_name);
 	rdev->rlc_fw = firmware_get(fw_name);
 	if (rdev->rlc_fw == NULL) {
 		err = -ENOENT;
diff --git a/src/dev/drm2/radeon/r600_cp.c b/src/dev/drm2/radeon/r600_cp.c
index 3c6462e..9382cb5 100644
--- a/src/dev/drm2/radeon/r600_cp.c
+++ b/src/dev/drm2/radeon/r600_cp.c
@@ -46,26 +46,26 @@ __FBSDID("$FreeBSD$");
 
 #ifdef __linux__
 /* Firmware Names */
-MODULE_FIRMWARE("radeon/R600_pfp.bin");
-MODULE_FIRMWARE("radeon/R600_me.bin");
-MODULE_FIRMWARE("radeon/RV610_pfp.bin");
-MODULE_FIRMWARE("radeon/RV610_me.bin");
-MODULE_FIRMWARE("radeon/RV630_pfp.bin");
-MODULE_FIRMWARE("radeon/RV630_me.bin");
-MODULE_FIRMWARE("radeon/RV620_pfp.bin");
-MODULE_FIRMWARE("radeon/RV620_me.bin");
-MODULE_FIRMWARE("radeon/RV635_pfp.bin");
-MODULE_FIRMWARE("radeon/RV635_me.bin");
-MODULE_FIRMWARE("radeon/RV670_pfp.bin");
-MODULE_FIRMWARE("radeon/RV670_me.bin");
-MODULE_FIRMWARE("radeon/RS780_pfp.bin");
-MODULE_FIRMWARE("radeon/RS780_me.bin");
-MODULE_FIRMWARE("radeon/RV770_pfp.bin");
-MODULE_FIRMWARE("radeon/RV770_me.bin");
-MODULE_FIRMWARE("radeon/RV730_pfp.bin");
-MODULE_FIRMWARE("radeon/RV730_me.bin");
-MODULE_FIRMWARE("radeon/RV710_pfp.bin");
-MODULE_FIRMWARE("radeon/RV710_me.bin");
+MODULE_FIRMWARE("radeon_R600_pfp.bin");
+MODULE_FIRMWARE("radeon_R600_me.bin");
+MODULE_FIRMWARE("radeon_RV610_pfp.bin");
+MODULE_FIRMWARE("radeon_RV610_me.bin");
+MODULE_FIRMWARE("radeon_RV630_pfp.bin");
+MODULE_FIRMWARE("radeon_RV630_me.bin");
+MODULE_FIRMWARE("radeon_RV620_pfp.bin");
+MODULE_FIRMWARE("radeon_RV620_me.bin");
+MODULE_FIRMWARE("radeon_RV635_pfp.bin");
+MODULE_FIRMWARE("radeon_RV635_me.bin");
+MODULE_FIRMWARE("radeon_RV670_pfp.bin");
+MODULE_FIRMWARE("radeon_RV670_me.bin");
+MODULE_FIRMWARE("radeon_RS780_pfp.bin");
+MODULE_FIRMWARE("radeon_RS780_me.bin");
+MODULE_FIRMWARE("radeon_RV770_pfp.bin");
+MODULE_FIRMWARE("radeon_RV770_me.bin");
+MODULE_FIRMWARE("radeon_RV730_pfp.bin");
+MODULE_FIRMWARE("radeon_RV730_me.bin");
+MODULE_FIRMWARE("radeon_RV710_pfp.bin");
+MODULE_FIRMWARE("radeon_RV710_me.bin");
 #endif
 
 
@@ -359,7 +359,7 @@ static int r600_cp_init_microcode(drm_radeon_private_t *dev_priv)
 	DRM_INFO("Loading %s CP Microcode\n", chip_name);
 	err = 0;
 
-	snprintf(fw_name, sizeof(fw_name), "radeon/%s_pfp.bin", chip_name);
+	snprintf(fw_name, sizeof(fw_name), "radeon_%s_pfp.bin", chip_name);
 	dev_priv->pfp_fw = firmware_get(fw_name);
 	if (dev_priv->pfp_fw == NULL) {
 		err = -ENOENT;
@@ -373,7 +373,7 @@ static int r600_cp_init_microcode(drm_radeon_private_t *dev_priv)
 		goto out;
 	}
 
-	snprintf(fw_name, sizeof(fw_name), "radeon/%s_me.bin", chip_name);
+	snprintf(fw_name, sizeof(fw_name), "radeon_%s_me.bin", chip_name);
 	dev_priv->me_fw = firmware_get(fw_name);
 	if (dev_priv->me_fw == NULL) {
 		err = -ENOENT;
diff --git a/src/dev/drm2/radeon/si.c b/src/dev/drm2/radeon/si.c
index 394b93a..1ac5d78 100644
--- a/src/dev/drm2/radeon/si.c
+++ b/src/dev/drm2/radeon/si.c
@@ -41,21 +41,21 @@ __FBSDID("$FreeBSD$");
 #define SI_MC_UCODE_SIZE 7769
 
 #ifdef __linux__
-MODULE_FIRMWARE("radeon/TAHITI_pfp.bin");
-MODULE_FIRMWARE("radeon/TAHITI_me.bin");
-MODULE_FIRMWARE("radeon/TAHITI_ce.bin");
-MODULE_FIRMWARE("radeon/TAHITI_mc.bin");
-MODULE_FIRMWARE("radeon/TAHITI_rlc.bin");
-MODULE_FIRMWARE("radeon/PITCAIRN_pfp.bin");
-MODULE_FIRMWARE("radeon/PITCAIRN_me.bin");
-MODULE_FIRMWARE("radeon/PITCAIRN_ce.bin");
-MODULE_FIRMWARE("radeon/PITCAIRN_mc.bin");
-MODULE_FIRMWARE("radeon/PITCAIRN_rlc.bin");
-MODULE_FIRMWARE("radeon/VERDE_pfp.bin");
-MODULE_FIRMWARE("radeon/VERDE_me.bin");
-MODULE_FIRMWARE("radeon/VERDE_ce.bin");
-MODULE_FIRMWARE("radeon/VERDE_mc.bin");
-MODULE_FIRMWARE("radeon/VERDE_rlc.bin");
+MODULE_FIRMWARE("radeon_TAHITI_pfp_bin");
+MODULE_FIRMWARE("radeon_TAHITI_me_bin");
+MODULE_FIRMWARE("radeon_TAHITI_ce_bin");
+MODULE_FIRMWARE("radeon_TAHITI_mc_bin");
+MODULE_FIRMWARE("radeon_TAHITI_rlc_bin");
+MODULE_FIRMWARE("radeon_PITCAIRN_pfp_bin");
+MODULE_FIRMWARE("radeon_PITCAIRN_me_bin");
+MODULE_FIRMWARE("radeon_PITCAIRN_ce_bin");
+MODULE_FIRMWARE("radeon_PITCAIRN_mc_bin");
+MODULE_FIRMWARE("radeon_PITCAIRN_rlc_bin");
+MODULE_FIRMWARE("radeon_VERDE_pfp_bin");
+MODULE_FIRMWARE("radeon_VERDE_me_bin");
+MODULE_FIRMWARE("radeon_VERDE_ce_bin");
+MODULE_FIRMWARE("radeon_VERDE_mc_bin");
+MODULE_FIRMWARE("radeon_VERDE_rlc_bin");
 #endif
 
 #ifdef FREEBSD_WIP /* FreeBSD: to please GCC 4.2. */
@@ -325,7 +325,7 @@ static int si_init_microcode(struct radeon_device *rdev)
 	DRM_INFO("Loading %s Microcode\n", chip_name);
 	err = 0;
 
-	snprintf(fw_name, sizeof(fw_name), "radeon/%s_pfp.bin", chip_name);
+	snprintf(fw_name, sizeof(fw_name), "radeon_%s_pfp_bin", chip_name);
 	rdev->pfp_fw = drm_firmware_get(fw_name, rdev->dev);
 	if (rdev->pfp_fw == NULL) {
 		err = -ENOENT;
@@ -339,7 +339,7 @@ static int si_init_microcode(struct radeon_device *rdev)
 		goto out;
 	}
 
-	snprintf(fw_name, sizeof(fw_name), "radeon/%s_me.bin", chip_name);
+	snprintf(fw_name, sizeof(fw_name), "radeon_%s_me_bin", chip_name);
 	rdev->me_fw = drm_firmware_get(fw_name, rdev->dev);
 	if (rdev->me_fw == NULL) {
 		err = -ENOENT;
@@ -352,7 +352,7 @@ static int si_init_microcode(struct radeon_device *rdev)
 		err = -EINVAL;
 	}
 
-	snprintf(fw_name, sizeof(fw_name), "radeon/%s_ce.bin", chip_name);
+	snprintf(fw_name, sizeof(fw_name), "radeon_%s_ce_bin", chip_name);
 	rdev->ce_fw = drm_firmware_get(fw_name, rdev->dev);
 	if (rdev->ce_fw == NULL) {
 		err = -ENOENT;
@@ -365,7 +365,7 @@ static int si_init_microcode(struct radeon_device *rdev)
 		err = -EINVAL;
 	}
 
-	snprintf(fw_name, sizeof(fw_name), "radeon/%s_rlc.bin", rlc_chip_name);
+	snprintf(fw_name, sizeof(fw_name), "radeon_%s_rlc_bin", rlc_chip_name);
 	rdev->rlc_fw = drm_firmware_get(fw_name, rdev->dev);
 	if (rdev->rlc_fw == NULL) {
 		err = -ENOENT;
@@ -378,7 +378,7 @@ static int si_init_microcode(struct radeon_device *rdev)
 		err = -EINVAL;
 	}
 
-	snprintf(fw_name, sizeof(fw_name), "radeon/%s_mc.bin", chip_name);
+	snprintf(fw_name, sizeof(fw_name), "radeon_%s_mc_bin", chip_name);
 	rdev->mc_fw = drm_firmware_get(fw_name, rdev->dev);
 	if (rdev->mc_fw == NULL) {
 		err = -ENOENT;



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20180928173627.GA1082>