Looks good, but...
On 01/31/2011 04:50 PM, ext Kishon Vijay Abraham I wrote:
> Modify OMAP2+ McBSP to use omap hwmod framework APIs
>
> Signed-off-by: Kishon Vijay Abraham I <[email protected]>
> Signed-off-by: Charulatha V <[email protected]>
> Signed-off-by: Shubhrajyoti D <[email protected]>
> ---
> arch/arm/mach-omap2/mcbsp.c | 685
> +++----------------------------------------
> 1 files changed, 43 insertions(+), 642 deletions(-)
...
> + if (oh->class->rev == MCBSP_CONFIG_TYPE3) {
> + if (id == 2)
> + pdata->buffer_size = 0x500; /*FIFO size is 1024 + 256*/
> + else
> + pdata->buffer_size = 0x80; /*FIFO size is 128*/
> + }
I would add spaces around these comments (after /* and before */).
--
Péter
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
