diff --git a/arch/arm/src/lpc17xx_40xx/lpc17_40_i2c.c b/arch/arm/src/lpc17xx_40xx/lpc17_40_i2c.c index 5b65332917628..b64a290dfadd3 100644 --- a/arch/arm/src/lpc17xx_40xx/lpc17_40_i2c.c +++ b/arch/arm/src/lpc17xx_40xx/lpc17_40_i2c.c @@ -528,11 +528,8 @@ struct i2c_master_s *lpc17_40_i2cbus_initialize(int port) { struct lpc17_40_i2cdev_s *priv; - irqstate_t flags; uint32_t regval; - flags = spin_lock_irqsave(&priv->spinlock); - #ifdef CONFIG_LPC17_40_I2C0 if (port == 0) { @@ -621,13 +618,10 @@ struct i2c_master_s *lpc17_40_i2cbus_initialize(int port) else #endif { - spin_unlock_irqrestore(&priv->spinlock, flags); i2cerr("ERROR: LPC I2C Only supports ports 0, 1 and 2\n"); return NULL; } - spin_unlock_irqrestore(&priv->spinlock, flags); - putreg32(I2C_CONSET_I2EN, priv->base + LPC17_40_I2C_CONSET_OFFSET); /* Attach Interrupt Handler */ diff --git a/arch/arm/src/lpc43xx/lpc43_i2c.c b/arch/arm/src/lpc43xx/lpc43_i2c.c index 8d1c8a478ebee..25a0a62e18329 100644 --- a/arch/arm/src/lpc43xx/lpc43_i2c.c +++ b/arch/arm/src/lpc43xx/lpc43_i2c.c @@ -459,11 +459,8 @@ struct i2c_master_s *lpc43_i2cbus_initialize(int port) return NULL; } - irqstate_t flags; uint32_t regval; - flags = spin_lock_irqsave(&priv->spinlock); - #ifdef CONFIG_LPC43_I2C0 if (port == 0) { @@ -523,12 +520,9 @@ struct i2c_master_s *lpc43_i2cbus_initialize(int port) else #endif { - spin_unlock_irqrestore(&priv->spinlock, flags); return NULL; } - spin_unlock_irqrestore(&priv->spinlock, flags); - putreg32(I2C_CONSET_I2EN, priv->base + LPC43_I2C_CONSET_OFFSET); /* Attach Interrupt Handler */ diff --git a/arch/arm/src/lpc54xx/lpc54_i2c_master.c b/arch/arm/src/lpc54xx/lpc54_i2c_master.c index ead84fadfc109..815d9d444ec55 100644 --- a/arch/arm/src/lpc54xx/lpc54_i2c_master.c +++ b/arch/arm/src/lpc54xx/lpc54_i2c_master.c @@ -909,13 +909,10 @@ static int lpc54_i2c_reset(struct i2c_master_s *dev) struct i2c_master_s *lpc54_i2cbus_initialize(int port) { struct lpc54_i2cdev_s *priv; - irqstate_t flags; uint32_t regval; i2cinfo("port=%d\n", port); - flags = spin_lock_irqsave(&priv->spinlock); - /* Configure the requestin I2C peripheral */ /* NOTE: The basic FLEXCOMM initialization was performed in @@ -1263,13 +1260,10 @@ struct i2c_master_s *lpc54_i2cbus_initialize(int port) else #endif { - spin_unlock_irqrestore(&priv->spinlock, flags); i2cerr("ERROR: Unsupported port=%d\n", port); return NULL; } - spin_unlock_irqrestore(&priv->spinlock, flags); - /* Install our operations */ priv->dev.ops = &lpc54_i2c_ops;