Re: [hatari-devel] MODE SENSE patch |
[ Thread Index |
Date Index
| More lists.tuxfamily.org/hatari-devel Archives
]
- To: hatari-devel@xxxxxxxxxxxxxxxxxxx
- Subject: Re: [hatari-devel] MODE SENSE patch
- From: Thomas Huth <th.huth@xxxxxxxxx>
- Date: Sun, 20 Mar 2022 07:26:12 +0000
- Dkim-signature: v=1; a=rsa-sha256; c=relaxed/simple; d=posteo.de; s=2017; t=1647761177; bh=mV1v53UWaxoY4kzdpvU9F8DtHmMYzjbUviRrPuf5hHI=; h=Date:From:To:Subject:From; b=gnU+2IXDJYmSNOXw8ag8Ewk8AIWp+zmVzAdcooAoDR9nbq1Jzq5yVDh63ro2jdi4W ZaWZGuzmfuhnsxHnhMKIl3YNzO8658PhyWEWJISGLs2UzEw8qHW4BHzleWjN5XuqU4 kNKEaNP2dOPiIncAUOCG2kmdUaI+/MekrQlw51x6hsYn1cFXIVXqJcFU32UrydUs4z 88y3HkpYs0uqFPS2q6uTe+JsX+Y11Z8s9d2irjQ5ls4ijaY1Vg9qfdEY/mFqxC40So +G687oGtzqo8+C39nTP4yeX2l8JsD2NEtlByJjDtuFK06Cb1MHBpm6eLRTDLtkbtfx oEvv/2nZZrTuw==
Am Thu, 17 Mar 2022 22:55:07 +0100
schrieb Uwe Seimet <Uwe.Seimet@xxxxxxxxx>:
> This is my (consolidated) patch. I hope I did not miss anything. As already
> mentioned the DBD bit is not checked, but it was also not checked before,
> and I did not want to spend more time on this.
Honestly, it's better to change only one thing at a time if they are
unrelated. Otherwise, patch review (and debugging later) gets way more
complicated.
So it would be better to have one patch to update the magic constants, one
patch to fix mode sense, etc.
So for this patch:
static int HDC_GetCommandByteCount(SCSI_CTRLR *ctr)
{
- if (ctr->opcode == 0x88 || ctr->opcode == 0x8a || ctr->opcode == 0x8f ||
- ctr->opcode == 0x91 || ctr->opcode == 0x9e || ctr->opcode == 0x9f) {
- return 16;
- }
- else if (ctr->opcode == 0xa0) {
+ if (ctr->opcode == HD_REPORT_LUNS) {
return 12;
}
What happened to the "return 16"? Why is it ok to remove it? ==> This
information is missing in the patch description.
Thomas