mirror of
https://github.com/Telecominfraproject/OpenCellular.git
synced 2025-12-28 10:45:22 +00:00
af77729737
We've been declaring a bunch of statically-sized arrays:
extern struct foo_t foo[FOO_COUNT];
And then initializing them like so:
struct foo_t foo[FOO_COUNT] = {
/* blah */
};
That only catches cases where we initialize with too many entries. It
doesn't catch cases where we haven't initialized enough. This change tests
for both cases like so:
extern struct foo_t foo[];
struct foo_t foo[] = {
/* blah */
};
BUILD_ASSERT(ARRAY_SIZE(foo) == FOO_COUNT);
The affected arrays are:
adc_channels[ADC_CH_COUNT]
gpio_list[GPIO_COUNT]
temp_sensors[TEMP_SENSOR_COUNT]
x86_signal_list[X86_SIGNAL_COUNT]
i2c_ports[I2C_PORTS_USED]
BUG=chrome-os-partner:18343
BRANCH=falco,peppy
TEST=build all platforms
All platforms should still build, all tests should still pass.
Change-Id: Ibb16dc3201f32df7cdc875648e89ba4ffb09f733
Signed-off-by: Bill Richardson <wfrichar@chromium.org>
Reviewed-on: https://gerrit.chromium.org/gerrit/63833
Reviewed-by: Randall Spangler <rspangler@chromium.org>
62 lines
1.6 KiB
C
62 lines
1.6 KiB
C
/* Copyright (c) 2012 The Chromium OS Authors. All rights reserved.
|
|
* Use of this source code is governed by a BSD-style license that can be
|
|
* found in the LICENSE file.
|
|
*/
|
|
|
|
/* Temperature sensor module for Chrome EC */
|
|
|
|
#ifndef __CROS_EC_TEMP_SENSOR_H
|
|
#define __CROS_EC_TEMP_SENSOR_H
|
|
|
|
#include "common.h"
|
|
|
|
/* "enum temp_sensor_id" must be defined for each board in board.h. */
|
|
enum temp_sensor_id;
|
|
|
|
/* Type of temperature sensors. */
|
|
enum temp_sensor_type {
|
|
/* Ignore this temperature sensor. */
|
|
TEMP_SENSOR_TYPE_IGNORED = -1,
|
|
/* CPU temperature sensors. */
|
|
TEMP_SENSOR_TYPE_CPU = 0,
|
|
/* Other on-board temperature sensors. */
|
|
TEMP_SENSOR_TYPE_BOARD,
|
|
/* Case temperature sensors. */
|
|
TEMP_SENSOR_TYPE_CASE,
|
|
|
|
TEMP_SENSOR_TYPE_COUNT
|
|
};
|
|
|
|
struct temp_sensor_t {
|
|
const char* name;
|
|
/* Temperature sensor type. */
|
|
enum temp_sensor_type type;
|
|
/* Read sensor value in K into temp_ptr; return non-zero if error. */
|
|
int (*read)(int idx, int *temp_ptr);
|
|
/* Index among the same kind of sensors. */
|
|
int idx;
|
|
/* Delay between reading temperature and taking action about it,
|
|
* in seconds. */
|
|
int action_delay_sec;
|
|
};
|
|
|
|
#ifdef CONFIG_TEMP_SENSOR
|
|
/*
|
|
* Defined in board_temp_sensor.c. Must be in the same order as
|
|
* in enum temp_sensor_id.
|
|
*/
|
|
extern const struct temp_sensor_t temp_sensors[];
|
|
#endif
|
|
|
|
/**
|
|
* Get the most recently measured temperature for the sensor.
|
|
*
|
|
* @param id Sensor ID
|
|
* @param temp_ptr Destination for temperature
|
|
*
|
|
* @return EC_SUCCESS, or non-zero if error.
|
|
*/
|
|
int temp_sensor_read(enum temp_sensor_id id, int *temp_ptr);
|
|
|
|
#endif /* __CROS_EC_TEMP_SENSOR_H */
|