Re: [RFC PATCH 3/8] mtd: rawnand: ams-delta: Set port direction once per transfer

From: Janusz Krzysztofik
Date: Fri Jul 20 2018 - 14:11:27 EST


On Thursday, July 19, 2018 8:23:18 AM CEST Boris Brezillon wrote:
> On Thu, 19 Jul 2018 01:57:05 +0200
> Janusz Krzysztofik <jmkrzyszt@xxxxxxxxx> wrote:
>
> > In its current shape, the driver sets data port direction before each
> > byte read/write operation, even during multi-byte transfers. Optimize
> > that by setting the port direction only on first byte of each transfer.
>
> Sounds like premature optimization for something you'll rework when
> fully switching to the GPIO consumer API to control the DATA bus.

Indeed, this optimization was crucial for getting acceptable performance of
data transfers over GPIO. I'm only not sure if there is any action in
response to your comment expected on my side, e.g., did you want to say I
should modify the patch description, or change the order of patches?

Thanks,
Janusz