]> Gitweb @ Texas Instruments - Open Source Git Repositories - git.TI.com/gitweb - ipc/ipcdev.git/log
ipc/ipcdev.git
10 years agoQNX: Stop building early for unsupported platforms
Chris Ring [Thu, 3 Apr 2014 20:47:24 +0000 (13:47 -0700)]
QNX: Stop building early for unsupported platforms

Catch an unsupported platform sooner with a simple check.

Without this check, the build proceeds and fails halfway through with
a confusing error - in my case it was when the loader was being built
and some of the required preprocessor defines weren't set.

10 years agoFixed 'use' help text to add new -t option in ipc_trace_daemon for QNX
vwan@ti.com [Thu, 3 Apr 2014 18:27:41 +0000 (11:27 -0700)]
Fixed 'use' help text to add new -t option in ipc_trace_daemon for QNX

Minor update of the help text to add the option, in case the user uses the
'use' command for help.

This addresses CQ SDOCM00107256.

Signed-off-by: VW <vwan@ti.com>
10 years agoAdd new resource type and corresponding struct for internal memory
Robert Tivy [Fri, 21 Mar 2014 00:03:41 +0000 (17:03 -0700)]
Add new resource type and corresponding struct for internal memory

A new resource type TYPE_INTMEM is added that allows specifying internal
memory resources that the kernel will map and later on load ELF file
sections.

This new type replaces TYPE_CRASHDUMP, which is not used or supported.

10 years agoFix VirtQueue_getAvailBuf on QNX to avoid race condition
vwan@ti.com [Tue, 1 Apr 2014 23:20:27 +0000 (16:20 -0700)]
Fix VirtQueue_getAvailBuf on QNX to avoid race condition

This is to address SDOCM00106156.

In commit c98d2c5624aac20bcdd738f5a54fbddca045fd69, we have fixed a race
condition on the BIOS side by clearing the VRING_USED_F_NO_NOTIFY flag
prior to checking whether there are remaining buffers. We are now doing
the same on the host side.

Note that this code is not currently exercised on the host, but is simply
maintained to ensure that the VirtQueue module is complete in functionality.

Signed-off-by: VW <vwan@ti.com>
10 years agoLinux: Quiet "unused variable" build warning
Chris Ring [Tue, 1 Apr 2014 23:40:14 +0000 (16:40 -0700)]
Linux: Quiet "unused variable" build warning

For Linux-based devices without GateMP support (e.g. OMAP5, TCI6638),
remove an unused variable declaration.

10 years agoDocs: Minor spec clarifications for MmRpc
Chris Ring [Thu, 20 Mar 2014 23:04:06 +0000 (16:04 -0700)]
Docs: Minor spec clarifications for MmRpc

Clarify that the 'handle' parameter for various MmRpc APIs must
be valid.

10 years agoBuild: Removed unused ti.sdo.ipc.build pkg 3.22.00.01_eng 3.22.00.02_eng
Chris Ring [Mon, 17 Mar 2014 18:14:52 +0000 (11:14 -0700)]
Build: Removed unused ti.sdo.ipc.build pkg

This package is no longer used and can be removed.

10 years agoSDOCM00106949 IPC interrupt preempts itself on ARP32 (EVE)
Ramsey Harris [Fri, 14 Mar 2014 22:19:09 +0000 (15:19 -0700)]
SDOCM00106949 IPC interrupt preempts itself on ARP32 (EVE)

Changed default interrupt vector to actual values. When creating Hwi
instances, compute a mask which blocks both interrupt vectors. Set
the mask in Hwi create params.

10 years agoSDOCM00106428 Notify mailbox driver support on DRA7xx (DSP, EVE)
Ramsey Harris [Mon, 17 Feb 2014 22:14:54 +0000 (14:14 -0800)]
SDOCM00106428 Notify mailbox driver support on DRA7xx (DSP, EVE)

New NotifyDriverMbx module; initial implementation of notify mailbox
driver on dsp and eve only. Added support for multiple notify driver
configs in NotifySetup; requires new dispatch isr for notify events.
The isr will dispatch to the appropriate driver (shm or mbx). Improved
the NotifyDriverShm performance because it checks only the mailboxes
which might have raised the given event instead of always checking all
mailboxes. Added ROV view of NotifyDriverMbx. Added a mailbox tab
which shows inbound and outbound mailbox status. Better ISA handling in
source code and script code. Updated Build module with NotifyDriverMbx.c
to include new driver in IPC product libraries for C66 and ARP32.

10 years agoBIOS: NameServerRemoteNotify callback may have uninitialized var
Arnie Reynoso [Fri, 14 Mar 2014 17:04:10 +0000 (10:04 -0700)]
BIOS: NameServerRemoteNotify callback may have uninitialized var

The callback function for NameServerRemoteNotify may end up with an
uninitialized swiHandle.  Though this case should never happen, the handle
should be initialized and an assert must be issued if it remains NULL.
This also fixes a Coverity error with the variable.

10 years agoBIOS: NameServerRemote must copy names with 1 char less than max
Arnie Reynoso [Thu, 13 Mar 2014 21:08:00 +0000 (14:08 -0700)]
BIOS: NameServerRemote must copy names with 1 char less than max

The strncpy length must be max number of characters - 1 bytes. The last
character will be the null termination thus shouldn't be copied. This
fix addresses a Coverity buffer overrun error.

10 years agoBIOS: NameServerMessageQ names need proper termination
Arnie Reynoso [Thu, 13 Mar 2014 20:58:02 +0000 (13:58 -0700)]
BIOS: NameServerMessageQ names need proper termination

The NameServerMessageQ instanceName and name strings weren't being
properly null terminated when the names were 1 character less than the
maximum size. This addresses the following bug:

SDOCM00106336: NameServerMessageQ copy of name is not copying null
terminated character

10 years agoRemove dead code that appears to be trying to set L1D cache size to 0
Robert Tivy [Thu, 13 Mar 2014 00:44:46 +0000 (17:44 -0700)]
Remove dead code that appears to be trying to set L1D cache size to 0

The way to set an L1/L2 cache size is through the Build.platformTable
platform instance creation.  The dead code that is removed here was not
only trying to change the cache size at too late a time, it was doing
so by setting a local variable that went nowhere.

10 years agoEliminate need to specify Syslink_Override_Params on QNX
vwan@ti.com [Fri, 7 Mar 2014 21:26:11 +0000 (13:26 -0800)]
Eliminate need to specify Syslink_Override_Params on QNX

This commit changes Syslink_Override_Params in Platform.c to an empty
string. It removes the need to specify carveouts in the string, which
was only used for verification purposes when there is a slave-side MMU.
Code associated with the verification of the resource table entries
against these carveouts is removed.

This commit also changes the default for mmuEnable to TRUE, since that
is currently the only mode we support.

This addresses CQ SDOCM00105667.

Signed-off-by: VW <vwan@ti.com>
10 years agoAdd timestamp prefix to LOG[0|1|2]() macros
Robert Tivy [Fri, 28 Feb 2014 00:19:11 +0000 (16:19 -0800)]
Add timestamp prefix to LOG[0|1|2]() macros

For the timestamp prefix, subtract the start time (which is established
when the logFile is opened) so that the massive "since the epoch" value
is not displayed.  For this, subtract only the timeval.tv_sec (seconds)
value so we don't have to worry about the "borrow" that results when
start_tv.tv_usec > tv.tv_usec (essentially, treat start_tv.tv_usec as 0,
which skews this "relative" time ahead by start_tv.tv_usec microseconds).

10 years agoTest: Fix NameServerApp config 3.21 ipc-3.21-next 3.21.00.07 3.22.00.00_eng
Chris Ring [Tue, 4 Mar 2014 21:23:16 +0000 (13:23 -0800)]
Test: Fix NameServerApp config

IpcMgr_ipcStartup() was being added twice to the BIOS startup
function - once in nameserverapp.cfg (which is removed in this
commit) and again in messageq_common.cfg.xs (which is included
by nameserverapp.cfg).  In addition to just being wrong, this
resulted in an abort on Keystone 2 devices.

Also, the MessageQ.SetupTransportProxy was errantly not assigned
in the nameserverapp.cfg script.  This resulted in MessageQ.xs
pulling in 'default' MessageQ configuration, which pulled in much
more content than was necessary for this test (for example,
TransportShm, which then pulled in GateMP, SharedRegion, etc).  By
explicitly assigning MessageQ.SetupTransportProxy to the
ti.ipc.transports.TransportRpmsgSetup module, these extra [unused]
modules are no longer pulled in.

10 years agoRevert "SDOCM00106428 Add notify mailbox driver support on DRA7xx (Part 1)"
Chris Ring [Fri, 28 Feb 2014 17:40:43 +0000 (09:40 -0800)]
Revert "SDOCM00106428 Add notify mailbox driver support on DRA7xx (Part 1)"

This reverts commit 3be15bd0ee6a26a8c1d712e698f4055b94f47043.

10 years agoRevert "SDOCM00106428 Add notify mailbox driver support on DRA7xx (Part 2)"
Chris Ring [Fri, 28 Feb 2014 17:40:08 +0000 (09:40 -0800)]
Revert "SDOCM00106428 Add notify mailbox driver support on DRA7xx (Part 2)"

This reverts commit 73882d944bb7b1865f4c35e4bdb35c05bc3da9e5.

10 years agoRevert "SDOCM00106428 Add notify mailbox driver support on DRA7xx (Part 3)"
Chris Ring [Fri, 28 Feb 2014 17:38:21 +0000 (09:38 -0800)]
Revert "SDOCM00106428 Add notify mailbox driver support on DRA7xx (Part 3)"

This reverts commit 65a4b382ea3fc702dedc51f02471149e642772b8.

10 years agoFix misuse of SET_BIT macro to clear mailbox-interrupt-status-clear registers
vwan@ti.com [Thu, 27 Feb 2014 22:49:06 +0000 (14:49 -0800)]
Fix misuse of SET_BIT macro to clear mailbox-interrupt-status-clear registers

The mailbox interrupt handling code in QNX IPC uses the SET_BIT macro to
clear status bits from the MAILBOX_IRQSTATUS_CLR_u registers. This is wrong,
because the macro does as follows:

  >#define SET_BIT(num,pos)            ((num) |= (1u << (pos)))

It will read the register and end up clearing *ALL* bits that are pending
during the write-back, as opposed to the specific one it needs to clear. If the
host is in the midst of processing an interrupt from a given core when a second
core writes to its (sub)mailbox, the second payload may be lost.

This commit changes the SET_BIT call to a direct register assignment of the
appropriate bit mask.

Signed-off-by: VW <vwan@ti.com>
10 years agoIncrease NAMESERVER_GET_TIMEOUT from 20000 to 40000 microseconds
Robert Tivy [Thu, 27 Feb 2014 02:08:05 +0000 (18:08 -0800)]
Increase NAMESERVER_GET_TIMEOUT from 20000 to 40000 microseconds

Product testing revealed that a remote core was occasionally taking
slighly longer than 20000 microseconds to respond (in the negative)
to a NameServer_getRemote() query.  We're not sure why it's taking
so long in some cases, as normally the remote core will respond in a
few hundred microseconds.

Doubling the timeout will affect the total query time across multiple
remote cores when some of those cores aren't running NameServer, but
this should be OK since it affects "startup" times with MessageQ_open()
and shouldn't affect normal "execute" times.

10 years agoRemove duplicated call to GateMP_close in gatempapp on BIOS side
vwan@ti.com [Wed, 26 Feb 2014 21:54:16 +0000 (13:54 -0800)]
Remove duplicated call to GateMP_close in gatempapp on BIOS side

Signed-off-by: VW <vwan@ti.com>
10 years agoBuild: Switch from ?= assignments to =
Chris Ring [Thu, 27 Feb 2014 00:34:57 +0000 (16:34 -0800)]
Build: Switch from ?= assignments to =

products.mak should use the [less confusing] = assignments, rather
than the set-if-not-already-set ?= assignment.  This helps with user
readability of the assignments.

10 years agoSDOCM00106664: Reduce slave-side trace from tests
Chris Ring [Wed, 26 Feb 2014 21:41:37 +0000 (13:41 -0800)]
SDOCM00106664: Reduce slave-side trace from tests

The slave-side trace enabled by the tests is too high.  It obfuscates
actual test output, as well as, in at least one case, was slowing
down the slave-side execution enough to cause host-side timeouts
waiting for replies.

This commit changes some Registry-related masks from ALWAYS_ON
to RUNTIME_OFF to help quiet things down.

10 years agoEnable gptimers in PD_IPU domain 3.21.00.06_eng
vwan@ti.com [Tue, 25 Feb 2014 00:09:30 +0000 (16:09 -0800)]
Enable gptimers in PD_IPU domain

This commit adds workaround to enable gptimers in PD_IPU domain on Vayu in QNX.
It enables example ex41_forwardmsg to run successfully.

Signed-off-by: VW <vwan@ti.com>
10 years agoSDOCM00106450 HeapMemMP_free fails to cache invalidate freed...
Ramsey Harris [Fri, 21 Feb 2014 18:48:15 +0000 (10:48 -0800)]
SDOCM00106450 HeapMemMP_free fails to cache invalidate freed...

The freed buffer content (after the header) was left in cache. If
buffer content was dirty, it would eventually be evicted with a
write-back possibly overwriting memory in use by a different core,
and cause heap corruption.

10 years agoSDOCM00106428 Add notify mailbox driver support on DRA7xx (Part 3)
Ramsey Harris [Fri, 21 Feb 2014 17:51:14 +0000 (09:51 -0800)]
SDOCM00106428 Add notify mailbox driver support on DRA7xx (Part 3)

Added notify mailbox driver support for ARP32. The IPC interrupt on
arp32 will dispatch to the appropriate driver (shm or mbx). This
improves the NotifyDriverShm performance because it checks only the
mailboxes which might have raised the given event instead of always
checking all mailboxes. Improved ROV view of NotifyDriverMbx. Added
a mailbox tab which shows inbound and outbound mailbox status. Better
ISA handling in source code and script code.

10 years agoAdd configurable polling interval to ipc_trace_daemon on QNX
vwan@ti.com [Fri, 21 Feb 2014 22:21:57 +0000 (14:21 -0800)]
Add configurable polling interval to ipc_trace_daemon on QNX

This commit adds a command line parameter to ipc_trace_daemon to set the
polling interval. The default is now set to 0.5 sec as the previous
default was too long. This addresses SDOCM00105472.

Signed-off-by: VW <vwan@ti.com>
10 years agoLinux/Qnx: MessageQBench test must check initial MSG put
Arnie Reynoso [Wed, 19 Feb 2014 22:56:54 +0000 (14:56 -0800)]
Linux/Qnx: MessageQBench test must check initial MSG put

The MessageQBench test must ensure the initial MessageQ_put during
the applications handshaking returns successful before continuing.

10 years agoAdd IPU1 support in QNX
vwan@ti.com [Wed, 19 Feb 2014 19:51:31 +0000 (11:51 -0800)]
Add IPU1 support in QNX

This commit adds support for IPU1 in QNX. It has the current limitation that
the core cannot be put into deep sleep in the idle task, so the function
IpcPower_idle() must be removed from the idle loop. This issue will be fixed
at a later time.

Signed-off-by: VW <vwan@ti.com>
10 years agoSDOCM00106428 Add notify mailbox driver support on DRA7xx (Part 2)
Ramsey Harris [Wed, 19 Feb 2014 02:16:50 +0000 (18:16 -0800)]
SDOCM00106428 Add notify mailbox driver support on DRA7xx (Part 2)

Updated Build module with NotifyDriverMbx.c to include new driver
in IPC product libraries (C66, M4, ARP32, A15g).

10 years agoVAYU QNX: Prevent race condition when host is slow to clear interrupts from IPU
vwan@ti.com [Sat, 15 Feb 2014 01:33:27 +0000 (17:33 -0800)]
VAYU QNX: Prevent race condition when host is slow to clear interrupts from IPU

This commit prevents the same race condition that was fixed in commit id
feb7a8486ed537033c5aed5b170f52f758bddabe, this time for the IPUs as
the other commit only dealt with the DSP.

Signed-off-by: VW <vwan@ti.com>
10 years agoSDOCM00106428 Add notify mailbox driver support on DRA7xx (Part 1)
Ramsey Harris [Mon, 17 Feb 2014 22:14:54 +0000 (14:14 -0800)]
SDOCM00106428 Add notify mailbox driver support on DRA7xx (Part 1)

New NotifyDriverMbx module; initial implementation of notify mailbox
driver on dsp only. Added support for multiple notify driver configs
in NotifySetup; requires new dispatch isr for notify events.

10 years agoLinux/Qnx: MessageQ msg cannot be freed if put fails
Arnie Reynoso [Fri, 14 Feb 2014 19:53:31 +0000 (11:53 -0800)]
Linux/Qnx: MessageQ msg cannot be freed if put fails

MessageQ msg being put to the transport layer can only be freed
if the msg is successfully placed. Otherwise, it's the responsibility
of the application to free the msg.

10 years agoBIOS: Only build rpmsg libs when useful
Chris Ring [Fri, 14 Feb 2014 00:12:50 +0000 (16:12 -0800)]
BIOS: Only build rpmsg libs when useful

Many of the rpmsg-protocol-related IPC packages build for
all targets thrown at them (like good packages should).

However, we know many supported targets will never use the rpmsg
protocol, and we are therefore unnecessarily building libraries
that will never be used.

This commit changes some of the rpmsg-specific packages to not
build for targets that don't have any devices containing an
rpmsg-using HLOS-based host.

10 years agoLinux: Fix pkg-config dependency name 3.21.00.05_eng
Chris Ring [Thu, 13 Feb 2014 00:59:59 +0000 (16:59 -0800)]
Linux: Fix pkg-config dependency name

The name of dependent packages includes the 'lib' prefix - this
fixes pkg-config when run on libtiipc.pc.

10 years agoLinux: Fix warnings in LAD for non GateMP builds
Arnie Reynoso [Mon, 10 Feb 2014 22:28:27 +0000 (14:28 -0800)]
Linux: Fix warnings in LAD for non GateMP builds

To eliminate build warnings in LAD, variables used for GateMP enabled
code need to be encapsulated.

10 years agoLinux: Update autotools generated files
Chris Ring [Tue, 11 Feb 2014 19:41:38 +0000 (11:41 -0800)]
Linux: Update autotools generated files

Check in autoreconf-generated files after the change from
C66AK2E to 66AK2E.

10 years ago66AK2E: Linux: Remove leading 'C' in device name
Chris Ring [Tue, 11 Feb 2014 19:36:28 +0000 (11:36 -0800)]
66AK2E: Linux: Remove leading 'C' in device name

The initial Linux support for 66AK2E (commit
f45b1ad1a591b188eb1321102a3b8ed479eec490) went in with
the leading 'C66AK2E' identifier.  After review by the MCSDK team,
the leading 'C' was incorrect and removed.

Note that autotools' condition variables must start with a
leading character, so internally the variable 'C66AK2E' is
still used.  But the user only sees '66AK2E' when setting
the PLATFORM variable.

The BIOS-side changes are in a separate commit.

10 years ago66AK2E: BIOS: Remove leading 'C' in device name
Chris Ring [Tue, 11 Feb 2014 18:58:52 +0000 (10:58 -0800)]
66AK2E: BIOS: Remove leading 'C' in device name

The initial support for 66AK2E (commit
867efc6fa15f91639820614f6f9ca6ee6751ffa6) went in with
the 'C66AK2E' identifier.  After review by the MCSDK team,
the leading 'C' was incorrect and removed.

The XDC platform and catalog includes this leading 'C', so
most of the internal config-related support did not require
changes.

Finally, this is only the BIOS-side changes.  The Linux-side
changes will be done in a subsequent commit.

10 years agoSDOCM00106383: Remove C6x COFF support
Chris Ring [Tue, 11 Feb 2014 17:25:41 +0000 (09:25 -0800)]
SDOCM00106383: Remove C6x COFF support

C6x COFF support is being [softly] removed from IPC 3.21.  This
commit deletes C6x COFF-related variables from products.mak, but
leaves everything else intact.  If a user finds they need COFF
support, they can easily re-enable it themselves by setting the
variable(s) removed from products.mak and rebuilding.

Eventually the intent is to remove C6x COFF support altogether -
including the build support in ipc-bios.mak and ipc-bios.bld, and
possibly elsewhere.

10 years agoBIOS: GateMP host support must ensure maxNameLen is 32
Arnie Reynoso [Mon, 10 Feb 2014 19:49:16 +0000 (11:49 -0800)]
BIOS: GateMP host support must ensure maxNameLen is 32

When GateMP host support is enabled, the default GateMP must ensure
the size of the maxNameLen must be at least 32. The size must correspond
to the expected size on the host OS (Linux, Qnx, Android) which is
currently 32.

The default size for maxNameLen on the BIOS-side is 32. Thus this issue
hadn't be encountered.

10 years agoLinux/Android: Remove existing response FIFO in LAD
Arnie Reynoso [Mon, 10 Feb 2014 19:30:50 +0000 (11:30 -0800)]
Linux/Android: Remove existing response FIFO in LAD

In rare instances when LAD is improperly terminated, the PID-based
response FIFO will not be removed.  This will cause startup failures
of subsequent execution of LAD. The existing response FIFO must be
removed before creating new ones with the same PID. (SDOCM00106318)

10 years agoAndroid: Change LAD's working directory
Arnie Reynoso [Mon, 10 Feb 2014 19:24:24 +0000 (11:24 -0800)]
Android: Change LAD's working directory

Depending on user privileges when the LAD daemon is started, the
working directory may not be writable. The directory has been moved
to /data/lad.  File permissions at this level are easier to dictate
(SDOCM00106314).

10 years agoVirtQueue: Remove "A9" from #defines
Chris Ring [Mon, 10 Feb 2014 18:44:43 +0000 (10:44 -0800)]
VirtQueue: Remove "A9" from #defines

Many of our supported devices do not use A9s, so generalize
the definitions that previously used A9 in them.

10 years agoDRA7XX: Remove unnecessary rpmsg-related code
Chris Ring [Mon, 10 Feb 2014 17:47:21 +0000 (09:47 -0800)]
DRA7XX: Remove unnecessary rpmsg-related code

Similar to commit 16adcaad which cleaned up OMAP5 code, this
removes dead code for DRA7XX.  Most of the changes remove
untested, and likely broken, non-SMP code.

In at least one case, the code cleanup may fix previously
undiscovered issues.  For example, in VirtQueue_isr(), the first
switch() block was [likely inadvertently] skipped for DSP2.  There
is no known issue because of this, but now with this cleanup, DSP2
will (and should!) participate in this initial switch statement.

10 years agoDRA7XX: Use XDC preprocessor definitions
Chris Ring [Mon, 10 Feb 2014 17:41:59 +0000 (09:41 -0800)]
DRA7XX: Use XDC preprocessor definitions

Switch from IPC created preprocessor definitions (set by
package.bld) to standard XDC-provided definitions.

Also remove some minor dead code.

This is just code cleanup, no intended functional change.

10 years agoC66AK2E: BIOS: Fix C66AK2E test
Chris Ring [Fri, 7 Feb 2014 01:34:51 +0000 (17:34 -0800)]
C66AK2E: BIOS: Fix C66AK2E test

Building on 867efc6fa15f91639820614f6f9ca6ee6751ffa6, this
fixes a test that was failing to build for C66AK2E.

10 years agoVAYU QNX: Prevent race condition when ARM is slow to handle mailbox interrupts
vwan@ti.com [Thu, 6 Feb 2014 22:09:43 +0000 (14:09 -0800)]
VAYU QNX: Prevent race condition when ARM is slow to handle mailbox interrupts

This patch fixes a race condition where the host side in QNX is slow to clear a
mailbox interrupt. If the slave were to try to send another interrupt before
the ARM has cleared the first one, the code in InterruptDsp_intSend()
will simply drop it. This is bad because on QNX, the code in VirtQueue_ISR
interprets the received arg as the VirtQueue id. Hence if consecutive
messages were sent with different ids, the second message would be missed.

The fix is to poll whenever the mailbox is occupied, and send the message
only when the mailbox is free. (SDOCM00106278)

Signed-off-by: VW <vwan@ti.com>
10 years agoBIOS: Fix typo in variable name
Chris Ring [Thu, 6 Feb 2014 21:06:56 +0000 (13:06 -0800)]
BIOS: Fix typo in variable name

SZIE?  Really?  This stuff kills me.

10 years agoLinux: Update autotools generated files
Chris Ring [Tue, 4 Feb 2014 22:25:35 +0000 (14:25 -0800)]
Linux: Update autotools generated files

Check in autoreconf-generated files after the pkg-config
changes.

10 years agoSDOCM00103659: Add pkg-config support
Chris Ring [Tue, 4 Feb 2014 22:21:57 +0000 (14:21 -0800)]
SDOCM00103659: Add pkg-config support

This builds on commit 6348f9de, which added pkg-config support for
the mmrpc lib.  This commit adds pkg-config support
for the tiipc and tiipcutils libraries.

Also updates .gitignore to ignore the generated *.pc files.

10 years agoDRA7XX: Minor test cfg script refactoring
Chris Ring [Tue, 4 Feb 2014 18:23:04 +0000 (10:23 -0800)]
DRA7XX: Minor test cfg script refactoring

Only MmServicMgr-using slaves require the
ti.sysbios.xdcruntime.GateThreadSupport Module.  This moves
the dependency on that package from the common Dsp_vayu.cfg into
the one MmServiceMgr-based test.

10 years agoDRA7XX: Consolidate DSP1/2 config scripts
Chris Ring [Tue, 4 Feb 2014 18:05:07 +0000 (10:05 -0800)]
DRA7XX: Consolidate DSP1/2 config scripts

Lots of duplication between DRA7XX's ti/ipc/tests/Dsp(1|2).cfg
scripts.  This commit merges the two into Dsp_vayu.cfg, which
encapsulates the single "real" difference, which was the
MultiProc config name.  It also adjusts users of these Dsp .cfg
scripts to now use the common Dsp_vayu.cfg script.

10 years agoOMAP5: Adjust IPU carveout size and linker map
Suman Anna [Wed, 18 Dec 2013 20:30:28 +0000 (14:30 -0600)]
OMAP5: Adjust IPU carveout size and linker map

The sizes for the IPU data section has been reduced, in line
with the changes on the host OS side. There is no change in
the starting addresses of either the DSP or IPU. The linker
memory map file used for building OMAP5 samples has also been
adjusted accordingly.

Signed-off-by: Suman Anna <s-anna@ti.com>
10 years agoDRA7xx: Adjust carveout sizes and addresses
Suman Anna [Tue, 17 Dec 2013 22:07:26 +0000 (16:07 -0600)]
DRA7xx: Adjust carveout sizes and addresses

The starting addresses for the resource table carveouts have
been adjusted in line with the changes on the host OS side.
The IPU2 size has been decreased, and the starting addresses
of the vrings have been reshuffled.

The linker memory map has also been adjusted for IPU2. No memory
map changes were made to IPU1 and DSP1 processors even though
the host-side can support a little bit more larger footprint
images - the current memory maps for these processors is a
minimal memory map.

Signed-off-by: Suman Anna <s-anna@ti.com>
10 years agoVayu: Minor comment wordsmithing 3.21.00.03_eng 3.21.00.04_eng
Chris Ring [Sat, 1 Feb 2014 17:51:40 +0000 (09:51 -0800)]
Vayu: Minor comment wordsmithing

Fix typo, 80 char alignment.

10 years agoTests: Remove MPU mapping from gatempapp sample's resource table
Suman Anna [Fri, 31 Jan 2014 23:45:46 +0000 (17:45 -0600)]
Tests: Remove MPU mapping from gatempapp sample's resource table

The L4_PER mapping entry is split into three individual L4_PERx
entries. This is done to remove any mappings for the MPU-relevant
address spaces into the slave processor MMUs.

This change incorporates the same changes to the gatempapp resource
table as in commit,
e4b6067 "Remoteproc: DRA7xx: Remove MPU mapping from resource table".

Signed-off-by: Suman Anna <s-anna@ti.com>
10 years agoAndroid: Adds GateMP build support
Arnie Reynoso [Fri, 31 Jan 2014 16:32:10 +0000 (08:32 -0800)]
Android: Adds GateMP build support

The necessary build files have been updated to add GateMP for
Android.  The functionality has been verified manually using
breakpoints to ensure GateMP APIs are behaving as expected.

10 years agoLinux: LAD requires -l option for logging
Arnie Reynoso [Thu, 30 Jan 2014 23:32:47 +0000 (15:32 -0800)]
Linux: LAD requires -l option for logging

With the addition of GateMP support, the LAD daemon now processes
arguments differently. An 'l' option is now required to enable
logging. The 'g' option enables GateMP support. A usage output has
been added with an 'h' option.

Examples:
    lad_<platform> -h
    lad_<platform> -l log.txt
    lad_<platform> -g -l log.txt

10 years agoOMAP5: Minor test cfg cleanup
Chris Ring [Fri, 31 Jan 2014 16:31:33 +0000 (08:31 -0800)]
OMAP5: Minor test cfg cleanup

Minor config refactoring.  The GateThreadSupport module is only
needed by grcm users, but was brought in through Dsp.cfg which
is used by non-grcm-using tests.  Also, took the opportunity to
fixup some syntax buglets and remove some dead comments.

10 years agoLinux: Improve MessageQApp cmdline error handling
Chris Ring [Thu, 30 Jan 2014 22:16:01 +0000 (14:16 -0800)]
Linux: Improve MessageQApp cmdline error handling

In the MessageQApp, if an errant 2nd arg is passed on the command
line (e.g. "DSP" instead of "2"), the calculated procId is 0
(which is the host), and HOST<->HOST messaging is attempted.  This
isn't supported.

This commit catches that errant cmdline and emits an error.

10 years agoOMAP5: Remove unnecessary rpmsg-related code
Chris Ring [Thu, 30 Jan 2014 19:58:10 +0000 (11:58 -0800)]
OMAP5: Remove unnecessary rpmsg-related code

The original OMAP5 code import included support for running the
IPU in non-SMP BIOS.  This was untested in the context of IPC 3.x,
and is not supported.  The unsupported/dead stuff was obfuscating
the live stuff, so this commit removes it.

There is no intentional change in behavior, other than slight
performance improvement due to small bits of code removal.

10 years agoBIOS: Enable xdc__strict and fix issues
Chris Ring [Wed, 29 Jan 2014 01:35:50 +0000 (17:35 -0800)]
BIOS: Enable xdc__strict and fix issues

XDC provides a xdc__strict feature.  Defining that pre-processor
variable prior to #including <xdc/std.h> will disable deprecated
data types (e.g. Uint32 - note the lowercase 'i') and change
NULL from '0' to '(void *)0'.

This commit throws that xdc__strict definition when building the
BIOS-side code, and fixes issues that were found enabling it.

Interestingly, there were a few bugs that turned up during this
exercise (rare in the wild, but still good cleanup).

10 years agoGateMP: Revert ti_sdo_ipc_GateMP_getSharedAddr
vwan@ti.com [Fri, 24 Jan 2014 18:44:32 +0000 (10:44 -0800)]
GateMP: Revert ti_sdo_ipc_GateMP_getSharedAddr

This function was inadvertently modified in a previous commit,
and needs to be reverted back to its original state.
(SDOCM00106117)

Signed-off-by: VW <vwan@ti.com>
10 years agoLinux: Update autotools-generated files
Chris Ring [Thu, 23 Jan 2014 22:32:38 +0000 (14:32 -0800)]
Linux: Update autotools-generated files

10 years agoLinux: Fix gateMPApp unconditional build
Arnie Reynoso [Thu, 23 Jan 2014 17:10:53 +0000 (09:10 -0800)]
Linux: Fix gateMPApp unconditional build

The tests directory Makefile needs to ensure CMEM_INSTALL_DIR
variable is set before attempting to build the gateMPApp for DRA7XX.

10 years agoDocs: Update MmRpc doxygen documentation
Chris Ring [Wed, 15 Jan 2014 20:37:14 +0000 (12:37 -0800)]
Docs: Update MmRpc doxygen documentation

10 years agoLinux: Update to autotools files for GateMP addition 3.21.00.02_eng
Chris Ring [Wed, 22 Jan 2014 00:32:22 +0000 (16:32 -0800)]
Linux: Update to autotools files for GateMP addition

10 years agoLinux: Added GateMP support for DRA7XX devices
Arnie Reynoso [Tue, 21 Jan 2014 22:17:44 +0000 (14:17 -0800)]
Linux: Added GateMP support for DRA7XX devices

GateMP support has been added using the device's spinlocks.

To enable GateMP on the host, throw -g option when launching the
LAD daemon as follows:

        ./lad_dra7xx log.txt -g

To enable GateMP on the slave core, the core must be configured as
the owner of SR0 and GateMP.hostSupport config param must be set
to 'true' as follows:

         var GateMP = xdc.useModule('ti.sdo.ipc.GateMP');
         GateMP.hostSupport = true;

The slave core with GateMP enabled must the loaded (remoteproc)
prior to the execution of LAD on the host. This adds a restriction
on LAD that wasn't previously there.

Note: Initial validation has only been performed on DRA7XX DSP1.

10 years agoLinux: Update autotools generated files
Chris Ring [Tue, 21 Jan 2014 21:41:42 +0000 (13:41 -0800)]
Linux: Update autotools generated files

Check in autoreconf-generated files for C66AK2E support.

10 years agoC66AK2E: Linux: Introduce support for C66AK2E
Chris Ring [Tue, 21 Jan 2014 21:31:52 +0000 (13:31 -0800)]
C66AK2E: Linux: Introduce support for C66AK2E

The C66AK2E device is an A15 + C66 DSP, similar in hardware
to the TCI663X family of devices.  This commit adds Linux-side
support.

Note that this commit doesn't update the autotools-generated
files.  That will be done in a separate commit.

10 years agoC66AK2E: BIOS: Introduce support for C66AK2E
Chris Ring [Tue, 21 Jan 2014 21:19:39 +0000 (13:19 -0800)]
C66AK2E: BIOS: Introduce support for C66AK2E

The C66AK2E device is an A15 + C66 DSP, similar in hardware
to the TCI663X family of devices.  This commit adds BIOS-side
support.

10 years agoLinux: Update autotools generated files
Chris Ring [Tue, 21 Jan 2014 19:20:10 +0000 (11:20 -0800)]
Linux: Update autotools generated files

Check in autoreconf-generated files.

10 years agoTCI6630: Linux: Introduce support for TCI6630K2L
Chris Ring [Fri, 17 Jan 2014 01:07:41 +0000 (17:07 -0800)]
TCI6630: Linux: Introduce support for TCI6630K2L

The TCI6630K2L device is an A15 + 4 C66 DSPs, very similar
to TCH6638K2K (A15 + 8 C66 DSPs).  This commit adds
Linux support.

Note that this commit doesn't update the autotools-generated
files.  That will be done in a separate commit.

10 years agoTCI6630: BIOS: Introduce support for TCI6630K2L
Chris Ring [Wed, 15 Jan 2014 21:09:22 +0000 (13:09 -0800)]
TCI6630: BIOS: Introduce support for TCI6630K2L

The TCI6630K2L device is an A15 + 4 C66 DSPs, very similar
to TCI6638K2K (A15 + 8 C66 DSPs).  This commit adds
BIOS-side support.

10 years agoIncrease the max name length for GateMP's NameServer on QNX to 32
vwan@ti.com [Thu, 16 Jan 2014 19:41:03 +0000 (11:41 -0800)]
Increase the max name length for GateMP's NameServer on QNX to 32

This is to match up with the BIOS side default configuration in GateMP, and to
accommodate for the name of the default gate, which is 17-byte long.

Signed-off-by: VW <vwan@ti.com>
10 years agoLinux: Update autotools generated files
Arnie Reynoso [Thu, 16 Jan 2014 17:09:50 +0000 (09:09 -0800)]
Linux: Update autotools generated files

The autotools generated files have been update to reflect the addition
of a pkgconfig file for the mmrpc library.

Including the generated files in the development branch eliminates the
need for users to have autotools (autoconf, autoamke, and libtool) to
build ipc Linux sources.

10 years agoLinux: Add package config information to mmrpc
Arnie Reynoso [Thu, 16 Jan 2014 17:05:52 +0000 (09:05 -0800)]
Linux: Add package config information to mmrpc

This patch adds the libmmrpc.pc to package config.
This aides in identifying the correct CFLAGS for any user or mmrpc
as below.

pkg-config --cflags libmmrpc

Using the above command, the applications using libmmrpc can
find the cflags automatically using pkg-config thus avoiding to
explicitly set the path to MmRpc.h

10 years agoSDOCM00106019 IPC makefile template does not honor Build.customCCOpts
Ramsey Harris [Wed, 15 Jan 2014 23:33:12 +0000 (15:33 -0800)]
SDOCM00106019 IPC makefile template does not honor Build.customCCOpts

The makefile template had an incorrect reference to the BIOS module.
This caused the wrong customCCOpts to be used in the makefile.

10 years agoLAD: Dead code removal
Chris Ring [Sat, 14 Dec 2013 00:53:16 +0000 (16:53 -0800)]
LAD: Dead code removal

Removed some dead code.

10 years agoRemoteproc: DRA7xx: Cleanup references to OMAP5
Suman Anna [Thu, 19 Dec 2013 01:12:26 +0000 (17:12 -0800)]
Remoteproc: DRA7xx: Cleanup references to OMAP5

OMAP5 has it own resource table files, so clean up any references
to OMAP5 from the DRA7xx resource table files.

Signed-off-by: Suman Anna <s-anna@ti.com>
10 years agoIpcMgr: Fix comment typo 3.21.00.01_eng
Chris Ring [Fri, 13 Dec 2013 20:09:44 +0000 (12:09 -0800)]
IpcMgr: Fix comment typo

No functional change - just fixed a typo.

10 years agoDRA7XX: Improve SMP detection and error messages
Chris Ring [Fri, 13 Dec 2013 19:35:48 +0000 (11:35 -0800)]
DRA7XX: Improve SMP detection and error messages

Specifically for the ti.ipc.family.vayu package, which is limited
to use on HLOS-based systems, only SMP BIOS is supported on the
IPUs.  This commit improves the package's SMP detection (in
getLibs()) and error reporting (in validate()).

10 years agoIgnore 'remote get' timeouts in NameServer_get when procId array is supplied in QNX.
vwan@ti.com [Fri, 13 Dec 2013 01:50:03 +0000 (17:50 -0800)]
Ignore 'remote get' timeouts in NameServer_get when procId array is supplied in QNX.

This patch fixes an oversight in commit 2427c7d168fb42a3e277f7d4c7827e7cdc955ac2,
where 'remote get' timeouts were ignored when procId array is NULL, but not
for the case when the procId array is supplied.

10 years agoChange NameServer_get to ignore 'remote get' timeouts in BIOS
vwan@ti.com [Fri, 13 Dec 2013 01:56:21 +0000 (17:56 -0800)]
Change NameServer_get to ignore 'remote get' timeouts in BIOS

This patch allows NameServer requests to succeed even when some of the other
remote cores are not yet ready to handle NameServer requests.

10 years agoFix wrong assertion in packages/ti/sdo/ipc/family/vayu/InterruptDsp.c
vwan@ti.com [Fri, 13 Dec 2013 01:59:22 +0000 (17:59 -0800)]
Fix wrong assertion in packages/ti/sdo/ipc/family/vayu/InterruptDsp.c

10 years agoMerge remote-tracking branch 'origin/ipc-3.20-next' into ipc-next 3.21.00.00_eng
Chris Ring [Mon, 9 Dec 2013 20:54:15 +0000 (12:54 -0800)]
Merge remote-tracking branch 'origin/ipc-3.20-next' into ipc-next

10 years agoMerge remote-tracking branch 'origin/ipc-3.10-next' into ipc-3.20-next ipc-3.20-next
Chris Ring [Mon, 9 Dec 2013 20:51:40 +0000 (12:51 -0800)]
Merge remote-tracking branch 'origin/ipc-3.10-next' into ipc-3.20-next

10 years agoMerge remote-tracking branch 'origin/3.00' into ipc-3.10-next ipc-3.10-next
Chris Ring [Mon, 9 Dec 2013 20:46:10 +0000 (12:46 -0800)]
Merge remote-tracking branch 'origin/3.00' into ipc-3.10-next

Conflicts:
linux/src/tests/Makefile.am
linux/src/tests/Makefile.in

10 years agoSDOCM00105512: Support unnamed Linux MessageQs
Chris Ring [Fri, 6 Dec 2013 00:52:42 +0000 (16:52 -0800)]
SDOCM00105512: Support unnamed Linux MessageQs

The MessageQ API supports unnamed MessageQs; update Linux's
LAD-based MessageQ creation command to handle this.

Previously a segment fault would occur due to strncpy() being
passed a NULL src string array.

10 years agoRemoves all references to OMAP4 and TI81xx devices in QNX codebase
vwan@ti.com [Sat, 23 Nov 2013 01:03:28 +0000 (17:03 -0800)]
Removes all references to OMAP4 and TI81xx devices in QNX codebase

This is to address SDOCM00101830.

Signed-off-by: VW <vwan@ti.com>
10 years agoLinux: Update autotools to require KERNEL path 3.00 ipc-3.00-next 3.00.04.29
Arnie Reynoso [Wed, 16 Oct 2013 17:14:10 +0000 (10:14 -0700)]
Linux: Update autotools to require KERNEL path

The IPC Linx-side build now requires KERNEL path to be set to
build all libraries and test applications redardless of platform.

This change is needed since rpmsg has been made Linux kernel version
aware.
(cherry picked from commit ffd46a7143ff4e25ea14085b44ff5105c9912769)

Conflicts:
linux/src/tests/Makefile.am
linux/src/tests/Makefile.in
products.mak

10 years agoLinux: Update user AF_RPMSG define for 3.9 kernel
Arnie Reynoso [Wed, 16 Oct 2013 17:11:11 +0000 (10:11 -0700)]
Linux: Update user AF_RPMSG define for 3.9 kernel

IPC's Linux user space support contains a copy of a few
kernel-defined structs and definitions.  This is necessary
because the kernel headers are not include-able by user space.
One of these values is the socket address family used for
rpmsg-based sockets - AF_RPMSG.  This definition is always
'one more' than the last address family defined by the kernel's
socket.h.

Prior to Linux 3.9, this was aways 40.  In Linux 3.9, a new
address family was introduced (AF_VSOCK), and as a result we
needed to set AF_RPMSG to 41.  To enable this, the user space
header reflects on the kernel version and appropriately assigns
the value of AF_RPMSG.

Note that in addition to adding 3.9+ support, this commit also
cleans up some dead code.

10 years agoRelease the OMAP5 DSP MMU from reset after loading the core 3.20 3.20.00.06
vwan@ti.com [Wed, 13 Nov 2013 18:16:28 +0000 (10:16 -0800)]
Release the OMAP5 DSP MMU from reset after loading the core

This patch reverts the behavior from a previous patch that moved the MMU release
to be done prior to the load. It was causing the IPC resource manager to crash when
reloading the Tesla multiple times.

Signed-off-by: VW <vwan@ti.com>
10 years agoList supported core ids for QNX IPC manager in help/use text.
vwan@ti.com [Tue, 12 Nov 2013 21:36:42 +0000 (13:36 -0800)]
List supported core ids for QNX IPC manager in help/use text.

Signed-off-by: VW <vwan@ti.com>
10 years agoSDOCM00104726 Fix RcmServer.c compile warning
Ramsey Harris [Tue, 12 Nov 2013 18:37:39 +0000 (10:37 -0800)]
SDOCM00104726 Fix RcmServer.c compile warning

The C28 is a 16-bit processor. I added a cast to promote the key
to a 32-bit value to give sufficient room for the shift operation.
This promotion was happening implicitly on 32-bit processors.

10 years agoAdd support for loading to IPU internal memory on QNX OMAP5 3.20.00.05_eng
vwan@ti.com [Fri, 8 Nov 2013 02:02:43 +0000 (18:02 -0800)]
Add support for loading to IPU internal memory on QNX OMAP5

Add static memory entry to inform loader of internal memory area on IPU.
Also remove some code that was overriding the static entries.
This addresses SDOCM00104907.

Signed-off-by: VW <vwan@ti.com>
10 years agoAdd support for loading to slaves' internal memory on QNX Vayu
vwan@ti.com [Fri, 8 Nov 2013 00:47:44 +0000 (16:47 -0800)]
Add support for loading to slaves' internal memory on QNX Vayu

Add static memory entry to inform loader of internal memory areas on DSP1 and IPU2.
This addresses SDOCM00104907.

Signed-off-by: VW <vwan@ti.com>
10 years agoSDOCM00099002 Make LibType_Debug build profile same as custom.
Ramsey Harris [Wed, 6 Nov 2013 19:56:10 +0000 (11:56 -0800)]
SDOCM00099002 Make LibType_Debug build profile same as custom.

Merged updates in ti.sysbios.Build module into ti.ipc.sdo.Build.
Turn off package library builds when building IPC for product
release. Added ti.sdo.ipc.Build.libType config param. Added source
files to Pkg.otherFiles for source only release. Added package
debug library build support. Removed package ti.sdo.utils from
ti.sdo.ipc.Build file list, it builds separately now.