On 21.12.2021 22:24:52, Thomas.Kopp@xxxxxxxxxxxxx wrote:
> diff --git a/drivers/net/can/spi/mcp251xfd/mcp251xfd-regmap.c b/drivers/net/can/spi/mcp251xfd/mcp251xfd-regmap.c
> index 297491516a26..e5bc897f37e8 100644
> --- a/drivers/net/can/spi/mcp251xfd/mcp251xfd-regmap.c
> +++ b/drivers/net/can/spi/mcp251xfd/mcp251xfd-regmap.c
> @@ -332,12 +332,10 @@ mcp251xfd_regmap_crc_read(void *context,
> *
> * If the highest bit in the lowest byte is flipped
> * the transferred CRC matches the calculated one. We
> - * assume for now the CRC calculation in the chip
> - * works on wrong data and the transferred data is
> - * correct.
> + * assume for now the CRC operates on the correct data.
> */
> if (reg == MCP251XFD_REG_TBC &&
> - (buf_rx->data[0] == 0x0 || buf_rx->data[0] == 0x80)) {
> + ((buf_rx->data[0] & 0xF8) == 0x0 || (buf_rx->data[0] & 0xF8) == 0x80)) {
With this change the read of the TBC on the mcp2517fd becomes much more
stable. No more single bit flips in the 1st data byte that can be fixed
with xor 0x80.
regards,
Marc
--
Pengutronix e.K. | Marc Kleine-Budde |
Embedded Linux | https://www.pengutronix.de |
Vertretung West/Dortmund | Phone: +49-231-2826-924 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
Attachment:
signature.asc
Description: PGP signature
