Re: [PATCH 2/3] soundwire: intel: add SDW_MASTER_QUIRKS_CLEAR_INITIAL_CLASH quirk

2021-02-02 Thread Pierre-Louis Bossart
+ prop->quirks = SDW_MASTER_QUIRKS_CLEAR_INITIAL_CLASH; Should this not be last 'enabling' the quirk patch in series :) Sorry, I don't understand the comment. Do you mind clarifying Vinod? Sure, I would like to series built as, first defining the quirk along/followed by bus changes

Re: [PATCH 2/3] soundwire: intel: add SDW_MASTER_QUIRKS_CLEAR_INITIAL_CLASH quirk

2021-02-01 Thread Vinod Koul
On 01-02-21, 10:20, Pierre-Louis Bossart wrote: > > > On 2/1/21 4:42 AM, Vinod Koul wrote: > > On 26-01-21, 16:37, Bard Liao wrote: > > > There is nothing we can do to handle the bus clash interrupt before > > > interrupt mask is enabled. > > > > > > Signed-off-by: Bard Liao > > > Reviewed-by:

Re: [PATCH 2/3] soundwire: intel: add SDW_MASTER_QUIRKS_CLEAR_INITIAL_CLASH quirk

2021-02-01 Thread Pierre-Louis Bossart
On 2/1/21 4:42 AM, Vinod Koul wrote: On 26-01-21, 16:37, Bard Liao wrote: There is nothing we can do to handle the bus clash interrupt before interrupt mask is enabled. Signed-off-by: Bard Liao Reviewed-by: Rander Wang Reviewed-by: Pierre-Louis Bossart --- drivers/soundwire/intel.c | 2

Re: [PATCH 2/3] soundwire: intel: add SDW_MASTER_QUIRKS_CLEAR_INITIAL_CLASH quirk

2021-02-01 Thread Vinod Koul
On 26-01-21, 16:37, Bard Liao wrote: > There is nothing we can do to handle the bus clash interrupt before > interrupt mask is enabled. > > Signed-off-by: Bard Liao > Reviewed-by: Rander Wang > Reviewed-by: Pierre-Louis Bossart > --- > drivers/soundwire/intel.c | 2 ++ > 1 file changed, 2 inse