From patchwork Wed Jun 14 20:40:42 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: HMS Incident Management X-Patchwork-Id: 13280449 Received: from lindbergh.monkeyblade.net (lindbergh.monkeyblade.net [23.128.96.19]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 1B76EC2C8 for ; Wed, 14 Jun 2023 20:40:54 +0000 (UTC) Received: from EUR04-DB3-obe.outbound.protection.outlook.com (mail-db3eur04on2068.outbound.protection.outlook.com [40.107.6.68]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 30F7326B8; Wed, 14 Jun 2023 13:40:45 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=bAaty/TUrVeXCRRBUKiI7Sn5QVxEYMwCP3KSqN8dTxa6Y4b4SweHPRhF3ki6AVB2umtT0rFPMmpQa8ZOf+S7CK3e+JCemoi5pnlBzMydc7vAbKeTmBprSURYI7JvlhvgRijABcrWlUvX2s0ZkVBBCdyMe/6tIBSO3O7Qrd1WV7cch23C6C4eShmmxNEfZl0U7cIpwYDqJs2RCrwI3HeQCd2FZSsipQUluZF46ujk2WFZmSuirm7dTdtDJoAHoEloVeMN/XRKmi4JY/kgcXn2EZH1dBggMCgE321MmRj45P7Q6qqsWkh0GmiQtdn3q2sUk5SyeepCMzgt3zPqULkN4Q== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=VxF6aO35+M+sme2Qg1Fe9eUrqv+idvReFp5nkQhtTSM=; b=eLgjBXsk3T4JCHTpB1zFlyTrkN2/0TT4l5vS4FnpY+lh38ZVB/GPkkrn+LwR+GTibhgci1hUEi6nB8FhbsI6bQHNqsAavonoPJm59PV9F0YeSGf34c0LTiRKG0JN3JgL0xlPY+7vn5rQmHkdzcA+5G/kFZI2NZvl9X90Jc9/JAkWlZhjMrby4V+vzu1e6qOdPLQ1oBEfslqin4xzoF5sVAlVm0Q92zmeZ+l44vZXJU0ovrT5YMlHbkWQZa1+N7/eiUFWacvaKF6sC9AeOQ0vSkhVYhdmZJKJ2WSAFhoCLjVo9PDnyf09Pa3SY9c/QbvfnkuL4v6KK9t1veoP023+6A== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=hms.se; dmarc=pass action=none header.from=hms.se; dkim=pass header.d=hms.se; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=hms.se; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=VxF6aO35+M+sme2Qg1Fe9eUrqv+idvReFp5nkQhtTSM=; b=LMXUV0sJHQHjjvTcYzn8lfc4i7zoVQPFBKAVzUT9wDVsHvjSi8QDwy31MgOciHXqTJLHCOQPW8od/suVobugyQUusYbPiAtADBhUgihbrKOGI1ESAN5L1E+XUxoKkRWS/soVpJ4lCTAkKeWgpjMV3+YvpYryRzDdQ2e8XMHxenM= Received: from PAVPR10MB7209.EURPRD10.PROD.OUTLOOK.COM (2603:10a6:102:311::14) by AS2PR10MB7226.EURPRD10.PROD.OUTLOOK.COM (2603:10a6:20b:60a::12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6477.37; Wed, 14 Jun 2023 20:40:43 +0000 Received: from PAVPR10MB7209.EURPRD10.PROD.OUTLOOK.COM ([fe80::7fb8:41fa:a2ba:6b25]) by PAVPR10MB7209.EURPRD10.PROD.OUTLOOK.COM ([fe80::7fb8:41fa:a2ba:6b25%4]) with mapi id 15.20.6455.045; Wed, 14 Jun 2023 20:40:42 +0000 From: HMS Incident Management To: "mkl@pengutronix.de" , "linux-can@vger.kernel.org" , "Thomas.Kopp@microchip.com" CC: "socketcan@hartkopp.net" , "netdev@vger.kernel.org" , "marex@denx.de" , "simon.horman@corigine.com" , "linux-kernel@vger.kernel.org" , "mailhol.vincent@wanadoo.fr" Subject: [PATCH v5 1/3] can: length: fix bitstuffing count Thread-Topic: [PATCH v5 1/3] can: length: fix bitstuffing count Thread-Index: AQHZnwB80da4Mh/1cU+tsGKhSv3dyA== Date: Wed, 14 Jun 2023 20:40:42 +0000 Message-ID: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=hms.se; x-ms-publictraffictype: Email x-ms-traffictypediagnostic: PAVPR10MB7209:EE_|AS2PR10MB7226:EE_ x-ms-office365-filtering-correlation-id: 96670e17-0a6f-4aa8-faf5-08db6d179f6e x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: siVputgqGTK4xuvfuVFv0ruV3Nl6WUg09hr30eWjssM/7YEsJFEzYspRoB2QpSL9PZsoFPCGSIGXCb0ybrskAU2qWlNpakJaQyny7ezk4IulEhBi6WPzahRChCUYwazxrK+masJXUzdhra5Qp3VpuGOxye5DW+VdopaGv7WxclCKwvODsGDZFFuW1O60BdmPTCDd69NB88lhrC0z/OedROnGHQi16ZSnnAQoRxkJHnUjJuEWmVKTAJuIlBxuPcEPHOiPcwKCIhaXmXK5Fny2gl+ZLBb/UqaMWekNQBfd9gkhUorzmwToGpwWZwcLVpZo/ojohQdeEzsjTayB7AYIIWxY7s3JmId438hyFM+uXjUQGgM2WKsvmzf+JUXTXclUKq27SMoi34NMnvAcuez7SM0q+5r7X/Vtqh9/hhMgb7+SbLsNcdX97ZzTJ2pKjqD3U/2sumpdfwFjOdqOOkkYXlzYp0tSRdpy+Tb7Tr4jrVJVRAHnLqM5Tz3McCflYgZLvPAZQinz+PVfd3/8HMEwUAJt+JqTREd3v9hcG629UHEfq17EgFLhEKS1vTJ54UUFAphXfgKpKDn/o8sJW+UK7pJo8eHCjqWLRd6A96rgEgaC+py0UEh4op4NBzIZWXtj x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:PAVPR10MB7209.EURPRD10.PROD.OUTLOOK.COM;PTR:;CAT:NONE;SFS:(13230028)(4636009)(39850400004)(376002)(396003)(366004)(346002)(136003)(451199021)(7696005)(71200400001)(55016003)(186003)(6506007)(26005)(9686003)(66946007)(66556008)(4326008)(64756008)(66476007)(66446008)(41300700001)(8676002)(8936002)(122000001)(38070700005)(2906002)(38100700002)(316002)(5660300002)(52536014)(76116006)(83380400001)(54906003)(86362001)(110136005)(33656002)(478600001);DIR:OUT;SFP:1101; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: BqXCG+HsRuUS1UNaaP4YE39VapRH1uouqTJegB8HsEQ978OUNdVUi86SyH1PRcWZhgesAE1qgMUz3DUZCZy9KqopC/k1pN+dxR5bGjBaj4rhkNba1kkT83HJcY4VWq9nFzruc4lG2Je3MowWP7UMRU0y89NA/9kXonHBmkWYOXI7dqZPSIZ7KgHdIIR4ryljT8OtkEnaPYL0wB86+wCoBYxx4o9Mv2wDDIU31oeEzlbYMjentgGDa+IGpNJsRp1GJuwvaJvCCrRORtUoG+gi2hy4oHHUNN1Ng5fhPeCgL115NHg3FoZ9I13cSA/ccbfVgIGQjYsTpk/byJ2QO33THuQOlQztNweBkON52Ap2/02U5WRw503mmMWFkwgbToSF/15IG70WcO8IEDcIvbl2cQktHrqi7Bp0+P9FTZ3gd7V5PZt0FFitgMuweiTj2y8jGxZvQwjjSzVKDF2Eun9kTGKMhnL71yzBpeUMQXfEkxnqEwRj70H6kQd8cOCYaYYdXZ5pLaKutIO34xzIcZGdCYBWEftWKB5qY/9nMMC6iuGckn6xucVxQeimEoE9ETNE82+r0kpyM+xAbfRm8q8CNvVbaNSL5JLD3zY7xhQr6HCKLeZlovq1kExejit7DcZC3wiJWZPO/c9Ni1mrvEQx1AKbBeidYsWLO2dNKA7S1zb4qG1HQe2jm8wROFI0ajr9z4mz73hrGBGdrNVfy/ZCZNC1r1seBJjQv+vPqFIJphbjJb98ckU8RrjrJY292xp05QqVpI+oB1ros2tXRMReWB+Lypl3Rxfzu6X0J0MAgrFwNKFm574AC44WpBWvWqFLo5FFzsOXzwyfHtXl+wdQpiUQHpZRGkZgirWAujfO5hM+wM87UbqmMH3d0XmhpzKLLUD1UQa7Oi6FEH15TpbZYCKzxCBlpYr18uVHhJ/eU2a18V1AVy8+dt29MZ+F60CoVklDJ4RrLd1jj1G/Qfke0ksuBNjDTZ7bqw7Aeb5NtkkFUrfWRNXnQ3ML1kM5xs9SVbv8MFBCRVYXoB6+6k7HWiLsFr7nuGJrXjrQ6S9pN+afocCXQ0+gF/4H1r6Aa2JG0wSAUFhSur6+kkQHstArvAAZTxtPunHx6u+YLMQ42uk6yFn8vFygIs3lVrbSuScIunvRci7NC+vas/lAzBeO20/jCv0njRy9VugOus+1alsRHOplmTMfbLhIMnSDoYHeb5nO6W+jd7ngs1szvvZyg4dRXTiVVK8JSi14oi5y0+MU5sBajsWlIXVgLTidxMa6a7tyBv1ea2p8Kt0XpQF4rEyRo4AzzBfWYVGKdS+hZyJH5FIFZy9szImqsDuOwOqIDOyUhWdAk4CECKzKdH1YHao3ZpK1miav6KdQO+G7K+wFerQpQeg0HDsoYB6LnlUITCjr/3w26ngWKLlnsKZL8u5hFvYFirtxhHKqXtrTSMBIk5Xp7tNDw66yoTwXRAvXgc+KczIkoU1+cenPJ6o4tRYmC+A1hgblzNRDR440v+RqnhtFyyDsDdH/wZeZeTpYGxNvnSOS1ZIAesmAtDig1W3rGi+8aGEkQUt+n376xBW2i+AX4kb0WYkLAq5usvg0Q3FHv4RZ0HjN1K2QFMHauc3QBZ1OGIUop/GItYOGZ3DQnQm+azoYEZdlqw55/YaQ Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-OriginatorOrg: hms.se X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: PAVPR10MB7209.EURPRD10.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-Network-Message-Id: 96670e17-0a6f-4aa8-faf5-08db6d179f6e X-MS-Exchange-CrossTenant-originalarrivaltime: 14 Jun 2023 20:40:42.9112 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 5c7c1590-4488-4e42-bc9c-15218f8ac994 X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: w6c2URbV/sIOG45KA42N+RLD3jvnbPav/sUgGxJimWvGTsANd+VX33yS4WhJTmf0gYy901afUh9RBqQhp4erlt5C2WU/eElRIkuOfST1EdOUCBPYAHiCpTzJTvdo/5xXO6nn2h/vD/Xc+0nJeHEmbQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS2PR10MB7226 X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE, URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net **We apologize for the delay in delivering this email, which was caused by a mail incident that occurred over the weekend on June 10th. This email was originally sent from vincent.mailhol@gmail.com on 06/11/2023 02:58:08 The Stuff Bit Count is always coded on 4 bits [1]. Update the Stuff Bit Count size accordingly. In addition, the CRC fields of CAN FD Frames contain stuff bits at fixed positions called fixed stuff bits [2]. The CRC field starts with a fixed stuff bit and then has another fixed stuff bit after each fourth bit [2], which allows us to derive this formula: FSB count = 1 + round_down(len(CRC field)/4) The length of the CRC field is [1]: len(CRC field) = len(Stuff Bit Count) + len(CRC) = 4 + len(CRC) with len(CRC) either 17 or 21 bits depending of the payload length. In conclusion, for CRC17: FSB count = 1 + round_down((4 + 17)/4) = 6 and for CRC 21: FSB count = 1 + round_down((4 + 21)/4) = 7 Add a Fixed Stuff bits (FSB) field with above values and update CANFD_FRAME_OVERHEAD_SFF and CANFD_FRAME_OVERHEAD_EFF accordingly. [1] ISO 11898-1:2015 section 10.4.2.6 "CRC field": The CRC field shall contain the CRC sequence followed by a recessive CRC delimiter. For FD Frames, the CRC field shall also contain the stuff count. Stuff count If FD Frames, the stuff count shall be at the beginning of the CRC field. It shall consist of the stuff bit count modulo 8 in a 3-bit gray code followed by a parity bit [...] [2] ISO 11898-1:2015 paragraph 10.5 "Frame coding": In the CRC field of FD Frames, the stuff bits shall be inserted at fixed positions; they are called fixed stuff bits. There shall be a fixed stuff bit before the first bit of the stuff count, even if the last bits of the preceding field are a sequence of five consecutive bits of identical value, there shall be only the fixed stuff bit, there shall not be two consecutive stuff bits. A further fixed stuff bit shall be inserted after each fourth bit of the CRC field [...] Fixes: 85d99c3e2a13 ("can: length: can_skb_get_frame_len(): introduce function to get data length of frame in data link layer") Suggested-by: Thomas Kopp Signed-off-by: Vincent Mailhol Reviewed-by: Thomas Kopp --- include/linux/can/length.h | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/include/linux/can/length.h b/include/linux/can/length.h index 69336549d24f..b8c12c83bc51 100644 --- a/include/linux/can/length.h +++ b/include/linux/can/length.h @@ -72,17 +72,18 @@ * Error Status Indicator (ESI) 1 * Data length code (DLC) 4 * Data field 0...512 - * Stuff Bit Count (SBC) 0...16: 4 20...64:5 + * Stuff Bit Count (SBC) 4 * CRC 0...16: 17 20...64:21 * CRC delimiter (CD) 1 + * Fixed Stuff bits (FSB) 0...16: 6 20...64:7 * ACK slot (AS) 1 * ACK delimiter (AD) 1 * End-of-frame (EOF) 7 * Inter frame spacing 3 * - * assuming CRC21, rounded up and ignoring bitstuffing + * assuming CRC21, rounded up and ignoring dynamic bitstuffing */ -#define CANFD_FRAME_OVERHEAD_SFF DIV_ROUND_UP(61, 8) +#define CANFD_FRAME_OVERHEAD_SFF DIV_ROUND_UP(67, 8) /* * Size of a CAN-FD Extended Frame @@ -101,17 +102,18 @@ * Error Status Indicator (ESI) 1 * Data length code (DLC) 4 * Data field 0...512 - * Stuff Bit Count (SBC) 0...16: 4 20...64:5 + * Stuff Bit Count (SBC) 4 * CRC 0...16: 17 20...64:21 * CRC delimiter (CD) 1 + * Fixed Stuff bits (FSB) 0...16: 6 20...64:7 * ACK slot (AS) 1 * ACK delimiter (AD) 1 * End-of-frame (EOF) 7 * Inter frame spacing 3 * - * assuming CRC21, rounded up and ignoring bitstuffing + * assuming CRC21, rounded up and ignoring dynamic bitstuffing */ -#define CANFD_FRAME_OVERHEAD_EFF DIV_ROUND_UP(80, 8) +#define CANFD_FRAME_OVERHEAD_EFF DIV_ROUND_UP(86, 8) /* * Maximum size of a Classical CAN frame From patchwork Wed Jun 14 20:40:36 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: HMS Incident Management X-Patchwork-Id: 13280448 Received: from lindbergh.monkeyblade.net (lindbergh.monkeyblade.net [23.128.96.19]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 90CC311C86 for ; Wed, 14 Jun 2023 20:40:40 +0000 (UTC) Received: from EUR04-DB3-obe.outbound.protection.outlook.com (mail-db3eur04on2065.outbound.protection.outlook.com [40.107.6.65]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 4B41526A1; Wed, 14 Jun 2023 13:40:38 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=a4wCMXDNzySqCpXeoCJJT4eMLYOsCJK3sTEpH9frvU75BHNkbRqML23MGfaKoSvPpNWjzyNnx/p+XrN+b1DIz4OLHv7cBYIGYxXzyp08+os4IFlSGkKt+N3d7F8aimWuAExcjFsODoEewHLjkspv2XPNioE1cqSwT1JM59fXps+JLWj7ulbdinY6lHaFtCrilunGjrEZb01+/OKvf9frElkw0hbDpjvGUyyoK+g48KpgtAgKU09Jrkbd+D/c0WK2tztsdJflcysUsbLdWxJJZ7bPCh92kiZhrJqij15Y3NHRLIbAJHaRP6kPsMW3yoaEg4fMqADpxfnrEoljbGkQgg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=+1fdZXd27DmTfmHg9STrcI+H4kCAA4nzhDZFOdxb9WQ=; b=fnCa+C2BPOGG5d9QfMMG9/Z0nM6R3gQ6qyPkST2eUQN/hkWtw8zJvU8InU5nRxopq+EgA15EruP7LYK4Cd0xD4BkEDH8YhHZSOgyLA9szec6rBcNufWkivX/a20r+hM+fz2br9fEAXyFvsyldeIcdz+gEUgY6gjvtn8V/5OEeZ0CkNyAkF5vG8jv/RMa0AKUKcgFqe42lMa7rSfwdWEU5Q6w1uhM3zHzjidlljc6ew4JjoUwEF5Xp0zXeSYozUVo28yiYoo/Bfw5ISmFaXBXimaRfwbFE3hu+K7W29e4uqqScb2AwcQqqyQB5phySCursVw2X/q76JJ8bpBpNEqYFA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=hms.se; dmarc=pass action=none header.from=hms.se; dkim=pass header.d=hms.se; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=hms.se; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=+1fdZXd27DmTfmHg9STrcI+H4kCAA4nzhDZFOdxb9WQ=; b=eTGzS23dxPdsp7cSYZTR9EfxARRd4+xcbLoQrmaUCZLTmJ2n2CQCxZn06YtihU88yG6+F2CWGBqqJ1gBz87LCIsQjA0f1VrXbApLMCfvBib2Nlw65Q+0qTNlYl0D2Gcn+Q0bKipQ2YfNie3By/EoHzV8G147qbgIVc9vzBYPGCY= Received: from PAVPR10MB7209.EURPRD10.PROD.OUTLOOK.COM (2603:10a6:102:311::14) by AS2PR10MB7226.EURPRD10.PROD.OUTLOOK.COM (2603:10a6:20b:60a::12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6477.37; Wed, 14 Jun 2023 20:40:36 +0000 Received: from PAVPR10MB7209.EURPRD10.PROD.OUTLOOK.COM ([fe80::7fb8:41fa:a2ba:6b25]) by PAVPR10MB7209.EURPRD10.PROD.OUTLOOK.COM ([fe80::7fb8:41fa:a2ba:6b25%4]) with mapi id 15.20.6455.045; Wed, 14 Jun 2023 20:40:36 +0000 From: HMS Incident Management To: "mkl@pengutronix.de" , "linux-can@vger.kernel.org" , "Thomas.Kopp@microchip.com" CC: "socketcan@hartkopp.net" , "netdev@vger.kernel.org" , "marex@denx.de" , "simon.horman@corigine.com" , "linux-kernel@vger.kernel.org" , "mailhol.vincent@wanadoo.fr" Subject: [PATCH v5 2/3] can: length: fix description of the RRS field Thread-Topic: [PATCH v5 2/3] can: length: fix description of the RRS field Thread-Index: AQHZnwB4UV8wLasOFEWs7GpEoVhhWw== Date: Wed, 14 Jun 2023 20:40:36 +0000 Message-ID: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=hms.se; x-ms-publictraffictype: Email x-ms-traffictypediagnostic: PAVPR10MB7209:EE_|AS2PR10MB7226:EE_ x-ms-office365-filtering-correlation-id: 2b9673cf-56ae-4c94-5c98-08db6d179b5f x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: Jku7fS7hNFlvQqfp61BbccrfZ+scGf6EYI+bDs4InekLd97uUlrgyKriAt8E0cKft/kq8JUFPcI0U+WdbZ1FeXq7Wagl73AtcEjpkmiXa4YuS6gln5S259Crh8ZLUuNiJoJmhMoPXyWiPskKOCzHfb4Wp3DIxjk/LCwzrRHWOVLKma/R/a+gAoCKOdwqLQA+dAL9FjY1wZscTTFjSgLTAQJUMFtF3RnKEAZBgTD6N2+q8DIMDhGCchWWZNDsjOZ20xJUhamOtziUbleXB9t8q7WcDe6vU5OptL9yrx/MGf8OfxbSrZEaMxdGSMWQD/rPpSWMHDzL9Vl8sb0yyQZERK8rDGcezVSRdHInBT8+NnL1klujXoMN9o5tTArxMpYLDtmvfUZdYnerMENocXDNKEkpsTQlyaiRzltnYScdZx+c+de1WYIEm6eNW+nzRjqUwZD+jP80XPFEfqLAOJRYN65CzuuuLxWV/RQkKUauZE7koVPYkAQMeqkuTWAvPP/BbNBWJp6dqjFJFmNUXJKMEBZ21Zdkv1PkfqZRfTiTuSS5xEECdc0qNrN+ia40oiMXTDI/54k+x3ukBGs9HdLFcnws4hXUlSEwkrLo+coKVwXCTAJfYJu58jkZI2mLOMMz x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:PAVPR10MB7209.EURPRD10.PROD.OUTLOOK.COM;PTR:;CAT:NONE;SFS:(13230028)(4636009)(39850400004)(376002)(396003)(366004)(346002)(136003)(451199021)(7696005)(71200400001)(55016003)(186003)(6506007)(26005)(9686003)(66946007)(66556008)(4326008)(64756008)(66476007)(66446008)(41300700001)(8676002)(8936002)(122000001)(38070700005)(2906002)(38100700002)(316002)(5660300002)(52536014)(76116006)(83380400001)(54906003)(86362001)(110136005)(33656002)(478600001);DIR:OUT;SFP:1101; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: hNNuont6JeWuopr2WFUQ7unxbeGlSLp3BQgJYqBbL6HT8XVPaTxtgfkmd4R2fWleaP9GrL07gBvwhMRnf+PSd0w7q6WoD1rECTNGqW2mwwtF+Xm1uiwv5kP9Rj8DtnBlSqdj1IA4kz6cj+mpG/tvLIdJUqs4IQJbaIDAqk2AgfME6kb3oIapHEHqtr2l1IAwPkOgF7oNKnZEwI4JZjQemI8Erv7nVZUZJ11gBxJ9KfPuUs3qTbIjgS2l7IvaYru2uvDks3MVPZK1zgkJeGr7wZ1bwwmwQ//28+0DRz4P1ef7MVJi2n6I2G4E+IiMj34/zXiOoQQTC1Dm2ngyTqGjVsxUDARYtP4n3oOnnZ5TN9qEuP6YUK+Eu+2rnNPDy7QcuSYRfiKdiHUlTwFwJyF+QkAout3tomdovYuWygFXBF6newrYsjyJSZQVH0T7xGU3vF8+KLZh03PqT6im+axMc/6niUPNUkXHDkE+kyGFPkDkQgae84JE+0E7htaUGyNva/7lG2iOIvJUbNjIR2jl+7gUxdV6Wzt4ppu36tAl+u1Oqze1ykh3lUuqP+LL+Wk1hl4yb1u6qJ9gii2Fvi6VBSEghC/8BDaZvl4+PM3wM0oPiwXQQmu6nRmQ8LRVbXGel1DrGf06ngoo8vqtm6fGE8jyd25Zla2g6oD/L+9QZyIYgGRt1NrXSCUiFSGDrk2cs4qtgNXeo9fnrfImTeXPWJQemSZQ4VLWH1t4XyLgPDtiVn0mvkPXpstQOHyrYgbW4A/WYyak0wQ7iyuipjykGOFeM/hsv1Waj314EVS1npG7tRWFsJWnGc4LyKjAplvQfCAbCztCoAEWk5bww6ebD0a+6NONpVpIHsXtGt8ibKzAfE09ME2qwYOUO7a1vY53nVKP2sBjgYy/IkSyJdvRUhEF6PpAXb+nC7JbcNSjGZ8zVbcDrfYxmxoF0A9u+tKbXMW0ohub15j+ItXivjypqwiU/hbkreS+wVeB9YkZxklHehnj/7mWhRU76Bg1SEu5Qm7h/cpPYCzhyTOJdX72ZnzC4FzITGY1+0eDneQ12MAap9qN1okeQZSDh/4YDhL8+ZKhNMk6L3tyOEsG0T6DNasrS9rX5FDOWfksyf3ceJShpPciu851PmnbNKBXqJTQd6ffLPJlN/f5WWOXrJIVT8Djhr7JVsyYQkxEUfx/AaPXoFFNOw0H/Ie0uRNlDwCBf2WmPgP5/t3WxsC/gyQvPkqzzzJGASP233uPOwWIhWJzY2B2c78fF8B4IAUMmJJodCAZy0yaKn/mC50TRtYtVzewILvMMi6qH2P2h9khtU3BGfrP4HjbDGNByPQisRP64fWZLHLz86INZS50TYJkJKD32tE0cbU4syYtT3uKGuawC0/qOeQrzNYarSDas93e9RED/7Wz+P8uCTWKaQnqhx729MbXDSOgVz9zFPjYcIJ19NcTdynxzlklIR+SRqlFW5ereKqheZ5x02H/ohlqsobUHpO3Illb8ATQjkRV9OSCnmIivPRc9ve7c1vPjXljrcpfOTR8vkL7wEBeEazkEKj+HmpqQXI0/Zih+FuHFaRdcgzIlF5wZ+hoHLGK+HpbYwNhpoFDEScjHLrdRVGsW01TZVa+faxIw98OR0qG4Hu+1flB9XYhm67hAtmSQYje Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-OriginatorOrg: hms.se X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: PAVPR10MB7209.EURPRD10.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-Network-Message-Id: 2b9673cf-56ae-4c94-5c98-08db6d179b5f X-MS-Exchange-CrossTenant-originalarrivaltime: 14 Jun 2023 20:40:36.1046 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 5c7c1590-4488-4e42-bc9c-15218f8ac994 X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: P782T4SGBWBdEPb/5llLxWMaPQ0YFT4cQb+FQ8G/DtXZaGC+A5ouVL4uwY3sYA+mdH1oPiZ+3e+MjhHla3Tn4T9hcPXRvuE8AZkeRiHGf7voGWtts80bO+qm54GXAtfNBBxV6Gs64SRYklFLAzxIxg== X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS2PR10MB7226 X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE, URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net **We apologize for the delay in delivering this email, which was caused by a mail incident that occurred over the weekend on June 10th. This email was originally sent from vincent.mailhol@gmail.com on 06/11/2023 02:58:14 The CAN-FD frames only have one reserved bit. The bit corresponding to Classical CAN frame's RTR bit is called the "Remote Request Substitution (RRS)" [1]. N.B. The RRS is not to be confused with the Substitute Remote Request (SRR). Fix the description in the CANFD_FRAME_OVERHEAD_SFF/EFF macros. The total remains unchanged, so this is just a documentation fix. In addition to the above add myself as copyright owner for 2020 (as coauthor of the initial version, c.f. Fixes tag). [1] ISO 11898-1:2015 paragraph 10.4.2.3 "Arbitration field": RSS bit [only in FD Frames] The RRS bit shall be transmitted in FD Frames at the position of the RTR bit in Classical Frames. The RRS bit shall be transmitted dominant, but receivers shall accept recessive and dominant RRS bits. Fixes: 85d99c3e2a13 ("can: length: can_skb_get_frame_len(): introduce function to get data length of frame in data link layer") Signed-off-by: Vincent Mailhol Reviewed-by: Thomas Kopp --- include/linux/can/length.h | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/include/linux/can/length.h b/include/linux/can/length.h index b8c12c83bc51..521fdbce2d69 100644 --- a/include/linux/can/length.h +++ b/include/linux/can/length.h @@ -1,6 +1,7 @@ /* SPDX-License-Identifier: GPL-2.0 */ /* Copyright (C) 2020 Oliver Hartkopp * Copyright (C) 2020 Marc Kleine-Budde + * Copyright (C) 2020 Vincent Mailhol */ #ifndef _CAN_LENGTH_H @@ -64,7 +65,7 @@ * --------------------------------------------------------- * Start-of-frame 1 * Identifier 11 - * Reserved bit (r1) 1 + * Remote Request Substitution (RRS) 1 * Identifier extension bit (IDE) 1 * Flexible data rate format (FDF) 1 * Reserved bit (r0) 1 @@ -95,7 +96,7 @@ * Substitute remote request (SRR) 1 * Identifier extension bit (IDE) 1 * Identifier B 18 - * Reserved bit (r1) 1 + * Remote Request Substitution (RRS) 1 * Flexible data rate format (FDF) 1 * Reserved bit (r0) 1 * Bit Rate Switch (BRS) 1 From patchwork Wed Jun 14 20:40:29 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: HMS Incident Management X-Patchwork-Id: 13280446 X-Patchwork-Delegate: kuba@kernel.org Received: from lindbergh.monkeyblade.net (lindbergh.monkeyblade.net [23.128.96.19]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 6CC1629A1 for ; Wed, 14 Jun 2023 20:40:34 +0000 (UTC) Received: from EUR04-VI1-obe.outbound.protection.outlook.com (mail-vi1eur04on2041.outbound.protection.outlook.com [40.107.8.41]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id BDF77268E; Wed, 14 Jun 2023 13:40:31 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=lc2nv0dVHqrPJeMHPOkAIifonJZrLB/GvYTDrT+UShMIbkVKjOp9/4lNy1DbW1kDHLwAoJy0vxdOj5ioqt4/vlmqXQNQcEwR5RR284TuB0I42c4CYzfXFg9paOxTNzkIuATYwzai+ziBbSo4RmByWxVOgwOyF2ur1taUaEunXPHFHjOPcPHtiyXcD326gvsGPzrZdXkzDJVDt2njwWt+UvQEMHUIQQSRQUor7HqXOdwvavihc3bzuOnkQCC5FT1FCfykcas+9rkdlup9N735P9hYNDTsm2JP3ZrGSZYwOAYqnqQOSjW4yWGrsMu/yKCZsvHpqRF0PaoikErZdQu/+g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=NxFAuTQI96ktjPvSl0YpBLLA1BR4br9/RdPNk1sn9eI=; b=ggmlkKxBYoj7tPTibKtFuwsu6JkilN4g7zklL7hs+JDfCEKlP6rFt+eaiIYEZldKI5qcX48xcB+OiHSN9lRmnMC3neB1u3sVpOrXCZ3WSORMIvw1YVPrZPam3IkGpbVp0XobG9eJWFSw+yu0+oCJ9pszpetvqtD5VKKy3G9noj8yujd/IfpX+jRi8p1n2HKbHfln0jPdxK+H8qY1/NgvECaXEqIgyk/9VZ+mQ+BpnLyRnnWPkDZn8bTsTaTCGcd4QMDqjZHPDKsHKVdBjFedZOZuoOtoUeUQHYu0bwlJgWkdvPYDUjjpDe55jU/dCUZhb2jrNJ6pdtbMnBfuWS+WPg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=hms.se; dmarc=pass action=none header.from=hms.se; dkim=pass header.d=hms.se; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=hms.se; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=NxFAuTQI96ktjPvSl0YpBLLA1BR4br9/RdPNk1sn9eI=; b=dehq4AN6qgJ9EZPjSK8387pKvSE+rzQjzLsRAlLoBBKAGHn+bJf+BgK5e6R9ofFZl3JJiGr2Yle1aWZZKDY3fDTwoiZ3DW121J48od1auMv/dSm/GegTH20vBFvZBmWimpMQocc3+9ro+ni8qDNGTdt28ubhd8QIAqqFQhnvJ48= Received: from PAVPR10MB7209.EURPRD10.PROD.OUTLOOK.COM (2603:10a6:102:311::14) by AS2PR10MB7226.EURPRD10.PROD.OUTLOOK.COM (2603:10a6:20b:60a::12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6477.37; Wed, 14 Jun 2023 20:40:29 +0000 Received: from PAVPR10MB7209.EURPRD10.PROD.OUTLOOK.COM ([fe80::7fb8:41fa:a2ba:6b25]) by PAVPR10MB7209.EURPRD10.PROD.OUTLOOK.COM ([fe80::7fb8:41fa:a2ba:6b25%4]) with mapi id 15.20.6455.045; Wed, 14 Jun 2023 20:40:29 +0000 From: HMS Incident Management To: "mkl@pengutronix.de" , "linux-can@vger.kernel.org" , "Thomas.Kopp@microchip.com" CC: "socketcan@hartkopp.net" , "netdev@vger.kernel.org" , "marex@denx.de" , "simon.horman@corigine.com" , "linux-kernel@vger.kernel.org" , "mailhol.vincent@wanadoo.fr" Subject: [PATCH v5 3/3] can: length: refactor frame lengths definition to add size in bits Thread-Topic: [PATCH v5 3/3] can: length: refactor frame lengths definition to add size in bits Thread-Index: AQHZnwB0tJzSdRUtgES6oX/99kpc1A== Date: Wed, 14 Jun 2023 20:40:29 +0000 Message-ID: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=hms.se; x-ms-publictraffictype: Email x-ms-traffictypediagnostic: PAVPR10MB7209:EE_|AS2PR10MB7226:EE_ x-ms-office365-filtering-correlation-id: 3f384a5f-5b14-4534-5b81-08db6d17974d x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: W7bufFP5o76NZhvyne4wdvCEOQkDylUiwgvQe5WMuqDFWjaf7h4pA7Iid5slo6oh07SOtHKJ2wrodGSZNNjNltLc3eZ2U1rZCVaW8PCHJcRxbk4miio5NXEXI4CAiVcgjdm4O9lSULd6p89aIaeA4YrKrLHrbG6ZKeN93Wv1xs99LY5ZxrClbXOb9GyY6cHSQlHN3/hpWGTzlRBU7W/MCN5+Vl07t7t5Z4wcVJ2hW1hPB2qMGAUzGSJIxrAyI387KjlEYgzAmxbWGNbzL7vgwenQEQNDz/egP54oDOVZTFfxpzh72NjM7CAd7Rers3W9SPHFhnoFzkD2HvkJC4IQPVBg/JZk3rPYh+Tbc0Jsmh5lCLCeF99zgE7racqvv3j/d39UN4kzmuZYaVyqDW33Y5qDb8dWvmezi74pVm72ddXumg9LBPcX/zPfJezBOiuYM1VRkGL96EIWOTOH076XTrhnrYrMQ6PWIQO0bqcWCpLRfKwsMttyjcqnURbqPJERc5csa3pt0q++x62zJpg3JEF4YzDvlQ1P6KUvsi7N0qyUecncwDuSZYYGIeNHfpGFpP2vBEGf1aSTMzDeCJ16s9oP3iyoTFZmJMp5f6d7aeIeh6Af1IY2iqDFhZlx7V4L x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:PAVPR10MB7209.EURPRD10.PROD.OUTLOOK.COM;PTR:;CAT:NONE;SFS:(13230028)(4636009)(39850400004)(376002)(396003)(366004)(346002)(136003)(451199021)(7696005)(71200400001)(55016003)(186003)(6506007)(26005)(9686003)(66946007)(66556008)(4326008)(64756008)(66476007)(66446008)(41300700001)(8676002)(8936002)(122000001)(38070700005)(2906002)(38100700002)(316002)(5660300002)(52536014)(30864003)(76116006)(83380400001)(54906003)(86362001)(110136005)(33656002)(478600001);DIR:OUT;SFP:1101; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: RsK/gKFxrP9fW4Mx55KyMcTvxXmGaT+ApBBj1Hk2ckxSPSFB2ERsR13XI1pRWjDVNt29m2CPFdtV0juV5gZ7IgHJMyN3iUocnmAgNRAI7pjLJb30QABl5LdQU50nDTpaQ6rmqsYLzgz7tmnj1aMxUt4CRPqHVEcgNioV0m6ddFAPR/khAjKZElN4PvNN4eHwrP7AhBzLfNl9qobvoUVhP7su6AhWmtyOKcoBdsDU49b4wJAUnzxehOxyy69cULeOO3ypX/JlA0yIKR3vWDvyQNYKeDCI/Y0t0OVN7oWwvYOwR0r/TFd4DNvqlEnU2cQ8Bfd+3Vgdw+gT8jKSlnHXgxw6i/KftV2m8BQdlAINoMqQtonPTXhOQAPwuiHRfugsDQg+mW+sUMlZhpVN57KsoYD6GfZxTjJONFJOoAOqAm/reK6F3yxMDDhXQ+P5Gf8bmwTigmcdoNWTslE7Ccb/13Zkn/Q8bP2BnX0/2IdUSZF8unOXdeLIZ7GWDZ8yxv8wz+AargL7EGRPJr6zE9MVJ9+RBUX/A46rbrrJnTZwjUz0Rweji76HfXVdreIItB5Ag3aE3SvPJeyjDeT5k/AwdOW3AIo3ScxKZf+exRgRRx8yZvGJ4eR/rABt3Fww+/FvqVIwzCaxtEwKt5SFAJgCvelT2HYdIauD9ruriJfuEL2KeCpNL5b4ys3DQF9i22/pFogB9jNGESUuNqfjDLFoDBY8zJ2l9jWEK9WD8o9qrHkJK81EbXoeOMPoK9W8c1o8ZqrHEUfxFe2Cp+hAZ0SVgVoU9gAa28/K/LO16f6aPrB5otQLCXeJUcKSWcMj4pEV8UitQQYj90vm3BTmlS4X8vmcEmCx/Mf3lN2pM3BQFJ9TZ5Ngh5KI0OSExl4PDbc+U4lCzDJLycKyBtJZV3mbLFwgeHGeYwKVHV4KVhD0wjbkblFat8YqZobZXv7WN87XocMQzViJxzM61l3CHvcFFzqDjnatKVb6BuLWUrWWNND1NnYNnQ7Vjz6iLCuO7+qRBuc2mWjMi+xFOgpR2Ul+x+9XdSKlCDowJN0c1MjZGZ9liaVxAGmQh/X850qOQTGPYVqvxzNjc72zcJeh0iPbrMS9kmb9CPOI0AFRndq10+uX7DOByJgD8voP9e30oqkdlPYRsxrVzFge5wR3AYHaUwN/NEcgeSjMHowr/ZDM99h/aWDxogWLjCufZ706uivwf0kkfH9nBN00dvBBH9ypNnA/8A+Inv6XNMolusmSK984fDk1LRHNLsfsdNz8cdEwwAaTWdoO98g44qbqRAtK8NNVlCDmaKRaTOUd1isaIF6tKETb5bYfjsFaF4bSG1YPEXnEvpkAAlD8SeoLoKOfnM3rA+QsWxPA37Xajdp1I3aug+em3D7EfQjV2NQ0rTmcNQMnG09EbKOydsXElC7c51CXitrKIS+r0xE25J0l8FHUqtdyL6TU846ZptBjxxrRjCRNS7c9/zWN8GqjvZxVQsdTcZrhS3NgKxU2Xpw/P9p+LA38PiPUFXIL5W5kYzIrTB8lcMJrvTiPG9q9E58dqdVi0TkjeSv49c8m3gvlCl0MQTXFN/TFZj/N0r7hjkWr8ZxMPl15u8E04P/AiMWu0IqMMtzbL/jtigdjqo1zrSIPXF5BUUpQJwUjIhTTcL32 Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-OriginatorOrg: hms.se X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: PAVPR10MB7209.EURPRD10.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-Network-Message-Id: 3f384a5f-5b14-4534-5b81-08db6d17974d X-MS-Exchange-CrossTenant-originalarrivaltime: 14 Jun 2023 20:40:29.2574 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 5c7c1590-4488-4e42-bc9c-15218f8ac994 X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: MUEzWYC/aucXVv1JwtQmCEsALgch0IdxbMAYGPvtX5J2IDXbbsnYAjXAmu2er71N5hThAYRUPc+PnevETIU5TpHaJ9JYivwCRaC6uVbQvS4wtgjrjFpxA5hLBwcVSV4bU64DOY8R6XZuaCFoL7z3YQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS2PR10MB7226 X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE, URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net X-Patchwork-Delegate: kuba@kernel.org **We apologize for the delay in delivering this email, which was caused by a mail incident that occurred over the weekend on June 10th. This email was originally sent from vincent.mailhol@gmail.com on 06/11/2023 02:58:17 Introduce a method to calculate the exact size in bits of a CAN(-FD) frame with or without dynamic bitstuffing. These are all the possible combinations taken into account: - Classical CAN or CAN-FD - Standard or Extended frame format - CAN-FD CRC17 or CRC21 - Include or not intermission Instead of doing several individual macro definitions, declare the can_frame_bits() function-like macro. To this extent, do a full refactoring of the length definitions. In addition add the can_frame_bytes(). This function-like macro replaces the existing macro: - CAN_FRAME_OVERHEAD_SFF: can_frame_bytes(false, false, 0) - CAN_FRAME_OVERHEAD_EFF: can_frame_bytes(false, true, 0) - CANFD_FRAME_OVERHEAD_SFF: can_frame_bytes(true, false, 0) - CANFD_FRAME_OVERHEAD_EFF: can_frame_bytes(true, true, 0) Function-like macros were chosen over inline functions because they can be used to initialize const struct fields. The different maximum frame lengths (maximum data length, including intermission) are as follow: Frame type bits bytes ------------------------------------------------------- Classic CAN SFF no bitstuffing 111 14 Classic CAN EFF no bitstuffing 131 17 Classic CAN SFF bitstuffing 135 17 Classic CAN EFF bitstuffing 160 20 CAN-FD SFF no bitstuffing 579 73 CAN-FD EFF no bitstuffing 598 75 CAN-FD SFF bitstuffing 712 89 CAN-FD EFF bitstuffing 736 92 The macro CAN_FRAME_LEN_MAX and CANFD_FRAME_LEN_MAX are kept as an alias to, respectively, can_frame_bytes(false, true, CAN_MAX_DLEN) and can_frame_bytes(true, true, CANFD_MAX_DLEN). In addition to the above: - Use ISO 11898-1:2015 definitions for the names of the CAN frame fields. - Include linux/bits.h for use of BITS_PER_BYTE. - Include linux/math.h for use of mult_frac() and DIV_ROUND_UP(). N.B: the use of DIV_ROUND_UP() is not new to this patch, but the include was previously omitted. - Add copyright 2023 for myself. Suggested-by: Thomas Kopp Signed-off-by: Vincent Mailhol Reviewed-by: Thomas Kopp --- drivers/net/can/dev/length.c | 15 +- include/linux/can/length.h | 302 +++++++++++++++++++++++++---------- 2 files changed, 216 insertions(+), 101 deletions(-) diff --git a/drivers/net/can/dev/length.c b/drivers/net/can/dev/length.c index b48140b1102e..b7f4d76dd444 100644 --- a/drivers/net/can/dev/length.c +++ b/drivers/net/can/dev/length.c @@ -78,18 +78,7 @@ unsigned int can_skb_get_frame_len(const struct sk_buff *skb) else len = cf->len; - if (can_is_canfd_skb(skb)) { - if (cf->can_id & CAN_EFF_FLAG) - len += CANFD_FRAME_OVERHEAD_EFF; - else - len += CANFD_FRAME_OVERHEAD_SFF; - } else { - if (cf->can_id & CAN_EFF_FLAG) - len += CAN_FRAME_OVERHEAD_EFF; - else - len += CAN_FRAME_OVERHEAD_SFF; - } - - return len; + return can_frame_bytes(can_is_canfd_skb(skb), cf->can_id & CAN_EFF_FLAG, + false, len); } EXPORT_SYMBOL_GPL(can_skb_get_frame_len); diff --git a/include/linux/can/length.h b/include/linux/can/length.h index 521fdbce2d69..abc978b38f79 100644 --- a/include/linux/can/length.h +++ b/include/linux/can/length.h @@ -1,132 +1,258 @@ /* SPDX-License-Identifier: GPL-2.0 */ /* Copyright (C) 2020 Oliver Hartkopp * Copyright (C) 2020 Marc Kleine-Budde - * Copyright (C) 2020 Vincent Mailhol + * Copyright (C) 2020, 2023 Vincent Mailhol */ #ifndef _CAN_LENGTH_H #define _CAN_LENGTH_H +#include #include #include +#include /* - * Size of a Classical CAN Standard Frame + * Size of a Classical CAN Standard Frame header in bits * - * Name of Field Bits + * Name of Field Bits * --------------------------------------------------------- - * Start-of-frame 1 - * Identifier 11 - * Remote transmission request (RTR) 1 - * Identifier extension bit (IDE) 1 - * Reserved bit (r0) 1 - * Data length code (DLC) 4 - * Data field 0...64 - * CRC 15 - * CRC delimiter 1 - * ACK slot 1 - * ACK delimiter 1 - * End-of-frame (EOF) 7 - * Inter frame spacing 3 + * Start Of Frame (SOF) 1 + * Arbitration field: + * base ID 11 + * Remote Transmission Request (RTR) 1 + * Control field: + * IDentifier Extension bit (IDE) 1 + * FD Format indicator (FDF) 1 + * Data Length Code (DLC) 4 + * + * including all fields preceding the data field, ignoring bitstuffing + */ +#define CAN_FRAME_HEADER_SFF_BITS 19 + +/* + * Size of a Classical CAN Extended Frame header in bits * - * rounded up and ignoring bitstuffing + * Name of Field Bits + * --------------------------------------------------------- + * Start Of Frame (SOF) 1 + * Arbitration field: + * base ID 11 + * Substitute Remote Request (SRR) 1 + * IDentifier Extension bit (IDE) 1 + * ID extension 18 + * Remote Transmission Request (RTR) 1 + * Control field: + * FD Format indicator (FDF) 1 + * Reserved bit (r0) 1 + * Data length code (DLC) 4 + * + * including all fields preceding the data field, ignoring bitstuffing */ -#define CAN_FRAME_OVERHEAD_SFF DIV_ROUND_UP(47, 8) +#define CAN_FRAME_HEADER_EFF_BITS 39 /* - * Size of a Classical CAN Extended Frame + * Size of a CAN-FD Standard Frame in bits + * + * Name of Field Bits + * --------------------------------------------------------- + * Start Of Frame (SOF) 1 + * Arbitration field: + * base ID 11 + * Remote Request Substitution (RRS) 1 + * Control field: + * IDentifier Extension bit (IDE) 1 + * FD Format indicator (FDF) 1 + * Reserved bit (res) 1 + * Bit Rate Switch (BRS) 1 + * Error Status Indicator (ESI) 1 + * Data length code (DLC) 4 + * + * including all fields preceding the data field, ignoring bitstuffing + */ +#define CANFD_FRAME_HEADER_SFF_BITS 22 + +/* + * Size of a CAN-FD Extended Frame in bits + * + * Name of Field Bits + * --------------------------------------------------------- + * Start Of Frame (SOF) 1 + * Arbitration field: + * base ID 11 + * Substitute Remote Request (SRR) 1 + * IDentifier Extension bit (IDE) 1 + * ID extension 18 + * Remote Request Substitution (RRS) 1 + * Control field: + * FD Format indicator (FDF) 1 + * Reserved bit (res) 1 + * Bit Rate Switch (BRS) 1 + * Error Status Indicator (ESI) 1 + * Data length code (DLC) 4 + * + * including all fields preceding the data field, ignoring bitstuffing + */ +#define CANFD_FRAME_HEADER_EFF_BITS 41 + +/* + * Size of a CAN CRC Field in bits * * Name of Field Bits * --------------------------------------------------------- - * Start-of-frame 1 - * Identifier A 11 - * Substitute remote request (SRR) 1 - * Identifier extension bit (IDE) 1 - * Identifier B 18 - * Remote transmission request (RTR) 1 - * Reserved bits (r1, r0) 2 - * Data length code (DLC) 4 - * Data field 0...64 - * CRC 15 - * CRC delimiter 1 - * ACK slot 1 - * ACK delimiter 1 - * End-of-frame (EOF) 7 - * Inter frame spacing 3 + * CRC sequence (CRC15) 15 + * CRC Delimiter 1 + * + * ignoring bitstuffing + */ +#define CAN_FRAME_CRC_FIELD_BITS 16 + +/* + * Size of a CAN-FD CRC17 Field in bits (length: 0..16) * - * rounded up and ignoring bitstuffing + * Name of Field Bits + * --------------------------------------------------------- + * Stuff Count 4 + * CRC Sequence (CRC17) 17 + * CRC Delimiter 1 + * Fixed stuff bits 6 */ -#define CAN_FRAME_OVERHEAD_EFF DIV_ROUND_UP(67, 8) +#define CANFD_FRAME_CRC17_FIELD_BITS 28 /* - * Size of a CAN-FD Standard Frame + * Size of a CAN-FD CRC21 Field in bits (length: 20..64) * * Name of Field Bits * --------------------------------------------------------- - * Start-of-frame 1 - * Identifier 11 - * Remote Request Substitution (RRS) 1 - * Identifier extension bit (IDE) 1 - * Flexible data rate format (FDF) 1 - * Reserved bit (r0) 1 - * Bit Rate Switch (BRS) 1 - * Error Status Indicator (ESI) 1 - * Data length code (DLC) 4 - * Data field 0...512 - * Stuff Bit Count (SBC) 4 - * CRC 0...16: 17 20...64:21 - * CRC delimiter (CD) 1 - * Fixed Stuff bits (FSB) 0...16: 6 20...64:7 - * ACK slot (AS) 1 - * ACK delimiter (AD) 1 - * End-of-frame (EOF) 7 - * Inter frame spacing 3 - * - * assuming CRC21, rounded up and ignoring dynamic bitstuffing - */ -#define CANFD_FRAME_OVERHEAD_SFF DIV_ROUND_UP(67, 8) + * Stuff Count 4 + * CRC sequence (CRC21) 21 + * CRC Delimiter 1 + * Fixed stuff bits 7 + */ +#define CANFD_FRAME_CRC21_FIELD_BITS 33 /* - * Size of a CAN-FD Extended Frame + * Size of a CAN(-FD) Frame footer in bits * * Name of Field Bits * --------------------------------------------------------- - * Start-of-frame 1 - * Identifier A 11 - * Substitute remote request (SRR) 1 - * Identifier extension bit (IDE) 1 - * Identifier B 18 - * Remote Request Substitution (RRS) 1 - * Flexible data rate format (FDF) 1 - * Reserved bit (r0) 1 - * Bit Rate Switch (BRS) 1 - * Error Status Indicator (ESI) 1 - * Data length code (DLC) 4 - * Data field 0...512 - * Stuff Bit Count (SBC) 4 - * CRC 0...16: 17 20...64:21 - * CRC delimiter (CD) 1 - * Fixed Stuff bits (FSB) 0...16: 6 20...64:7 - * ACK slot (AS) 1 - * ACK delimiter (AD) 1 - * End-of-frame (EOF) 7 - * Inter frame spacing 3 - * - * assuming CRC21, rounded up and ignoring dynamic bitstuffing - */ -#define CANFD_FRAME_OVERHEAD_EFF DIV_ROUND_UP(86, 8) + * ACK slot 1 + * ACK delimiter 1 + * End Of Frame (EOF) 7 + * + * including all fields following the CRC field + */ +#define CAN_FRAME_FOOTER_BITS 9 + +/* + * First part of the Inter Frame Space + * (a.k.a. IMF - intermission field) + */ +#define CAN_INTERMISSION_BITS 3 + +/** + * can_bitstuffing_len() - Calculate the maximum length with bitstuffing + * @destuffed_len: length of a destuffed bit stream + * + * The worst bit stuffing case is a sequence in which dominant and + * recessive bits alternate every four bits: + * + * Destuffed: 1 1111 0000 1111 0000 1111 + * Stuffed: 1 1111o 0000i 1111o 0000i 1111o + * + * Nomenclature + * + * - "0": dominant bit + * - "o": dominant stuff bit + * - "1": recessive bit + * - "i": recessive stuff bit + * + * Aside from the first bit, one stuff bit is added every four bits. + * + * Return: length of the stuffed bit stream in the worst case scenario. + */ +#define can_bitstuffing_len(destuffed_len) \ + (destuffed_len + (destuffed_len - 1) / 4) + +#define __can_bitstuffing_len(bitstuffing, destuffed_len) \ + (bitstuffing ? can_bitstuffing_len(destuffed_len) : \ + destuffed_len) + +#define __can_cc_frame_bits(is_eff, bitstuffing, \ + intermission, data_len) \ +( \ + __can_bitstuffing_len(bitstuffing, \ + (is_eff ? CAN_FRAME_HEADER_EFF_BITS : \ + CAN_FRAME_HEADER_SFF_BITS) + \ + (data_len) * BITS_PER_BYTE + \ + CAN_FRAME_CRC_FIELD_BITS) + \ + CAN_FRAME_FOOTER_BITS + \ + (intermission ? CAN_INTERMISSION_BITS : 0) \ +) + +#define __can_fd_frame_bits(is_eff, bitstuffing, \ + intermission, data_len) \ +( \ + __can_bitstuffing_len(bitstuffing, \ + (is_eff ? CANFD_FRAME_HEADER_EFF_BITS : \ + CANFD_FRAME_HEADER_SFF_BITS) + \ + (data_len) * BITS_PER_BYTE) + \ + ((data_len) len. Should be zero for remote frames. No + * sanitization is done on @data_len and it shall have no side + * effects. + * + * Return: the numbers of bits on the wire of a CAN frame. + */ +#define can_frame_bits(is_fd, is_eff, bitstuffing, \ + intermission, data_len) \ +( \ + is_fd ? __can_fd_frame_bits(is_eff, bitstuffing, \ + intermission, data_len) : \ + __can_cc_frame_bits(is_eff, bitstuffing, \ + intermission, data_len) \ +) + +/* + * Number of bytes in a CAN frame + * (rounded up, including intermission) + */ +#define can_frame_bytes(is_fd, is_eff, bitstuffing, data_len) \ + DIV_ROUND_UP(can_frame_bits(is_fd, is_eff, bitstuffing, \ + true, data_len), \ + BITS_PER_BYTE) /* * Maximum size of a Classical CAN frame - * (rounded up and ignoring bitstuffing) + * (rounded up, ignoring bitstuffing but including intermission) */ -#define CAN_FRAME_LEN_MAX (CAN_FRAME_OVERHEAD_EFF + CAN_MAX_DLEN) +#define CAN_FRAME_LEN_MAX can_frame_bytes(false, true, false, CAN_MAX_DLEN) /* * Maximum size of a CAN-FD frame - * (rounded up and ignoring bitstuffing) + * (rounded up, ignoring dynamic bitstuffing but including intermission) */ -#define CANFD_FRAME_LEN_MAX (CANFD_FRAME_OVERHEAD_EFF + CANFD_MAX_DLEN) +#define CANFD_FRAME_LEN_MAX can_frame_bytes(true, true, false, CANFD_MAX_DLEN) /* * can_cc_dlc2len(value) - convert a given data length code (dlc) of a -- 2.39.3