mbox series

[V6,0/4] Add wakeup support over UART RX

Message ID 1600091917-7464-1-git-send-email-skakit@codeaurora.org (mailing list archive)
Headers show
Series Add wakeup support over UART RX | expand

Message

Satya Priya Sept. 14, 2020, 1:58 p.m. UTC
Changes in V2:
 - As per Matthias's comment added wakeup support for all the UARTs
   of SC7180.
 - Added sleep state in sc7180-idp.dts file.
 - Modify the if check in set_mctrl API in serial driver to avoid
   making RFR high during suspend.

Changes in V3:
 - As per Matthias's comments modify the idp dts pin config to keep
   only the required pin settings.
 - Remove the extra parentheses from serial driver patch.

Changes in V4:
 - As per Matthias's comments, change the commit text to mention why
   GPIO function needs to be selected in sleep.
 - Add separate patch for improvements made in pin conf settings.

Changes in V5:
 - Moved pinctrl and interrupt configuration to board specific files.
 - Added new patch for trogdor board specific changes.

Changes in V6:
 - As per Doug's comments, fixed nits in patch 2 and 3.
 - As per Bjorn's comments changed the commit text and rationale for
   rts, tx and rx in patch 2 and 3.

satya priya (4):
  arm64: dts: qcom: sc7180: Improve the uart3 pin config for sc7180-idp
  arm64: dts: qcom: sc7180: Add wakeup support for BT UART on sc7180-idp
  arm64: dts: qcom: sc7180-trogdor: Add wakeup support for BT UART
  tty: serial: qcom_geni_serial: Fix the UART wakeup issue

 arch/arm64/boot/dts/qcom/sc7180-idp.dts      | 65 +++++++++++++++++++++++++---
 arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi | 54 +++++++++++++++++++++++
 drivers/tty/serial/qcom_geni_serial.c        |  2 +-
 3 files changed, 114 insertions(+), 7 deletions(-)

Comments

Bjorn Andersson Sept. 15, 2020, 12:16 a.m. UTC | #1
On Mon 14 Sep 13:58 UTC 2020, satya priya wrote:

> Changes in V2:
>  - As per Matthias's comment added wakeup support for all the UARTs
>    of SC7180.
>  - Added sleep state in sc7180-idp.dts file.
>  - Modify the if check in set_mctrl API in serial driver to avoid
>    making RFR high during suspend.
> 
> Changes in V3:
>  - As per Matthias's comments modify the idp dts pin config to keep
>    only the required pin settings.
>  - Remove the extra parentheses from serial driver patch.
> 
> Changes in V4:
>  - As per Matthias's comments, change the commit text to mention why
>    GPIO function needs to be selected in sleep.
>  - Add separate patch for improvements made in pin conf settings.
> 
> Changes in V5:
>  - Moved pinctrl and interrupt configuration to board specific files.
>  - Added new patch for trogdor board specific changes.
> 
> Changes in V6:
>  - As per Doug's comments, fixed nits in patch 2 and 3.
>  - As per Bjorn's comments changed the commit text and rationale for
>    rts, tx and rx in patch 2 and 3.
> 
> satya priya (4):
>   arm64: dts: qcom: sc7180: Improve the uart3 pin config for sc7180-idp
>   arm64: dts: qcom: sc7180: Add wakeup support for BT UART on sc7180-idp
>   arm64: dts: qcom: sc7180-trogdor: Add wakeup support for BT UART

Applied these...

>   tty: serial: qcom_geni_serial: Fix the UART wakeup issue

...expecting Greg to pick this one up.

Thanks,
Bjorn

> 
>  arch/arm64/boot/dts/qcom/sc7180-idp.dts      | 65 +++++++++++++++++++++++++---
>  arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi | 54 +++++++++++++++++++++++
>  drivers/tty/serial/qcom_geni_serial.c        |  2 +-
>  3 files changed, 114 insertions(+), 7 deletions(-)
> 
> -- 
> QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member 
> of Code Aurora Forum, hosted by The Linux Foundation
>