diff mbox series

[1/1] mfd: lpc_ich: Use ALIGN_DOWN()

Message ID 20231229145059.6138-1-ilpo.jarvinen@linux.intel.com
State New
Headers
Series [1/1] mfd: lpc_ich: Use ALIGN_DOWN() |

Commit Message

Ilpo Järvinen Dec. 29, 2023, 2:50 p.m. UTC 
  Instead of open coding, use ALIGN_DOWN() for alignment.

Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
---
 drivers/mfd/lpc_ich.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Lee Jones Jan. 11, 2024, 10:43 a.m. UTC | #1 
On Fri, 29 Dec 2023 16:50:59 +0200, Ilpo Järvinen wrote:
> Instead of open coding, use ALIGN_DOWN() for alignment.
> 
> 

Applied, thanks!

[1/1] mfd: lpc_ich: Use ALIGN_DOWN()
      commit: 39852f0bbf9f29f2e4946d20c764ed7eb85d4a24

--
Lee Jones [李琼斯]
diff mbox series

Patch

diff --git a/drivers/mfd/lpc_ich.c b/drivers/mfd/lpc_ich.c
index 73a0e7f9bd31..f14901660147 100644
--- a/drivers/mfd/lpc_ich.c
+++ b/drivers/mfd/lpc_ich.c
@@ -38,6 +38,7 @@ 
 
 #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
 
+#include <linux/align.h>
 #include <linux/kernel.h>
 #include <linux/module.h>
 #include <linux/errno.h>
@@ -1321,7 +1322,7 @@  static int lpc_ich_init_spi(struct pci_dev *dev)
 	case INTEL_SPI_BYT:
 		pci_read_config_dword(dev, SPIBASE_BYT, &spi_base);
 		if (spi_base & SPIBASE_BYT_EN) {
-			res->start = spi_base & ~(SPIBASE_BYT_SZ - 1);
+			res->start = ALIGN_DOWN(spi_base, SPIBASE_BYT_SZ);
 			res->end = res->start + SPIBASE_BYT_SZ - 1;
 
 			info->set_writeable = lpc_ich_byt_set_writeable;