[PATCH 1/4] mmc: mmci: Bugfix in pio read for small packets
Ulf Hansson
ulf.hansson at stericsson.com
Fri Oct 14 04:07:54 EDT 2011
>>> So if they apply, they are independent AFAICT.
>> 7108/1 and 7109/1, have real dependencies. Otherwise there none.
>
> Ok, I assume that those depend on the first two patches.
>
> So, I tried applying 7110/1 .. 7112/1 but the first rejects because it
> doesn't have the non-power-of-2 support patch applied (7107/1). And
> it seems sensible that 7107/1 depends on 7106/1 (the PIO patch) which
> I believe to be a problem.
>
I see the problem.
The patches from the patch serie "Clarify code paths for how to modify
the power register" are independent from this "SDIO serie". Although the
patches has been based upon the patches from the SDIO serie, which is as
you say the reason to why they not apply cleanly.
I will upload a second version of the serie "Clarify code paths for how
to modify the power register" which is NOT based on the "SDIO serie".
Then when can discuss each serie separately.
> Therefore, I don't think any of these can be applied without the
> initial PIO patch.
>
> One thing I haven't yet mentioned is about the non-power-of-2 support -
> surely this can only be supported if blksz_datactrl16 is set? If so,
> shouldn't it key off that? I don't see how it could otherwise support
> non-power of 2 block sizes with just a log2 of the block size programmed
> into the data control register.
>
Your observation is correct, but at the same time do we really need to
have such a dependency check in the code? The hardware setup is handled
in the variant struct completely, so we believe that should be enough.
More information about the linux-arm-kernel
mailing list