RE: [PATCH v2] phy: renesas: phy-rcar-gen2: Fix the array off by one warning

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Kishon,

Thanks for the feedback

> -----Original Message-----
> From: Kishon Vijay Abraham I <kishon@xxxxxx>
> Sent: Wednesday, October 9, 2019 1:55 PM
> To: Biju Das <biju.das@xxxxxxxxxxxxxx>
> Cc: Simon Horman <horms+renesas@xxxxxxxxxxxx>; Wolfram Sang
> <wsa+renesas@xxxxxxxxxxxxxxxxxxxx>; Simon Horman
> <horms@xxxxxxxxxxxx>; Geert Uytterhoeven <geert+renesas@xxxxxxxxx>;
> Chris Paterson <Chris.Paterson2@xxxxxxxxxxx>; Fabrizio Castro
> <fabrizio.castro@xxxxxxxxxxxxxx>; linux-renesas-soc@xxxxxxxxxxxxxxx;
> Yoshihiro Shimoda <yoshihiro.shimoda.uh@xxxxxxxxxxx>
> Subject: Re: [PATCH v2] phy: renesas: phy-rcar-gen2: Fix the array off by one
> warning
> 
> Hi Biju,
> 
> On 01/10/19 12:09 PM, Biju Das wrote:
> > Hello Kishon,
> >
> > Are you happy with this patch? Please let me know.
> >
> > https://patchwork.kernel.org/patch/10946601/
> 
> 
> Can you resend the patch with the updated tags please?
> 

Yes. Will do

Regards,
Biju

> >
> >
> > Regards,
> > Biju
> >
> >> Subject: RE: [PATCH v2] phy: renesas: phy-rcar-gen2: Fix the array
> >> off by one warning
> >>
> >> Hi Biju-san,
> >>
> >>> From: Biju Das, Sent: Thursday, May 16, 2019 10:22 PM
> >>>
> >>> Fix the below smatch warning by adding variable check rather than
> >>> the hardcoded value.
> >>> warn: array off by one? 'data->select_value[channel_num]'
> >>>
> >>> Reported-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
> >>> Signed-off-by: Biju Das <biju.das@xxxxxxxxxxxxxx>
> >>> Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
> >>
> >> Thank you for the patch!
> >>
> >> Reviewed-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@xxxxxxxxxxx>
> >>
> >> Best regards,
> >> Yoshihiro Shimoda
> >




[Index of Archives]     [Linux Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux