[PATCH 1/5] MTD: mtdram: properly handle the phys argument in the point method

Nicolas Pitre nicolas.pitre at linaro.org
Wed Oct 11 14:53:22 PDT 2017


On Wed, 11 Oct 2017, Richard Weinberger wrote:

> On Wed, Oct 11, 2017 at 11:32 PM, Nicolas Pitre
> <nicolas.pitre at linaro.org> wrote:
> > In fact the above should be: *retlen = addr - *virt;
> 
> Hmm, yes.
> 
> While we are here, how did you test that code?

I didn't actually manage to trigger it as physical pages were always 
contiguous in my testing. So this is from inspection i.e. I just re-read 
the code from your reply.

> > Let me know if I should repost.
> 
> Yes, please. But wait a few days, hopefully we get more reviews.

Sure.


Nicolas



More information about the linux-mtd mailing list