Re: [PATCH 1/1] virtio-can: Add link to CAN specification from ISO.

Harald Mommer <[email protected]> Fri, 7 Jun 2024 15:49:38 +0200
Newsgroups dev.linux.lists.virtio-dev,org.kernel.vger.linux-can
Message-ID <[email protected]>
Got a justified complaint about causing too much noise, so took out 
[email protected] and [email protected] to shorten the CC.

On 07.06.24 05:43, Vincent Mailhol wrote:
> On Thu. 6 Jun. 2024 at 23:26, Harald Mommer
> <[email protected]>  wrote:
>> Add link to the CAN specification in the ISO shop.
>>
>>    ISO 11898-1:2015
>>    Road vehicles
>>    Controller area network (CAN)
>>    Part 1: Data link layer and physical signalling
>>
>> The specification is not freely obtainable there.
>> ---
>>   introduction.tex | 3 ++-
>>   1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/introduction.tex b/introduction.tex
>> index 8bcef03..72573d6 100644
>> --- a/introduction.tex
>> +++ b/introduction.tex
> Excuse my ignorance, this is not a patch toward the Linux tree, right?
> Could you let me know which git tree this patch is targetting?
targeted to git repository: https://github.com/oasis-tcs/virtio-spec.git 
branch: virtio-1.4 (37c6a406678a).
>> @@ -142,7 +142,8 @@ \section{Normative References}\label{sec:Normative References}
>>       TRANSMISSION CONTROL PROTOCOL
>>          \newline\url{https://ddec1-0-en-ctp.trendmicro.com:443/wis/clicktime/v1/query?url=https%3a%2f%2fwww.rfc%2deditor.org%2frfc%2frfc793%7d%5c%5c&umid=65841137-ddc9-4901-90a4-a0cb61c155d1&auth=53c7c7de28b92dfd96e93d9dd61a23e634d2fbec-04306aba055e4efd56e85e2894cfd163fc39b45a
>>          \phantomsection\label{intro:CAN}\textbf{[CAN]} &
>> -    ISO 11898-1:2015 Road vehicles -- Controller area network (CAN) -- Part 1: Data link layer and physical signalling\\
>> +    ISO 11898-1:2015 Road vehicles -- Controller area network (CAN) -- Part 1: Data link layer and physical signalling
>> +       \newline\url{https://ddec1-0-en-ctp.trendmicro.com:443/wis/clicktime/v1/query?url=https%3a%2f%2fwww.iso.org%2fstandard%2f63648.html%7d%5c%5c&umid=65841137-ddc9-4901-90a4-a0cb61c155d1&auth=53c7c7de28b92dfd96e93d9dd61a23e634d2fbec-0151c683df516f2ca3c42192c0ecb6e43e334d9f
>>   \end{longtable}
> I just realised that ISO 11898-1:2024 was published last month:
> https://ddec1-0-en-ctp.trendmicro.com:443/wis/clicktime/v1/query?url=https%3a%2f%2fwww.iso.org%2fstandard%2f86384.html&umid=65841137-ddc9-4901-90a4-a0cb61c155d1&auth=53c7c7de28b92dfd96e93d9dd61a23e634d2fbec-3887c3b5f94c9c2790492dc7ef4af9e35b9cc020.
>
> Just for confirmation, are you keeping the reference to ISO
> 11898-1:2015 until CAN XL support gets added? If yes, OK as-is.

Saw also yesterday the update at ISO when checking the link and thought 
a moment about it. The now superseded specification 11898-1:2015 is the 
one which was used when the virtio CAN specification was developed. So 
this is the one to be referenced now.

> On my side, I now need to read the new ISO 11898-1:2024.

On my side, before reading the new ISO 11898-1:2024 it first has to be 
ordered and to be delivered.

>>   \section{Non-Normative References}
>> --
>> 2.34.1
>>
> >