Re: [PATCH 2/4 - rev2] Add new timeval_to_sec function

2007-08-21 Thread Bob Beers
What if the name of the function was more descriptive of what the function actually does? Maybe timeval_ceil_sec()? - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH 2/4 - rev2] Add new timeval_to_sec function

2007-08-21 Thread Varun Chandramohan
Bob Beers wrote: What if the name of the function was more descriptive of what the function actually does? Maybe timeval_ceil_sec()? Looks ok, it can be done if everyone is ok with it. - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to [EMAIL

[PATCH 2/4 - rev2] Add new timeval_to_sec function

2007-08-20 Thread Varun Chandramohan
A new function for converting timeval to time_t is added in time.h. Its a common function used in different places. Signed-off-by: Varun Chandramohan [EMAIL PROTECTED] --- include/linux/time.h | 12 1 files changed, 12 insertions(+), 0 deletions(-) diff --git

Re: [PATCH 2/4 - rev2] Add new timeval_to_sec function

2007-08-20 Thread Stephen Hemminger
On Mon, 20 Aug 2007 13:45:36 +0530 Varun Chandramohan [EMAIL PROTECTED] wrote: A new function for converting timeval to time_t is added in time.h. Its a common function used in different places. Signed-off-by: Varun Chandramohan [EMAIL PROTECTED] --- include/linux/time.h | 12

Re: [PATCH 2/4 - rev2] Add new timeval_to_sec function

2007-08-20 Thread Varun Chandramohan
Stephen Hemminger wrote: On Mon, 20 Aug 2007 13:45:36 +0530 Varun Chandramohan [EMAIL PROTECTED] wrote: A new function for converting timeval to time_t is added in time.h. Its a common function used in different places. Signed-off-by: Varun Chandramohan [EMAIL PROTECTED] ---