[PATCH libdrm] radeon: add fallthrough annotation

[Eric Engestrom]

On Monday, 2017-07-31 03:05:47 +0000, Michel Dänzer wrote:
> On 31/07/17 05:34 AM, Eric Engestrom wrote:
> > GCC 7 started warning when a switch case has neither a `break` nor
> > a "fallthrough" comment.
> > Let's be explicit that we meant to fall through here.
> > 
> > Signed-off-by: Eric Engestrom <eric at engestrom.ch>
> > ---
> >  radeon/radeon_surface.c | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/radeon/radeon_surface.c b/radeon/radeon_surface.c
> > index 965be24c..04df77d6 100644
> > --- a/radeon/radeon_surface.c
> > +++ b/radeon/radeon_surface.c
> > @@ -2503,6 +2503,7 @@ static int radeon_surface_sanity(struct radeon_surface_manager *surf_man,
> >          if (surf->npix_y > 1) {
> >              return -EINVAL;
> >          }
> > +        /* fallthrough */
> >      case RADEON_SURF_TYPE_2D:
> >          if (surf->npix_z > 1) {
> >              return -EINVAL;
> > 
> 
> Reviewed-by: Michel Dänzer <michel.daenzer at amd.com>

Thanks!

> 
> P.S. Patches touching the radeon (and amdgpu) directory should (also) be
> sent to the amd-gfx mailing list for review.

Sure, making a note.

Do we want a `scripts/get_(maintainer|reviewer).pl` for libdrm?