Re: [PATCH] da850-evm, trivial: use da850_evm prefix for consistency

2010-11-23 Thread Kevin Hilman
Nori, Sekhar nsek...@ti.com writes:

 On Sat, Nov 20, 2010 at 03:13:04, Ben Gardiner wrote:
 There was a single case of 'da850evm' prefix in the board-da850-evm.c file
 where the reset of the prefixes were 'da850_evm'; change it to 'da850_evm' 
 for
 consistency.

 Signed-off-by: Ben Gardiner bengardi...@nanometrics.ca

 ---

 @Sekhar, you asked me to prefix all the static symbols I added to
 board-da850-evm.c with 'da850evm' -- but I noticed that the current 
 convention
 is a prefix of 'da850_evm' so I decided to stick with the convention and
 replace the only outlier with this patch.

 Thanks. I personally prefer da850evm, but consistency
 is more important so that a search-replace is possible
 later on. So I am okay with this too.

I'll take that as an Ack.

Applying, queuing for 2.6.38.

Kevin

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


RE: [PATCH] da850-evm, trivial: use da850_evm prefix for consistency

2010-11-22 Thread Nori, Sekhar
On Sat, Nov 20, 2010 at 03:13:04, Ben Gardiner wrote:
 There was a single case of 'da850evm' prefix in the board-da850-evm.c file
 where the reset of the prefixes were 'da850_evm'; change it to 'da850_evm' for
 consistency.

 Signed-off-by: Ben Gardiner bengardi...@nanometrics.ca

 ---

 @Sekhar, you asked me to prefix all the static symbols I added to
 board-da850-evm.c with 'da850evm' -- but I noticed that the current convention
 is a prefix of 'da850_evm' so I decided to stick with the convention and
 replace the only outlier with this patch.

Thanks. I personally prefer da850evm, but consistency
is more important so that a search-replace is possible
later on. So I am okay with this too.

Regards,
Sekhar

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