Commit Graph

75 Commits

Author SHA1 Message Date
Randall Spangler
028150a059 Increase link image size to 40KB
with x86 power module enabled, it no longer fits in 32KB.

Signed-off-by: Randall Spangler <rspangler@chromium.org>

BUG=none
TEST=make

Change-Id: I38c9054efd8a072cc5800cc87936e53e2df00e58
2012-01-26 13:36:07 -08:00
Vincent Palatin
09b937b767 Ensure the panic UART is set to the right UART
Preparatory work to introduce a second SoC : 2nd series 4/4

Add a build time assertion which checks whether the UART used in the C
uart code is the same one as the one defined for assembly panic code.

Signed-off-by: Vincent Palatin <vpalatin@chromium.org>

BUG=None
TEST=build for BDS with both good and bad address in CONFIG_UART_ADDRESS

Change-Id: I28dd6089bc938f06be0654d7bed75d7d698fafe0
2012-01-26 01:55:37 +00:00
Vincent Palatin
1f00fc154a Make more features optional
Preparatory work to introduce a second SoC : 2nd series 3/4

Some modules won't be used on other designs, make them optional.

Signed-off-by: Vincent Palatin <vpalatin@chromium.org>

BUG=None
TEST=run the EC firmware on BDS and check that the commands from the
optional features are still available and working.

Change-Id: I979864ed94dc4da90c1010bd2e4589d84bc2d046
2012-01-26 01:55:37 +00:00
Vincent Palatin
1008124533 Remove useless header includes
Preparatory work to introduce a second SoC : 2nd series 2/4

Avoid introducing platform specific dependencies in common files where
they are not necessary.

Signed-off-by: Vincent Palatin <vpalatin@chromium.org>

BUG=None
TEST=build for BDS and Link

Change-Id: If2ccd022e4956425222b55a5a48ca7522857e7f0
2012-01-26 01:32:30 +00:00
Vincent Palatin
4cca2932ef Move SoC-independant headers to another directory
Preparatory work to introduce a second SoC : 2nd series 1/4

The atomic operations are SoC independant since they are only using
LDREX/STREX instructions which are just core specific ARMv7-M).

The watchdog header defines the API which is common to all platforms.

Signed-off-by: Vincent Palatin <vpalatin@chromium.org>

BUG=None
TEST=run EC firmware on BDS and check a few console commands
2012-01-26 01:29:48 +00:00
Vincent Palatin
9a465855f8 NVIC registers are not SoC specific
Preparatory work to introduce a second SoC : 5/5

All Cortex-M3/4 have the same NVIC registers at the same address.

Signed-off-by: Vincent Palatin <vpalatin@chromium.org>

BUG=None
TEST=run EC firmware on BDS and check a few console commands

Change-Id: I6b03c4c1fb21850be8c8afb711ea44134c8cdea1
2012-01-25 22:50:07 +00:00
Vincent Palatin
9301cef981 Add configuration parameters for the panic UART code
Preparatory work to introduce a second SoC : 4/5

Allow to use the common code for most SoC.
Also simplify the UART code, we don't need speed on the panic path.

Signed-off-by: Vincent Palatin <vpalatin@chromium.org>

BUG=None
TEST=trigger a panic and check the UART output on BDS

Change-Id: I11f7bbc571ab9efbc21fb7b805bf4e271b192c3b
2012-01-25 22:50:07 +00:00
Vincent Palatin
cf9fcef328 Move OS files to a CPU specific directory
Preparatory work to introduce a second SoC : 3/5

We split the drivers files which contain SoC specific drivers from the
OS files which only depend the actual CPU core.

Signed-off-by: Vincent Palatin <vpalatin@chromium.org>

BUG=None
TEST=run EC firmware on BDS and test a few commands on the console.

Change-Id: I598f8b23e074da9bd6b0e2ce6689c1075fe854f0
2012-01-25 22:50:07 +00:00
Vincent Palatin
645dad5d3f Split the timer code between OS code and hardware dependant code.
Preparatory work to introduce a second SoC : 2/5

The hwtimer.* files implement the driver for the SoC timer block.
The timer.* files provides the OS level clock/timer functions.

Signed-off-by: Vincent Palatin <vpalatin@chromium.org>

BUG=None
TEST=on BDS, check 'waitms' and 'gettime' on the EC console.

Change-Id: Icbc58d9be59ee268e2d5a94f8b20de0cabcdc91d
2012-01-25 22:50:07 +00:00
Vincent Palatin
c89bea4a5b Go back to SoC independant IRQ vectors declaration
Preparatory work to introduce a second SoC : 1/5

Instead of putting hardcoded IRQ SoC name in the vector table,
upgrade the DECLARE_IRQ macro to expand its argument.

Also add a parameter to set the size of the NVIC table to save flash
memory.

Signed-off-by: Vincent Palatin <vpalatin@chromium.org>

BUG=None
TEST=run EC on BDS and see timer IRQs firing.

Change-Id: I44fefdabdd37d756492a71f24554979c72c1b50f
2012-01-25 22:47:12 +00:00
Randall Spangler
59d55ece06 Indicate which signals in GPIOGET have changed since the last call
Signed-off-by: Randall Spangler <rspangler@chromium.org>

BUG=none
TEST=at EC console, gpioget twice, then toggle an output using gpioset, then gpioget again

May see *'s the first time.  Second time, should see no *'s.  Third
time, should see a * only the toggled output (and any input signals
which respond to it).

Change-Id: Ibc1870839201008592b7982049cc352c1779a0e3
2012-01-25 12:17:10 -08:00
Randall Spangler
959c38da68 Define non-present interrupt handlers as null
This fixes linker errors when the X86_POWER and/or POWERBTN tasks are disabled.

Signed-off-by: Randall Spangler <rspangler@chromium.org>

BUG=none
TEST=remove X86POWER and POWERBTN tasks from ec.tasklist and make

Change-Id: I8a95020925e32ac4f80b9363f5aa6ab0a2d9ccd1
2012-01-25 11:04:51 -08:00
Vincent Palatin
e0633d274c Merge "mutex: add unit testing" 2012-01-25 10:53:53 -08:00
Vincent Palatin
636d1257a3 Merge "Initial mutex implementation" 2012-01-25 10:53:00 -08:00
Vincent Palatin
47c740e0b2 mutex: add unit testing
Exercise all basic use cases on mutexes.

Signed-off-by: Vincent Palatin <vpalatin@chromium.org>

BUG=None
TEST=make qemu-tests

Change-Id: I40de3ae59862db96b40a067c9ad54a978f5646b8
2012-01-25 18:23:48 +00:00
Vincent Palatin
16e43a3d0d Initial mutex implementation
They are designed to protect shared hardware resources (e.g. I2C
controller).
Please refrain using them as a general purpose synchronization primitive
for the tasks to avoid unintended slippery effects (e.g. priority inversion),
use the provided message-passing functions instead for that purpose.

The mutex variable (ie the "struct mutex") should be initially filled
with 0, but this is the default compiler behavior if you declare it as a
global variable.

Signed-off-by: Vincent Palatin <vpalatin@chromium.org>

BUG=None
TEST=make qemu-tests

Change-Id: I328f7eadf5257560944dbbbeda0b99d5b24520e8
2012-01-25 18:23:48 +00:00
Vincent Palatin
dc97a7ec1e Add more openOCD automation as a TCL script
This script is automatically loaded when launching 'openocd -f
openocd.cfg'.
It adds 'flash_bds', 'ramboot_bds' commands to the openOCD console. The
former is writing the current EC firmware inside the internal flash, the
latter is loading a RAM only firmware on the chip.
There are similar commands for the Link Proto-0 board :
'flash_link', 'ramboot_link'.

Signed-off-by: Vincent Palatin <vpalatin@chromium.org>

BUG=none
TEST=type 'flash_bds' from openocd telnet console

Change-Id: Ic63281a99ee1d083986696675ed0a82da7c033ee
2012-01-24 16:18:52 -08:00
Vincent Palatin
d356dea61e Add modularity to the build
You can now enable/disable tasks more easily.
To conditionally compile a C file depending on the task FOO activation,
just write something like that in the build.mk file :
common-$(CONFIG_TASK_FOO)+=foo_source.o

Signed-off-by: Vincent Palatin <vpalatin@chromium.org>

BUG=None
TEST=make all BOARD=link && make qemu-tests

Change-Id: I760fb248e1599d13190ccd937a68ef47da17b510
2012-01-24 23:17:07 +00:00
Randall Spangler
396a94c43f Merge changes I94d64a53,If88610f3
* changes:
  Add openocd script to upload code to link proto0
  Help command prints a sorted multi-column list of commands
2012-01-24 14:08:47 -08:00
Randall Spangler
8b7bdbdf9d Add openocd script to upload code to link proto0
Signed-off-by: Randall Spangler <rspangler@chromium.org>

BUG=none
TEST=type 'script up-link.tcl' from openocd telnet console

Change-Id: I94d64a5356269628bec1a7f6809651d70398afa9
2012-01-24 13:55:27 -08:00
Randall Spangler
87a228b354 Merge "Fix UART1 (x86 COMx output) on link proto0" 2012-01-24 13:52:19 -08:00
Randall Spangler
8c56cc4fa7 Help command prints a sorted multi-column list of commands
Signed-off-by: Randall Spangler <rspangler@chromium.org>

BUG=none
TEST=type 'help' from console

Change-Id: If88610f35337243ca2550de2851bd1924083344d
2012-01-24 10:32:29 -08:00
Randall Spangler
431622d0b9 Fix UART1 (x86 COMx output) on link proto0
Signed-off-by: Randall Spangler <rspangler@chromium.org>

BUG=chrome-os-partner:7703
TEST=reboot; look for "Hello on UART1" message on UART1 (which is uart2 on servo)

Change-Id: Ie497af48e62c28174b69adca5bea52d2f68d494f
2012-01-24 09:49:26 -08:00
Randall Spangler
19c08c6f3e Read all 4 temperature sensors on link
Signed-off-by: Randall Spangler <rspangler@chromium.org>

BUG=chrome-os-partner:7527
TEST=run 'temps' from console.  Should print info on 4 sensors.

Change-Id: I8e0165235f9a12233bc3ac1fbde55c8eb3cfbb00
2012-01-24 09:16:53 -08:00
Vincent Palatin
c21f07e58e register console commands at compile-time
Instead of using a runtime callback to register the console commands,
put them in a special linker section. So we can do a macro to "register"
them during the build.

It saves 684 bytes and a few microseconds at startup.

Signed-off-by: Vincent Palatin <vpalatin@chromium.org>

BUG=None
TEST=run a few commands from the BDS command line.

Change-Id: Id33ea210b9035bf76ed720373c74c5dd24ccd1b1
2012-01-24 00:50:08 +00:00
Randall Spangler
a2a85365d6 Use correct ADC channel for charger current on link
Signed-off-by: Randall Spangler <rspangler@chromium.org>

BUG=chrome-os-partner:7697
TEST=if it runs, it works

Change-Id: I36ab37a8cf1c3e4bf41bfb38e622e766cee8a4c4
2012-01-23 16:39:54 -08:00
Randall Spangler
cdc3fbf2cf Calibrate internal oscillator using hibernate clock
This works around a chip errata where the internal oscillator on early
EC parts (as used on proto0) is untrimmed.

Signed-off-by: Randall Spangler <rspangler@chromium.org>

BUG=chrome-os-partner:7693
TEST=if it runs, it works

Change-Id: Ie82a524543f4cf25efd0de7998dbdae103bd126b
2012-01-23 14:58:56 -08:00
Randall Spangler
51df9457f4 Add (disabled) support for compiling code for RAM.
When code is compiled for RAM (by re-enabling the flag in board.mk),
use the following openocd commands to load it:

reset halt
load_image ../../../build/link/ec.RO.flat 0x20000000 bin
reg 15 0x20000400
resume

Note that you'll also usually need to disable a bunch of modules to make
the code small enough to fit in RAM.

Signed-off-by: Randall Spangler <rspangler@chromium.org>

BUG=chrome-os-partner:7681
TEST=if it runs, it works

Change-Id: I2b3cc69b361ad73706af3ff6de1ce952e8d5a0a9
2012-01-23 12:40:29 -08:00
chrome-bot
e471361dda Merge "Fix linker script for code segment" 2012-01-23 11:22:53 -08:00
Vincent Palatin
3c9ed0f8c2 Fix linker script for code segment
Ensure that all the code ends up in the expected place.

This bug beat us for the RAM only version but it is present
even in the flash version (though probably harmless).

Signed-off-by: Vincent Palatin <vpalatin@chromium.org>

BUG=None
TEST=run the resulting image on BDS and inspect manually the .map file.

Change-Id: Iefe505bc507547b1043cb493ba7e8f210bb1ebc4
2012-01-23 18:35:47 +00:00
Randall Spangler
24ac6b98c3 Use internal oscillator
Signed-off-by: Randall Spangler <rspangler@chromium.org>

BUG=chrome-os-partner:7677
TEST=if it runs, it works

Change-Id: Ib4f28505f6b9a21375f9c3ba97c6e8d56722c051
2012-01-23 09:37:06 -08:00
Todd Broch
b561ef5918 Add openocd cfg for use to program EC via servo V2
BUG=none
TEST=manual,
<connect servo v2 to board w/ applicable EC>
cd src/platform/ec/chip/lm4/openocd
sudo openocd -f servo_v2.cfg

Should see proper access to TAP like:
Info : 88 289 core.c:948 jtag_examine_chain_display(): JTAG tap:
  lm4f232h.cpu tap/device found: 0x4ba00477 (mfg: 0x23b, part: 0xba00, ver: 0x4)

Change-Id: Ie36daba6a197e9ebe6c6ee875cf8fc711d64f7b4
Signed-off-by: Todd Broch <tbroch@chromium.org>
2012-01-19 18:01:05 -08:00
Vic Yang
af8026cdf9 USB Charging control
Implement TPS2543 USB charging control.
It contains routine for setting each USB port as dedicated charging port
or standard downstream port. To allow us controlling the current
distributed to each port, we can select whether to allow 500mA or 1500mA
for each port.

BUG=chrome-os-partner:7476
TEST=Added USB port definition for BDS and tested GPIO output voltage
level is correct for all modes.

Change-Id: I19bc4b30d333aa802f868ebfc3a398b30e99ba0f
2012-01-19 10:54:37 +08:00
Randall Spangler
37dcc1ab56 Handle all GPIO IRQs. Interrupts no longer enabled by default.
Signed-off-by: Randall Spangler <rspangler@chromium.org>

BUG=chrome-os-partner:7456
TEST=if it runs, it works

Change-Id: Ib82afab7d53203af31eefc9887feb98679266ac1
2012-01-17 15:21:20 -08:00
Randall Spangler
002bc4278b Add x86 power state machine
For bringup, this powers on the x86 unconditionally.

Signed-off-by: Randall Spangler <rspangler@chromium.org>

BUG=chrome-os-partner:7528
TEST=none

Change-Id: Ib23e56d38ab42f8d8a4dbd1ba9dce12f0c3eeec9
2012-01-17 12:19:47 -08:00
Randall Spangler
ff3ebed7a8 GPIO interrupts are disabled by default
Added gpio_enable_interrupt() to enable them.  This ensures that a
module which handles GPIO interrupts doesn't get them until it's
ready.

Signed-off-by: Randall Spangler <rspangler@chromium.org>

BUG=chrome-os-partner:7456
TEST=toggle power button while rebooting; without this fix it triggers a hard fault.

Change-Id: I35d926053963a70dd9246ce46a4913603b2b2489
2012-01-13 10:34:09 -08:00
Randall Spangler
3c90a7f2ac Move board-specific GPIO lists to board-specific files
Signed-off-by: Randall Spangler <rspangler@chromium.org>

BUG=chrome-os-partner:7528
TEST=none

Change-Id: I47fd5d709a9575e41fdcdf21a7440ebbb762cef5
2012-01-12 16:47:46 -08:00
Randall Spangler
70a9928add Configure all GPIOs
Signed-off-by: Randall Spangler <rspangler@chromium.org>

BUG=chrome-os-partner:7528
TEST=none

Change-Id: I0a9be4c689fb72507edcf202073b23c58902d7de
2012-01-12 15:54:35 -08:00
Randall Spangler
a8afa116be Tidy GPIO configuration and board-specific configs for modules.
Signed-off-by: Randall Spangler <rspangler@chromium.org>

BUG=chrome-os-partner:7528
TEST=none

Change-Id: Ia06707db78ea9a9313b49a93e8732a7fc9fcc191
2012-01-12 13:11:57 -08:00
Randall Spangler
d244f97bb9 Set up I2C for link board
Also fixes a typo in power_button.c when compiled for link

Signed-off-by: Randall Spangler <rspangler@chromium.org>

BUG=chrome-os-partner:7528
TEST=none

Change-Id: Ib43ef9edefe0b0caba246faa4a0d12d66ff9e205
2012-01-12 09:57:56 -08:00
chrome-bot
c386a8ff37 Merge "Add ectool option to specify single temp sensor to read data." 2012-01-12 08:19:31 -08:00
Randall Spangler
c9352ce6ca Add constants for all GPIOs.
Signed-off-by: Randall Spangler <rspangler@chromium.org>

BUG=chrome-os-partner:7528
TEST=none

Change-Id: I3b77cbbb7f0cc12a4daae7ababd603b5d7af32d1
2012-01-11 16:30:56 -08:00
Randall Spangler
1f2f8627c3 Add JTAG module
This just ensures the JTAG pins are reset to JTAG function on warm reboot.

Signed-off-by: Randall Spangler <rspangler@chromium.org>

BUG=chrome-os-partner:7448
TEST=none

Change-Id: I0cccdbe7a68c228db7f354898ed30598e9fabff0
2012-01-11 11:37:55 -08:00
Randall Spangler
a767d9b22b Add GPIO get/set commands
Signed-off-by: Randall Spangler <rspangler@chromium.org>

BUG=chrome-os-partner:7528
TEST=from debug console,

gpioget --> prints current level.  Run a few times to see DEBUG_LED
value toggle.

gpioset debug_led 1 --> turns debug LED on.  Run repeatedly to
override the idle task toggling it off.

Change-Id: I7c64044228697e052a9c20eb052d37a1f640f6e7
2012-01-11 10:59:59 -08:00
Randall Spangler
39f86d2fad Split power button code into its own file
Signed-off-by: Randall Spangler <rspangler@chromium.org>

BUG=chrome-os-partner:7499
TEST=press and release power button; should see debug messages

Change-Id: I8909ae4643afc98753edb690771618ad43135e3e
2012-01-10 14:14:59 -08:00
Randall Spangler
55ffdb465b Clean up labels and TODOs
Signed-off-by: Randall Spangler <rspangler@chromium.org>

BUG=none
TEST=none

Change-Id: I8d6e99b3e2b60d32ea3719468590b055e692a67e
2012-01-10 10:31:56 -08:00
Randall Spangler
edc50a4295 Add openocd config files
Signed-off-by: Randall Spangler <rspangler@chromium.org>

BUG=none
TEST=none

Change-Id: I211d23d61db37d7ed40621276f456e6935462c3d
2012-01-05 14:20:34 -08:00
Vincent Palatin
e80eb4d558 Fix tests compilation
add newly added tasks.

Signed-off-by: Vincent Palatin <vpalatin@chromium.org>

BUG=None
TEST=make tests ; make qemu-tests

Change-Id: Ia0719303e20d4bc1befc7a9bb13df49a063df11a
2012-01-04 17:38:42 +00:00
Vic Yang
0650392f53 Add ectool option to specify single temp sensor to read data.
Add an option to read temperature from only specified temperature
sensor. With this we can prevent ectool returning error if we already
know a sensor is not connected and do not want ectool to read it.

BUG=none
TEST=ectool temps; ectool temps temp_sensor_ec_internal

Change-Id: I221f1e3390d17f4a3ae99f58eb701a99c7b566b9
2012-01-04 16:21:39 +08:00
Louis Yung-Chieh Lo
6fc214c1e3 Implement power button behavior.
Implement the state machine for PWRBTN# input and output to PCH. The state
begins while power button is pressed and stops while it is released or 8 secs
pass. The state machine is maintained inside the infinite loop of gpio_task().

Remember to #define EVT after the board comes back.

Also, the power-up sequence is not completed yet. Remember to come back when
it is done. Search TODO in code.

BUG=none
TEST=Manual run on evaluation board (via console printf):
  #---------------------------
  # power button pressed and release after 8s
  [3070341] New State: START   next_ts:3070340
  # wait for next pwrbtn_sm_handle() is run.
  [3071428] set_pwrbtn_to_pch(LOW)
  [3071482] New State: T0   next_ts:3103426
  [3103963] set_pwrbtn_to_pch(HIGH)
  [3104023] New State: T1   next_ts:7071961
  [7072044] set_pwrbtn_to_pch(LOW)
  [7072103] New State: T2   next_ts:11072042
  [11072661] set_pwrbtn_to_pch(HIGH)
  [11072723] New State: STOPPED   next_ts:11072042
  # power button released
  [14336515] New State: STOPPING   next_ts:14336514
  [14337604] set_pwrbtn_to_pch(HIGH)
  [14337662] New State: SOPPED   next_ts:14336514
  #---------------------------
  # Power button released during T1
  [378324436] New State: START   next_ts:378324435
  [378325530] set_pwrbtn_to_pch(LOW)
  [378325589] New State: T0   next_ts:378357528
  [378358080] set_pwrbtn_to_pch(HIGH)
  [378358143] New State: T1   next_ts:382326078
  [378889269] New State: STOPPING   next_ts:378889268
  [378890361] set_pwrbtn_to_pch(HIGH)
  [378890420] New State: STOPPED   next_ts:378889268
  #---------------------------
  # Power button released during T2
  [448592391] New State: START   next_ts:448592390
  [448593483] set_pwrbtn_to_pch(LOW)
  [448593542] New State: T0   next_ts:448625481
  [448626041] set_pwrbtn_to_pch(HIGH)
  [448626104] New State: T1   next_ts:452594039
  [452594076] set_pwrbtn_to_pch(LOW)
  [452594138] New State: T2   next_ts:456594074
  [453199165] New State: STOPPING   next_ts:453199164
  [453200257] set_pwrbtn_to_pch(HIGH)
  [453200316] New State: STOPPED   next_ts:453199164

Change-Id: I5a12f3e46fa8086c530123a81ee743d3664b91f4
2011-12-30 16:38:16 +08:00