drm/amd/powerplay: make fiji command table constant.

Commit Message

Dave Airlie April 11, 2016, 9:11 p.m. UTC

From: Dave Airlie <[e-mail]>

On my otherly hacked up kernel module this does:
1090519	 141169	   1532	1233220	 12d144	drivers/gpu/drm/amd/amdgpu/amdgpu.ko

1213447	  18225	   1532	1233204	 12d134	drivers/gpu/drm/amd/amdgpu/amdgpu.ko

clearly having less data is better.

Signed-off-by: Dave Airlie <[e-mail]>
---
 drivers/gpu/drm/amd/powerplay/inc/fiji_pwrvirus.h  | 2 +-
 drivers/gpu/drm/amd/powerplay/smumgr/fiji_smumgr.c | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

Comments

Dave Airlie April 11, 2016, 9:20 p.m. UTC | #1

On 12 April 2016 at 07:11, Dave Airlie <[e-mail]> wrote:
> From: Dave Airlie <[e-mail]>

Oh ignore these, I see Nils has sent some that do the same and a lot more!

Dave.

Patch

diff --git a/drivers/gpu/drm/amd/powerplay/inc/fiji_pwrvirus.h b/drivers/gpu/drm/amd/powerplay/inc/fiji_pwrvirus.h
index 0262ad3..8a31665 100644
--- a/drivers/gpu/drm/amd/powerplay/inc/fiji_pwrvirus.h
+++ b/drivers/gpu/drm/amd/powerplay/inc/fiji_pwrvirus.h
@@ -46,7 +46,7 @@  struct PWR_Command_Table
 typedef struct PWR_Command_Table PWR_Command_Table;
 
 #define PWR_VIRUS_TABLE_SIZE  10243
-static PWR_Command_Table PwrVirusTable[PWR_VIRUS_TABLE_SIZE] =
+static const PWR_Command_Table PwrVirusTable[PWR_VIRUS_TABLE_SIZE] =
 {
     { PwrCmdWrite, 0x100100b6, mmPCIE_INDEX                               },
     { PwrCmdWrite, 0x00000000, mmPCIE_DATA                                },
diff --git a/drivers/gpu/drm/amd/powerplay/smumgr/fiji_smumgr.c b/drivers/gpu/drm/amd/powerplay/smumgr/fiji_smumgr.c
index cdbb9f8..070f2a2 100644
--- a/drivers/gpu/drm/amd/powerplay/smumgr/fiji_smumgr.c
+++ b/drivers/gpu/drm/amd/powerplay/smumgr/fiji_smumgr.c
@@ -665,7 +665,7 @@  int fiji_setup_pwr_virus(struct pp_smumgr *smumgr)
 {
 	int i, result = -1;
 	uint32_t reg, data;
-	PWR_Command_Table *virus = PwrVirusTable;
+	const PWR_Command_Table *virus = PwrVirusTable;
 	struct fiji_smumgr *priv = (struct fiji_smumgr *)(smumgr->backend);
 
 	priv->avfs.AvfsBtcStatus = AVFS_LOAD_VIRUS;