mbox series

[v2,0/3] Revert "usb: gadget: u_audio: add real feedback implementation"

Message ID 20210826185739.3868-1-ftoth@exalondelft.nl (mailing list archive)
Headers show
Series Revert "usb: gadget: u_audio: add real feedback implementation" | expand

Message

Ferry Toth Aug. 26, 2021, 6:57 p.m. UTC
Although there is a patch resolving this issue (see 
https://lore.kernel.org/linux-usb/1jilzsy8g7.fsf@starbuckisacylon.baylibre.com/T/#u)
it needs a little work and will not be ready for v5.14.0 release. Until then
revert the series.

v2:
- Added SoB (Balbi)


Ferry Toth (3):
  Revert "usb: gadget: u_audio: add real feedback implementation"
  Revert "usb: gadget: f_uac2: add adaptive sync support for capture"
  Revert "usb: gadget: f_uac2/u_audio: add feedback endpoint support"

 .../ABI/testing/configfs-usb-gadget-uac2      |   2 -
 Documentation/usb/gadget-testing.rst          |   2 -
 drivers/usb/gadget/function/f_uac2.c          | 144 +----------
 drivers/usb/gadget/function/u_audio.c         | 225 +-----------------
 drivers/usb/gadget/function/u_audio.h         |  12 -
 drivers/usb/gadget/function/u_uac2.h          |   4 -
 6 files changed, 6 insertions(+), 383 deletions(-)

Comments

Jerome Brunet Aug. 27, 2021, 7:59 a.m. UTC | #1
On Thu 26 Aug 2021 at 20:57, Ferry Toth <ftoth@exalondelft.nl> wrote:

> Although there is a patch resolving this issue (see 
> https://lore.kernel.org/linux-usb/1jilzsy8g7.fsf@starbuckisacylon.baylibre.com/T/#u)
> it needs a little work and will not be ready for v5.14.0 release. Until then
> revert the series.

Seems like a quite messy solution when the fix available :/
Change with the updated commit description is avaialable BTW [0]

[0]: https://lore.kernel.org/20210827075853.266912-1-jbrunet@baylibre.com

>
> v2:
> - Added SoB (Balbi)
>
>
> Ferry Toth (3):
>   Revert "usb: gadget: u_audio: add real feedback implementation"
>   Revert "usb: gadget: f_uac2: add adaptive sync support for capture"
>   Revert "usb: gadget: f_uac2/u_audio: add feedback endpoint support"
>
>  .../ABI/testing/configfs-usb-gadget-uac2      |   2 -
>  Documentation/usb/gadget-testing.rst          |   2 -
>  drivers/usb/gadget/function/f_uac2.c          | 144 +----------
>  drivers/usb/gadget/function/u_audio.c         | 225 +-----------------
>  drivers/usb/gadget/function/u_audio.h         |  12 -
>  drivers/usb/gadget/function/u_uac2.h          |   4 -
>  6 files changed, 6 insertions(+), 383 deletions(-)
Greg Kroah-Hartman Aug. 27, 2021, 8:27 a.m. UTC | #2
On Thu, Aug 26, 2021 at 08:57:36PM +0200, Ferry Toth wrote:
> Although there is a patch resolving this issue (see 
> https://lore.kernel.org/linux-usb/1jilzsy8g7.fsf@starbuckisacylon.baylibre.com/T/#u)
> it needs a little work and will not be ready for v5.14.0 release. Until then
> revert the series.

revert the series for what?  5.14-final needs these reverts?  Or are
these for 5.15-rc1?

confused,

greg k-h
Andy Shevchenko Aug. 27, 2021, 9:15 a.m. UTC | #3
On Fri, Aug 27, 2021 at 11:05 AM Jerome Brunet <jbrunet@baylibre.com> wrote:
> On Thu 26 Aug 2021 at 20:57, Ferry Toth <ftoth@exalondelft.nl> wrote:
>
> > Although there is a patch resolving this issue (see
> > https://lore.kernel.org/linux-usb/1jilzsy8g7.fsf@starbuckisacylon.baylibre.com/T/#u)
> > it needs a little work and will not be ready for v5.14.0 release. Until then
> > revert the series.
>
> Seems like a quite messy solution when the fix available :/

Is it? AFAIR Thing pointed out to some problems with the proposed
solution. Have those been addressed?

> Change with the updated commit description is avaialable BTW [0]

> [0]: https://lore.kernel.org/20210827075853.266912-1-jbrunet@baylibre.com

Dunno, but this gives me 404.
Ferry Toth Aug. 27, 2021, 8:44 p.m. UTC | #4
Op 27-08-2021 om 11:15 schreef Andy Shevchenko:
> On Fri, Aug 27, 2021 at 11:05 AM Jerome Brunet <jbrunet@baylibre.com> wrote:
>> On Thu 26 Aug 2021 at 20:57, Ferry Toth <ftoth@exalondelft.nl> wrote:
>>
>>> Although there is a patch resolving this issue (see
>>> https://lore.kernel.org/linux-usb/1jilzsy8g7.fsf@starbuckisacylon.baylibre.com/T/#u)
>>> it needs a little work and will not be ready for v5.14.0 release. Until then
>>> revert the series.
>>
>> Seems like a quite messy solution when the fix available :/
> 
> Is it? AFAIR Thing pointed out to some problems with the proposed
> solution. Have those been addressed?
> 
>> Change with the updated commit description is avaialable BTW [0]
> 
>> [0]: https://lore.kernel.org/20210827075853.266912-1-jbrunet@baylibre.com
> 
> Dunno, but this gives me 404.
> 
I think:
https://lore.kernel.org/linux-usb/1jfsuvw817.fsf@starbuckisacylon.baylibre.com/T/#m922149b7e74204c8ed1bed2c624910ab4eafd43c
Ferry Toth Aug. 27, 2021, 8:49 p.m. UTC | #5
Hi
Op 27-08-2021 om 10:27 schreef Greg Kroah-Hartman:
> On Thu, Aug 26, 2021 at 08:57:36PM +0200, Ferry Toth wrote:
>> Although there is a patch resolving this issue (see
>> https://lore.kernel.org/linux-usb/1jilzsy8g7.fsf@starbuckisacylon.baylibre.com/T/#u)
>> it needs a little work and will not be ready for v5.14.0 release. Until then
>> revert the series.
> 
> revert the series for what?  5.14-final needs these reverts?  Or are
> these for 5.15-rc1?
> 
> confused,

Apologies. Yes, the idea was to revert for 5.14-final as the series 
creates a regression.

However, an updated patch "usb: gadget: f_uac2: fixup feedback endpoint 
stop"
https://lore.kernel.org/linux-usb/1jfsuvw817.fsf@starbuckisacylon.baylibre.com/T/#m922149b7e74204c8ed1bed2c624910ab4eafd43c
now has been acked by Felipe.

If we can get that it for 5.14-final it would be less messy.

Can we?

> greg k-h
>