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: Uwe Seimet <Uwe.Seimet@xxxxxxxxx>
- Date: Sun, 20 Mar 2022 09:12:37 +0100
- Authentication-results: strato.com; dkim=none
- Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; t=1647763958; s=strato-dkim-0002; d=seimet.de; h=In-Reply-To:References:Message-ID:Subject:To:From:Date:Cc:Date:From: Subject:Sender; bh=XTT0xjjX2xdM2yFpjjrHVBzyz5CTfPm3/sbLFApGgh0=; b=KWmPbyloFzr5rrbljXCmPM3YX8piwSEJP913+gkSJExE8B37/Gvv+TY3168NLLAyED 3YTH7BAqOb6mYprZQeSKtonR+W0IjGRCoSzZW3Pv9vBlnQi+IU2Wp2ZjefVTsREUxm4d f9OnsIZ8A8TSwpPBaNcZJIaHEFeGfV5Pjr2OYSGcxxaazTh3Q3w/WjklgcbbDtzS4JE4 bLCukVTcpVfJ6Rxv3pS0oxSaJa/9cissWx79kKZBtBPF6zJatWO6G4qdawmGY4AJtPtX omMA8sPNTcQwHSroclfHSY/KJzaRm8GJy5OtqurjgE5meZzA6lnk22VBJPtlPazvPpCa gJHQ==
Hi,
> So it would be better to have one patch to update the magic constants, one
> patch to fix mode sense, etc.
I know what you mean, but it would have made everything even more
complicated from my perspective. It took very long for my first patch to
be reviewed, and at that time there were already other minor changes in my
pipeline.
> - 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.
This would have prepared future command extensions, but were not directly
related to the MODE SENSE and REPORT LUNS changes.
Anyway, please just discard all of my recent patches. I understand the points
you are making, but all in all the Hatari development workflow is not
compatible with me (anymore). Even though it might be close to the Linux
one ;-).
I am sorry that others had to spend that much time on my patches.
Best regards
Uwe