> On 14.12.2015, at 16:20, ker...@martin.sperl.org wrote:
> +     /* the number of bytes by which we need to shift to align */
> +     alignment_shift = BIT(alignment) - alignment_to_correct;

This actually needs to be:
        alignment_shift = alignment - alignment_to_correct

> +
> +     /* calculate the values */
> +     alignment_mask = (1 << alignment) - 1;
This actually needs to be:
         alignment_mask = alignment - 1

This was due to a misconception of @spi_master.dma_alignment...

I will wait for other feedback before creating v4 of the
patchset.

Martin
--
To unsubscribe from this list: send the line "unsubscribe linux-spi" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to