From e4fa41e2adf052bc04016ba62e35b08f70dc3ccf Mon Sep 17 00:00:00 2001 From: Henrik Brix Andersen Date: Mon, 26 May 2025 10:39:11 +0000 Subject: [PATCH] drivers: can: mcp2515: merge nested if-statements into one Merge two nested if-statements into one. Signed-off-by: Henrik Brix Andersen --- drivers/can/can_mcp2515.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/drivers/can/can_mcp2515.c b/drivers/can/can_mcp2515.c index 8b3e7808242..a3b8b79452c 100644 --- a/drivers/can/can_mcp2515.c +++ b/drivers/can/can_mcp2515.c @@ -918,11 +918,9 @@ static int mcp2515_init(const struct device *dev) k_mutex_init(&dev_data->mutex); k_sem_init(&dev_data->tx_sem, MCP2515_TX_CNT, MCP2515_TX_CNT); - if (dev_cfg->common.phy != NULL) { - if (!device_is_ready(dev_cfg->common.phy)) { - LOG_ERR("CAN transceiver not ready"); - return -ENODEV; - } + if (dev_cfg->common.phy != NULL && !device_is_ready(dev_cfg->common.phy)) { + LOG_ERR("CAN transceiver not ready"); + return -ENODEV; } if (!spi_is_ready_dt(&dev_cfg->bus)) {