Mark A. Greer wrote:

"Mark A. Greer" <mgr...@mvista.com> writes:

[...]

diff --git a/arch/arm/mach-davinci/clock.h b/arch/arm/mach-davinci/clock.h
index 91c24e6..9d09105 100644
--- a/arch/arm/mach-davinci/clock.h
+++ b/arch/arm/mach-davinci/clock.h
@@ -66,6 +66,7 @@ struct clk {
        u8                      usecount;
        u8                      flags;
        u8                      lpsc;
+       u8                      ctlr;

How about psc_ctlr.  Just a little more clear since it's in a 'struct
clk' it might be assumed it's a clock controller.

Good idea.

As if 'psc' didn't include the controller already... That 'lpsc' was pretty badly chosen name, it shoiuld've been 'module' instead, and we then could use 'lpsc' ISO 'ctlr'. How about fixing this now -- at last?

WBR, Sergei

_______________________________________________
Davinci-linux-open-source mailing list
Davinci-linux-open-source@linux.davincidsp.com
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source

Reply via email to