linux-kernel - RE: [LINUX PATCH v12 2/2] memory: pl353: Add driver for arm pl353 static memory controller

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <MWHPR02MB262379941E7314EA6C3FC68DAFA90@MWHPR02MB2623.namprd02.prod.outlook.com>
Date:   Thu, 6 Dec 2018 08:29:57 +0000
From:   Naga Sureshkumar Relli <nagasure@...inx.com>
To:     Michal Simek <michals@...inx.com>,
        "mmayer@...adcom.com" <mmayer@...adcom.com>,
        "f.fainelli@...il.com" <f.fainelli@...il.com>,
        "ladis@...ux-mips.org" <ladis@...ux-mips.org>,
        "treding@...dia.com" <treding@...dia.com>,
        "digetx@...il.com" <digetx@...il.com>,
        "d-gerlach@...com" <d-gerlach@...com>
CC:     "linus.walleij@...aro.org" <linus.walleij@...aro.org>,
        Michal Simek <michals@...inx.com>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [LINUX PATCH v12 2/2] memory: pl353: Add driver for arm pl353
 static memory controller

Hi Michal,

Ok, will update and will send new patch.

Thanks,
Naga Sureshkumar Relli

> -----Original Message-----
> From: Michal Simek [mailto:michal.simek@...inx.com]
> Sent: Wednesday, December 5, 2018 8:17 PM
> To: Naga Sureshkumar Relli <nagasure@...inx.com>; mmayer@...adcom.com;
> f.fainelli@...il.com; ladis@...ux-mips.org; treding@...dia.com; digetx@...il.com; d-
> gerlach@...com
> Cc: linus.walleij@...aro.org; Michal Simek <michals@...inx.com>; linux-
> kernel@...r.kernel.org
> Subject: Re: [LINUX PATCH v12 2/2] memory: pl353: Add driver for arm pl353 static
> memory controller
> 
> On 29. 11. 18 13:57, Naga Sureshkumar Relli wrote:
> > Add driver for arm pl353 static memory controller. This controller is
> > used in Xilinx Zynq SoC for interfacing the NAND and NOR/SRAM memory devices.
> >
> > Signed-off-by: Naga Sureshkumar Relli
> > <naga.sureshkumar.relli@...inx.com>
> > Reviewed-by: Linus Walleij <linus.walleij@...aro.org>
> > ---
> >  drivers/memory/Kconfig     |   9 +
> >  drivers/memory/Makefile    |   1 +
> >  drivers/memory/pl353-smc.c | 468
> > +++++++++++++++++++++++++++++++++++++++++++++
> >  include/linux/pl353-smc.h  |  30 +++
> >  4 files changed, 508 insertions(+)
> >  create mode 100644 drivers/memory/pl353-smc.c  create mode 100644
> > include/linux/pl353-smc.h
> >
> 
> Missing changelog which was there in v11
> https://lore.kernel.org/patchwork/patch/961604/
> 
> > diff --git a/drivers/memory/Kconfig b/drivers/memory/Kconfig index
> > 63389f0..2d91b00 100644
> > --- a/drivers/memory/Kconfig
> > +++ b/drivers/memory/Kconfig
> > @@ -145,6 +145,15 @@ config DA8XX_DDRCTL
> >  	  Texas Instruments da8xx SoCs. It's used to tweak various memory
> >  	  controller configuration options.
> >
> > +config PL353_SMC
> > +	tristate "ARM PL35X Static Memory Controller(SMC) driver"
> > +	default y
> > +	depends on ARM
> > +	depends on ARM_AMBA
> > +	help
> > +	  This driver is for the ARM PL351/PL353 Static Memory
> > +	  Controller(SMC) module.
> > +
> >  source "drivers/memory/samsung/Kconfig"
> >  source "drivers/memory/tegra/Kconfig"
> >
> > diff --git a/drivers/memory/Makefile b/drivers/memory/Makefile index
> > a01ab3e..90161de 100644
> > --- a/drivers/memory/Makefile
> > +++ b/drivers/memory/Makefile
> > @@ -19,6 +19,7 @@ obj-$(CONFIG_MVEBU_DEVBUS)	+= mvebu-devbus.o
> >  obj-$(CONFIG_JZ4780_NEMC)	+= jz4780-nemc.o
> >  obj-$(CONFIG_MTK_SMI)		+= mtk-smi.o
> >  obj-$(CONFIG_DA8XX_DDRCTL)	+= da8xx-ddrctl.o
> > +obj-$(CONFIG_PL353_SMC)		+= pl353-smc.o
> >
> >  obj-$(CONFIG_SAMSUNG_MC)	+= samsung/
> >  obj-$(CONFIG_TEGRA_MC)		+= tegra/
> > diff --git a/drivers/memory/pl353-smc.c b/drivers/memory/pl353-smc.c
> > new file mode 100644 index 0000000..6738a84
> > --- /dev/null
> > +++ b/drivers/memory/pl353-smc.c
> > @@ -0,0 +1,468 @@
> > +// SPDX-License-Identifier: GPL-2.0
> > +/*
> > + * ARM PL353 SMC driver
> > + *
> > + * Copyright (C) 2012 - 2018 Xilinx, Inc
> > + * Author: Punnaiah Choudary Kalluri <punnaiah@...inx.com>
> > + * Author: Naga Sureshkumar Relli <nagasure@...inx.com>
> > + *
> 
> remove line above.
> 
> and then fix these two
> 
> CHECK: Please use a blank line after function/struct/union/enum declarations
> #359: FILE: drivers/memory/pl353-smc.c:299:
> +}
> +static struct amba_driver pl353_smc_driver;
> 
> CHECK: Alignment should match open parenthesis
> #369: FILE: drivers/memory/pl353-smc.c:309:
> +void pl353_smc_init_nand_interface(struct amba_device *adev,
> +					struct device_node *nand_node)
> 
> total: 0 errors, 2 warnings, 2 checks, 520 lines checked
> 
> NOTE: For some of the reported defects, checkpatch may be able to
> 
> and also
> this doesn't right too.
> 
> drivers/memory/pl353-smc.c:308:6: warning: symbol 'pl353_smc_init_nand_interface' was
> not declared. Should it be static?
>   CC      drivers/memory/pl353-smc.o
> drivers/memory/pl353-smc.c:308:6: warning: no previous prototype for
> ‘pl353_smc_init_nand_interface’ [-Wmissing-prototypes]  void
> pl353_smc_init_nand_interface(struct amba_device *adev,
>       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> drivers/memory/pl353-smc.c:439:34: warning: ‘pl353_smc_of_match’ defined but not used
> [-Wunused-const-variable=]  static const struct of_device_id pl353_smc_of_match[] = {
> 
> Thanks,
> Michal