On Thu, 1 Jun 2017 15:21:15 +0200 Boris Brezillon <[email protected]> wrote:
> On Thu, 1 Jun 2017 15:12:32 +0200 > Pavel Machek <[email protected]> wrote: > > > On Thu 2017-06-01 01:09:17, Darwin Dingel wrote: > > > Hi Pavel, > > > > > > Just a minor thing. Sorry about the late comment. > > > > > > On 01/06/17 08:59, Pavel Machek wrote: > > > > + > > > > + res = check_erased_page(chip, buf); > > > > + return res; > > > > + } > > > > > > Can we just remove 'res' and change this line to: > > > return check_erased_page(chip, buf); > > > > Well... I originally had a printk there, and yes, it can be > > simplified. I can roll v3, if required, but I'd leave it as is -- it > > is still convenient place to add debugging to. > > I can fix it when applying, no need to send a new version. Applied. Thanks, Boris
