Messages in this thread | | | From | Linus Walleij <> | Date | Tue, 13 Sep 2016 13:49:25 +0200 | Subject | Re: [PATCH] gpio: pca953x: initialize ret to zero to avoid returning garbage |
| |
On Tue, Sep 13, 2016 at 6:04 AM, Phil Reid <preid@electromag.com.au> wrote: > On 12/09/2016 21:40, Linus Walleij wrote: >> >> On Fri, Sep 9, 2016 at 10:31 AM, Colin King <colin.king@canonical.com> >> wrote: >> >>> From: Colin Ian King <colin.king@canonical.com> >>> >>> ret is not initialized so it contains garbage. Ensure garbage >>> is not returned in the case that pdata && pdata->teardown is false >>> by initializing ret to 0. >>> >>> Signed-off-by: Colin Ian King <colin.king@canonical.com> >> >> >> Patch applied. > > G'day Linus, > > > I believe this does the same as Arnd Bergmanns patch from 26 Aug. > [PATCH 1/5] gpio: pca954x: fix undefined error code from remove > > IMO Colin's is the cleaner solution. > Arnd's could be removed.
OK you're right. I reverted Arnd's patch.
Yours, Linus Walleij
| |