| Message ID | 20240315143621.v2.4.Ia5dd755f81d7cc9a4393c43f77b9be4ed2278ee3@changeid (mailing list archive) |
|---|---|
| State | Not Applicable |
| Headers | show |
| Series | drm/msm/dp: Improve DP AUX transfer vs. HPD interactions | expand |
Quoting Douglas Anderson (2024-03-15 14:36:32) > This is a no-op change to just fix a typo in the name of a static function. > > Signed-off-by: Douglas Anderson <dianders@chromium.org> > --- > > Changes in v2: > - ("Fix typo in static function (ststus => status)") new for v2. This was sent at https://lore.kernel.org/r/20240306193515.455388-1-quic_abhinavk@quicinc.com
Hi, On Mon, Mar 18, 2024 at 12:26 PM Stephen Boyd <swboyd@chromium.org> wrote: > > Quoting Douglas Anderson (2024-03-15 14:36:32) > > This is a no-op change to just fix a typo in the name of a static function. > > > > Signed-off-by: Douglas Anderson <dianders@chromium.org> > > --- > > > > Changes in v2: > > - ("Fix typo in static function (ststus => status)") new for v2. > > This was sent at > https://lore.kernel.org/r/20240306193515.455388-1-quic_abhinavk@quicinc.com Whoops! I guess we both noticed it at about the same time. My patch should be dropped then. The rest of my series (patches #1 - #3) are still relevant. I won't repost them since they can be applied just fine even if this patch is dropped. -Doug
On 3/18/2024 12:37 PM, Doug Anderson wrote: > Hi, > > On Mon, Mar 18, 2024 at 12:26 PM Stephen Boyd <swboyd@chromium.org> wrote: >> >> Quoting Douglas Anderson (2024-03-15 14:36:32) >>> This is a no-op change to just fix a typo in the name of a static function. >>> >>> Signed-off-by: Douglas Anderson <dianders@chromium.org> >>> --- >>> >>> Changes in v2: >>> - ("Fix typo in static function (ststus => status)") new for v2. >> >> This was sent at >> https://lore.kernel.org/r/20240306193515.455388-1-quic_abhinavk@quicinc.com > > Whoops! I guess we both noticed it at about the same time. My patch > should be dropped then. The rest of my series (patches #1 - #3) are > still relevant. I won't repost them since they can be applied just > fine even if this patch is dropped. > > -Doug Thanks for the patch. I will pick up https://lore.kernel.org/r/20240306193515.455388-1-quic_abhinavk@quicinc.com for -fixes so you can drop this one.
diff --git a/drivers/gpu/drm/msm/dp/dp_display.c b/drivers/gpu/drm/msm/dp/dp_display.c index b0f3e2ef5a6b..78e702f66ed2 100644 --- a/drivers/gpu/drm/msm/dp/dp_display.c +++ b/drivers/gpu/drm/msm/dp/dp_display.c @@ -494,7 +494,7 @@ static void dp_display_handle_video_request(struct dp_display_private *dp) } } -static int dp_display_handle_port_ststus_changed(struct dp_display_private *dp) +static int dp_display_handle_port_status_changed(struct dp_display_private *dp) { int rc = 0; @@ -551,7 +551,7 @@ static int dp_display_usbpd_attention_cb(struct device *dev) drm_dbg_dp(dp->drm_dev, "hpd_state=%d sink_request=%d\n", dp->hpd_state, sink_request); if (sink_request & DS_PORT_STATUS_CHANGED) - rc = dp_display_handle_port_ststus_changed(dp); + rc = dp_display_handle_port_status_changed(dp); else rc = dp_display_handle_irq_hpd(dp); }
This is a no-op change to just fix a typo in the name of a static function. Signed-off-by: Douglas Anderson <dianders@chromium.org> --- Changes in v2: - ("Fix typo in static function (ststus => status)") new for v2. drivers/gpu/drm/msm/dp/dp_display.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)