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
| ||
|
Date: Thu, 9 Feb 2017 15:09:01 +1030 From: Joel Stanley <joel@....id.au> To: Cyril Bur <cyrilbur@...il.com> Cc: Jassi Brar <jassisinghbrar@...il.com>, linux-kernel@...r.kernel.org, Benjamin Herrenschmidt <benh@...nel.crashing.org>, Michael Neuling <mikey@...ling.org>, OpenBMC Maillist <openbmc@...ts.ozlabs.org> Subject: Re: [PATCH v2] drivers/mailbox: Add Aspeed mailbox driver On Wed, Feb 8, 2017 at 10:06 AM, Cyril Bur <cyrilbur@...il.com> wrote: > This provides access to the mbox registers on the ast2400 and ast2500 > SoCs. > > This driver allows arbitrary reads and writes to the 16 data registers as > the other end may have configured the mbox hardware to provide an > interrupt when a specific register gets written to. > > Signed-off-by: Cyril Bur <cyrilbur@...il.com> > --- > V2: > s/ASpeed/Aspeed/ > Reword Kconfig options > Use tristate for config symbol > > drivers/mailbox/Kconfig | 8 + > drivers/mailbox/Makefile | 2 + > drivers/mailbox/aspeed-mbox.c | 334 ++++++++++++++++++++++++++++++++++++++++++ > 3 files changed, 344 insertions(+) > create mode 100644 drivers/mailbox/aspeed-mbox.c > > diff --git a/drivers/mailbox/Kconfig b/drivers/mailbox/Kconfig > index ceff415f201c..e24044d5c219 100644 > --- a/drivers/mailbox/Kconfig > +++ b/drivers/mailbox/Kconfig > @@ -152,4 +152,12 @@ config BCM_PDC_MBOX > Mailbox implementation for the Broadcom PDC ring manager, > which provides access to various offload engines on Broadcom > SoCs. Say Y here if you want to use the Broadcom PDC. > + > +config ASPEED_LPC_MBOX > + depends on (ARCH_ASPEED || COMPILE_TEST) && REGMAP && MFD_SYSCON > + tristate "Aspeed LPC Mailbox Controller" > + ---help--- > + Provides a driver for the MBOX registers found on Aspeed SOCs > + (AST2400 and AST2500). This driver provides a device for aspeed > + mbox registers You repeat yourself. Mention that this driver is for the the BMC side. > diff --git a/drivers/mailbox/aspeed-mbox.c b/drivers/mailbox/aspeed-mbox.c > +static int aspeed_mbox_probe(struct platform_device *pdev) > +{ > + struct aspeed_mbox *mbox; > + struct device *dev; > + int rc; > + > + dev = &pdev->dev; > + > + mbox = devm_kzalloc(dev, sizeof(*mbox), GFP_KERNEL); > + if (!mbox) > + return -ENOMEM; > + > + dev_set_drvdata(&pdev->dev, mbox); > + > + rc = of_property_read_u32(dev->of_node, "reg", &mbox->base); > + if (rc) { > + dev_err(dev, "Couldn't read reg device-tree property\n"); > + return rc; > + } I think we want to use of_address_to_resource here. > + > +static const struct of_device_id aspeed_mbox_match[] = { > + { .compatible = "aspeed,ast2400-mbox" }, > + { .compatible = "aspeed,ast2500-mbox" }, > + { }, I didn't see the bindings in this series. Have they already been accepted? Cheers, Joel
Powered by blists - more mailing lists