Re: [PATCH 2/2] Revert "ASoC: ux500: drop platform DAI assignments"

2017-07-18 Thread Mark Brown
On Tue, Jul 18, 2017 at 12:36:29PM +0200, Johan Hovold wrote: > On Tue, Jul 18, 2017 at 11:06:55AM +0100, Mark Brown wrote: > > Reverts shouldn't be special - they're just regular patches and should > > have sensible changelogs like any others. > Stating that you're reverting a commit and which

Re: [PATCH 2/2] Revert "ASoC: ux500: drop platform DAI assignments"

2017-07-18 Thread Mark Brown
On Tue, Jul 18, 2017 at 12:36:29PM +0200, Johan Hovold wrote: > On Tue, Jul 18, 2017 at 11:06:55AM +0100, Mark Brown wrote: > > Reverts shouldn't be special - they're just regular patches and should > > have sensible changelogs like any others. > Stating that you're reverting a commit and which

Re: [PATCH 2/2] Revert "ASoC: ux500: drop platform DAI assignments"

2017-07-18 Thread Johan Hovold
On Tue, Jul 18, 2017 at 11:06:55AM +0100, Mark Brown wrote: > On Tue, Jul 18, 2017 at 10:21:18AM +0200, Johan Hovold wrote: > > On Mon, Jul 17, 2017 at 03:51:27PM +0100, Mark Brown wrote: > > > > Please submit patches using subject lines reflecting the style for the > > > subsystem. This makes

Re: [PATCH 2/2] Revert "ASoC: ux500: drop platform DAI assignments"

2017-07-18 Thread Johan Hovold
On Tue, Jul 18, 2017 at 11:06:55AM +0100, Mark Brown wrote: > On Tue, Jul 18, 2017 at 10:21:18AM +0200, Johan Hovold wrote: > > On Mon, Jul 17, 2017 at 03:51:27PM +0100, Mark Brown wrote: > > > > Please submit patches using subject lines reflecting the style for the > > > subsystem. This makes

Re: [PATCH 2/2] Revert "ASoC: ux500: drop platform DAI assignments"

2017-07-18 Thread Mark Brown
On Tue, Jul 18, 2017 at 10:21:18AM +0200, Johan Hovold wrote: > On Mon, Jul 17, 2017 at 03:51:27PM +0100, Mark Brown wrote: > > Please submit patches using subject lines reflecting the style for the > > subsystem. This makes it easier for people to identify relevant > > patches. Look at what

Re: [PATCH 2/2] Revert "ASoC: ux500: drop platform DAI assignments"

2017-07-18 Thread Mark Brown
On Tue, Jul 18, 2017 at 10:21:18AM +0200, Johan Hovold wrote: > On Mon, Jul 17, 2017 at 03:51:27PM +0100, Mark Brown wrote: > > Please submit patches using subject lines reflecting the style for the > > subsystem. This makes it easier for people to identify relevant > > patches. Look at what

Re: [PATCH 2/2] Revert "ASoC: ux500: drop platform DAI assignments"

2017-07-18 Thread Johan Hovold
On Mon, Jul 17, 2017 at 03:51:27PM +0100, Mark Brown wrote: > On Wed, Jul 12, 2017 at 05:55:30PM +0200, Johan Hovold wrote: > > This reverts commit f1013cdeeeb9 ("ASoC: ux500: drop platform DAI > > assignments"), which seems to have been based on a misunderstanding and > > prevents the platform

Re: [PATCH 2/2] Revert "ASoC: ux500: drop platform DAI assignments"

2017-07-18 Thread Johan Hovold
On Mon, Jul 17, 2017 at 03:51:27PM +0100, Mark Brown wrote: > On Wed, Jul 12, 2017 at 05:55:30PM +0200, Johan Hovold wrote: > > This reverts commit f1013cdeeeb9 ("ASoC: ux500: drop platform DAI > > assignments"), which seems to have been based on a misunderstanding and > > prevents the platform

Re: [PATCH 2/2] Revert "ASoC: ux500: drop platform DAI assignments"

2017-07-17 Thread Mark Brown
On Wed, Jul 12, 2017 at 05:55:30PM +0200, Johan Hovold wrote: > This reverts commit f1013cdeeeb9 ("ASoC: ux500: drop platform DAI > assignments"), which seems to have been based on a misunderstanding and > prevents the platform driver callbacks from being made (e.g. to > preallocate DMA memory).

Re: [PATCH 2/2] Revert "ASoC: ux500: drop platform DAI assignments"

2017-07-17 Thread Mark Brown
On Wed, Jul 12, 2017 at 05:55:30PM +0200, Johan Hovold wrote: > This reverts commit f1013cdeeeb9 ("ASoC: ux500: drop platform DAI > assignments"), which seems to have been based on a misunderstanding and > prevents the platform driver callbacks from being made (e.g. to > preallocate DMA memory).

Re: [PATCH 2/2] Revert "ASoC: ux500: drop platform DAI assignments"

2017-07-17 Thread Johan Hovold
On Fri, Jul 14, 2017 at 03:36:57PM +0200, Linus Walleij wrote: > On Wed, Jul 12, 2017 at 5:55 PM, Johan Hovold wrote: > > > This reverts commit f1013cdeeeb9 ("ASoC: ux500: drop platform DAI > > assignments"), which seems to have been based on a misunderstanding and > > prevents

Re: [PATCH 2/2] Revert "ASoC: ux500: drop platform DAI assignments"

2017-07-17 Thread Johan Hovold
On Fri, Jul 14, 2017 at 03:36:57PM +0200, Linus Walleij wrote: > On Wed, Jul 12, 2017 at 5:55 PM, Johan Hovold wrote: > > > This reverts commit f1013cdeeeb9 ("ASoC: ux500: drop platform DAI > > assignments"), which seems to have been based on a misunderstanding and > > prevents the platform

Re: [PATCH 2/2] Revert "ASoC: ux500: drop platform DAI assignments"

2017-07-14 Thread Linus Walleij
On Wed, Jul 12, 2017 at 5:55 PM, Johan Hovold wrote: > This reverts commit f1013cdeeeb9 ("ASoC: ux500: drop platform DAI > assignments"), which seems to have been based on a misunderstanding and > prevents the platform driver callbacks from being made (e.g. to > preallocate DMA

Re: [PATCH 2/2] Revert "ASoC: ux500: drop platform DAI assignments"

2017-07-14 Thread Linus Walleij
On Wed, Jul 12, 2017 at 5:55 PM, Johan Hovold wrote: > This reverts commit f1013cdeeeb9 ("ASoC: ux500: drop platform DAI > assignments"), which seems to have been based on a misunderstanding and > prevents the platform driver callbacks from being made (e.g. to > preallocate DMA memory). > > The

[PATCH 2/2] Revert "ASoC: ux500: drop platform DAI assignments"

2017-07-12 Thread Johan Hovold
This reverts commit f1013cdeeeb9 ("ASoC: ux500: drop platform DAI assignments"), which seems to have been based on a misunderstanding and prevents the platform driver callbacks from being made (e.g. to preallocate DMA memory). The real culprit for the warnings about attempts to create duplicate

[PATCH 2/2] Revert "ASoC: ux500: drop platform DAI assignments"

2017-07-12 Thread Johan Hovold
This reverts commit f1013cdeeeb9 ("ASoC: ux500: drop platform DAI assignments"), which seems to have been based on a misunderstanding and prevents the platform driver callbacks from being made (e.g. to preallocate DMA memory). The real culprit for the warnings about attempts to create duplicate