| Message ID | 20250209-fd-hdmi-hpd-v4-3-6224568ed87f@linaro.org |
|---|---|
| State | New |
| Headers | show |
| Series | [v4,01/16] dt-bindings: display/msm/hdmi: drop obsolete GPIOs from schema | expand |
On 2/8/2025 9:04 PM, Dmitry Baryshkov wrote: > As a preparation to the next patches convert 'static const char *' > arrays to 'static const char * const', as required by the checkpatch.pl > > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> Reviewed-by: Jessica Zhang <quic_jesszhan@quicinc.com> > --- > drivers/gpu/drm/msm/hdmi/hdmi.c | 10 +++++----- > drivers/gpu/drm/msm/hdmi/hdmi.h | 8 ++++---- > 2 files changed, 9 insertions(+), 9 deletions(-) > > diff --git a/drivers/gpu/drm/msm/hdmi/hdmi.c b/drivers/gpu/drm/msm/hdmi/hdmi.c > index 248541ff449204c72cd444458dadb9ae4a0a53d1..9e9900882687fa2ae4a734d5cf10b5bae5af2f87 100644 > --- a/drivers/gpu/drm/msm/hdmi/hdmi.c > +++ b/drivers/gpu/drm/msm/hdmi/hdmi.c > @@ -224,17 +224,17 @@ int msm_hdmi_modeset_init(struct hdmi *hdmi, > .item ## _names = item ##_names_ ## entry, \ > .item ## _cnt = ARRAY_SIZE(item ## _names_ ## entry) > > -static const char *hpd_reg_names_8960[] = {"core-vdda"}; > -static const char *hpd_clk_names_8960[] = {"core", "master_iface", "slave_iface"}; > +static const char * const hpd_reg_names_8960[] = {"core-vdda"}; > +static const char * const hpd_clk_names_8960[] = {"core", "master_iface", "slave_iface"}; > > static const struct hdmi_platform_config hdmi_tx_8960_config = { > HDMI_CFG(hpd_reg, 8960), > HDMI_CFG(hpd_clk, 8960), > }; > > -static const char *pwr_reg_names_8x74[] = {"core-vdda", "core-vcc"}; > -static const char *pwr_clk_names_8x74[] = {"extp", "alt_iface"}; > -static const char *hpd_clk_names_8x74[] = {"iface", "core", "mdp_core"}; > +static const char * const pwr_reg_names_8x74[] = {"core-vdda", "core-vcc"}; > +static const char * const pwr_clk_names_8x74[] = {"extp", "alt_iface"}; > +static const char * const hpd_clk_names_8x74[] = {"iface", "core", "mdp_core"}; > static unsigned long hpd_clk_freq_8x74[] = {0, 19200000, 0}; > > static const struct hdmi_platform_config hdmi_tx_8974_config = { > diff --git a/drivers/gpu/drm/msm/hdmi/hdmi.h b/drivers/gpu/drm/msm/hdmi/hdmi.h > index a5f481c39277631b7a19d294b086d6208be26511..381f957b34305494cb4da0b7dccb73b6ac3a1377 100644 > --- a/drivers/gpu/drm/msm/hdmi/hdmi.h > +++ b/drivers/gpu/drm/msm/hdmi/hdmi.h > @@ -84,20 +84,20 @@ struct hdmi { > /* platform config data (ie. from DT, or pdata) */ > struct hdmi_platform_config { > /* regulators that need to be on for hpd: */ > - const char **hpd_reg_names; > + const char * const *hpd_reg_names; > int hpd_reg_cnt; > > /* regulators that need to be on for screen pwr: */ > - const char **pwr_reg_names; > + const char * const *pwr_reg_names; > int pwr_reg_cnt; > > /* clks that need to be on for hpd: */ > - const char **hpd_clk_names; > + const char * const *hpd_clk_names; > const long unsigned *hpd_freq; > int hpd_clk_cnt; > > /* clks that need to be on for screen pwr (ie pixel clk): */ > - const char **pwr_clk_names; > + const char * const *pwr_clk_names; > int pwr_clk_cnt; > }; > >
diff --git a/drivers/gpu/drm/msm/hdmi/hdmi.c b/drivers/gpu/drm/msm/hdmi/hdmi.c index 248541ff449204c72cd444458dadb9ae4a0a53d1..9e9900882687fa2ae4a734d5cf10b5bae5af2f87 100644 --- a/drivers/gpu/drm/msm/hdmi/hdmi.c +++ b/drivers/gpu/drm/msm/hdmi/hdmi.c @@ -224,17 +224,17 @@ int msm_hdmi_modeset_init(struct hdmi *hdmi, .item ## _names = item ##_names_ ## entry, \ .item ## _cnt = ARRAY_SIZE(item ## _names_ ## entry) -static const char *hpd_reg_names_8960[] = {"core-vdda"}; -static const char *hpd_clk_names_8960[] = {"core", "master_iface", "slave_iface"}; +static const char * const hpd_reg_names_8960[] = {"core-vdda"}; +static const char * const hpd_clk_names_8960[] = {"core", "master_iface", "slave_iface"}; static const struct hdmi_platform_config hdmi_tx_8960_config = { HDMI_CFG(hpd_reg, 8960), HDMI_CFG(hpd_clk, 8960), }; -static const char *pwr_reg_names_8x74[] = {"core-vdda", "core-vcc"}; -static const char *pwr_clk_names_8x74[] = {"extp", "alt_iface"}; -static const char *hpd_clk_names_8x74[] = {"iface", "core", "mdp_core"}; +static const char * const pwr_reg_names_8x74[] = {"core-vdda", "core-vcc"}; +static const char * const pwr_clk_names_8x74[] = {"extp", "alt_iface"}; +static const char * const hpd_clk_names_8x74[] = {"iface", "core", "mdp_core"}; static unsigned long hpd_clk_freq_8x74[] = {0, 19200000, 0}; static const struct hdmi_platform_config hdmi_tx_8974_config = { diff --git a/drivers/gpu/drm/msm/hdmi/hdmi.h b/drivers/gpu/drm/msm/hdmi/hdmi.h index a5f481c39277631b7a19d294b086d6208be26511..381f957b34305494cb4da0b7dccb73b6ac3a1377 100644 --- a/drivers/gpu/drm/msm/hdmi/hdmi.h +++ b/drivers/gpu/drm/msm/hdmi/hdmi.h @@ -84,20 +84,20 @@ struct hdmi { /* platform config data (ie. from DT, or pdata) */ struct hdmi_platform_config { /* regulators that need to be on for hpd: */ - const char **hpd_reg_names; + const char * const *hpd_reg_names; int hpd_reg_cnt; /* regulators that need to be on for screen pwr: */ - const char **pwr_reg_names; + const char * const *pwr_reg_names; int pwr_reg_cnt; /* clks that need to be on for hpd: */ - const char **hpd_clk_names; + const char * const *hpd_clk_names; const long unsigned *hpd_freq; int hpd_clk_cnt; /* clks that need to be on for screen pwr (ie pixel clk): */ - const char **pwr_clk_names; + const char * const *pwr_clk_names; int pwr_clk_cnt; };
As a preparation to the next patches convert 'static const char *' arrays to 'static const char * const', as required by the checkpatch.pl Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> --- drivers/gpu/drm/msm/hdmi/hdmi.c | 10 +++++----- drivers/gpu/drm/msm/hdmi/hdmi.h | 8 ++++---- 2 files changed, 9 insertions(+), 9 deletions(-)