Marek Vasut
cd3c5896b6
imx: i2c: Zap unnecessary malloc() calls
...
The malloc() calls are unnecessary, just allocate the stuff on stack.
While at it, reorder the code a little, so that only one variable is
used for the text, use snprintf() instead of sprintf() and use %01d
as a formatting string to avoid any possible overflows.
Signed-off-by: Marek Vasut <marex@denx.de >
Cc: Igor Grinberg <grinberg@compulab.co.il >
Cc: Nikita Kiryanov <nikita@compulab.co.il >
Cc: Sean Cross <xobs@kosagi.com >
Cc: Simon Glass <sjg@chromium.org >
Cc: Stefano Babic <sbabic@denx.de >
Cc: Tim Harvey <tharvey@gateworks.com >
Reviewed-by: Christian Gmeiner <christian.gmeiner@gmail.com >
2014-12-30 14:05:48 +01:00
Simon Glass
c6f3f32356
dm: imx: i2c: Use gpio_request() to request GPIOs
...
GPIOs should be requested before use. Without this, driver model will
not permit the GPIO to be used.
Cc: Igor Grinberg <grinberg@compulab.co.il >
Signed-off-by: Simon Glass <sjg@chromium.org >
Signed-off-by: Nikita Kiryanov <nikita@compulab.co.il >
Acked-by: Igor Grinberg <grinberg@compulab.co.il >
2014-10-22 10:36:55 -06:00
Simon Glass
edbf8b4f8c
imx: Add error checking to setup_i2c()
...
Since this function can fail, check its return value.
Signed-off-by: Simon Glass <sjg@chromium.org >
Tested-by: Nikita Kiryanov <nikita@compulab.co.il >
2014-10-22 10:36:55 -06:00
Wolfgang Denk
1a4596601f
Add GPL-2.0+ SPDX-License-Identifier to source files
...
Signed-off-by: Wolfgang Denk <wd@denx.de >
[trini: Fixup common/cmd_io.c]
Signed-off-by: Tom Rini <trini@ti.com >
2013-07-24 09:44:38 -04:00
Stefano Babic
ea00e59be0
MX: set a common place to share code for Freescale i.MX
...
Up now only MX5 and MX6 can share code, because they have
a common source directory in cpu/armv7. Other not armv7
i.MX can profit of the same shared code. Move these files
into a directory accessible for all, similar to plat-mxc
in linux.
Signed-off-by: Stefano Babic <sbabic@denx.de >
2012-09-10 14:24:29 +02:00