Skip to contentSkip to conversation

Revert "serial/amba-pl011: Leave the TX IRQ alone when the UART is not open"

PATCH New Greg Kroah-Hartman 2 participants 3 messages May 7, 2015 – May 8, 2015 1 message not in the archive TTY layer and serial drivers

lore
Share & tools
lore

A message that is not in the archive (sent off-list or not yet received): [email protected]

Greg Kroah-Hartman

author maintainer
#1 lore raw

PATCH 1/1 Revert "serial/amba-pl011: Leave the TX IRQ alone when the UART is not open"

Quoted 12 lines · Dave Martin

Calling serial maintainers...

Please consider the following commit for merging in v4.1.

f28c1d0a78a8c6a217ac5362cb719efa169ae2a7 Revert "serial/amba-pl011: Leave the TX IRQ alone when the UART is not open"

This backs out a misguided optimisation that is causing pl011 driver deadlocks for some people in certain scenarios (my bad).

The revert is self-contained and independent of the other changes I posted.

Quoted 7 lines

Thanks
---Dave

Original commit message:

This reverts commit f2ee6dfa0e8597eea8b98d240b0033994e20d215.

Please send this in a format I can apply it in...

#2 lore raw

PATCH 1/1 Revert "serial/amba-pl011: Leave the TX IRQ alone when the UART is not open"

Quoted 17 lines · Greg Kroah-Hartman
Quoted 16 lines · Dave Martin

Calling serial maintainers...

Please consider the following commit for merging in v4.1.

f28c1d0a78a8c6a217ac5362cb719efa169ae2a7 Revert "serial/amba-pl011: Leave the TX IRQ alone when the UART is not open"

This backs out a misguided optimisation that is causing pl011 driver deadlocks for some people in certain scenarios (my bad).

The revert is self-contained and independent of the other changes I posted.

Thanks
---Dave

Quoted 5 lines
Quoted 3 lines

Original commit message:

This reverts commit f2ee6dfa0e8597eea8b98d240b0033994e20d215.

Please send this in a format I can apply it in...

The revert has been posted twice on this list and you already applied it to tty-next with commit ID f28c1d0a78a8c6a217ac5362cb719efa169ae2a7 (thanks btw). I could have made that clearer...

I'm happy to post it again, but I assumed (wrongly?) that you'd prefer to cherry-pick it from your own branch. If you really want me to post a duplicate, please confirm.

I was concerned that posting a duplicate of a patch you already picked up would risk further confusion...

Cheers
---Dave

author maintainer
#3 lore raw

PATCH 1/1 Revert "serial/amba-pl011: Leave the TX IRQ alone when the UART is not open"

Quoted 20 lines · Dave P Martin
Quoted 19 lines · Greg Kroah-Hartman
Quoted 18 lines · Dave Martin

Calling serial maintainers...

Please consider the following commit for merging in v4.1.

f28c1d0a78a8c6a217ac5362cb719efa169ae2a7 Revert "serial/amba-pl011: Leave the TX IRQ alone when the UART is not open"

This backs out a misguided optimisation that is causing pl011 driver deadlocks for some people in certain scenarios (my bad).

The revert is self-contained and independent of the other changes I posted.

Thanks
---Dave

Original commit message:

Quoted 7 lines
Quoted 3 lines
Quoted 1 line

This reverts commit f2ee6dfa0e8597eea8b98d240b0033994e20d215.

Please send this in a format I can apply it in...

The revert has been posted twice on this list and you already applied it to tty-next with commit ID f28c1d0a78a8c6a217ac5362cb719efa169ae2a7 (thanks btw). I could have made that clearer...

Yes, as I had no idea that is what you wanted here.

Quoted 3 lines

I'm happy to post it again, but I assumed (wrongly?) that you'd prefer to cherry-pick it from your own branch. If you really want me to post a duplicate, please confirm.

Please resend, and be more explicit, remember, I get over 1000 emails a day...

greg k-h