linux-i2c.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Re: Re: [PATCH]  I2C: Change the value of octeon i2c adapter timeout value
@ 2013-04-19  9:13 EUNBONG SONG
  2013-04-19 18:32 ` Wolfram Sang
  0 siblings, 1 reply; 2+ messages in thread
From: EUNBONG SONG @ 2013-04-19  9:13 UTC (permalink / raw)
  To: Wolfram Sang
  Cc: linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org



On Fri, Apr 19, 2013 at 12:01:04AM +0000, EUNBONG SONG wrote:
>> 
>>  I think HZ/50 is better than 2 for adapter timeout.

> Basically OK. But why HZ/50? Most drivers use HZ.

Actually, I just translated 2 jiffies because HZ is 100 in default cavium config. 
You can find that in "arch/mips/configs/cavium_octeon_defconfig". 
And i have been using this value over 1 year in octeon board without problem.
Thanks. 

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: Re: [PATCH]  I2C: Change the value of octeon i2c adapter timeout value
  2013-04-19  9:13 Re: [PATCH] I2C: Change the value of octeon i2c adapter timeout value EUNBONG SONG
@ 2013-04-19 18:32 ` Wolfram Sang
  0 siblings, 0 replies; 2+ messages in thread
From: Wolfram Sang @ 2013-04-19 18:32 UTC (permalink / raw)
  To: EUNBONG SONG
  Cc: linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org

On Fri, Apr 19, 2013 at 09:13:54AM +0000, EUNBONG SONG wrote:
> 
> 
> On Fri, Apr 19, 2013 at 12:01:04AM +0000, EUNBONG SONG wrote:
> >> 
> >>  I think HZ/50 is better than 2 for adapter timeout.
> 
> > Basically OK. But why HZ/50? Most drivers use HZ.
> 
> Actually, I just translated 2 jiffies because HZ is 100 in default cavium config. 
> You can find that in "arch/mips/configs/cavium_octeon_defconfig". 
> And i have been using this value over 1 year in octeon board without problem.

Have you been writing to EEPROMS? Their erase/write cycle might be
longer. But I am not forcing you to change the value, just giving some
suggestions.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2013-04-19 18:32 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-04-19  9:13 Re: [PATCH] I2C: Change the value of octeon i2c adapter timeout value EUNBONG SONG
2013-04-19 18:32 ` Wolfram Sang

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).