Thread (44 messages) 44 messages, 5 authors, 2016-09-05

Re: [PATCH v8 01/18] remoteproc: st_slim_rproc: add a slimcore rproc driver

From: Lee Jones <hidden>
Date: 2016-08-31 11:22:22
Also in: dri-devel, linux-arm-kernel, linux-remoteproc, lkml

On Tue, 30 Aug 2016, Peter Griffin wrote:
On Tue, 30 Aug 2016, Lee Jones wrote:
quoted
On Fri, 26 Aug 2016, Peter Griffin wrote:
quoted
slim core is used as a basis for many IPs in the STi
chipsets such as fdma and demux. To avoid duplicating
the elf loading code in each device driver a slim
rproc driver has been created.

This driver is designed to be used by other device drivers
such as fdma, or demux whose IP is based around a slim core.
The device driver can call slim_rproc_alloc() to allocate
a slim rproc and slim_rproc_put() when finished.

This driver takes care of ioremapping the slim
registers (dmem, imem, slimcore, peripherals), whose offsets
and sizes can change between IP's. It also obtains and enables
any clocks used by the device. This approach avoids having
a double mapping of the registers as slim_rproc does not register
its own platform device. It also maps well to device tree
abstraction as it allows us to have one dt node for the whole
device.

All of the generic rproc elf loading code can be reused, and
we provide start() stop() hooks to start and stop the slim
core once the firmware has been loaded. This has been tested
successfully with fdma driver.
Nit.  It would be good to use a constant line-wrap.

'M-x post-mode' will help with this.
Can you provide the magic which makes this happen for GIT commit messages?
I tend to do it manually.  However a 3 second Google search produced
[0], which looks like it could be fun/useful.

[0] https://www.emacswiki.org/emacs/Git

[...]
quoted
quoted
+ * License terms:  GNU General Public License (GPL), version 2
Are you sure ST are okay with the shortened version of the GPL?
Do you mean the banner should be like this?

 * This program is free software; you can redistribute it and/or modify
 * it under the terms of the GNU General Public License as published by
 * the Free Software Foundation; either version 2 of the License, or
 * (at your option) any later version.
Yes, exactly.

[...]
quoted
quoted
+/* slimcore registers */
What's it called? slimcore, slim core, ST Slim?
It is usually referred to as SLIM core, or SLIM CPU in the various functional
specifications.
quoted
Please be consistent.  Use the name from the datasheet.
OK. The datasheet isn't consistent either, so we will settle on SLIM core and
SLIM CPU.
Perfect.

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help