include/device/device.h: Remove CHIP_NAME() macro

Macros can be confusing on their own; hiding commas make things worse.
This can sometimes be downright misleading. A "good" example would be
the code in soc/intel/xeon_sp/spr/chip.c:

CHIP_NAME("Intel SapphireRapids-SP").enable_dev = chip_enable_dev,

This appears as CHIP_NAME() being some struct when in fact these are
defining 2 separate members of the same struct.

It was decided to remove this macro altogether, as it does not do
anything special and incurs a maintenance burden.

Change-Id: Iaed6dfb144bddcf5c43634b0c955c19afce388f0
Signed-off-by: Nicholas Sudsgaard <devel+coreboot@nsudsgaard.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/80239
Reviewed-by: Yidi Lin <yidilin@google.com>
Reviewed-by: Felix Singer <service+coreboot-gerrit@felixsinger.de>
Reviewed-by: Jakub Czapiga <czapiga@google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Matt DeVillier <matt.devillier@amd.corp-partner.google.com>

6 files changed, 6 insertions, 6 deletions

diff --git a/src/drivers/soundwire/alc1308/alc1308.c b/src/drivers/soundwire/alc1308/alc1308.c
index f16d3a2de5..5449fe8648 100644
--- a/src/drivers/soundwire/alc1308/alc1308.c
+++ b/src/drivers/soundwire/alc1308/alc1308.c
@@ -146,6 +146,6 @@ static void soundwire_alc1308_enable(struct device *dev)
 }
 
 struct chip_operations drivers_soundwire_alc1308_ops = {
-	CHIP_NAME("Realtek ALC1308 SoundWire Codec")
+	.name = "Realtek ALC1308 SoundWire Codec",
 	.enable_dev = soundwire_alc1308_enable
 };
diff --git a/src/drivers/soundwire/alc5682/alc5682.c b/src/drivers/soundwire/alc5682/alc5682.c
index c1aaff6cb7..0d8a3ab061 100644
--- a/src/drivers/soundwire/alc5682/alc5682.c
+++ b/src/drivers/soundwire/alc5682/alc5682.c
@@ -174,6 +174,6 @@ static void soundwire_alc5682_enable(struct device *dev)
 }
 
 struct chip_operations drivers_soundwire_alc5682_ops = {
-	CHIP_NAME("Realtek ALC5682 SoundWire Codec")
+	.name = "Realtek ALC5682 SoundWire Codec",
 	.enable_dev = soundwire_alc5682_enable
 };
diff --git a/src/drivers/soundwire/alc711/alc711.c b/src/drivers/soundwire/alc711/alc711.c
index 34d55c78e4..cfe28ade0c 100644
--- a/src/drivers/soundwire/alc711/alc711.c
+++ b/src/drivers/soundwire/alc711/alc711.c
@@ -151,6 +151,6 @@ static void soundwire_alc711_enable(struct device *dev)
 }
 
 struct chip_operations drivers_soundwire_alc711_ops = {
-	CHIP_NAME("Realtek ALC711 SoundWire Codec")
+	.name = "Realtek ALC711 SoundWire Codec",
 	.enable_dev = soundwire_alc711_enable
 };
diff --git a/src/drivers/soundwire/cs42l42/cs42l42.c b/src/drivers/soundwire/cs42l42/cs42l42.c
index 111844c171..e7c14cc434 100644
--- a/src/drivers/soundwire/cs42l42/cs42l42.c
+++ b/src/drivers/soundwire/cs42l42/cs42l42.c
@@ -226,6 +226,6 @@ static void soundwire_cs42l42_enable(struct device *dev)
 }
 
 struct chip_operations drivers_soundwire_cs42l42_ops = {
-	CHIP_NAME("Cirrus Logic CS42L42 SoundWire Codec")
+	.name = "Cirrus Logic CS42L42 SoundWire Codec",
 	.enable_dev = soundwire_cs42l42_enable
 };
diff --git a/src/drivers/soundwire/max98363/max98363.c b/src/drivers/soundwire/max98363/max98363.c
index 2963cdaa4e..7b90d7834b 100644
--- a/src/drivers/soundwire/max98363/max98363.c
+++ b/src/drivers/soundwire/max98363/max98363.c
@@ -136,6 +136,6 @@ static void soundwire_max98363_enable(struct device *dev)
 }
 
 struct chip_operations drivers_soundwire_max98363_ops = {
-	CHIP_NAME("Maxim MAX98363 SoundWire Codec")
+	.name = "Maxim MAX98363 SoundWire Codec",
 	.enable_dev = soundwire_max98363_enable
 };
diff --git a/src/drivers/soundwire/max98373/max98373.c b/src/drivers/soundwire/max98373/max98373.c
index fb202fe811..e3107c4a4c 100644
--- a/src/drivers/soundwire/max98373/max98373.c
+++ b/src/drivers/soundwire/max98373/max98373.c
@@ -160,6 +160,6 @@ static void soundwire_max98373_enable(struct device *dev)
 }
 
 struct chip_operations drivers_soundwire_max98373_ops = {
-	CHIP_NAME("Maxim MAX98373 SoundWire Codec")
+	.name = "Maxim MAX98373 SoundWire Codec",
 	.enable_dev = soundwire_max98373_enable
 };