Re: [PATCH v2 2/2] mtd: nand: atmel_nand: fix a possible NULL dereference

2015-11-24 Thread Josh Wu
Hi, Corentin On 11/24/2015 9:12 PM, LABBE Corentin wrote: On Fri, Nov 20, 2015 at 04:33:14PM +0800, Josh Wu wrote: Hi, Corentin Thanks for the patch. It looks fine to me, just one nitpick in below: On 11/20/2015 3:45 PM, LABBE Corentin wrote: of_match_device could return NULL, and so cause

Re: [PATCH v2 2/2] mtd: nand: atmel_nand: fix a possible NULL dereference

2015-11-24 Thread LABBE Corentin
On Fri, Nov 20, 2015 at 04:33:14PM +0800, Josh Wu wrote: > Hi, Corentin > > Thanks for the patch. It looks fine to me, just one nitpick in below: > > On 11/20/2015 3:45 PM, LABBE Corentin wrote: > > of_match_device could return NULL, and so cause a NULL pointer > > dereference later. > > > >

Re: [PATCH v2 2/2] mtd: nand: atmel_nand: fix a possible NULL dereference

2015-11-24 Thread Josh Wu
Hi, Corentin On 11/24/2015 9:12 PM, LABBE Corentin wrote: On Fri, Nov 20, 2015 at 04:33:14PM +0800, Josh Wu wrote: Hi, Corentin Thanks for the patch. It looks fine to me, just one nitpick in below: On 11/20/2015 3:45 PM, LABBE Corentin wrote: of_match_device could return NULL, and so cause

Re: [PATCH v2 2/2] mtd: nand: atmel_nand: fix a possible NULL dereference

2015-11-24 Thread LABBE Corentin
On Fri, Nov 20, 2015 at 04:33:14PM +0800, Josh Wu wrote: > Hi, Corentin > > Thanks for the patch. It looks fine to me, just one nitpick in below: > > On 11/20/2015 3:45 PM, LABBE Corentin wrote: > > of_match_device could return NULL, and so cause a NULL pointer > > dereference later. > > > >

Re: [PATCH v2 2/2] mtd: nand: atmel_nand: fix a possible NULL dereference

2015-11-20 Thread Josh Wu
Hi, Corentin Thanks for the patch. It looks fine to me, just one nitpick in below: On 11/20/2015 3:45 PM, LABBE Corentin wrote: of_match_device could return NULL, and so cause a NULL pointer dereference later. Signed-off-by: LABBE Corentin --- drivers/mtd/nand/atmel_nand.c | 5 +++-- 1

Re: [PATCH v2 2/2] mtd: nand: atmel_nand: fix a possible NULL dereference

2015-11-20 Thread Josh Wu
Hi, Corentin Thanks for the patch. It looks fine to me, just one nitpick in below: On 11/20/2015 3:45 PM, LABBE Corentin wrote: of_match_device could return NULL, and so cause a NULL pointer dereference later. Signed-off-by: LABBE Corentin ---

[PATCH v2 2/2] mtd: nand: atmel_nand: fix a possible NULL dereference

2015-11-19 Thread LABBE Corentin
of_match_device could return NULL, and so cause a NULL pointer dereference later. Signed-off-by: LABBE Corentin --- drivers/mtd/nand/atmel_nand.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/mtd/nand/atmel_nand.c b/drivers/mtd/nand/atmel_nand.c index

[PATCH v2 2/2] mtd: nand: atmel_nand: fix a possible NULL dereference

2015-11-19 Thread LABBE Corentin
of_match_device could return NULL, and so cause a NULL pointer dereference later. Signed-off-by: LABBE Corentin --- drivers/mtd/nand/atmel_nand.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/mtd/nand/atmel_nand.c