| Message ID | 20241213153543.v5.4.I6cff9d767b0f8ab6458d8940941e42c920902d49@changeid (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | Thunderbolt and DP altmode support for cros-ec-typec | expand |
On Fri, Dec 13, 2024 at 03:35:45PM -0800, Abhishek Pandit-Subedi wrote: > Print the error reason for typec_altmode_enter so users can understand > why displayport failed to enter. > > Signed-off-by: Abhishek Pandit-Subedi <abhishekpandit@chromium.org> Reviewed-by: Heikki Krogerus <heikki.krogerus@linux.intel.com> > --- > > Changes in v5: > - Put dev_err on single line. > - Slip clang-format a washington to look the other way. > > drivers/usb/typec/altmodes/displayport.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/usb/typec/altmodes/displayport.c b/drivers/usb/typec/altmodes/displayport.c > index 3245e03d59e6..ac84a6d64c2f 100644 > --- a/drivers/usb/typec/altmodes/displayport.c > +++ b/drivers/usb/typec/altmodes/displayport.c > @@ -252,7 +252,7 @@ static void dp_altmode_work(struct work_struct *work) > case DP_STATE_ENTER: > ret = typec_altmode_enter(dp->alt, NULL); > if (ret && ret != -EBUSY) > - dev_err(&dp->alt->dev, "failed to enter mode\n"); > + dev_err(&dp->alt->dev, "failed to enter mode: %d\n", ret); > break; > case DP_STATE_ENTER_PRIME: > ret = typec_cable_altmode_enter(dp->alt, TYPEC_PLUG_SOP_P, NULL); > -- > 2.47.1.613.gc27f4b7a9f-goog
On Fri, Dec 13, 2024 at 03:35:45PM -0800, Abhishek Pandit-Subedi wrote: > Print the error reason for typec_altmode_enter so users can understand > why displayport failed to enter. > > Signed-off-by: Abhishek Pandit-Subedi <abhishekpandit@chromium.org> Reviewed-by: Benson Leung <bleung@chromium.org> > --- > > Changes in v5: > - Put dev_err on single line. > - Slip clang-format a washington to look the other way. > > drivers/usb/typec/altmodes/displayport.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/usb/typec/altmodes/displayport.c b/drivers/usb/typec/altmodes/displayport.c > index 3245e03d59e6..ac84a6d64c2f 100644 > --- a/drivers/usb/typec/altmodes/displayport.c > +++ b/drivers/usb/typec/altmodes/displayport.c > @@ -252,7 +252,7 @@ static void dp_altmode_work(struct work_struct *work) > case DP_STATE_ENTER: > ret = typec_altmode_enter(dp->alt, NULL); > if (ret && ret != -EBUSY) > - dev_err(&dp->alt->dev, "failed to enter mode\n"); > + dev_err(&dp->alt->dev, "failed to enter mode: %d\n", ret); > break; > case DP_STATE_ENTER_PRIME: > ret = typec_cable_altmode_enter(dp->alt, TYPEC_PLUG_SOP_P, NULL); > -- > 2.47.1.613.gc27f4b7a9f-goog > >
diff --git a/drivers/usb/typec/altmodes/displayport.c b/drivers/usb/typec/altmodes/displayport.c index 3245e03d59e6..ac84a6d64c2f 100644 --- a/drivers/usb/typec/altmodes/displayport.c +++ b/drivers/usb/typec/altmodes/displayport.c @@ -252,7 +252,7 @@ static void dp_altmode_work(struct work_struct *work) case DP_STATE_ENTER: ret = typec_altmode_enter(dp->alt, NULL); if (ret && ret != -EBUSY) - dev_err(&dp->alt->dev, "failed to enter mode\n"); + dev_err(&dp->alt->dev, "failed to enter mode: %d\n", ret); break; case DP_STATE_ENTER_PRIME: ret = typec_cable_altmode_enter(dp->alt, TYPEC_PLUG_SOP_P, NULL);
Print the error reason for typec_altmode_enter so users can understand why displayport failed to enter. Signed-off-by: Abhishek Pandit-Subedi <abhishekpandit@chromium.org> --- Changes in v5: - Put dev_err on single line. - Slip clang-format a washington to look the other way. drivers/usb/typec/altmodes/displayport.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)