[U-Boot] [PATCH 02/11] dm: spi: Correct minor nits in ICH driver

[Bin Meng]

On Sun, Jun 7, 2015 at 10:50 PM, Simon Glass <sjg at chromium.org> wrote:
> Tidy up three minor problems in this file.
>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
>
>  drivers/spi/ich.c | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/spi/ich.c b/drivers/spi/ich.c
> index 50354fd..6b6cfbf 100644
> --- a/drivers/spi/ich.c
> +++ b/drivers/spi/ich.c
> @@ -422,7 +422,7 @@ static int ich_spi_xfer(struct udevice *dev, unsigned int bitlen,
>         int using_cmd = 0;
>         int ret;
>
> -       /* Ee don't support writing partial bytes. */
> +       /* We don't support writing partial bytes */
>         if (bitlen % 8) {
>                 debug("ICH SPI: Accessing partial bytes not supported\n");
>                 return -EPROTONOSUPPORT;
> @@ -601,7 +601,7 @@ static int ich_spi_xfer(struct udevice *dev, unsigned int bitlen,
>                         return status;
>
>                 if (status & SPIS_FCERR) {
> -                       debug("ICH SPI: Data transaction error\n");
> +                       debug("ICH SPI: Data transaction error %x\n", status);
>                         return -EIO;
>                 }
>
> @@ -619,7 +619,6 @@ static int ich_spi_xfer(struct udevice *dev, unsigned int bitlen,
>         return 0;
>  }
>
> -
>  /*
>   * This uses the SPI controller from the Intel Cougar Point and Panther Point
>   * PCH to write-protect portions of the SPI flash until reboot. The changes
> --

Reviewed-by: Bin Meng <bmeng.cn at gmail.com>