[PATCH v7 4/8] davinci: dm644x: Replace register base value with a defined macro

[Nori, Sekhar]

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 <manjunath.hadli at ti.com>
> Acked-by: Sekhar Nori <nsekhar at ti.com>

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 <manjunath.hadli at ti.com>
    Signed-off-by: Sekhar Nori <nsekhar at ti.com>

Thanks,
Sekhar