Hi Alex,

There are callers in the same file "info_packet.c" 

-----Original Message-----
From: Alex Deucher <alexdeuc...@gmail.com> 
Sent: August 27, 2019 10:20
To: Wentland, Harry <harry.wentl...@amd.com>
Cc: YueHaibing <yuehaib...@huawei.com>; Wentland, Harry 
<harry.wentl...@amd.com>; Li, Sun peng (Leo) <sunpeng...@amd.com>; Deucher, 
Alexander <alexander.deuc...@amd.com>; Koenig, Christian 
<christian.koe...@amd.com>; Zhou, David(ChunMing) <david1.z...@amd.com>; 
airl...@linux.ie; dan...@ffwll.ch; Lakha, Bhawanpreet 
<bhawanpreet.la...@amd.com>; Koo, Anthony <anthony....@amd.com>; Othman, Ahmad 
<ahmad.oth...@amd.com>; Bernstein, Eric <eric.bernst...@amd.com>; Cyr, Aric 
<aric....@amd.com>; alvin.l...@amd.com; Tatla, Harmanprit 
<harmanprit.ta...@amd.com>; dri-de...@lists.freedesktop.org; 
amd-gfx@lists.freedesktop.org; linux-ker...@vger.kernel.org
Subject: Re: [PATCH 2/3] drm/amd/display: remove unused function 
setFieldWithMask

On Tue, Aug 27, 2019 at 10:01 AM Harry Wentland <hwent...@amd.com> wrote:
>
> On 2019-08-27 3:09 a.m., YueHaibing wrote:
> > After commit a9f54ce3c603 ("drm/amd/display: Refactoring VTEM"), 
> > there is no caller in tree.
> >
> > Reported-by: Hulk Robot <hul...@huawei.com> Signed-off-by: 
> > YueHaibing <yuehaib...@huawei.com>
>
> Reviewed-by: Harry Wentland <harry.wentl...@amd.com>
>

Applied.  Thanks!

Alex

> Harry
>
> > ---
> >  .../drm/amd/display/modules/info_packet/info_packet.c | 19 
> > -------------------
> >  1 file changed, 19 deletions(-)
> >
> > diff --git 
> > a/drivers/gpu/drm/amd/display/modules/info_packet/info_packet.c 
> > b/drivers/gpu/drm/amd/display/modules/info_packet/info_packet.c
> > index 5f4b98d..d885d64 100644
> > --- a/drivers/gpu/drm/amd/display/modules/info_packet/info_packet.c
> > +++ b/drivers/gpu/drm/amd/display/modules/info_packet/info_packet.c
> > @@ -114,25 +114,6 @@ enum ColorimetryYCCDP {
> >       ColorimetryYCC_DP_ITU2020YCbCr  = 7,  };
> >
> > -void setFieldWithMask(unsigned char *dest, unsigned int mask, 
> > unsigned int value) -{
> > -     unsigned int shift = 0;
> > -
> > -     if (!mask || !dest)
> > -             return;
> > -
> > -     while (!((mask >> shift) & 1))
> > -             shift++;
> > -
> > -     //reset
> > -     *dest = *dest & ~mask;
> > -     //set
> > -     //dont let value span past mask
> > -     value = value & (mask >> shift);
> > -     //insert value
> > -     *dest = *dest | (value << shift);
> > -}
> > -
> >  void mod_build_vsc_infopacket(const struct dc_stream_state *stream,
> >               struct dc_info_packet *info_packet)  {
> >
> _______________________________________________
> amd-gfx mailing list
> amd-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to