Re: [PATCH][i3c-next] i3c: fix incorrect update to max_read_ds

2018-11-08 Thread Colin Ian King
On 08/11/2018 09:09, Boris Brezillon wrote: > On Wed, 7 Nov 2018 20:42:08 + > Hi Colin, > > Colin King wrote: > >> From: Colin Ian King >> >> Currently max_read_ds is being updated twice, which is incorrect. >> The second assignment should be to max_write_ds instead. Fix this. >> >>

Re: [PATCH][i3c-next] i3c: fix incorrect update to max_read_ds

2018-11-08 Thread Colin Ian King
On 08/11/2018 09:09, Boris Brezillon wrote: > On Wed, 7 Nov 2018 20:42:08 + > Hi Colin, > > Colin King wrote: > >> From: Colin Ian King >> >> Currently max_read_ds is being updated twice, which is incorrect. >> The second assignment should be to max_write_ds instead. Fix this. >> >>

Re: [PATCH][i3c-next] i3c: fix incorrect update to max_read_ds

2018-11-08 Thread Boris Brezillon
On Wed, 7 Nov 2018 20:42:08 + Hi Colin, Colin King wrote: > From: Colin Ian King > > Currently max_read_ds is being updated twice, which is incorrect. > The second assignment should be to max_write_ds instead. Fix this. > > Detected by CoverityScan, CID#1475379 ("Unused value") > >

Re: [PATCH][i3c-next] i3c: fix incorrect update to max_read_ds

2018-11-08 Thread Boris Brezillon
On Wed, 7 Nov 2018 20:42:08 + Hi Colin, Colin King wrote: > From: Colin Ian King > > Currently max_read_ds is being updated twice, which is incorrect. > The second assignment should be to max_write_ds instead. Fix this. > > Detected by CoverityScan, CID#1475379 ("Unused value") > >

[PATCH][i3c-next] i3c: fix incorrect update to max_read_ds

2018-11-07 Thread Colin King
From: Colin Ian King Currently max_read_ds is being updated twice, which is incorrect. The second assignment should be to max_write_ds instead. Fix this. Detected by CoverityScan, CID#1475379 ("Unused value") Fixes: 50cacdabeae1 ("i3c: Add core I3C infrastructure") Signed-off-by: Colin Ian

[PATCH][i3c-next] i3c: fix incorrect update to max_read_ds

2018-11-07 Thread Colin King
From: Colin Ian King Currently max_read_ds is being updated twice, which is incorrect. The second assignment should be to max_write_ds instead. Fix this. Detected by CoverityScan, CID#1475379 ("Unused value") Fixes: 50cacdabeae1 ("i3c: Add core I3C infrastructure") Signed-off-by: Colin Ian