Re: [PATCH 1/1] hso: fix problem with wrong status code sent byOPTION GTM601 during RING indication

From: David Miller
Date: Thu Oct 03 2013 - 16:00:56 EST


From: "Dr. H. Nikolaus Schaller" <hns@xxxxxxxxxxxxx>
Date: Thu, 3 Oct 2013 21:40:34 +0200

> I have made the bug observation from debug log that this bit is set in a response
> each time the modem has a RING message. It might be specific to this modem
> and firmware version, i.e. a firmware bug. But we have no means to verify or even
> change it in the firmware.
>
> I.e. the driver must handle it in a better way.
>
> Because the notification is rejected by the driver, the driver will hang up and the
> whole modem connection breaks down.
>
> With this patch, the problem was never observed again in ~2 years.
>
> I'd hope the maintainer of this driver can shed some light on it.

I think if you suspect the bit is set in response to RING messages
then you should define a macro so that the number is not just magic,
and put a descriptrive comment above the macro definition for that
bit.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/