[coreboot-gerrit] Change in ...coreboot[master]: drivers: [wip] Remove duplicated '&' on ptr's function

HAOUAS Elyes (Code Review) gerrit at coreboot.org
Tue Nov 27 17:02:55 CET 2018


HAOUAS Elyes has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/29868


Change subject: drivers: [wip] Remove duplicated '&' on ptr's function
......................................................................

drivers: [wip] Remove duplicated '&' on ptr's function

Change-Id: I7a99d0dcbc8ea1362a12a68fa519c49058d30a05
Signed-off-by: Elyes HAOUAS <ehaouas at noos.fr>
---
M src/drivers/generic/adau7002/adau7002.c
M src/drivers/generic/generic/generic.c
M src/drivers/generic/gpio_keys/gpio_keys.c
M src/drivers/generic/gpio_regulator/gpio_regulator.c
M src/drivers/generic/max98357a/max98357a.c
M src/drivers/i2c/da7219/da7219.c
M src/drivers/i2c/generic/generic.c
M src/drivers/i2c/hid/hid.c
M src/drivers/i2c/max98373/max98373.c
M src/drivers/i2c/nau8825/nau8825.c
M src/drivers/i2c/rt5663/rt5663.c
M src/drivers/i2c/sx9310/sx9310.c
M src/drivers/i2c/tpm/chip.c
M src/drivers/intel/mipi_camera/camera.c
14 files changed, 42 insertions(+), 42 deletions(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/68/29868/1

diff --git a/src/drivers/generic/adau7002/adau7002.c b/src/drivers/generic/adau7002/adau7002.c
index 2ea1086..b948d21 100644
--- a/src/drivers/generic/adau7002/adau7002.c
+++ b/src/drivers/generic/adau7002/adau7002.c
@@ -63,8 +63,8 @@
 	.set_resources			= DEVICE_NOOP,
 	.enable_resources		= DEVICE_NOOP,
 #if IS_ENABLED(CONFIG_HAVE_ACPI_TABLES)
-	.acpi_name			= &adau7002_acpi_name,
-	.acpi_fill_ssdt_generator	= &adau7002_fill_ssdt,
+	.acpi_name			= adau7002_acpi_name,
+	.acpi_fill_ssdt_generator	= adau7002_fill_ssdt,
 #endif
 };
 
@@ -75,5 +75,5 @@
 
 struct chip_operations drivers_generic_adau7002_ops = {
 	CHIP_NAME("Analog Digital DMIC")
-	.enable_dev = &adau7002_enable
+	.enable_dev = adau7002_enable
 };
diff --git a/src/drivers/generic/generic/generic.c b/src/drivers/generic/generic/generic.c
index 37fbab5..b5541a3 100644
--- a/src/drivers/generic/generic/generic.c
+++ b/src/drivers/generic/generic/generic.c
@@ -83,8 +83,8 @@
 	.read_resources			= DEVICE_NOOP,
 	.set_resources			= DEVICE_NOOP,
 	.enable_resources		= DEVICE_NOOP,
-	.acpi_name			= &generic_dev_acpi_name,
-	.acpi_fill_ssdt_generator	= &generic_dev_fill_ssdt_generator,
+	.acpi_name			= generic_dev_acpi_name,
+	.acpi_fill_ssdt_generator	= generic_dev_fill_ssdt_generator,
 };
 
 static void generic_dev_enable(struct device *dev)
@@ -94,5 +94,5 @@
 
 struct chip_operations drivers_generic_generic_ops = {
 	CHIP_NAME("Generic Device")
-	.enable_dev = &generic_dev_enable
+	.enable_dev = generic_dev_enable
 };
diff --git a/src/drivers/generic/gpio_keys/gpio_keys.c b/src/drivers/generic/gpio_keys/gpio_keys.c
index 6120981..267a4aa 100644
--- a/src/drivers/generic/gpio_keys/gpio_keys.c
+++ b/src/drivers/generic/gpio_keys/gpio_keys.c
@@ -114,8 +114,8 @@
 	.read_resources			= DEVICE_NOOP,
 	.set_resources			= DEVICE_NOOP,
 	.enable_resources		= DEVICE_NOOP,
-	.acpi_name			= &gpio_keys_acpi_name,
-	.acpi_fill_ssdt_generator	= &gpio_keys_fill_ssdt_generator,
+	.acpi_name			= gpio_keys_acpi_name,
+	.acpi_fill_ssdt_generator	= gpio_keys_fill_ssdt_generator,
 };
 
 static void gpio_keys_enable(struct device *dev)
@@ -125,5 +125,5 @@
 
 struct chip_operations drivers_generic_gpio_keys_ops = {
 	CHIP_NAME("GPIO Keys")
-	.enable_dev = &gpio_keys_enable
+	.enable_dev = gpio_keys_enable
 };
diff --git a/src/drivers/generic/gpio_regulator/gpio_regulator.c b/src/drivers/generic/gpio_regulator/gpio_regulator.c
index c038392..3b33525 100644
--- a/src/drivers/generic/gpio_regulator/gpio_regulator.c
+++ b/src/drivers/generic/gpio_regulator/gpio_regulator.c
@@ -75,8 +75,8 @@
 	.read_resources = DEVICE_NOOP,
 	.set_resources = DEVICE_NOOP,
 	.enable_resources = DEVICE_NOOP,
-	.acpi_name = &gpio_regulator_acpi_name,
-	.acpi_fill_ssdt_generator = &gpio_regulator_fill_ssdt_generator,
+	.acpi_name = gpio_regulator_acpi_name,
+	.acpi_fill_ssdt_generator = gpio_regulator_fill_ssdt_generator,
 };
 
 static void gpio_regulator_enable(struct device *dev)
@@ -86,5 +86,5 @@
 
 struct chip_operations drivers_generic_gpio_regulator_ops = {
 	CHIP_NAME("GPIO Regulator")
-	.enable_dev = &gpio_regulator_enable
+	.enable_dev = gpio_regulator_enable
 };
diff --git a/src/drivers/generic/max98357a/max98357a.c b/src/drivers/generic/max98357a/max98357a.c
index f9b2d6c..8e5ea0c 100644
--- a/src/drivers/generic/max98357a/max98357a.c
+++ b/src/drivers/generic/max98357a/max98357a.c
@@ -77,8 +77,8 @@
 	.set_resources		  = DEVICE_NOOP,
 	.enable_resources	  = DEVICE_NOOP,
 #if IS_ENABLED(CONFIG_HAVE_ACPI_TABLES)
-	.acpi_name                = &max98357a_acpi_name,
-	.acpi_fill_ssdt_generator = &max98357a_fill_ssdt,
+	.acpi_name                = max98357a_acpi_name,
+	.acpi_fill_ssdt_generator = max98357a_fill_ssdt,
 #endif
 };
 
@@ -105,5 +105,5 @@
 
 struct chip_operations drivers_generic_max98357a_ops = {
 	CHIP_NAME("Maxim Integrated 98357A Amplifier")
-	.enable_dev = &max98357a_enable
+	.enable_dev = max98357a_enable
 };
diff --git a/src/drivers/i2c/da7219/da7219.c b/src/drivers/i2c/da7219/da7219.c
index d2a4b78..ebc22e0 100644
--- a/src/drivers/i2c/da7219/da7219.c
+++ b/src/drivers/i2c/da7219/da7219.c
@@ -114,8 +114,8 @@
 	.set_resources		  = DEVICE_NOOP,
 	.enable_resources	  = DEVICE_NOOP,
 #if IS_ENABLED(CONFIG_HAVE_ACPI_TABLES)
-	.acpi_name		  = &da7219_acpi_name,
-	.acpi_fill_ssdt_generator = &da7219_fill_ssdt,
+	.acpi_name		  = da7219_acpi_name,
+	.acpi_fill_ssdt_generator = da7219_fill_ssdt,
 #endif
 };
 
@@ -126,5 +126,5 @@
 
 struct chip_operations drivers_i2c_da7219_ops = {
 	CHIP_NAME("Dialog Semiconductor DA7219 Audio Codec")
-	.enable_dev = &da7219_enable
+	.enable_dev = da7219_enable
 };
diff --git a/src/drivers/i2c/generic/generic.c b/src/drivers/i2c/generic/generic.c
index df929b7..ca2dfaf 100644
--- a/src/drivers/i2c/generic/generic.c
+++ b/src/drivers/i2c/generic/generic.c
@@ -193,8 +193,8 @@
 	.set_resources		  = DEVICE_NOOP,
 	.enable_resources	  = DEVICE_NOOP,
 #if IS_ENABLED(CONFIG_HAVE_ACPI_TABLES)
-	.acpi_name		  = &i2c_generic_acpi_name,
-	.acpi_fill_ssdt_generator = &i2c_generic_fill_ssdt_generator,
+	.acpi_name		  = i2c_generic_acpi_name,
+	.acpi_fill_ssdt_generator = i2c_generic_fill_ssdt_generator,
 #endif
 };
 
@@ -228,5 +228,5 @@
 
 struct chip_operations drivers_i2c_generic_ops = {
 	CHIP_NAME("I2C Device")
-	.enable_dev = &i2c_generic_enable
+	.enable_dev = i2c_generic_enable
 };
diff --git a/src/drivers/i2c/hid/hid.c b/src/drivers/i2c/hid/hid.c
index bccce75..952e3e4 100644
--- a/src/drivers/i2c/hid/hid.c
+++ b/src/drivers/i2c/hid/hid.c
@@ -51,8 +51,8 @@
 	.set_resources		  = DEVICE_NOOP,
 	.enable_resources	  = DEVICE_NOOP,
 #if IS_ENABLED(CONFIG_HAVE_ACPI_TABLES)
-	.acpi_name		  = &i2c_hid_acpi_name,
-	.acpi_fill_ssdt_generator = &i2c_hid_fill_ssdt_generator,
+	.acpi_name		  = i2c_hid_acpi_name,
+	.acpi_fill_ssdt_generator = i2c_hid_fill_ssdt_generator,
 #endif
 };
 
@@ -69,5 +69,5 @@
 
 struct chip_operations drivers_i2c_hid_ops = {
 	CHIP_NAME("I2C HID Device")
-	.enable_dev = &i2c_hid_enable
+	.enable_dev = i2c_hid_enable
 };
diff --git a/src/drivers/i2c/max98373/max98373.c b/src/drivers/i2c/max98373/max98373.c
index e019b70..e10ccf1 100644
--- a/src/drivers/i2c/max98373/max98373.c
+++ b/src/drivers/i2c/max98373/max98373.c
@@ -91,8 +91,8 @@
 	.read_resources		  = DEVICE_NOOP,
 	.set_resources		  = DEVICE_NOOP,
 	.enable_resources	  = DEVICE_NOOP,
-	.acpi_name		  = &max98373_acpi_name,
-	.acpi_fill_ssdt_generator = &max98373_fill_ssdt,
+	.acpi_name		  = max98373_acpi_name,
+	.acpi_fill_ssdt_generator = max98373_fill_ssdt,
 };
 
 static void max98373_enable(struct device *dev)
@@ -108,5 +108,5 @@
 
 struct chip_operations drivers_i2c_max98373_ops = {
 	CHIP_NAME("Maxim MAX98373 Codec")
-	.enable_dev = &max98373_enable
+	.enable_dev = max98373_enable
 };
diff --git a/src/drivers/i2c/nau8825/nau8825.c b/src/drivers/i2c/nau8825/nau8825.c
index 86d4306..3f90512 100644
--- a/src/drivers/i2c/nau8825/nau8825.c
+++ b/src/drivers/i2c/nau8825/nau8825.c
@@ -103,8 +103,8 @@
 	.set_resources		  = DEVICE_NOOP,
 	.enable_resources	  = DEVICE_NOOP,
 #if IS_ENABLED(CONFIG_HAVE_ACPI_TABLES)
-	.acpi_name                = &nau8825_acpi_name,
-	.acpi_fill_ssdt_generator = &nau8825_fill_ssdt,
+	.acpi_name                = nau8825_acpi_name,
+	.acpi_fill_ssdt_generator = nau8825_fill_ssdt,
 #endif
 };
 
@@ -115,5 +115,5 @@
 
 struct chip_operations drivers_i2c_nau8825_ops = {
 	CHIP_NAME("Nuvoton NAU8825 Codec")
-	.enable_dev = &nau8825_enable
+	.enable_dev = nau8825_enable
 };
diff --git a/src/drivers/i2c/rt5663/rt5663.c b/src/drivers/i2c/rt5663/rt5663.c
index 38b2a98..1ce5993 100644
--- a/src/drivers/i2c/rt5663/rt5663.c
+++ b/src/drivers/i2c/rt5663/rt5663.c
@@ -91,8 +91,8 @@
 	.read_resources			= DEVICE_NOOP,
 	.set_resources			= DEVICE_NOOP,
 	.enable_resources		= DEVICE_NOOP,
-	.acpi_name			= &rt5663_acpi_name,
-	.acpi_fill_ssdt_generator	= &rt5663_fill_ssdt,
+	.acpi_name			= rt5663_acpi_name,
+	.acpi_fill_ssdt_generator	= rt5663_fill_ssdt,
 };
 
 static void rt5663_enable(struct device *dev)
@@ -102,5 +102,5 @@
 
 struct chip_operations drivers_i2c_rt5663_ops = {
 	CHIP_NAME("Realtek RT5663 Codec")
-	.enable_dev = &rt5663_enable
+	.enable_dev = rt5663_enable
 };
diff --git a/src/drivers/i2c/sx9310/sx9310.c b/src/drivers/i2c/sx9310/sx9310.c
index 0e98bcb..afd9e35 100644
--- a/src/drivers/i2c/sx9310/sx9310.c
+++ b/src/drivers/i2c/sx9310/sx9310.c
@@ -85,8 +85,8 @@
 	.read_resources		  = DEVICE_NOOP,
 	.set_resources		  = DEVICE_NOOP,
 	.enable_resources	  = DEVICE_NOOP,
-	.acpi_name		  = &i2c_sx9310_acpi_name,
-	.acpi_fill_ssdt_generator = &i2c_sx9310_fill_ssdt,
+	.acpi_name		  = i2c_sx9310_acpi_name,
+	.acpi_fill_ssdt_generator = i2c_sx9310_fill_ssdt,
 };
 
 static void i2c_sx9310_enable(struct device *dev)
@@ -106,5 +106,5 @@
 
 struct chip_operations drivers_i2c_sx9310_ops = {
 	CHIP_NAME(I2C_SX9310_ACPI_NAME)
-	.enable_dev = &i2c_sx9310_enable
+	.enable_dev = i2c_sx9310_enable
 };
diff --git a/src/drivers/i2c/tpm/chip.c b/src/drivers/i2c/tpm/chip.c
index 3dbe811..687ce05 100644
--- a/src/drivers/i2c/tpm/chip.c
+++ b/src/drivers/i2c/tpm/chip.c
@@ -78,8 +78,8 @@
 	.read_resources		  = DEVICE_NOOP,
 	.set_resources		  = DEVICE_NOOP,
 	.enable_resources	  = DEVICE_NOOP,
-	.acpi_name		  = &i2c_tpm_acpi_name,
-	.acpi_fill_ssdt_generator = &i2c_tpm_fill_ssdt,
+	.acpi_name		  = i2c_tpm_acpi_name,
+	.acpi_fill_ssdt_generator = i2c_tpm_fill_ssdt,
 };
 
 static void i2c_tpm_enable(struct device *dev)
@@ -95,5 +95,5 @@
 
 struct chip_operations drivers_i2c_tpm_ops = {
 	CHIP_NAME("I2C TPM")
-	.enable_dev = &i2c_tpm_enable
+	.enable_dev = i2c_tpm_enable
 };
diff --git a/src/drivers/intel/mipi_camera/camera.c b/src/drivers/intel/mipi_camera/camera.c
index cdb9da3..0cea0ba 100644
--- a/src/drivers/intel/mipi_camera/camera.c
+++ b/src/drivers/intel/mipi_camera/camera.c
@@ -84,8 +84,8 @@
 	.read_resources			= DEVICE_NOOP,
 	.set_resources			= DEVICE_NOOP,
 	.enable_resources		= DEVICE_NOOP,
-	.acpi_name			= &camera_acpi_name,
-	.acpi_fill_ssdt_generator	= &camera_fill_ssdt,
+	.acpi_name			= camera_acpi_name,
+	.acpi_fill_ssdt_generator	= camera_fill_ssdt,
 };
 
 static void camera_enable(struct device *dev)
@@ -95,5 +95,5 @@
 
 struct chip_operations drivers_intel_mipi_camera_ops = {
 	CHIP_NAME("Intel MIPI Camera Device")
-	.enable_dev = &camera_enable
+	.enable_dev = camera_enable
 };

-- 
To view, visit https://review.coreboot.org/c/coreboot/+/29868
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I7a99d0dcbc8ea1362a12a68fa519c49058d30a05
Gerrit-Change-Number: 29868
Gerrit-PatchSet: 1
Gerrit-Owner: HAOUAS Elyes <ehaouas at noos.fr>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20181127/05a7aaee/attachment-0001.html>


More information about the coreboot-gerrit mailing list