Skip to content

Commit 28e3dc8

Browse files
Tom Rixandersson
authored andcommitted
soc: qcom: pdr: use static for servreg_* variables
Sparse reports this repesentative issue pdr_internal.h:31:22: warning: symbol 'servreg_location_entry_ei' was not declared. Should it be static? Similar for other servreg_* Global variables should not be defined in header files. This only works because pdr_internal.h is only included by pdr_interface.c. Single file use variables should be static so add static to their storage-class specifiers. Signed-off-by: Tom Rix <[email protected]> Signed-off-by: Bjorn Andersson <[email protected]> Link: https://lore.kernel.org/r/[email protected]
1 parent ec69dfb commit 28e3dc8

File tree

1 file changed

+10
-10
lines changed

1 file changed

+10
-10
lines changed

drivers/soc/qcom/pdr_internal.h

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ struct servreg_location_entry {
2828
u32 instance;
2929
};
3030

31-
struct qmi_elem_info servreg_location_entry_ei[] = {
31+
static struct qmi_elem_info servreg_location_entry_ei[] = {
3232
{
3333
.data_type = QMI_STRING,
3434
.elem_len = SERVREG_NAME_LENGTH + 1,
@@ -74,7 +74,7 @@ struct servreg_get_domain_list_req {
7474
u32 domain_offset;
7575
};
7676

77-
struct qmi_elem_info servreg_get_domain_list_req_ei[] = {
77+
static struct qmi_elem_info servreg_get_domain_list_req_ei[] = {
7878
{
7979
.data_type = QMI_STRING,
8080
.elem_len = SERVREG_NAME_LENGTH + 1,
@@ -116,7 +116,7 @@ struct servreg_get_domain_list_resp {
116116
struct servreg_location_entry domain_list[SERVREG_DOMAIN_LIST_LENGTH];
117117
};
118118

119-
struct qmi_elem_info servreg_get_domain_list_resp_ei[] = {
119+
static struct qmi_elem_info servreg_get_domain_list_resp_ei[] = {
120120
{
121121
.data_type = QMI_STRUCT,
122122
.elem_len = 1,
@@ -199,7 +199,7 @@ struct servreg_register_listener_req {
199199
char service_path[SERVREG_NAME_LENGTH + 1];
200200
};
201201

202-
struct qmi_elem_info servreg_register_listener_req_ei[] = {
202+
static struct qmi_elem_info servreg_register_listener_req_ei[] = {
203203
{
204204
.data_type = QMI_UNSIGNED_1_BYTE,
205205
.elem_len = 1,
@@ -227,7 +227,7 @@ struct servreg_register_listener_resp {
227227
enum servreg_service_state curr_state;
228228
};
229229

230-
struct qmi_elem_info servreg_register_listener_resp_ei[] = {
230+
static struct qmi_elem_info servreg_register_listener_resp_ei[] = {
231231
{
232232
.data_type = QMI_STRUCT,
233233
.elem_len = 1,
@@ -263,7 +263,7 @@ struct servreg_restart_pd_req {
263263
char service_path[SERVREG_NAME_LENGTH + 1];
264264
};
265265

266-
struct qmi_elem_info servreg_restart_pd_req_ei[] = {
266+
static struct qmi_elem_info servreg_restart_pd_req_ei[] = {
267267
{
268268
.data_type = QMI_STRING,
269269
.elem_len = SERVREG_NAME_LENGTH + 1,
@@ -280,7 +280,7 @@ struct servreg_restart_pd_resp {
280280
struct qmi_response_type_v01 resp;
281281
};
282282

283-
struct qmi_elem_info servreg_restart_pd_resp_ei[] = {
283+
static struct qmi_elem_info servreg_restart_pd_resp_ei[] = {
284284
{
285285
.data_type = QMI_STRUCT,
286286
.elem_len = 1,
@@ -300,7 +300,7 @@ struct servreg_state_updated_ind {
300300
u16 transaction_id;
301301
};
302302

303-
struct qmi_elem_info servreg_state_updated_ind_ei[] = {
303+
static struct qmi_elem_info servreg_state_updated_ind_ei[] = {
304304
{
305305
.data_type = QMI_SIGNED_4_BYTE_ENUM,
306306
.elem_len = 1,
@@ -336,7 +336,7 @@ struct servreg_set_ack_req {
336336
u16 transaction_id;
337337
};
338338

339-
struct qmi_elem_info servreg_set_ack_req_ei[] = {
339+
static struct qmi_elem_info servreg_set_ack_req_ei[] = {
340340
{
341341
.data_type = QMI_STRING,
342342
.elem_len = SERVREG_NAME_LENGTH + 1,
@@ -362,7 +362,7 @@ struct servreg_set_ack_resp {
362362
struct qmi_response_type_v01 resp;
363363
};
364364

365-
struct qmi_elem_info servreg_set_ack_resp_ei[] = {
365+
static struct qmi_elem_info servreg_set_ack_resp_ei[] = {
366366
{
367367
.data_type = QMI_STRUCT,
368368
.elem_len = 1,

0 commit comments

Comments
 (0)