Skip to content

Commit a9661f2

Browse files
committed
drm/vc4: kms: Rename NUM_CHANNELS
The NUM_CHANNELS define has a pretty generic name and was right before the function using it. Let's move to something that makes the hardware-specific nature more obvious, and to a more appropriate place. Signed-off-by: Maxime Ripard <maxime@cerno.tech> Tested-by: Hoegeun Kwon <hoegeun.kwon@samsung.com> Reviewed-by: Hoegeun Kwon <hoegeun.kwon@samsung.com> Reviewed-by: Thomas Zimmermann <tzimmermann@suse.de> Link: https://patchwork.freedesktop.org/patch/msgid/20201105135656.383350-4-maxime@cerno.tech
1 parent 213189d commit a9661f2

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

drivers/gpu/drm/vc4/vc4_kms.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,8 @@
2424
#include "vc4_drv.h"
2525
#include "vc4_regs.h"
2626

27+
#define HVS_NUM_CHANNELS 3
28+
2729
struct vc4_ctm_state {
2830
struct drm_private_state base;
2931
struct drm_color_ctm *ctm;
@@ -660,12 +662,10 @@ static int vc4_load_tracker_obj_init(struct vc4_dev *vc4)
660662
return drmm_add_action_or_reset(&vc4->base, vc4_load_tracker_obj_fini, NULL);
661663
}
662664

663-
#define NUM_CHANNELS 3
664-
665665
static int
666666
vc4_atomic_check(struct drm_device *dev, struct drm_atomic_state *state)
667667
{
668-
unsigned long unassigned_channels = GENMASK(NUM_CHANNELS - 1, 0);
668+
unsigned long unassigned_channels = GENMASK(HVS_NUM_CHANNELS - 1, 0);
669669
struct drm_crtc_state *old_crtc_state, *new_crtc_state;
670670
struct drm_crtc *crtc;
671671
int i, ret;

0 commit comments

Comments
 (0)