Re: [PATCH 06/15] drm/i915/tv: Use bools where appropriate

On Mon, Nov 12, 2018 at 06:59:50PM +0200, Ville Syrjala wrote:

From: Ville Syrjälä ville.syrjala@linux.intel.com

'component_only' is a bool. Initialize it like a bool.

Signed-off-by: Ville Syrjälä ville.syrjala@linux.intel.com

Reviewed-by: Imre Deak imre.deak@intel.com

---

drivers/gpu/drm/i915/intel_tv.c | 24 ++++++++++++------------ 1 file changed, 12 insertions(+), 12 deletions(-)

diff --git a/drivers/gpu/drm/i915/intel_tv.c b/drivers/gpu/drm/i915/intel_tv.c index 96257b29d07c..72de154b8627 100644 --- a/drivers/gpu/drm/i915/intel_tv.c +++ b/drivers/gpu/drm/i915/intel_tv.c @@ -380,7 +380,7 @@ static const struct tv_mode tv_modes[] = { .clock = 108000, .refresh = 59940, .oversample = 8,

• .component_only = 0,
  

• .component_only = false,
  

  /* 525 Lines, 60 Fields, 15.734KHz line, Sub-Carrier 3.580MHz */

  .hsync_end = 64, .hblank_end = 124,

@@ -423,7 +423,7 @@ static const struct tv_mode tv_modes[] = { .clock = 108000, .refresh = 59940, .oversample = 8,

• .component_only = 0,
  

• .component_only = false,
  

  /* 525 Lines, 60 Fields, 15.734KHz line, Sub-Carrier 4.43MHz */ .hsync_end = 64, .hblank_end = 124, .hblank_start = 836, .htotal = 857,

@@ -465,7 +465,7 @@ static const struct tv_mode tv_modes[] = { .clock = 108000, .refresh = 59940, .oversample = 8,

• .component_only = 0,
  

• .component_only = false,
  
  

  /* 525 Lines, 60 Fields, 15.734KHz line, Sub-Carrier 3.580MHz */ .hsync_end = 64, .hblank_end = 124,

@@ -508,7 +508,7 @@ static const struct tv_mode tv_modes[] = { .clock = 108000, .refresh = 59940, .oversample = 8,

• .component_only = 0,
  

• .component_only = false,
  
  

  /* 525 Lines, 60 Fields, 15.734KHz line, Sub-Carrier 3.580MHz */ .hsync_end = 64, .hblank_end = 124,

@@ -552,7 +552,7 @@ static const struct tv_mode tv_modes[] = { .clock = 108000, .refresh = 50000, .oversample = 8,

• .component_only = 0,
  

• .component_only = false,
  
  

  .hsync_end = 64, .hblank_end = 128, .hblank_start = 844, .htotal = 863,

@@ -597,7 +597,7 @@ static const struct tv_mode tv_modes[] = { .clock = 108000, .refresh = 50000, .oversample = 8,

• .component_only = 0,
  

• .component_only = false,
  
  

  .hsync_end = 64, .hblank_end = 142, .hblank_start = 844, .htotal = 863,

@@ -639,7 +639,7 @@ static const struct tv_mode tv_modes[] = { .clock = 108000, .refresh = 59940, .oversample = 4,

• .component_only = 1,
  

• .component_only = true,
  
  

  .hsync_end = 64, .hblank_end = 122, .hblank_start = 842, .htotal = 857,

@@ -663,7 +663,7 @@ static const struct tv_mode tv_modes[] = { .clock = 108000, .refresh = 50000, .oversample = 4,

• .component_only = 1,
  

• .component_only = true,
  
  

  .hsync_end = 64, .hblank_end = 139, .hblank_start = 859, .htotal = 863,

@@ -687,7 +687,7 @@ static const struct tv_mode tv_modes[] = { .clock = 148500, .refresh = 60000, .oversample = 2,

• .component_only = 1,
  

• .component_only = true,
  
  

  .hsync_end = 80, .hblank_end = 300, .hblank_start = 1580, .htotal = 1649,

@@ -711,7 +711,7 @@ static const struct tv_mode tv_modes[] = { .clock = 148500, .refresh = 50000, .oversample = 2,

• .component_only = 1,
  

• .component_only = true,
  
  

  .hsync_end = 80, .hblank_end = 300, .hblank_start = 1580, .htotal = 1979,

@@ -736,7 +736,7 @@ static const struct tv_mode tv_modes[] = { .clock = 148500, .refresh = 50000, .oversample = 2,

• .component_only = 1,
  

• .component_only = true,
  
  

  .hsync_end = 88, .hblank_end = 235, .hblank_start = 2155, .htotal = 2639,

@@ -762,7 +762,7 @@ static const struct tv_mode tv_modes[] = { .clock = 148500, .refresh = 60000, .oversample = 2,

• .component_only = 1,
  

• .component_only = true,
  
  

  .hsync_end = 88, .hblank_end = 235, .hblank_start = 2155, .htotal = 2199,

-- 2.18.1

---

dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel