On Sat, May 16, 2020 at 10:23:30PM +0100, Chris Wilson wrote: > drivers/gpu/drm/drm_dp_mst_topology.c:2898:6: warning: symbol > 'drm_dp_send_clear_payload_id_table' was not declared. Should it be static?
Compiler seems a bit confused, because there is a static declaration a bit further up. So should have complained about inconsistent decls, but then my understanding of C is rather shoddy. > drivers/gpu/drm/drm_dp_mst_topology.c:5451:37: warning: missing braces around > initializer > drivers/gpu/drm/drm_dp_mst_topology.c:5451:37: warning: missing braces around > initializer > > Signed-off-by: Chris Wilson <[email protected]> Reviewed-by: Daniel Vetter <[email protected]> > --- > drivers/gpu/drm/drm_dp_mst_topology.c | 7 ++++--- > 1 file changed, 4 insertions(+), 3 deletions(-) > > diff --git a/drivers/gpu/drm/drm_dp_mst_topology.c > b/drivers/gpu/drm/drm_dp_mst_topology.c > index 70455e304a26..1bdf3cfeeebb 100644 > --- a/drivers/gpu/drm/drm_dp_mst_topology.c > +++ b/drivers/gpu/drm/drm_dp_mst_topology.c > @@ -2895,8 +2895,9 @@ static int drm_dp_send_link_address(struct > drm_dp_mst_topology_mgr *mgr, > return ret < 0 ? ret : changed; > } > > -void drm_dp_send_clear_payload_id_table(struct drm_dp_mst_topology_mgr *mgr, > - struct drm_dp_mst_branch *mstb) > +static void > +drm_dp_send_clear_payload_id_table(struct drm_dp_mst_topology_mgr *mgr, > + struct drm_dp_mst_branch *mstb) > { > struct drm_dp_sideband_msg_tx *txmsg; > int ret; > @@ -5448,7 +5449,7 @@ struct drm_dp_aux *drm_dp_mst_dsc_aux_for_port(struct > drm_dp_mst_port *port) > { > struct drm_dp_mst_port *immediate_upstream_port; > struct drm_dp_mst_port *fec_port; > - struct drm_dp_desc desc = { 0 }; > + struct drm_dp_desc desc = {}; > u8 endpoint_fec; > u8 endpoint_dsc; > > -- > 2.20.1 > > _______________________________________________ > dri-devel mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/dri-devel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dri-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/dri-devel
