From c1e02ca2203a7889539a88570e142f16cfd265a8 Mon Sep 17 00:00:00 2001 From: Randall Spangler Date: Fri, 1 Nov 2013 11:39:00 -0700 Subject: cleanup: Yes, even more TODO comments Almost done. Comment changes only. BUG=chrome-os-partner:18343 BRANCH=none TEST=build all platforms Change-Id: I974dfc12aa264e2035b3bae35a089c19344e7d45 Signed-off-by: Randall Spangler Reviewed-on: https://chromium-review.googlesource.com/175484 Reviewed-by: Bill Richardson --- common/build.mk | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) (limited to 'common') diff --git a/common/build.mk b/common/build.mk index 540626e0d7..132fd62783 100644 --- a/common/build.mk +++ b/common/build.mk @@ -11,14 +11,15 @@ common-y+=memory_commands.o shared_mem.o system.o hooks.o common-y+=gpio.o version.o printf.o queue.o common-$(CONFIG_BACKLIGHT_LID)+=backlight_lid.o -# TODO(rspangler): Why do these include battery_common but the other batteries -# don't? Perhaps should use CONFIG_CMD_BATTERY instead, since all that's in -# battery.c is the battery console command? +# TODO(crosbug.com/p/23821): Why do these include battery_common but +# the other batteries don't? Perhaps should use CONFIG_CMD_BATTERY +# instead, since all that's in battery.c is the battery console +# command? common-$(CONFIG_BATTERY_BQ27541)+=battery.o common-$(CONFIG_BATTERY_SMART)+=battery.o common-$(CONFIG_CHARGER)+=charge_state.o charger.o -# TODO(rspangler): This is really the charge state machine for ARM, not the -# charger driver for the tps65090. Rename. +# TODO(crosbug.com/p/23815): This is really the charge state machine +# for ARM, not the charger driver for the tps65090. Rename. common-$(CONFIG_CHARGER_TPS65090)+=pmu_tps65090_charger.o common-$(CONFIG_PMU_POWERINFO)+=pmu_tps65090_powerinfo.o common-$(CONFIG_PMU_TPS65090)+=pmu_tps65090.o -- cgit v1.2.1