Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

soundwire: slave: simplify code with acpi_get_local_u64_address()

Now we have a helper so there's no need to open-code.

Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: Péter Ujfalusi <peter.ujfalusi@linux.intel.com>
Reviewed-by: Bard Liao <yung-chuan.liao@linux.intel.com>
Acked-by: Vinod Koul <vkoul@kernel.org>
Reviewed-by: Takashi Iwai <tiwai@suse.de>
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://lore.kernel.org/r/20240528192936.16180-3-pierre-louis.bossart@linux.intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>

authored by

Pierre-Louis Bossart and committed by
Mark Brown
b6212f9b 0b7e4481

+4 -9
+4 -9
drivers/soundwire/slave.c
··· 97 97 struct acpi_device *adev, 98 98 struct sdw_slave_id *id) 99 99 { 100 - u64 addr; 101 100 unsigned int link_id; 102 - acpi_status status; 101 + u64 addr; 102 + int ret; 103 103 104 - status = acpi_evaluate_integer(adev->handle, 105 - METHOD_NAME__ADR, NULL, &addr); 106 - 107 - if (ACPI_FAILURE(status)) { 108 - dev_err(bus->dev, "_ADR resolution failed: %x\n", 109 - status); 104 + ret = acpi_get_local_u64_address(adev->handle, &addr); 105 + if (ret < 0) 110 106 return false; 111 - } 112 107 113 108 if (bus->ops->override_adr) 114 109 addr = bus->ops->override_adr(bus, addr);