[freescale-fslc:5.10-2.1.x-imx 5408/15242] drivers/gpu/drm/bridge/cadence/cdns-hdmi-core.c:150:5: warning: this 'else' clause does not guard...
3:22 a.m.
Hi Sandor,
FYI, the error/warning still remains.
tree: https://github.com/Freescale/linux-fslc 5.10-2.1.x-imx
head: 3b81a70be1099d44fdafaa6766bf75a2bd9e297e
commit: fd6b9e1196cb2043cdd73a04477551353abba0e4 [5408/15242] MLK-25199-5: drm: bridge:
mhdp_hdcp: add HDMI TX HDCP driver
config: sparc64-randconfig-p001-20210927 (attached as .config)
compiler: sparc64-linux-gcc (GCC) 11.2.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O
~/bin/make.cross
chmod +x ~/bin/make.cross
#
https://github.com/Freescale/linux-fslc/commit/fd6b9e1196cb2043cdd73a0447...
git remote add freescale-fslc https://github.com/Freescale/linux-fslc
git fetch --no-tags freescale-fslc 5.10-2.1.x-imx
git checkout fd6b9e1196cb2043cdd73a04477551353abba0e4
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross ARCH=sparc64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
> drivers/gpu/drm/imx/imx-drm-core.c:46:6: warning: no previous
prototype for 'imx_drm_encoder_destroy' [-Wmissing-prototypes]
46 |
void imx_drm_encoder_destroy(struct drm_encoder *encoder)
| ^~~~~~~~~~~~~~~~~~~~~~~
--
> drivers/gpu/drm/bridge/cadence/cdns-mhdp-common.c:156:5: warning:
no previous prototype for 'mhdp_mailbox_read' [-Wmissing-prototypes]
156 | int mhdp_mailbox_read(struct cdns_mhdp_device *mhdp)
| ^~~~~~~~~~~~~~~~~
--
drivers/gpu/drm/bridge/cadence/cdns-hdmi-core.c: In function
'HDCPTX_Status_store':
> drivers/gpu/drm/bridge/cadence/cdns-hdmi-core.c:150:5: warning:
this 'else' clause does not guard... [-Wmisleading-indentation]
150 |
else
| ^~~~
drivers/gpu/drm/bridge/cadence/cdns-hdmi-core.c:152:17: note: ...this statement, but
the latter is misleadingly indented as if it were guarded by the 'else'
152 | return -1;
| ^~~~~~
drivers/gpu/drm/bridge/cadence/cdns-hdmi-core.c: At top level:
drivers/gpu/drm/bridge/cadence/cdns-hdmi-core.c:304:6: warning: no previous prototype
for 'cdns_hdmi_mode_set' [-Wmissing-prototypes]
304 | void cdns_hdmi_mode_set(struct cdns_mhdp_device *mhdp)
| ^~~~~~~~~~~~~~~~~~
drivers/gpu/drm/bridge/cadence/cdns-hdmi-core.c:607:6: warning: no previous prototype
for 'cdns_hdmi_bridge_mode_fixup' [-Wmissing-prototypes]
607 | bool cdns_hdmi_bridge_mode_fixup(struct drm_bridge *bridge,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~
--
> drivers/gpu/drm/bridge/cadence/cdns-mhdp-hdcp.c:155:5: warning:
no previous prototype for 'cdns_mhdp_hdcp_tx_config' [-Wmissing-prototypes]
155 | int cdns_mhdp_hdcp_tx_config(struct cdns_mhdp_device *mhdp, u8 config)
| ^~~~~~~~~~~~~~~~~~~~~~~~
> drivers/gpu/drm/bridge/cadence/cdns-mhdp-hdcp.c:162:5: warning:
no previous prototype for 'cdns_mhdp_hdcp2_tx_respond_km' [-Wmissing-prototypes]
162 | int cdns_mhdp_hdcp2_tx_respond_km(struct cdns_mhdp_device *mhdp,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> drivers/gpu/drm/bridge/cadence/cdns-mhdp-hdcp.c:170:5: warning:
no previous prototype for 'cdns_mhdp_hdcp_tx_status_req' [-Wmissing-prototypes]
170 | int cdns_mhdp_hdcp_tx_status_req(struct cdns_mhdp_device *mhdp,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
> drivers/gpu/drm/bridge/cadence/cdns-mhdp-hdcp.c:196:5: warning:
no previous prototype for 'cdns_mhdp_hdcp2_tx_is_km_stored_req'
[-Wmissing-prototypes]
196 | int cdns_mhdp_hdcp2_tx_is_km_stored_req(struct
cdns_mhdp_device *mhdp, u8 *data, u16 len)
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> drivers/gpu/drm/bridge/cadence/cdns-mhdp-hdcp.c:219:5: warning:
no previous prototype for 'cdns_mhdp_hdcp2_tx_store_km' [-Wmissing-prototypes]
219 | int cdns_mhdp_hdcp2_tx_store_km(struct cdns_mhdp_device *mhdp,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~
> drivers/gpu/drm/bridge/cadence/cdns-mhdp-hdcp.c:241:5: warning:
no previous prototype for 'cdns_mhdp_hdcp_tx_is_receiver_id_valid'
[-Wmissing-prototypes]
241 | int cdns_mhdp_hdcp_tx_is_receiver_id_valid(struct
cdns_mhdp_device *mhdp,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> drivers/gpu/drm/bridge/cadence/cdns-mhdp-hdcp.c:287:5: warning:
no previous prototype for 'cdns_mhdp_hdcp_tx_respond_receiver_id_valid'
[-Wmissing-prototypes]
287 | int cdns_mhdp_hdcp_tx_respond_receiver_id_valid(
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> drivers/gpu/drm/bridge/cadence/cdns-mhdp-hdcp.c:295:5: warning:
no previous prototype for 'cdns_mhdp_hdcp_tx_reauth' [-Wmissing-prototypes]
295 | int cdns_mhdp_hdcp_tx_reauth(struct cdns_mhdp_device *mhdp, u8 msg)
| ^~~~~~~~~~~~~~~~~~~~~~~~
--
drivers/gpu/drm/bridge/cadence/cdns-hdmi-hdcp.c: In function
'hdmi_hdcp_set_config':
> drivers/gpu/drm/bridge/cadence/cdns-hdmi-hdcp.c:225:24: warning:
variable 'retEvents' set but not used [-Wunused-but-set-variable]
225 |
u8 bus_config, retEvents;
| ^~~~~~~~~
drivers/gpu/drm/bridge/cadence/cdns-hdmi-hdcp.c: In function
'hdmi_hdcp_check_receviers':
> drivers/gpu/drm/bridge/cadence/cdns-hdmi-hdcp.c:329:20: warning:
variable 'events' set but not used [-Wunused-but-set-variable]
329 |
u8 events = 0;
| ^~~~~~
drivers/gpu/drm/bridge/cadence/cdns-hdmi-hdcp.c: At top level:
> drivers/gpu/drm/bridge/cadence/cdns-hdmi-hdcp.c:871:6: warning:
no previous prototype for 'hdmi_hdcp_dump_pairing' [-Wmissing-prototypes]
871 | void hdmi_hdcp_dump_pairing(struct seq_file *s, void *data)
| ^~~~~~~~~~~~~~~~~~~~~~
vim +/else +150 drivers/gpu/drm/bridge/cadence/cdns-hdmi-core.c
116
117 ssize_t HDCPTX_Status_store(struct device *dev,
118 struct device_attribute *attr, const char *buf, size_t count)
119 {
120 struct cdns_mhdp_device *mhdp = dev_get_drvdata(dev);
121 int value;
122
123 if (count == 2) {
124 sscanf(buf, "%d", &value);
125 if ((value >= HDCP_STATE_NO_AKSV) && (value <=
HDCP_STATE_AUTH_FAILED)) {
126 mhdp->hdcp.state = value;
127 return count;
128 } else {
129 dev_err(dev, "%s &hdp->state invalid\n", __func__);
130 return -1;
131 }
132 }
133
134 dev_info(dev, "%s &hdp->state desired %s count=%d\n ", __func__,
buf, (int)count);
135
136 if (strncmp(buf, "HDCP_STATE_NO_AKSV", count - 1) == 0)
137 mhdp->hdcp.state = HDCP_STATE_NO_AKSV;
138 else if (strncmp(buf, "HDCP_STATE_INACTIVE", count - 1) == 0)
139 mhdp->hdcp.state = HDCP_STATE_INACTIVE;
140 else if (strncmp(buf, "HDCP_STATE_ENABLING", count - 1) == 0)
141 mhdp->hdcp.state = HDCP_STATE_ENABLING;
142 else if (strncmp(buf, "HDCP_STATE_AUTHENTICATING", count - 1) == 0)
143 mhdp->hdcp.state = HDCP_STATE_AUTHENTICATING;
144 else if (strncmp(buf, "HDCP_STATE_AUTHENTICATED", count - 1) == 0)
145 mhdp->hdcp.state = HDCP_STATE_AUTHENTICATED;
146 else if (strncmp(buf, "HDCP_STATE_DISABLING", count - 1) == 0)
147 mhdp->hdcp.state = HDCP_STATE_DISABLING;
148 else if (strncmp(buf, "HDCP_STATE_AUTH_FAILED", count - 1) == 0)
149 mhdp->hdcp.state = HDCP_STATE_AUTH_FAILED;
150 else
151 dev_err(dev, "%s &hdp->state
invalid\n", __func__);
152 return -1;
153 return count;
154 }
155
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
312
days inactive
312
days old
0 comments
1 participants
participants (1)
-
kernel test robot