Hi,

On Thu, Mar 15, 2012 at 08:03:35PM +0530, Venkatraman S wrote:
> From: Balaji T K <balaj...@ti.com>
> @@ -766,6 +769,8 @@ omap_hsmmc_start_command(struct omap_hsmmc_host *host, 
> struct mmc_command *cmd,
>               cmdtype = 0x3;
>  
>       cmdreg = (cmd->opcode << 24) | (resptype << 16) | (cmdtype << 22);
> +     if ((host->flags & AUTO_CMD12) && mmc_op_multi(cmd->opcode))

This should have braces too.

> @@ -837,11 +842,16 @@ omap_hsmmc_xfer_done(struct omap_hsmmc_host *host, 
> struct mmc_data *data)
>       else
>               data->bytes_xfered = 0;
>  
> -     if (!data->stop) {
> +     if (data->stop && ((!(host->flags & AUTO_CMD12)) || data->error))
> +             omap_hsmmc_start_command(host, data->stop, NULL);
> +     else {
> +             if (data->stop)
> +                     data->stop->resp[0] = OMAP_HSMMC_READ(host->base,
> +                                                     RSP76);
>               omap_hsmmc_request_done(host, data->mrq);
> -             return;
>       }
> -     omap_hsmmc_start_command(host, data->stop, NULL);
> +
> +     return;

return is unnecessary.

-- 
balbi

Attachment: signature.asc
Description: Digital signature

Reply via email to