Commit e9d7dc62 authored by Dave Airlie's avatar Dave Airlie Committed by Alex Deucher

amdgpu/dc: don't check for 0 on register read/writes always.

This adds ~50k to the driver text segment, and 10k to data segment.

   text	   data	    bss	    dec	    hex	filename
2385556	  39681	   1045	2426282	 2505aa	drivers/gpu/drm/amd/amdgpu/amdgpu.o
  text	   data	    bss	    dec	    hex	filename
2336593	  28857	   1045	2366495	 241c1f	drivers/gpu/drm/amd/amdgpu/amdgpu.o
Signed-off-by: default avatarDave Airlie <airlied@redhat.com>
Reviewed-by: default avatarHarry Wentland <harry.wentland@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 5e0adbff
...@@ -96,6 +96,9 @@ irq_handler_idx dm_register_interrupt( ...@@ -96,6 +96,9 @@ irq_handler_idx dm_register_interrupt(
* *
*/ */
/* enable for debugging new code, this adds 50k to the driver size. */
/* #define DM_CHECK_ADDR_0 */
#define dm_read_reg(ctx, address) \ #define dm_read_reg(ctx, address) \
dm_read_reg_func(ctx, address, __func__) dm_read_reg_func(ctx, address, __func__)
...@@ -105,12 +108,12 @@ static inline uint32_t dm_read_reg_func( ...@@ -105,12 +108,12 @@ static inline uint32_t dm_read_reg_func(
const char *func_name) const char *func_name)
{ {
uint32_t value; uint32_t value;
#ifdef DM_CHECK_ADDR_0
if (address == 0) { if (address == 0) {
DC_ERR("invalid register read; address = 0\n"); DC_ERR("invalid register read; address = 0\n");
return 0; return 0;
} }
#endif
value = cgs_read_register(ctx->cgs_device, address); value = cgs_read_register(ctx->cgs_device, address);
return value; return value;
...@@ -125,10 +128,12 @@ static inline void dm_write_reg_func( ...@@ -125,10 +128,12 @@ static inline void dm_write_reg_func(
uint32_t value, uint32_t value,
const char *func_name) const char *func_name)
{ {
#ifdef DM_CHECK_ADDR_0
if (address == 0) { if (address == 0) {
DC_ERR("invalid register write. address = 0"); DC_ERR("invalid register write. address = 0");
return; return;
} }
#endif
cgs_write_register(ctx->cgs_device, address, value); cgs_write_register(ctx->cgs_device, address, value);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment