diff mbox series

[v2,26/29] edid-decode: remove extra vendor field

Message ID 20210926083330.5206-27-joevt@shaw.ca (mailing list archive)
State New, archived
Headers show
Series edid-decode: bug fixes, additions, changes | expand

Commit Message

joevt Sept. 26, 2021, 8:33 a.m. UTC
- Since we use the data_block_oui function to get the vendor for the Production Identification Data Block, we don't need to output the vendor OUI or ID separately. data_block_oui is better because it will get the OUI's name (if it is known) and it verifies that the ID is valid ASCII (don't want to print weird control characters).

Signed-off-by: Joe van Tunen <joevt@shaw.ca>
---
 parse-displayid-block.cpp | 6 ------
 1 file changed, 6 deletions(-)
diff mbox series

Patch

diff --git a/parse-displayid-block.cpp b/parse-displayid-block.cpp
index a38c90e..ba2c15e 100644
--- a/parse-displayid-block.cpp
+++ b/parse-displayid-block.cpp
@@ -81,12 +81,6 @@  void edid_state::parse_displayid_product_id(const unsigned char *x)
 	check_displayid_datablock_revision(x[1]);
 
 	dispid.has_product_identification = true;
-	if (dispid.version >= 0x20) {
-		unsigned oui = (x[3] << 16) | (x[4] << 8) | x[5];
-		printf("    Vendor OUI %s\n", ouitohex(oui).c_str());
-	} else {
-		printf("    Vendor ID: %c%c%c\n", x[3], x[4], x[5]);
-	}
 	printf("    Product Code: %u\n", x[6] | (x[7] << 8));
 	unsigned sn = x[8] | (x[9] << 8) | (x[10] << 16) | (x[11] << 24);
 	if (sn) {