On Mon, Nov 16, 2020 at 12:38 PM Lee Jones <lee.jo...@linaro.org> wrote: > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/radeon/dce6_afmt.c:32:5: warning: no previous prototype for > ‘dce6_endpoint_rreg’ [-Wmissing-prototypes] > 32 | u32 dce6_endpoint_rreg(struct radeon_device *rdev, > | ^~~~~~~~~~~~~~~~~~ > drivers/gpu/drm/radeon/dce6_afmt.c:46:6: warning: no previous prototype for > ‘dce6_endpoint_wreg’ [-Wmissing-prototypes] > 46 | void dce6_endpoint_wreg(struct radeon_device *rdev, > | ^~~~~~~~~~~~~~~~~~ > drivers/gpu/drm/radeon/dce6_afmt.c:110:6: warning: no previous prototype for > ‘dce6_afmt_select_pin’ [-Wmissing-prototypes] > 110 | void dce6_afmt_select_pin(struct drm_encoder *encoder) > | ^~~~~~~~~~~~~~~~~~~~ > drivers/gpu/drm/radeon/dce6_afmt.c:123:6: warning: no previous prototype for > ‘dce6_afmt_write_latency_fields’ [-Wmissing-prototypes] > 123 | void dce6_afmt_write_latency_fields(struct drm_encoder *encoder, > | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > drivers/gpu/drm/radeon/dce6_afmt.c:152:6: warning: no previous prototype for > ‘dce6_afmt_hdmi_write_speaker_allocation’ [-Wmissing-prototypes] > 152 | void dce6_afmt_hdmi_write_speaker_allocation(struct drm_encoder > *encoder, > | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > drivers/gpu/drm/radeon/dce6_afmt.c:177:6: warning: no previous prototype for > ‘dce6_afmt_dp_write_speaker_allocation’ [-Wmissing-prototypes] > 177 | void dce6_afmt_dp_write_speaker_allocation(struct drm_encoder *encoder, > | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > drivers/gpu/drm/radeon/dce6_afmt.c:202:6: warning: no previous prototype for > ‘dce6_afmt_write_sad_regs’ [-Wmissing-prototypes] > 202 | void dce6_afmt_write_sad_regs(struct drm_encoder *encoder, > | ^~~~~~~~~~~~~~~~~~~~~~~~ > drivers/gpu/drm/radeon/dce6_afmt.c:268:6: warning: no previous prototype for > ‘dce6_hdmi_audio_set_dto’ [-Wmissing-prototypes] > 268 | void dce6_hdmi_audio_set_dto(struct radeon_device *rdev, > | ^~~~~~~~~~~~~~~~~~~~~~~ > drivers/gpu/drm/radeon/dce6_afmt.c:287:6: warning: no previous prototype for > ‘dce6_dp_audio_set_dto’ [-Wmissing-prototypes] > 287 | void dce6_dp_audio_set_dto(struct radeo_device *rdev, > | ^~~~~~~~~~~~~~~~~~~~~ > > Cc: Alex Deucher <alexander.deuc...@amd.com> > Cc: "Christian König" <christian.koe...@amd.com> > Cc: David Airlie <airl...@linux.ie> > Cc: Daniel Vetter <dan...@ffwll.ch> > Cc: Slava Grigorev <slava.grigo...@amd.com> > Cc: amd-...@lists.freedesktop.org > Cc: dri-de...@lists.freedesktop.org > Signed-off-by: Lee Jones <lee.jo...@linaro.org>
Applied. Thanks! Alex > --- > drivers/gpu/drm/radeon/dce6_afmt.c | 1 + > drivers/gpu/drm/radeon/dce6_afmt.h | 52 +++++++++++++++++++++++++++ > drivers/gpu/drm/radeon/radeon_audio.c | 17 +-------- > 3 files changed, 54 insertions(+), 16 deletions(-) > create mode 100644 drivers/gpu/drm/radeon/dce6_afmt.h > > diff --git a/drivers/gpu/drm/radeon/dce6_afmt.c > b/drivers/gpu/drm/radeon/dce6_afmt.c > index eec5d7a627388..4a1d5447eac17 100644 > --- a/drivers/gpu/drm/radeon/dce6_afmt.c > +++ b/drivers/gpu/drm/radeon/dce6_afmt.c > @@ -22,6 +22,7 @@ > */ > #include <linux/hdmi.h> > > +#include "dce6_afmt.h" > #include "radeon.h" > #include "radeon_audio.h" > #include "sid.h" > diff --git a/drivers/gpu/drm/radeon/dce6_afmt.h > b/drivers/gpu/drm/radeon/dce6_afmt.h > new file mode 100644 > index 0000000000000..176a538664a27 > --- /dev/null > +++ b/drivers/gpu/drm/radeon/dce6_afmt.h > @@ -0,0 +1,52 @@ > +/* dce6_afmt.h -- Private header for radeon driver -*- linux-c -*- > + * > + * Copyright 2013 Advanced Micro Devices, Inc. > + * > + * Permission is hereby granted, free of charge, to any person obtaining a > + * copy of this software and associated documentation files (the "Software"), > + * to deal in the Software without restriction, including without limitation > + * the rights to use, copy, modify, merge, publish, distribute, sublicense, > + * and/or sell copies of the Software, and to permit persons to whom the > + * Software is furnished to do so, subject to the following conditions: > + * > + * The above copyright notice and this permission notice shall be included in > + * all copies or substantial portions of the Software. > + * > + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR > + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, > + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL > + * THE COPYRIGHT HOLDER(S) OR AUTHOR(S) BE LIABLE FOR ANY CLAIM, DAMAGES OR > + * OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, > + * ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR > + * OTHER DEALINGS IN THE SOFTWARE. > + * > + */ > + > +#ifndef __DCE6_AFMT_H__ > +#define __DCE6_AFMT_H__ > + > +struct cea_sad; > +struct drm_connector; > +struct drm_display_mode; > +struct drm_encoder; > +struct radeon_crtc; > +struct radeon_device; > + > +u32 dce6_endpoint_rreg(struct radeon_device *rdev, u32 offset, u32 reg); > +void dce6_endpoint_wreg(struct radeon_device *rdev, u32 offset, u32 reg, u32 > v); > +void dce6_afmt_write_sad_regs(struct drm_encoder *encoder, > + struct cea_sad *sads, int sad_count); > +void dce6_afmt_hdmi_write_speaker_allocation(struct drm_encoder *encoder, > + u8 *sadb, int sad_count); > +void dce6_afmt_dp_write_speaker_allocation(struct drm_encoder *encoder, > + u8 *sadb, int sad_count); > +void dce6_afmt_write_latency_fields(struct drm_encoder *encoder, > + struct drm_connector *connector, > + struct drm_display_mode *mode); > +void dce6_afmt_select_pin(struct drm_encoder *encoder); > +void dce6_hdmi_audio_set_dto(struct radeon_device *rdev, > + struct radeon_crtc *crtc, unsigned int clock); > +void dce6_dp_audio_set_dto(struct radeon_device *rdev, > + struct radeon_crtc *crtc, unsigned int clock); > + > +#endif /* __DCE6_AFMT_H__ */ > diff --git a/drivers/gpu/drm/radeon/radeon_audio.c > b/drivers/gpu/drm/radeon/radeon_audio.c > index 3cc53db04d645..785c23cb30589 100644 > --- a/drivers/gpu/drm/radeon/radeon_audio.c > +++ b/drivers/gpu/drm/radeon/radeon_audio.c > @@ -25,6 +25,7 @@ > #include <linux/gcd.h> > > #include <drm/drm_crtc.h> > +#include "dce6_afmt.h" > #include "evergreen_hdmi.h" > #include "radeon.h" > #include "atom.h" > @@ -34,26 +35,10 @@ void r600_audio_enable(struct radeon_device *rdev, struct > r600_audio_pin *pin, > u8 enable_mask); > void dce6_audio_enable(struct radeon_device *rdev, struct r600_audio_pin > *pin, > u8 enable_mask); > -u32 dce6_endpoint_rreg(struct radeon_device *rdev, u32 offset, u32 reg); > -void dce6_endpoint_wreg(struct radeon_device *rdev, > - u32 offset, u32 reg, u32 v); > -void dce6_afmt_write_sad_regs(struct drm_encoder *encoder, > - struct cea_sad *sads, int sad_count); > -void dce6_afmt_hdmi_write_speaker_allocation(struct drm_encoder *encoder, > - u8 *sadb, int sad_count); > -void dce6_afmt_dp_write_speaker_allocation(struct drm_encoder *encoder, > - u8 *sadb, int sad_count); > -void dce6_afmt_write_latency_fields(struct drm_encoder *encoder, > - struct drm_connector *connector, struct drm_display_mode > *mode); > struct r600_audio_pin* r600_audio_get_pin(struct radeon_device *rdev); > struct r600_audio_pin* dce6_audio_get_pin(struct radeon_device *rdev); > -void dce6_afmt_select_pin(struct drm_encoder *encoder); > void r600_hdmi_audio_set_dto(struct radeon_device *rdev, > struct radeon_crtc *crtc, unsigned int clock); > -void dce6_hdmi_audio_set_dto(struct radeon_device *rdev, > - struct radeon_crtc *crtc, unsigned int clock); > -void dce6_dp_audio_set_dto(struct radeon_device *rdev, > - struct radeon_crtc *crtc, unsigned int clock); > void r600_set_avi_packet(struct radeon_device *rdev, u32 offset, > unsigned char *buffer, size_t size); > void r600_hdmi_update_acr(struct drm_encoder *encoder, long offset, > -- > 2.25.1 > > _______________________________________________ > dri-devel mailing list > dri-de...@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel