Skip to content

Commit e4fa41e

Browse files
henrikbrixandersenkartben
authored andcommitted
drivers: can: mcp2515: merge nested if-statements into one
Merge two nested if-statements into one. Signed-off-by: Henrik Brix Andersen <[email protected]>
1 parent c4247c6 commit e4fa41e

File tree

1 file changed

+3
-5
lines changed

1 file changed

+3
-5
lines changed

drivers/can/can_mcp2515.c

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -918,11 +918,9 @@ static int mcp2515_init(const struct device *dev)
918918
k_mutex_init(&dev_data->mutex);
919919
k_sem_init(&dev_data->tx_sem, MCP2515_TX_CNT, MCP2515_TX_CNT);
920920

921-
if (dev_cfg->common.phy != NULL) {
922-
if (!device_is_ready(dev_cfg->common.phy)) {
923-
LOG_ERR("CAN transceiver not ready");
924-
return -ENODEV;
925-
}
921+
if (dev_cfg->common.phy != NULL && !device_is_ready(dev_cfg->common.phy)) {
922+
LOG_ERR("CAN transceiver not ready");
923+
return -ENODEV;
926924
}
927925

928926
if (!spi_is_ready_dt(&dev_cfg->bus)) {

0 commit comments

Comments
 (0)