mbox series

[0/3] phy: phy-brcm-usb: Fixes for phy-brcm-usb driver

Message ID 20211201180653.35097-1-alcooperx@gmail.com
Headers show
Series phy: phy-brcm-usb: Fixes for phy-brcm-usb driver | expand

Message

Alan Cooper Dec. 1, 2021, 6:06 p.m. UTC
A few fixes for the phy-brcm-usb driver.

Al Cooper (3):
  phy: usb: Leave some clocks running during suspend
  usb: Add "wake on" functionality for newer Synopsis XHCI controllers
  phy: broadcom: Kconfig: Fix PHY_BRCM_USB config option

 drivers/phy/broadcom/Kconfig                  |  3 +-
 .../phy/broadcom/phy-brcm-usb-init-synopsys.c | 46 +++++++++++++++----
 drivers/phy/broadcom/phy-brcm-usb.c           | 38 +++++++++++++++
 3 files changed, 77 insertions(+), 10 deletions(-)


base-commit: f080815fdb3e3cff5a004ca83b3815ac17ef71b1

Comments

Florian Fainelli Jan. 19, 2022, 11:26 p.m. UTC | #1
Hi Vinod,

On 12/1/21 10:06 AM, Al Cooper wrote:
> A few fixes for the phy-brcm-usb driver.
> 
> Al Cooper (3):
>   phy: usb: Leave some clocks running during suspend
>   usb: Add "wake on" functionality for newer Synopsis XHCI controllers
>   phy: broadcom: Kconfig: Fix PHY_BRCM_USB config option

Are you able to pick up those patches? Thank you

> 
>  drivers/phy/broadcom/Kconfig                  |  3 +-
>  .../phy/broadcom/phy-brcm-usb-init-synopsys.c | 46 +++++++++++++++----
>  drivers/phy/broadcom/phy-brcm-usb.c           | 38 +++++++++++++++
>  3 files changed, 77 insertions(+), 10 deletions(-)
> 
> 
> base-commit: f080815fdb3e3cff5a004ca83b3815ac17ef71b1
>
Vinod Koul Jan. 23, 2022, 11:45 a.m. UTC | #2
On 19-01-22, 15:26, Florian Fainelli wrote:
> Hi Vinod,
> 
> On 12/1/21 10:06 AM, Al Cooper wrote:
> > A few fixes for the phy-brcm-usb driver.
> > 
> > Al Cooper (3):
> >   phy: usb: Leave some clocks running during suspend
> >   usb: Add "wake on" functionality for newer Synopsis XHCI controllers
> >   phy: broadcom: Kconfig: Fix PHY_BRCM_USB config option
> 
> Are you able to pick up those patches? Thank you

Applied phy patches to phy-fixes, thanks
Florian Fainelli Jan. 28, 2022, 1:01 a.m. UTC | #3
On 1/23/2022 3:45 AM, Vinod Koul wrote:
> On 19-01-22, 15:26, Florian Fainelli wrote:
>> Hi Vinod,
>>
>> On 12/1/21 10:06 AM, Al Cooper wrote:
>>> A few fixes for the phy-brcm-usb driver.
>>>
>>> Al Cooper (3):
>>>    phy: usb: Leave some clocks running during suspend
>>>    usb: Add "wake on" functionality for newer Synopsis XHCI controllers
>>>    phy: broadcom: Kconfig: Fix PHY_BRCM_USB config option
>>
>> Are you able to pick up those patches? Thank you
> 
> Applied phy patches to phy-fixes, thanks

Thanks! I can see patches 1 and 3 there but patch 2 is not in phy-fixes 
or your next branch. Can you pick it up if it is satisfactory? Thanks!
Vinod Koul Feb. 2, 2022, 4:57 a.m. UTC | #4
On 27-01-22, 17:01, Florian Fainelli wrote:
> 
> 
> On 1/23/2022 3:45 AM, Vinod Koul wrote:
> > On 19-01-22, 15:26, Florian Fainelli wrote:
> > > Hi Vinod,
> > > 
> > > On 12/1/21 10:06 AM, Al Cooper wrote:
> > > > A few fixes for the phy-brcm-usb driver.
> > > > 
> > > > Al Cooper (3):
> > > >    phy: usb: Leave some clocks running during suspend
> > > >    usb: Add "wake on" functionality for newer Synopsis XHCI controllers
> > > >    phy: broadcom: Kconfig: Fix PHY_BRCM_USB config option
> > > 
> > > Are you able to pick up those patches? Thank you
> > 
> > Applied phy patches to phy-fixes, thanks
> 
> Thanks! I can see patches 1 and 3 there but patch 2 is not in phy-fixes or
> your next branch. Can you pick it up if it is satisfactory? Thanks!

That is usb patch right. I wont pick unless Greg acks it to go thru phy
tree
Florian Fainelli Feb. 2, 2022, 4:24 p.m. UTC | #5
On 2/1/2022 8:57 PM, Vinod Koul wrote:
> On 27-01-22, 17:01, Florian Fainelli wrote:
>>
>>
>> On 1/23/2022 3:45 AM, Vinod Koul wrote:
>>> On 19-01-22, 15:26, Florian Fainelli wrote:
>>>> Hi Vinod,
>>>>
>>>> On 12/1/21 10:06 AM, Al Cooper wrote:
>>>>> A few fixes for the phy-brcm-usb driver.
>>>>>
>>>>> Al Cooper (3):
>>>>>     phy: usb: Leave some clocks running during suspend
>>>>>     usb: Add "wake on" functionality for newer Synopsis XHCI controllers
>>>>>     phy: broadcom: Kconfig: Fix PHY_BRCM_USB config option
>>>>
>>>> Are you able to pick up those patches? Thank you
>>>
>>> Applied phy patches to phy-fixes, thanks
>>
>> Thanks! I can see patches 1 and 3 there but patch 2 is not in phy-fixes or
>> your next branch. Can you pick it up if it is satisfactory? Thanks!
> 
> That is usb patch right. I wont pick unless Greg acks it to go thru phy
> tree

This is an USB PHY patch, I suppose the subject including "usb: " did 
somewhat mislead you. Here is the diffstat just for that patch:

  .../phy/broadcom/phy-brcm-usb-init-synopsys.c | 46 +++++++++++++++----
  1 file changed, 38 insertions(+), 8 deletions(-)

definitively falling into the PHY subsystem.
Vinod Koul Feb. 3, 2022, 12:12 a.m. UTC | #6
On 02-02-22, 08:24, Florian Fainelli wrote:
> 
> 
> On 2/1/2022 8:57 PM, Vinod Koul wrote:
> > On 27-01-22, 17:01, Florian Fainelli wrote:
> > > 
> > > 
> > > On 1/23/2022 3:45 AM, Vinod Koul wrote:
> > > > On 19-01-22, 15:26, Florian Fainelli wrote:
> > > > > Hi Vinod,
> > > > > 
> > > > > On 12/1/21 10:06 AM, Al Cooper wrote:
> > > > > > A few fixes for the phy-brcm-usb driver.
> > > > > > 
> > > > > > Al Cooper (3):
> > > > > >     phy: usb: Leave some clocks running during suspend
> > > > > >     usb: Add "wake on" functionality for newer Synopsis XHCI controllers
> > > > > >     phy: broadcom: Kconfig: Fix PHY_BRCM_USB config option
> > > > > 
> > > > > Are you able to pick up those patches? Thank you
> > > > 
> > > > Applied phy patches to phy-fixes, thanks
> > > 
> > > Thanks! I can see patches 1 and 3 there but patch 2 is not in phy-fixes or
> > > your next branch. Can you pick it up if it is satisfactory? Thanks!
> > 
> > That is usb patch right. I wont pick unless Greg acks it to go thru phy
> > tree
> 
> This is an USB PHY patch, I suppose the subject including "usb: " did
> somewhat mislead you. Here is the diffstat just for that patch:
> 
>  .../phy/broadcom/phy-brcm-usb-init-synopsys.c | 46 +++++++++++++++----
>  1 file changed, 38 insertions(+), 8 deletions(-)
> 
> definitively falling into the PHY subsystem.

Yeah that was it! Can you please change the title and resend...

> -- 
> Florian