On Fri, Oct 02, 2020 at 02:22:41PM +0200, Łukasz Stelmach wrote: > Make sure the cur_speed value used in s3c64xx_enable_datapath() > to configure DMA channel and in s3c64xx_wait_for_*() to calculate the > transfer timeout is set to the actual value of (half) the clock speed. > > Don't change non-CMU case, because no frequency calculation errors have > been reported. > > Reviewed-by: Krzysztof Kozlowski <k...@kernel.org> > Suggested-by: Tomasz Figa <tomasz.f...@gmail.com> > Signed-off-by: Łukasz Stelmach <l.stelm...@samsung.com>
For the future, tags are added in chronological order, so first is suggested (as someone suggested to make a patch), then your SoB (as you wrote it) and then my review (because you had to write a patch before I could review). All other patches here have these mixed up. No need to resend, but keep it in mind for the future. Best regards, Krzysztof
