drm/i915/display: Clean up zero initializers
authorVille Syrjälä <ville.syrjala@linux.intel.com>
Thu, 12 Oct 2023 12:24:37 +0000 (15:24 +0300)
committerVille Syrjälä <ville.syrjala@linux.intel.com>
Mon, 16 Oct 2023 16:09:22 +0000 (19:09 +0300)
Just use a simple {} to zero initialize arrays/structs instead
of the hodgepodge of stuff we are using currently.

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20231012122442.15718-2-ville.syrjala@linux.intel.com
Reviewed-by: Jani Nikula <jani.nikula@intel.com>
drivers/gpu/drm/i915/display/intel_acpi.c
drivers/gpu/drm/i915/display/intel_cx0_phy.c
drivers/gpu/drm/i915/display/intel_dp.c
drivers/gpu/drm/i915/display/intel_dp_aux_backlight.c
drivers/gpu/drm/i915/display/intel_dp_mst.c
drivers/gpu/drm/i915/display/intel_gmbus.c
drivers/gpu/drm/i915/display/intel_plane_initial.c
drivers/gpu/drm/i915/display/intel_sdvo.c
drivers/gpu/drm/i915/display/intel_snps_phy.c
drivers/gpu/drm/i915/display/intel_wm.c

index 9df78e7caa2bd7886c4df3376802179f6be91532..0aa3999374e255e9734f67cecc059ce1d37889c8 100644 (file)
@@ -153,7 +153,7 @@ static acpi_handle intel_dsm_pci_probe(struct pci_dev *pdev)
 static bool intel_dsm_detect(void)
 {
        acpi_handle dhandle = NULL;
-       char acpi_method_name[255] = { 0 };
+       char acpi_method_name[255] = {};
        struct acpi_buffer buffer = {sizeof(acpi_method_name), acpi_method_name};
        struct pci_dev *pdev = NULL;
        int vga_count = 0;
index 6e6a1818071eab1311f1b92178b24a899c5c0f4f..d414f6b7f993c220aaf78aa2efc718a09724d8f0 100644 (file)
@@ -3009,7 +3009,7 @@ void intel_c10pll_state_verify(struct intel_atomic_state *state,
        struct drm_i915_private *i915 = to_i915(state->base.dev);
        const struct intel_crtc_state *new_crtc_state =
                intel_atomic_get_new_crtc_state(state, crtc);
-       struct intel_c10pll_state mpllb_hw_state = { 0 };
+       struct intel_c10pll_state mpllb_hw_state = {};
        const struct intel_c10pll_state *mpllb_sw_state = &new_crtc_state->cx0pll_state.c10;
        struct intel_encoder *encoder;
        enum phy phy;
index d324204c1ce6f1ff87e1815e8783c1cb0db93d25..6f46b6692e43d4a27153646d1166647147b64b4c 100644 (file)
@@ -1583,7 +1583,7 @@ int intel_dp_dsc_compute_max_bpp(const struct intel_connector *connector,
 {
        struct drm_i915_private *i915 = to_i915(connector->base.dev);
        int i, num_bpc;
-       u8 dsc_bpc[3] = {0};
+       u8 dsc_bpc[3] = {};
        u8 dsc_max_bpc;
 
        dsc_max_bpc = intel_dp_dsc_max_src_input_bpc(i915);
@@ -2006,7 +2006,7 @@ static int intel_dp_dsc_compute_pipe_bpp(struct intel_dp *intel_dp,
        u8 max_req_bpc = conn_state->max_requested_bpc;
        u8 dsc_max_bpc, dsc_max_bpp;
        u8 dsc_min_bpc, dsc_min_bpp;
-       u8 dsc_bpc[3] = {0};
+       u8 dsc_bpc[3] = {};
        int forced_bpp, pipe_bpp;
        int num_bpc, i, ret;
 
@@ -2940,7 +2940,7 @@ intel_edp_init_source_oui(struct intel_dp *intel_dp, bool careful)
 {
        struct drm_i915_private *i915 = dp_to_i915(intel_dp);
        u8 oui[] = { 0x00, 0xaa, 0x01 };
-       u8 buf[3] = { 0 };
+       u8 buf[3] = {};
 
        /*
         * During driver init, we want to be careful and avoid changing the source OUI if it's
index 95cc5251843e999d70c9a1728c1a5035fe168255..26ea7e9f1b899914e7349aca03f81f69f94f066d 100644 (file)
@@ -169,7 +169,7 @@ intel_dp_aux_hdr_get_backlight(struct intel_connector *connector, enum pipe pipe
        struct intel_panel *panel = &connector->panel;
        struct intel_dp *intel_dp = enc_to_intel_dp(connector->encoder);
        u8 tmp;
-       u8 buf[2] = { 0 };
+       u8 buf[2] = {};
 
        if (drm_dp_dpcd_readb(&intel_dp->aux, INTEL_EDP_HDR_GETSET_CTRL_PARAMS, &tmp) != 1) {
                drm_err(&i915->drm, "[CONNECTOR:%d:%s] Failed to read current backlight mode from DPCD\n",
@@ -204,7 +204,7 @@ intel_dp_aux_hdr_set_aux_backlight(const struct drm_connector_state *conn_state,
        struct intel_connector *connector = to_intel_connector(conn_state->connector);
        struct drm_device *dev = connector->base.dev;
        struct intel_dp *intel_dp = enc_to_intel_dp(connector->encoder);
-       u8 buf[4] = { 0 };
+       u8 buf[4] = {};
 
        buf[0] = level & 0xFF;
        buf[1] = (level & 0xFF00) >> 8;
index 47f1efb50ea26b68400a961b56fa3aad98050e6e..7b4628f4f124040eae309b7367d2f26c59933e53 100644 (file)
@@ -192,7 +192,7 @@ static int intel_dp_dsc_mst_compute_link_config(struct intel_encoder *encoder,
                &crtc_state->hw.adjusted_mode;
        int slots = -EINVAL;
        int i, num_bpc;
-       u8 dsc_bpc[3] = {0};
+       u8 dsc_bpc[3] = {};
        int min_bpp, max_bpp, sink_min_bpp, sink_max_bpp;
        u8 dsc_max_bpc;
        bool need_timeslot_recalc = false;
index 801fabbccf7ebe208b586312d9d84c04d7814af5..40d7b6f3f4891c8cbe6e6590f90243ca77291581 100644 (file)
@@ -781,7 +781,7 @@ int intel_gmbus_output_aksv(struct i2c_adapter *adapter)
        struct intel_gmbus *bus = to_intel_gmbus(adapter);
        struct drm_i915_private *i915 = bus->i915;
        u8 cmd = DRM_HDCP_DDC_AKSV;
-       u8 buf[DRM_HDCP_KSV_LEN] = { 0 };
+       u8 buf[DRM_HDCP_KSV_LEN] = {};
        struct i2c_msg msgs[] = {
                {
                        .addr = DRM_HDCP_DDC_ADDR,
index 451a642e106e0952c6c973b453e0341ad5b9816f..a55c09cbd0e484ee7af1316a52d9d0eacb887514 100644 (file)
@@ -166,7 +166,7 @@ intel_alloc_initial_plane_obj(struct intel_crtc *crtc,
 {
        struct drm_device *dev = crtc->base.dev;
        struct drm_i915_private *dev_priv = to_i915(dev);
-       struct drm_mode_fb_cmd2 mode_cmd = { 0 };
+       struct drm_mode_fb_cmd2 mode_cmd = {};
        struct drm_framebuffer *fb = &plane_config->fb->base;
        struct i915_vma *vma;
 
index 35137e9785914ce765ee18f08b2330df04e4ebb0..a636f42ceae555bd2c34ec8746128273f3e2c00d 100644 (file)
@@ -649,7 +649,7 @@ intel_sdvo_get_value(struct intel_sdvo *intel_sdvo, u8 cmd, void *value, int len
 
 static bool intel_sdvo_set_target_input(struct intel_sdvo *intel_sdvo)
 {
-       struct intel_sdvo_set_target_input_args targets = {0};
+       struct intel_sdvo_set_target_input_args targets = {};
        return intel_sdvo_set_value(intel_sdvo,
                                    SDVO_CMD_SET_TARGET_INPUT,
                                    &targets, sizeof(targets));
index c0285365efae0bf98d289258984191f89fb76a90..ce5a73a4cc8934a4d70dae69c85da3533a0198e7 100644 (file)
@@ -1998,7 +1998,7 @@ void intel_mpllb_state_verify(struct intel_atomic_state *state,
        struct drm_i915_private *i915 = to_i915(state->base.dev);
        const struct intel_crtc_state *new_crtc_state =
                intel_atomic_get_new_crtc_state(state, crtc);
-       struct intel_mpllb_state mpllb_hw_state = { 0 };
+       struct intel_mpllb_state mpllb_hw_state = {};
        const struct intel_mpllb_state *mpllb_sw_state = &new_crtc_state->mpllb_state;
        struct intel_encoder *encoder;
 
index b615449e70b47e3603ea6922ae9a278c0629a843..82c4933ad5073e267f95757aad6969516df41b2c 100644 (file)
@@ -290,7 +290,7 @@ static ssize_t wm_latency_write(struct file *file, const char __user *ubuf,
 {
        struct seq_file *m = file->private_data;
        struct drm_i915_private *dev_priv = m->private;
-       u16 new[8] = { 0 };
+       u16 new[8] = {};
        int level;
        int ret;
        char tmp[32];