talos-op-linux/drivers/mailbox, branch v4.8.6

fix:mailbox:bcm-pdc-mailbox:mark symbols static where possible

2016-08-29T13:13:39+00:00

We get 2 warnings when biuld kernel with W=1: drivers/mailbox/bcm-pdc-mailbox.c:472:6: warning: no previous prototype for 'pdc_setup_debugfs' [-Wmissing-prototypes] drivers/mailbox/bcm-pdc-mailbox.c:488:6: warning: no previous prototype for 'pdc_free_debugfs' [-Wmissing-prototypes] In fact, these functions are only used in the file in which they are declared and don't need a declaration, but can be made static. so this patch marks these functions with 'static'. Signed-off-by: Baoyou Xie Acked-by: Arnd Bergmann Signed-off-by: Jassi Brar

mailbox: bcm-pdc: potential NULL dereference in pdc_shutdown()

2016-08-29T13:11:15+00:00

We can't pass NULL pointers to pdc_ring_free() so I moved the check for NULL. Signed-off-by: Dan Carpenter Signed-off-by: Jassi Brar

mailbox: Add HAS_DMA Kconfig dependency to BCM_PDC_MBOX

2016-08-29T13:09:46+00:00

Add HAS_DMA Kconfig dependency to BCM_PDC_MBOX to avoid link error on some platforms. Reported-by: Fengguang Wu Signed-off-by: Rob Rice Acked-by: Geert Uytterhoeven Signed-off-by: Jassi Brar

mailbox: Fix format and type mismatches in Broadcom PDC driver

2016-07-28T15:57:31+00:00

Fix format and type mismatches in a couple debug prints in the Broadcom PDC driver. Use %pad for dma_addr_t and %pa for resource_size_t. Signed-off-by: Rob Rice Reported-by: Fengguang Wu Signed-off-by: Jassi Brar

mailbox: Add Broadcom PDC mailbox driver

2016-07-28T04:04:47+00:00

The Broadcom PDC mailbox driver is a mailbox controller that manages data transfers to and from one or more offload engines. Signed-off-by: Rob Rice Reviewed-by: Scott Branden Reviewed-by: Ray Jui Signed-off-by: Jassi Brar

mailbox: pl320: remove __raw IO

2016-06-24T08:02:31+00:00

The use of __raw IO accesors is not endian safe and should be used sparingly. The relaxed variants should be as lightweight and also are endian safe. Note, this has not been run-time tested. Signed-off-by: Ben Dooks Acked-by: Rob Herring Signed-off-by: Jassi Brar

mailbox: mailbox-test: set tdev->signal to NULL after freeing

2016-06-12T09:43:04+00:00

tdev->signal is not set NULL after it's freed. This will cause random exceptions when the stale pointer is accessed after tdev->signal is freed. Also, since tdev->signal allocation is skipped the next time it's written, this leads to continuous fault finally leading to the total death of the system. Fixes: d1c2f87c9a8f ("mailbox: mailbox-test: Prevent memory leak") Signed-off-by: Sudeep Holla Acked-by: Lee Jones Signed-off-by: Jassi Brar

mailbox: Fix devm_ioremap_resource error detection code

2016-05-08T17:14:46+00:00

devm_ioremap_resource returns an ERR_PTR value, not NULL, on failure. The Coccinelle semantic patch used to make this change is as follows: @@ expression e,e1; statement S; @@ *e = devm_ioremap_resource(...); if (!e1) S Signed-off-by: Amitoj Kaur Chawla Signed-off-by: Jassi Brar

mailbox/omap: kill omap_mbox_{save/restore}_ctx() functions

2016-04-26T04:49:38+00:00

The omap_mbox_save_ctx() and omap_mbox_restore_ctx() API were previously provided to OMAP mailbox clients to save and restore the mailbox context during system suspend/resume. The save and restore functionality is now implemented through System PM driver callbacks, and there is no need for these functions, so kill these API. Signed-off-by: Suman Anna Signed-off-by: Jassi Brar

mailbox/omap: check for any unread messages during suspend

2016-04-26T04:49:38+00:00

The OMAP mailbox driver is used by clients to communicate with remote processors in general. The mailbox clients are expected to have stopped communicating with these remote processors during a system suspend. The OMAP mailbox fifos are expected to not have any messages as such. Add a check for any pending unprocessed messages in the suspend callback, to detect any communication protocol issues of the mailbox clients. The system suspend is aborted if any messages are found. Signed-off-by: Suman Anna Signed-off-by: Jassi Brar