Re: [PATCH 01/31] swim: Assert strobe with stable outputs

Le 21/07/2026 =C3=A0 09:37, Finn Thain a =C3=A9crit=C2=A0:
>=20
> On Tue, 21 Jul 2026, Laurent Vivier wrote:
>=20
>> Le 21/07/2026 =C3=A0 07:41, Finn Thain a =C3=A9crit=C2=A0:
>>>
>>> On Mon, 20 Jul 2026, Laurent Vivier wrote:
>>>
>>>> Le 16/07/2026 =C3=A0 12:02, Finn Thain a =C3=A9crit=C2=A0:
>>>>
>>>>> Keep the four phase pin directions set to output and hold their leve=
ls
>>>>> constant during LSTRB signalling.
>>>>
>>>> Perhaps you can also add a short comment to describe the changes you
>>>> made to the defines.
>>>>
>>>
>>> Can you clarify please? I think what you're asking is, shouldn't the P=
H3
>>> pin be an input when it's not being used for LSTRB signalling?
>>
>> My question is simpler:
>> You added the directions bits in the defines and it's not clear in the =
commit
>> message.
>> For instance, I can see you update all the defines, but at first glance=
 I
>> didn't understand why.
>>
>=20
> OK -- I see what you mean. It might be more idiomatic and more readable =
if
> the the pin direction nybble were defined in a separate macro rather tha=
n
> baked into every macro definition. E.g. #define PHASE_PIN_DIRECTION 0xF0
> The writes to the phase register would then get a bitwise OR with this n=
ew
> macro. Would that work for you?

Yes, perfectly.

Thanks,
Laurent