view blobs/patches/frame_na7_db_ir-systrace.patch @ 629:3231dd9b38c1

armio.c: make GPIOs 8 & 13 outputs driving 1 on all "classic" targets Calypso GPIOs 8 & 13 are pinmuxed with MCUEN1 & MCUEN2, respectively, and on powerup these pins are MCUEN, i.e., outputs driving 1. TI's code for C-Sample and earlier turns them into GPIOs configured as outputs also driving 1 - so far, so good - but TI's code for BOARD 41 (which covers D-Sample, Leonardo and all real world Calypso devices derived from the latter) switches them from MCUEN to GPIOs, but then leaves them as inputs. Given that the hardware powerup state of these two pins is outputs driving 1, every Calypso board design MUST be compatible with such driving; typically these GPIO signals will be either unused and unconnected or connected as outputs driving some peripheral. Turning these pins into GPIO inputs will result in floating inputs on every reasonably-wired board, thus I am convinced that this configuration is nothing but a bug on the part of whoever wrote this code at TI. This floating input bug had already been fixed earlier for GTA modem and FCDEV3B targets; the present change makes the fix unconditional for all "classic" targets. The newly affected targets are D-Sample, Leonardo, Tango and GTM900.
author Mychaela Falconia <falcon@freecalypso.org>
date Thu, 02 Jan 2020 05:38:26 +0000
parents acb07ce22054
children
line wrap: on
line source

# TI's original version of the pf_TaskEntry() function in the guts of GPF
# does this number once all GPF tasks have entered their main loop:
#
#    TraceMask[0] = 0;
#
# The effect of this line is that "system" trace output gets disabled;
# in particular, "system primitive" requests like querying for existing
# TRACECLASS of various entities or for the GPF version elicit no response
# whatsoever until one sends a TRACECLASS setting sysprim which restores
# the TC_SYSTEM bit in TraceMask[0].
#
# In FreeCalypso we consider this clearing of TraceMask[0] to be either
# a bug or a misfeature, and we would like to change it.  In the case of
# the GPF code in question, we do have a copy of frame.c source (the module
# where the offending pf_TaskEntry() function is implemented) which we believe
# is the same code as what's in the TCS211 version of GPF, but TCS211 is not
# set up to recompile any part of GPF from source, and there are other modules
# in the same frame_na7_db_ir.lib for which we don't have the original source
# (os_*), hence it is easier to do a binary lib patch.
#
# The present patch is a binary form of:
#
# -TraceMask[0] = 0;
# +TraceMask[0] = TC_SYSTEM;

[frame.obj]

.text 2ee 10