From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko Schocher Date: Sun, 05 Jul 2015 08:15:29 +0200 Subject: [U-Boot] [PATCH 07/55] exynos: i2c: Correct bug in pinmux selection In-Reply-To: <1435882592-487-8-git-send-email-sjg@chromium.org> References: <1435882592-487-1-git-send-email-sjg@chromium.org> <1435882592-487-8-git-send-email-sjg@chromium.org> Message-ID: <5598CB81.5040404@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hello Simon, Am 03.07.2015 um 02:15 schrieb Simon Glass: > When driver model is not used the current code does not correctly select > the pinmux for the I2C bus. This bug was introduced by this commit: > > 8dfcbaa dm: i2c: s3c24x0: adjust to dm-i2c api > > Signed-off-by: Simon Glass > --- > > drivers/i2c/s3c24x0_i2c.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Acked-by: Heiko Schocher As it is a bugfix, this should go in the current release ... right? bye, Heiko > > diff --git a/drivers/i2c/s3c24x0_i2c.c b/drivers/i2c/s3c24x0_i2c.c > index c053e84..9a04e48 100644 > --- a/drivers/i2c/s3c24x0_i2c.c > +++ b/drivers/i2c/s3c24x0_i2c.c > @@ -1035,7 +1035,7 @@ static void process_nodes(const void *blob, int node_list[], int count, > CONFIG_SYS_I2C_S3C24X0_SPEED); > bus->node = node; > bus->bus_num = i; > - exynos_pinmux_config(PERIPH_ID_I2C0 + bus->id, flags); > + exynos_pinmux_config(bus->id, flags); > > /* Mark position as used */ > node_list[i] = -1; > -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany