Hi Justin, On Mon, Jul 23, 2012 at 8:49 AM, Justin P. Mattock <justinmattock@xxxxxxxxx> wrote: > diff --git a/drivers/staging/tidspbridge/core/tiomap3430.c b/drivers/staging/tidspbridge/core/tiomap3430.c > index f9609ce..2c82d5a 100644 > --- a/drivers/staging/tidspbridge/core/tiomap3430.c > +++ b/drivers/staging/tidspbridge/core/tiomap3430.c > @@ -328,7 +328,7 @@ static int bridge_brd_read(struct bridge_dev_context *dev_ctxt, > ul_num_bytes, mem_type); > return status; > } > - /* copy the data from DSP memory, */ > + /* copy the data from DSP memory, */ I guess we can get rid of the comma (,) at the end of this sentence. ... > diff --git a/drivers/staging/tidspbridge/include/dspbridge/proc.h b/drivers/staging/tidspbridge/include/dspbridge/proc.h > index a82380e..9cd5022 100644 > --- a/drivers/staging/tidspbridge/include/dspbridge/proc.h > +++ b/drivers/staging/tidspbridge/include/dspbridge/proc.h > @@ -263,7 +263,7 @@ extern int proc_get_processor_id(void *proc, u32 * proc_id); > * Returns: > * 0 : Success. > * -EFAULT : Invalid processor handle. > - * -EPERM : General failure while retireving processor trace > + * -EPERM : General failure while retireing processor trace This was meant to be 'retrieving'. And given that this is focused on tidspbridge, could you change the patch subject? To: staging: tidspbridge: fix typos. Cheers, Omar _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel