Use ipname.regname format for --lookup

index 17c99b4296a4..0a44c4399bb8 100644
@@ -502,7 +502,8 @@ if (strcmp(argv[i], "--vm-write") && strcmp(argv[i], "-vw")) {
 "\n\t--enumerate, -e\n\t\tEnumerate all AMDGPU devices detected.\n"
 "\n\t--list-blocks, -lb\n\t\tList IP blocks discovered for this device.\n"
 "\n\t--list-regs, -lr <string>\n\t\tList registers for a given IP block (can use '-O bits' to list bitfields).\n"
-"\n\t--lookup, -lu <address_or_regname> <value>\n\t\tLook up bit decoding of an MMIO register by address (with 0x prefix) or by register name.\n"
+"\n\t--lookup, -lu <address_or_regname> <value>\n\t\tLook up bit decoding of an MMIO register by address (with 0x prefix) or by register name."
+	"\n\t\tThe register name string must include the ipname, e.g., uvd6.mmUVD_CONTEXT_ID.\n"
 "\n\t--write, -w <address> <number>\n\t\tWrite a value in hex to a register specified as a register path in the"
 	"\n\t\tform <asicname.ipname.regname>.  For instance \"tonga.uvd5.mmUVD_SOFT_RESET\"."
 	"\n\t\tCan be used multiple times to set multiple registers.  You can"

index d07130ab0dfa..f88d0b6bce77 100644
@@ -47,16 +47,31 @@ void umr_lookup(struct umr_asic *asic, char *address, char *value)
 				}
 			}
 	} else {
+		char ipname[256], regname[256], *p;
+
+		memset(ipname, 0, sizeof ipname);
+		memset(regname, 0, sizeof regname);
+		p = strstr(address, ".");
+		if (!p) {
+			fprintf(stderr, "[ERROR]: Must specify ipname.regname for umr_lookup()\n");
+			return;
+		}
+		memcpy(ipname, address, p - address);
+		strcpy(regname, p + 1);
+
 		for (i = 0; i < asic->no_blocks; i++)
-		for (j = 0; j < asic->blocks[i]->no_regs; j++)
-			if (asic->blocks[i]->regs[j].type == REG_MMIO &&
-			    !strcmp(asic->blocks[i]->regs[j].regname, address)) {
-				printf("%s.%s => 0x%08lx\n", asic->blocks[i]->ipname, asic->blocks[i]->regs[j].regname, (unsigned long)num);
-				for (k = 0; k < asic->blocks[i]->regs[j].no_bits; k++) {
-					uint32_t v;
-					v = (1UL << (asic->blocks[i]->regs[j].bits[k].stop + 1 - asic->blocks[i]->regs[j].bits[k].start)) - 1;
-					v &= (num >> asic->blocks[i]->regs[j].bits[k].start);
-					asic->blocks[i]->regs[j].bits[k].bitfield_print(asic, asic->asicname, asic->blocks[i]->ipname, asic->blocks[i]->regs[j].regname, asic->blocks[i]->regs[j].bits[k].regname, asic->blocks[i]->regs[j].bits[k].start, asic->blocks[i]->regs[j].bits[k].stop, v);
+			if (!strcmp(asic->blocks[i]->ipname, ipname)) {
+				for (j = 0; j < asic->blocks[i]->no_regs; j++) {
+					if (asic->blocks[i]->regs[j].type == REG_MMIO &&
+					    !strcmp(asic->blocks[i]->regs[j].regname, regname)) {
+						printf("%s.%s => 0x%08lx\n", asic->blocks[i]->ipname, asic->blocks[i]->regs[j].regname, (unsigned long)num);
+						for (k = 0; k < asic->blocks[i]->regs[j].no_bits; k++) {
+							uint32_t v;
+							v = (1UL << (asic->blocks[i]->regs[j].bits[k].stop + 1 - asic->blocks[i]->regs[j].bits[k].start)) - 1;
+							v &= (num >> asic->blocks[i]->regs[j].bits[k].start);
+							asic->blocks[i]->regs[j].bits[k].bitfield_print(asic, asic->asicname, asic->blocks[i]->ipname, asic->blocks[i]->regs[j].regname, asic->blocks[i]->regs[j].bits[k].regname, asic->blocks[i]->regs[j].bits[k].start, asic->blocks[i]->regs[j].bits[k].stop, v);
+						}
+					}
 				}
 			}
 	}