Hi Manju,
On Wed, Dec 21, 2011 at 19:13:37, Hadli, Manjunath wrote:
> Replace hard coded value of vpss register base to a define macro
> DM644X_VPSS_BASE for proper readability
>
> Signed-off-by: Manjunath Hadli <[email protected]>
> Acked-by: Sekhar Nori <[email protected]>
The subject line does not follow the format we have in place now.
Also in the subject, the term "defined macro" is odd. You have
fixed some indentation issues with earlier code in this patch.
That should find a mention too. I made these (and some other minor)
improvements to the patch text while committing.
Please have a look and take care next time onward. Here is the
new text:
ARM: davinci: dm644x: improve readability using macro
Replace the VPSS base address "magic number" with the
define "DM644X_VPSS_BASE" to improve readability.
While at it, fix the indentation done using spaces in
this part of code.
Signed-off-by: Manjunath Hadli <[email protected]>
Signed-off-by: Sekhar Nori <[email protected]>
Thanks,
Sekhar
_______________________________________________
Davinci-linux-open-source mailing list
[email protected]
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
