[PATCH] w1: w1-gpio: Loosen timings to improve cold boot reliability

[Eugen.Hristev at microchip.com]

On 11/8/21 5:07 PM, Chris Morgan wrote:
> From: Chris Morgan <macromorgan at hotmail.com>
> 
> On my NTC CHIP whenever I do a cold boot any attached DIPs cannot be
> found. Rebooting on the other hand appears to fix the issue. I found
> that if I modified the timing slightly (but still within spec) the
> w1 identification on cold boot became far more reliable.
> 
> Signed-off-by: Chris Morgan <macromorgan at hotmail.com>
> ---
>   drivers/w1/w1-gpio.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/w1/w1-gpio.c b/drivers/w1/w1-gpio.c
> index 9346f810ce..5565de2a92 100644
> --- a/drivers/w1/w1-gpio.c
> +++ b/drivers/w1/w1-gpio.c
> @@ -22,8 +22,8 @@
>   #define W1_TIMING_E    9
>   #define W1_TIMING_F    55
>   #define W1_TIMING_G    0
> -#define W1_TIMING_H    480
> -#define W1_TIMING_I    70
> +#define W1_TIMING_H    600
> +#define W1_TIMING_I    100
>   #define W1_TIMING_J    410
> 
>   struct w1_gpio_pdata {
> --
> 2.30.2
> 

Hi,

Thanks for your patch,

I will test this on my boards at some point, but I can't promise exactly 
when.
If there are any other users of this protocol, I think they should test 
it as well.

Eugen