[PATCH 1/1] include/ata.h: remove invalid links
xypron.glpk at gmx.de
Wed Feb 26 20:00:32 CET 2020
On 2/26/20 5:05 AM, Tom Rini wrote:
> On Wed, Feb 26, 2020 at 04:57:51AM +0100, Heinrich Schuchardt wrote:
>> On 2/25/20 10:30 PM, Tom Rini wrote:
>>> On Tue, Feb 25, 2020 at 10:04:53PM +0100, Heinrich Schuchardt wrote:
>>>> Remove invalid URLs for "Information Technology - AT Attachment-3 Interface
>>>> Signed-off-by: Heinrich Schuchardt <xypron.glpk at gmx.de>
>>>> include/ata.h | 4 ----
>>>> 1 file changed, 4 deletions(-)
>>>> diff --git a/include/ata.h b/include/ata.h
>>>> index 3f4e4a0234..aecb9cd589 100644
>>>> --- a/include/ata.h
>>>> +++ b/include/ata.h
>>>> @@ -7,10 +7,6 @@
>>>> * Most of the following information was derived from the document
>>>> * "Information Technology - AT Attachment-3 Interface (ATA-3)"
>>>> - * which can be found at:
>>>> - * http://www.dt.wdc.com/ata/ata-3/ata3r5v.zip
>>>> - * ftp://poctok.iae.nsk.su/pub/asm/Documents/IDE/ATA3R5V.ZIP
>>>> - * ftp://ftp.fee.vutbr.cz/pub/doc/io/ata/ata-3/ata3r5v.zip
>>>> #ifndef _ATA_H
>>> In this case, it looks like some of the relevant information about the
>>> spec, "Revision 5v" is encoded in the URL. I can't find a new link for
>>> it, but we should keep that information in the comment. Thanks!
>> Hello Tom,
>> revision 5v is an early draft of ANSI X3.298-1997. On Jan 27th, 1997
>> there was a draft 7b. Google finds it when you look for 'ata-3-std.pdf
>> The final standard ANSI X3.298-1997 is for sale here:
>> So I think we should simply point to the standard.
> I think we need to note what revision of the standard we wrote to, for
> future "hey, wait, there's a corner case we get wrong because we coded
> to an old draft" issues.
ata.h includes libata.h which is at least partially based on ATA/ATAPI-7
(a later standard than ATA-3). Many defines in ata.h are simply
duplicate or unused and should be removed. ata.h contains definitions of
variables that are not in the draft 7b ATA-3 standard.
I have no document indicating if there ever was a revision 5v or if this
is just our assumption due to the file name.
As ata.h is not based on a single revision of the ATA standard a
reference to some draft revision will not provide guidance to any future
developer. The future developer anyway will not have access to such a
More information about the U-Boot