drm/xe: Remove TODO from rtp infra
authorLucas De Marchi <lucas.demarchi@intel.com>
Mon, 23 Jan 2023 17:38:27 +0000 (09:38 -0800)
committerRodrigo Vivi <rodrigo.vivi@intel.com>
Tue, 19 Dec 2023 23:27:48 +0000 (18:27 -0500)
The function pointer is already present as match_func, inside
struct xe_rtp_rule and handled as so instead of inside rtp_regval as
originally thought out when this was written.

Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
drivers/gpu/drm/xe/xe_rtp_types.h

index b55b556a24958b15af3ffd71a11bd661d83a853f..630a2ec53fc6f0cc7ecdf49afff5f63a3a8cfcb8 100644 (file)
@@ -24,10 +24,6 @@ enum {
 struct xe_rtp_regval {
        /** @reg: Register */
        u32             reg;
-       /*
-        * TODO: maybe we need a union here with a func pointer for cases
-        * that are too specific to be generalized
-        */
        /** @clr_bits: bits to clear when updating register */
        u32             clr_bits;
        /** @set_bits: bits to set when updating register */