On Mon, Jul 7, 2014 at 9:46 PM, Simon Glass <s...@chromium.org> wrote:
> An incorrect message version is passed to the EC in some cases and the
> parameters of one function are switched.
>
> Fix these problems.
>
> Signed-off-by: Simon Glass <s...@chromium.org>
> ---
>
> drivers/misc/cros_ec_spi.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/misc/cros_ec_spi.c b/drivers/misc/cros_ec_spi.c
> index 7df709c..015333f 100644
> --- a/drivers/misc/cros_ec_spi.c
> +++ b/drivers/misc/cros_ec_spi.c
> @@ -98,7 +98,7 @@ int cros_ec_spi_command(struct cros_ec_dev *dev, uint8_t
> cmd, int cmd_version,
> }
>
> out = dev->dout;
> - out[0] = cmd_version;
> + out[0] = EC_CMD_VERSION0 + cmd_version;
> out[1] = cmd;
> out[2] = (uint8_t)dout_len;
> memcpy(out + 3, dout, dout_len);
> @@ -165,7 +165,7 @@ int cros_ec_spi_decode_fdt(struct cros_ec_dev *dev, const
> void *blob)
> */
> int cros_ec_spi_init(struct cros_ec_dev *dev, const void *blob)
> {
> - dev->spi = spi_setup_slave_fdt(blob, dev->parent_node, dev->node);
> + dev->spi = spi_setup_slave_fdt(blob, dev->node, dev->parent_node);
> if (!dev->spi) {
> debug("%s: Could not setup SPI slave\n", __func__);
> return -1;
> --
> 2.0.0.526.g5318336
>
> _______________________________________________
> U-Boot mailing list
> U-Boot@lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
Able to test display on peach_pit(which needs tps65090) with these patches.
Also, crosec commands from u-boot prompt works as expected on peach_pit.
Tested-by: Ajay Kumar <ajaykumar...@samsung.com>
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
