[igt-dev] [PATCH i-g-t] lib/debugfs: Nuke enum intel_pipe_crc_source

Chris Wilson chris at chris-wilson.co.uk
Fri Feb 15 19:54:24 UTC 2019


Quoting Ville Syrjala via igt-dev (2019-02-15 13:26:38)
>  static igt_pipe_crc_t *
> -pipe_crc_new(int fd, enum pipe pipe, enum intel_pipe_crc_source source, int flags)
> +pipe_crc_new(int fd, enum pipe pipe, const char *source, int flags)
>  {
>         igt_pipe_crc_t *pipe_crc;
>         char buf[128];
> @@ -679,7 +661,7 @@ pipe_crc_new(int fd, enum pipe pipe, enum intel_pipe_crc_source source, int flag
>         pipe_crc->fd = fd;
>         pipe_crc->dir = debugfs;
>         pipe_crc->pipe = pipe;
> -       pipe_crc->source = source;
> +       pipe_crc->source = strdup(source);

xstrdup()
igt_assert(pipe_crc->source);

Probably merits an igt_assert(source) on entry as well.

> -enum intel_pipe_crc_source {
> -        INTEL_PIPE_CRC_SOURCE_NONE,
> -        INTEL_PIPE_CRC_SOURCE_PLANE1,
> -        INTEL_PIPE_CRC_SOURCE_PLANE2,
> -        INTEL_PIPE_CRC_SOURCE_PF,
> -        INTEL_PIPE_CRC_SOURCE_PIPE,
> -        INTEL_PIPE_CRC_SOURCE_TV,
> -        INTEL_PIPE_CRC_SOURCE_DP_B,
> -        INTEL_PIPE_CRC_SOURCE_DP_C,
> -        INTEL_PIPE_CRC_SOURCE_DP_D,
> -        INTEL_PIPE_CRC_SOURCE_AUTO,
> -        INTEL_PIPE_CRC_SOURCE_MAX,
> -};
> +#define INTEL_PIPE_CRC_SOURCE_AUTO "auto"

Ah that's the magic behind the patch being so tiny.

Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk>
-Chris


More information about the igt-dev mailing list