Commit Graph

706 Commits

Author SHA1 Message Date
Hung-Te Lin
6654374f31 ec: Remove *.dis from default target.
*.dis files can always be generated by *.elf files, so we can remove it from
default build targets, and only generate that on demand (make dis).

This also speeds up building time from 6.637s to 4.9s.

BUG=chromium-os:31379
TEST=emerge-link chromeos-ec
      make # no *.dis
      make dis # get *.dis

Change-Id: Ibc5305501ae72a0733f401863ea1d4c1f17aa34f
Signed-off-by: Hung-Te Lin <hungte@chromium.org>
2012-05-30 14:00:16 +08:00
Louis Yung-Chieh Lo
2835e5594e Improve the timeout when running on non-Google EC boards.
The udelay() has big overhead so that repeating calling leads big errors
(expect 1 sec of timeout, but actually 12 secs of timeout).

So, the improvement is to double the udelay count when BUSY bit is set.

Even better, if we can check the I/O port content before really running
the EC command, it can save more time.

BUG=chrome-os-partner:10003
TEST=tested on link, alex, zgb, lumpy, stumpy and mario.
Only mario takes 1 second to timeout.
Others stop when checking ports (takes around 0.01 second).

Change-Id: I96c6d8cbe6226d05428a2ab126815e934942f5a9
Signed-off-by: Louis Yung-Chieh Lo <yjlou@chromium.org>
2012-05-30 10:10:16 +08:00
Gerrit
8bb9091f2e Merge "Add host command to simulate key press" 2012-05-29 18:19:17 -07:00
Gerrit
fa8788e8c0 Merge "Jump to RAM before entering hibernate" 2012-05-29 11:19:47 -07:00
Gerrit
63da7ee730 Merge "Use correct EEPROM timeout" 2012-05-29 10:31:35 -07:00
Randall Spangler
e31d9a6660 Jump to RAM before entering hibernate
Works around LM4 errata where EEPROM access is unstable while powering down.

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

BUG=chrome-os-partner:9996
TEST=hibernate 1

Change-Id: I99d21ec8ab5a06fb0972edebec3cc58ca9f60fa9
2012-05-29 10:21:36 -07:00
Gerrit
7f66786b76 Merge "usleep() chains to udelay() if called before task_start()" 2012-05-29 10:12:37 -07:00
Randall Spangler
d42877300a Use correct EEPROM timeout
Signed-off-by: Randall Spangler <rspangler@chromium.org>

BUG=chrome-os-partner:10200
TEST=manual

eeread 2 0 --> note original value
eewrite 2 0 0x1234
eeread 2 0 --> should be 0x1234
eewrite 2 0 (original value)
eeread 2 0 --> should be original value again

Change-Id: Ibb72426663122b22b2bfe87c821c374eab334450
2012-05-29 10:02:34 -07:00
Randall Spangler
ef46903dea usleep() chains to udelay() if called before task_start()
...so I can use usleep() for eeprom delays in the CL coming next...

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

BUG=chrome-os-partner:10200
TEST=if it boots, it worked

Change-Id: I564578f24452a4ac39abe79ff28cfff4b665ad2f
2012-05-29 09:38:58 -07:00
Gerrit
cbd18dbe5b Merge "Add a way to set indiviual sensitivity factor for each TMP006 sensor" 2012-05-29 09:16:04 -07:00
Vic Yang
abce21c6cf Add host command to simulate key press
In order to enable automatic keyboard testing, let's add key press
simulating command to ectool.

BUG=chrome-os-partner:9188
TEST='ectool kbpress 4 6 1' and see 'j' pressed.
     'ectool kbpress 4 6 0' and see 'j' released.

Change-Id: I5a445e13aad2bd09aa6e9a1d62995cf34b782aeb
2012-05-29 17:11:23 +08:00
Rong Chang
f267899861 Fix temperature comparison issue in discharging state
When discharging, EC checks if battery temperature is in working
range. The battery pack module should provide the working range
in deci-Kelvin, not Celcius.

Signed-off-by: Rong Chang <rongchang@chromium.org>
BUG=chrome-os-partner:9485
TEST=manual
  Unplug AC adapter, and check
    /sys/class/power_supply/BAT0/charge_now

Change-Id: Ib6a312941cb1f3622c6f18d2c58bc50a06feafaf
2012-05-28 18:07:07 +08:00
Vic Yang
86cfeb5232 Add a way to set indiviual sensitivity factor for each TMP006 sensor
Each TMP006 temperature sensor has different sensitivity factor. Let's
add a field to set different sensitivity factor for each sensor. Also
update the factors to get more reasonable temperature readings, but
still need more precise calibration.

BUG=chrome-os-partner:9599
TEST=Build and read tempearture succeeded.

Change-Id: Ib4feea3b78b71f6d37c9a02668ffa7bd9e63d390
2012-05-28 16:42:17 +08:00
Vic Yang
b85a7ce9d6 Make ectool correctly report when keyboard backlight is off
When keyboard backlight is disabled, make 'ectool pwmgetkblight' reports
'disabled'.

BUG=chrome-os-partner:9966
TEST='ectool pwmgetkblight' shows 'Keyboard backlight disabled' when
lid closed.

Change-Id: Ica690159e30431ccb530275fcc2311fb8f54a9aa
2012-05-26 20:29:54 +08:00
Randall Spangler
7ecd1d6d3c Add system_is_locked() to prevent sysjump on consumer systems
This returns true when both HW and SW write protect are enabled.

Once WP is enabled, sysjump will be locked out.

system_is_locked() can be used to gate other dangerous-ish commands too.

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

BUG=chrome-os-partner:7468
TEST=manual

sysinfo -> unlocked, copy A
sysjump B -> works
flashwp lock
reboot
(make sure flashinfo shows WP asserted and flash locked; note there is a
 HW bug on proto1 which makes this flaky)
sysinfo -> locked, copy A
sysjump B -> fails
(remove WP screw)
reboot hard
flashwp unlock

Change-Id: I849b573675c2c1cb4c44b9a05d6973e38247ca23
2012-05-25 15:03:47 -07:00
Randall Spangler
e704c712ad Better help for console commands
Additional help messages and usage are gated by
CONFIG_CONSOLE_CMDHELP, so we can turn it on if there's space (adds
about 3KB to image size) and turn it off when there isn't.

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

BUG=none
TEST=manual

1) help
2) help list
3) help gpioset
4) gpioset -> wrong number of params
5) gpioset fred 0 -> param1 bad
6) gpioset cpu_prochot fred -> param2 bad

Change-Id: Ibe99f37212020f763ebe65a068e6aa83a809a370
2012-05-25 13:34:06 -07:00
Gerrit
89e1d5a121 Merge "Add mmapinfo command to power_button.c" 2012-05-25 12:47:48 -07:00
Gerrit
755ac3de91 Merge "stm32: fix keyboard FIFO" 2012-05-25 11:59:06 -07:00
Gerrit
d4fb531066 Merge "Cleanup - includes go in include dir" 2012-05-25 10:40:56 -07:00
Bill Richardson
cc778c70d6 Add mmapinfo command to power_button.c
BUG=none
TEST=none

Change-Id: Id6ced0b87b354136e0b80026a537958586bb1325
Signed-off-by: Bill Richardson <wfrichar@chromium.org>
2012-05-25 10:20:25 -07:00
Randall Spangler
4f34d3f9ef Cleanup - includes go in include dir
Signed-off-by: Randall Spangler <rspangler@chromium.org>

BUG=none
TEST=if it builds, it works

Change-Id: I2064f3eed4790051312a5a53ef742dcf79c4ee9d
2012-05-25 09:59:58 -07:00
Vincent Palatin
c7fcad2924 Init task contexts/stacks at runtime
Instead of storing task contexts in .data and wasting several kB of
flash with mostly 0s, move them to .bss and fill the initial context at EC
startup. The runtime overhead is small enough.

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

BUG=chrome-os-partner:9839
TEST=run on Link and check verified boot and chromeOS startup are OK.

Change-Id: Iaef23d46a4e3e80e49886dfbf7ab1f537c587362
2012-05-25 16:11:41 +00:00
Vincent Palatin
38bab6b9f1 stm32: fix keyboard FIFO
When the FIFO is empty, returns the last read entry not the next one.

also rewrite the FIFO index increment to generate slightly better code.

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

BUG=chrome-os-partner:8869
TEST=On Snow, in U-Boot using "stdin=mkbp-keyb" type on internal
keyboard and see the correct text.

Change-Id: I189d230053de40dd563ce672db82dd6217e545e3
2012-05-25 16:04:50 +00:00
Gerrit
61902efd16 Merge "Fix disabling columns when power button is pressed" 2012-05-25 02:41:26 -07:00
Gerrit
09f23c8aa8 Merge "Add I2C host communication" 2012-05-24 18:45:18 -07:00
Gerrit
6b03f5389d Merge "Split communication functions from host tools" 2012-05-24 17:36:39 -07:00
Randall Spangler
b62a71a7eb Fix disabling columns when power button is pressed
This works around a problem where columns are not tri-stated when
calling select_column(COLUMN_ASSERT_ALL) with enable_scanning=0.

Also removes polling for power button released; we can use the same
task wait for that as we do for a keypress.

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

BUG=chrome-os-partner:7486
TEST=manual

1) press g then press power button repeatedly.  EC is not rebooted.
2) press power button the g repeatedly.  EC is not rebooted.
3) press power+refresh (or ESC on proto1).  EC is rebooted.

Change-Id: I43a0beae1a6c0ef8fa9379a8fff47b6006e63c8c
2012-05-24 16:42:34 -07:00
Vincent Palatin
2a5e284960 Add I2C host communication
Allow to use EC tool on ARM based platforms.

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

BUG=None
TEST=On Daisy, ectool version

Change-Id: If7f52de827d0bcffb39af0553245cce4e02b9b48
2012-05-24 23:34:01 +00:00
Vincent Palatin
304d207117 Split communication functions from host tools
Preparatory work to re-use the tools on ARM boards using I2C
communications.

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

BUG=None
TEST=make BOARD=link && make BOARD=bds && make BOard=DAISY

Change-Id: I31d41f30c3231a4a9349b939bf6bba871ed4c383
2012-05-24 23:34:01 +00:00
Randall Spangler
9d46eb75a2 Enable brown-out reset
This resets the EC when power falls below the brown-out threshold

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

BUG=chrome-os-partner:9952
TEST=hopefully srikanth can; requires hardware mods to test.

Change-Id: I161e49003409cb68eb43303c3c8de0eb4cc27104
2012-05-24 15:33:49 -07:00
Randall Spangler
b0fe455608 Fix I2C port frequency calculation
Also fix 'i2cscan' output, and detect/ignore busy ports

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

BUG=chrome-os-partner:9928
TEST=manual

1) check i2cscan
    i2cscan -> thermal port should report busy
    gpioset enable_vs 1
    i2cscan -> thermal port should show 4 devices

2) probe EC_SMB_CK[012] signals to verify clock frequency

Change-Id: I52e165de77eb9ed2619d648d84f251152772fe1c
2012-05-24 12:25:34 -07:00
Gerrit
00360ddc17 Merge "Add a way to pass in define flag when 'make'" 2012-05-24 09:59:05 -07:00
Randall Spangler
0d6203adbf Update backlight in lid open/close funcs
...so lidopen/lidclose debug command also update the backlight

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

BUG=chrome-os-partner:9188
TEST=manual:

powerbtn -> boot system, display backlight on
kblight 100 -> keyboard backlight on
lidclose -> backlights off
lidopen -> backlights on

Change-Id: I0d092d3061ba8a00c8790b3d1b8380eecd2d1628
2012-05-24 09:28:03 -07:00
Vic Yang
b8a2a4b575 Add a way to pass in define flag when 'make'
In order to pass in different config from ebuild and enable easier
debugging/testing, let's add a way to pass in define flags to make.

BUG=chrome-os-partner:9936
TEST='V=1 EXTRA_CFLAGS=-DFACTORY_MODE make' and see '-DFACTORY_MODE'.

Change-Id: I18dc5a3dc61bda1e63e96081def3f077ba431938
2012-05-24 23:06:28 +08:00
Gerrit
0a77eea2e4 Merge "Keyboard scan must stop driving columns when power button is pressed." 2012-05-23 23:07:09 -07:00
Louis Yung-Chieh Lo
29d25d807c Keyboard scan must stop driving columns when power button is pressed.
Tristage all columns as soon as possible right after the power button is
pressed. This can avoid the silego chip to reset the EC.

Resume the interrupt mode after power button is released so that the
deasserted columns doesn't trigger EC reset.

Also change some function names for export.

BUG=chrome-os-partner:7486
TEST=
press g first, then power button. repeat many times. system is not rebooted.
press power button first, then g. repeat many times. system is not rebooted.
ESC+power is still reseting system.
The long press power button still shutdowns system.

Change-Id: Ie5dec20ec3d3c4ffbd4acf5a4dd7f63eec38af56
2012-05-24 13:32:35 +08:00
Gerrit
e5333535f4 Merge "Handle multiple pending lightbar events." 2012-05-23 11:48:56 -07:00
Gerrit
eb7765c0c4 Merge "stm32: try to be more robust against malformed commands" 2012-05-22 19:52:56 -07:00
Gerrit
abc3b2ddce Merge "Add EC command to report board version" 2012-05-22 18:21:59 -07:00
Vincent Palatin
d9954b304a stm32: try to be more robust against malformed commands
Timeout properly when the AP doesn't want our bytes.

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

BUG=chrome-os-partner:8865
TEST=On Snow, boot with an old kernel.

Change-Id: Iac4fa5c3606f2e8731927326fad291dae26a615c
2012-05-23 00:52:36 +00:00
Bill Richardson
967e942084 Handle multiple pending lightbar events.
Since the events are requests for new patterns, we only want to do the
latest one. This does that.

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

No test: multiple events are rare. I had to fake them to see the problem.

Change-Id: I7e7de94655576e16c442a97097a7303e422a551a
Signed-off-by: Bill Richardson <wfrichar@chromium.org>
2012-05-22 17:36:07 -07:00
Gerrit
8b468a8a26 Merge "stm32: ensure I2C properly behave after sysjump" 2012-05-22 15:50:09 -07:00
Gerrit
37732ced8c Merge "snow: ensure we have proper size in the FMAP" 2012-05-22 15:50:09 -07:00
Vadim Bendebury
d80c362969 Add EC command to report board version
The main CPU might need to know the particular hardware version the
system is running on. This information is available to the EC, this
change adds a mechanism for the main CPU to request this information.

The board version is defined as a flat 16 bit number.

BUG=chrome-os-partner:8722
TEST=manual
  . this change was tested in concert with the appropriate coreboot
    modification, the proto1 board returns board version of 0. The
    code was modified manually to return a one off version - and the
    coreboot indeed observed board version of 1

Change-Id: If434d33f43783b18cb202ea15819bd5804694b8e
Signed-off-by: Vadim Bendebury <vbendeb@chromium.org>
2012-05-22 15:10:54 -07:00
Vincent Palatin
4a7774c122 stm32: ensure I2C properly behave after sysjump
Sometimes when requesting a sysjump from the AP, the I2C bus gets
wedged. Ensure we restore it in a proper state during the driver
initialization.

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

BUG=chrome-os-partner:8865
TEST=On Snow, use flashrom to program the EC.

Change-Id: I6279afba2bdeed4473ace42220a0342c4addaf1d
2012-05-22 21:42:36 +00:00
Vincent Palatin
d819f6cee4 snow: ensure we have proper size in the FMAP
The size of the RW partition is size of RW-A + size of RW-B.
When B is de-activated, we need to nullify its size to get the correct
total size.

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

BUG=chrome-os-partner:8865
TEST=On Snow, flashrom -p internal:bus=lpc -i EC_RW -w ec.bin

Change-Id: Id015af512f072efce5fcafc5a630845c5d9600bb
2012-05-22 21:42:35 +00:00
Bill Richardson
a7d62b4fd4 Add LPC command to handle vboot stuff.
This adds "ectool vboot", which works like so:

  # ./ectool vboot
  0x06 image=A fake_dev=1

  # ./ectool vboot 0
  # ./ectool vboot
  0x02 image=A fake_dev=0

  # ./ectool vboot ff
  # ./ectool vboot
  0x06 image=A fake_dev=1

You can set or unset the fake dev-switch, and see which firmware image the
EC is running from.

BUG=chrome-os-partner:8313
TEST=manual

Boot the host, use crossystem to see the devsw_boot state. Change it with
the ectool command, reboot, see that it's changed.

Change-Id: Iaac40267338c6a07bc47b80e925e829bf1e1ae0c
Signed-off-by: Bill Richardson <wfrichar@chromium.org>
2012-05-22 14:22:13 -07:00
Gerrit
43e26da794 Merge "do not advertise RW-B in FMAP if it doesn't exist" 2012-05-22 14:05:04 -07:00
Gerrit
5177cf3714 Merge "Notify lightbar task when CPU enters/leaves S5" 2012-05-22 14:05:04 -07:00
Gerrit
1089394038 Merge "Add support for printing fixed-point numbers" 2012-05-22 13:43:29 -07:00