[bug report] mtd: rawnand: allocate dynamically ONFI parameters during detection

[dan.carpenter@xxxxxxxxxx (Dan Carpenter)]

Hello Miquel Raynal,

This is a semi-automatic email about new static checker warnings.

The patch 0ff393463e9c: "mtd: rawnand: allocate dynamically ONFI 
parameters during detection" from Jul 25, 2018, leads to the 
following Smatch complaint:

    drivers/mtd/nand/raw/nand_micron.c:93 micron_nand_onfi_init()
     warn: variable dereferenced before check 'p->onfi' (see line 91)

drivers/mtd/nand/raw/nand_micron.c
    90		struct nand_parameters *p = &chip->parameters;
    91		struct nand_onfi_vendor_micron *micron = (void *)p->onfi->vendor;
                                                                 ^^^^^^^^^^^^^^^
The code dereferences "p->onfi"

    92	
    93		if (p->onfi) {
                    ^^^^^^^
but the patch adds a new check for NULL.

    94			chip->read_retries = micron->read_retry_options;
    95			chip->setup_read_retry = micron_nand_setup_read_retry;

regards,
dan carpenter