[PATCH V4 2/4] MTD : add the common code for GPMI controller driver

Veli-Pekka Peltola veli-pekka.peltola at bluegiga.com
Tue Apr 12 07:03:06 EDT 2011


Hi,

I have one correction to your NAND driver. Please see inlined comments.

> These files contain the common code for the GPMI driver.
> 
> Signed-off-by: Huang Shijie <b32955 at freescale.com>
> ---
>  drivers/mtd/nand/gpmi-nfc/gpmi-nfc-main.c | 2453
> +++++++++++++++++++++++++++++
>  drivers/mtd/nand/gpmi-nfc/gpmi-nfc.h      |  551 +++++++
>  2 files changed, 3004 insertions(+), 0 deletions(-)
>  create mode 100644 drivers/mtd/nand/gpmi-nfc/gpmi-nfc-main.c
>  create mode 100644 drivers/mtd/nand/gpmi-nfc/gpmi-nfc.h
> 
> diff --git a/drivers/mtd/nand/gpmi-nfc/gpmi-nfc-main.c
> b/drivers/mtd/nand/gpmi-nfc/gpmi-nfc-main.c
> new file mode 100644
> index 0000000..7bbc448
> --- /dev/null
> +++ b/drivers/mtd/nand/gpmi-nfc/gpmi-nfc-main.c
> @@ -0,0 +1,2453 @@

<snip>

> +#ifdef CONFIG_PM
> +static int gpmi_nfc_suspend(struct platform_device *pdev,
pm_message_t
> state)
> +{
> +	return 0;
> +}
> +
> +static int gpmi_nfc_resume(struct platform_device *pdev)
> +{
> +	return 0;
> +}
> +#else
> +#define suspend  NULL
> +#define resume   NULL
> +#endif

If CONFIG_PM is disabled your defines don't match function names.

<snip>

> +/* This structure represents this driver to the platform management
> system. */
> +static struct platform_driver gpmi_nfc_driver = {
> +	.driver = {
> +		.name = GPMI_NFC_DRIVER_NAME,
> +	},
> +	.probe   = gpmi_nfc_probe,
> +	.remove  = __exit_p(gpmi_nfc_remove),
> +	.suspend = gpmi_nfc_suspend,
> +	.resume  = gpmi_nfc_resume,
> +	.id_table = gpmi_ids,
> +};

This section fails to compile as gpmi_nfc_suspend and gpmi_nfc_resume
are undefined. It compiles fine if I add gpmi_nfc_ prefix to defines.

--
Veli-Pekka Peltola
R&D Engineer
Bluegiga Technologies 




More information about the linux-mtd mailing list