Message-ID: <0cbcee69-d0dc-1db9-bfa1-0ba2ec0812d6@amd.com>
Date:   Wed, 22 Feb 2023 02:10:57 +0530
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.7.1
Subject: Re: [PATCH V3 7/8] soundwire: amd: handle SoundWire wake enable
 interrupt
To:     Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>,
        vkoul@kernel.org
Cc:     alsa-devel@alsa-project.org, amadeuszx.slawinski@linux.intel.com,
        Mario.Limonciello@amd.com, Sunil-kumar.Dommati@amd.com,
        Basavaraj.Hiregoudar@amd.com, Mastan.Katragadda@amd.com,
        Arungopal.kondaveeti@amd.com,
        Bard Liao <yung-chuan.liao@linux.intel.com>,
        Sanyog Kale <sanyog.r.kale@intel.com>,
        open list <linux-kernel@vger.kernel.org>
References: <20230220100418.76754-1-Vijendar.Mukunda@amd.com>
 <20230220100418.76754-8-Vijendar.Mukunda@amd.com>
 <8a002459-37aa-47c2-4e7d-50adac6015f6@linux.intel.com>
Content-Language: en-US
From:   "Mukunda,Vijendar" <vijendar.mukunda@amd.com>
In-Reply-To: <8a002459-37aa-47c2-4e7d-50adac6015f6@linux.intel.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?dVEwWU9HSzZYaUxwa2d5eXBwNS9sN3NGb0ZoQ0IrRGdpTTIxVktzUXM0RXht?=
 =?utf-8?B?UGNVN3o2cFRoMC9uWFoxUTBaSjQ2VWtFUEFzdjNvNVlhQkd3bTBRUy9EWDlN?=
 =?utf-8?B?L3lCOUsrWFJUM0FsYm1hNCtlUUJXWlFYdklWNlNBdlVhM1ZaaUNpU3NZeXlJ?=
 =?utf-8?B?NVhMQWhmL0hja3dJYUVBU2FCeWZWZTFyc3FJU2hHTGlMYkFVdU1yRnVuR2hr?=
 =?utf-8?B?ek9YTU1Dd1MzZzFsWXc0d3dhWXp6Z2U3dDVBNFZ5V1ZjSVV3OURhdTJrbkVZ?=
 =?utf-8?B?V3liVjVBUzZXd1dqQXBWdU4ydC8wQnhsd2NmYkVTWVNnQm1iWGQvR3BZVHcr?=
 =?utf-8?B?TTFvbU9IeTNQdWlrOE5SRzNzZFdiOVRHdUVCMmcwNGt3RDh2YnFvQ0VjbTAz?=
 =?utf-8?B?b3lqT0YvNTZucVg1dUJuZVZGemgyTG5TSmhzV252R25BdC9FQjZLSElVakRz?=
 =?utf-8?B?S2RaNkFyaUxick1FZ2szT01oNjZmb21FelgvRHc4UWxuNDNPR0lBK1JIRVZs?=
 =?utf-8?B?TkVwS0RtS2ZpYzFmdTBBRDR1OUxDZGJVbnhaRTl1WWhzeVhTd0R5MGxQY1RC?=
 =?utf-8?B?V2pSWU9lNkw3Qy81YzBKRk5pSWZMcW5YOEUwYnozMFVhZWQ3SUFZc0hKQkhP?=
 =?utf-8?B?NGNWelllUjUwTlNpQ1d2WnBoQ09BbG9sM2kvOXJQSGlMU2FvYVg2MTVyckRC?=
 =?utf-8?B?MUsyM0ozdUZsQnR0Tld0QVJpejZqWGQ2SVYyY3hXTVdRTnYvMGtjNFRRY3pJ?=
 =?utf-8?B?UFJ2YktGNUN0ZFVWRGJWWUFHYUI1RXJvMXlmeWd2QzQ0MEFKOFhQMWk3ZjU0?=
 =?utf-8?B?Q2NST1gvRE5oSmc2WTNwQjBxckNtaVFxN3RkbjMwT1luOHE3Y2U4VHYwWVNY?=
 =?utf-8?B?TjhLTCswTTNYWWJtVW11V2hUcHN1TzRmOVlpUmliWjZ3Z1JsQmRUNlF2aUlw?=
 =?utf-8?B?Ri9DS0E3cW1jMURuWlZFWEp6REc2OVRReWpyZ3lESVRIcEhFNW5kV202RlBr?=
 =?utf-8?B?bjZSSUpYc2wrR2Jrb0FVWVc4SEtEVVZyUElGTExBa0J0ZWxubEF3MlkxeXhB?=
 =?utf-8?B?dGlKUU14eElpRDB1bTU3MWZGbDd5MS8zY244d3Bvdnp4aWYzUTg0dnkzUHYv?=
 =?utf-8?B?b1BRZGVmam50M1lEclNDQWszZzFjUk4xQ2NKcURjNXNpbVlBU2RjWlQ1dTA1?=
 =?utf-8?B?YTFjbXpuZDlmN0c5b1NNeVFGVEFkWlFTTTNXUDJOQURvMnlEYVV6TkpnVmF4?=
 =?utf-8?B?cmVna20vdFNYL1Q5WHEyZHN2WVl4M0IrVk1MR044K1k4OUt2TDRMdmNHc0VH?=
 =?utf-8?B?Q1EzeXZJaDJIc0V3Y2tSZng1RWZDL1BKMERoUVFsdndDeTRCVTBJRW8rd0Vj?=
 =?utf-8?B?dUx1Z1R4a2pYUlUvUzZsVUlCQzArZjkxc3hDcWtoeStXTitNV3czNGhaeUV2?=
 =?utf-8?B?cWRiZjhDemc1Q0ZtTzBGSnhITFhZWDh6N083RVd5clJaeXU4aXBScVBEdldV?=
 =?utf-8?B?RGI4S3U3OEsyVCtFQzFJRk50bFhlTlJzd0oxZ2xmcE5hVDVZVUJzWDgyQzdF?=
 =?utf-8?B?aDdFbUJHQ0JUREZpbEtkSWxPMWRHclhvOXo1U0JNK1BseW1pSjd4Wk1QMmhk?=
 =?utf-8?B?YzVPdTRTNjJET216czVCZjVnQmNJQmpMdWRZQnJBS2xvSHBpYkRrVThpd092?=
 =?utf-8?B?NFV0UllsRHY2ZCtMMVQrN1l0VVE2NXBwZXNpT0tkR05jV2pYMEpOUkZTbGhx?=
 =?utf-8?B?VEQ3R3lpQmdvVkhYMlh5WEtUc3pyTk1wWkdLcjNpRFIzSFkrMzhmaWpMVU9Z?=
 =?utf-8?B?RWxzb3RpV0JFekpaSTJTOUZrZDFsZ24zREY3UFR1SHRTNHVFM0Z6bm5QQVFx?=
 =?utf-8?B?YVVmSXF3cEFNSk8raFNxZ1NVcUIycEpVU2VnM1l6WHlFamNUL0xORk1DY1pV?=
 =?utf-8?B?bjdTZVZqTTBrVkp6VkNPWm1TTERzRE42Q1RhQnh3cGgrK1FYVXFZWXpPaThK?=
 =?utf-8?B?b1hZbysrNUJpb29FcjRRdzBRSXJKWnovZTJLNUhFZmZxeVE5YncyL2R4OXh6?=
 =?utf-8?B?L2pVdlc4UFluVnlHUXI3V2VmVS8xZTFsNW4xTStSVWVYYUQwQmFWUUVSQXhr?=
 =?utf-8?Q?Ajw6tsZmATqjx8qpkucGfVVah?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 732a9ed3-2e63-47ad-9e5a-08db144b9bdc
X-MS-Exchange-CrossTenant-AuthSource: DM6PR12MB4123.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 21 Feb 2023 20:38:37.5430
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: lmpzSQN86sAlsnE0f5CNQ+Ile6lWVa80kkfDtNYseHQ4UE1chKepl09B2Jzgj6jE8dYggW+B/I+CniqfIh76xg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL1PR12MB5271
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 21/02/23 21:43, Pierre-Louis Bossart wrote:
>
> On 2/20/23 05:04, Vijendar Mukunda wrote:
>> Add wake enable interrupt support for both the SoundWire manager
>> instances.
>>
>> Signed-off-by: Vijendar Mukunda <Vijendar.Mukunda@amd.com>
>> Signed-off-by: Mastan Katragadda <Mastan.Katragadda@amd.com>
>> ---
>>  drivers/soundwire/amd_manager.c | 10 ++++++++++
>>  drivers/soundwire/amd_manager.h |  1 +
>>  2 files changed, 11 insertions(+)
>>
>> diff --git a/drivers/soundwire/amd_manager.c b/drivers/soundwire/amd_manager.c
>> index 3322adeca0d8..a7182aa78652 100644
>> --- a/drivers/soundwire/amd_manager.c
>> +++ b/drivers/soundwire/amd_manager.c
>> @@ -932,6 +932,13 @@ static void amd_sdw_update_slave_status(u32 status_change_0to7, u32 status_chang
>>  	}
>>  }
>>  
>> +static void amd_sdw_process_wake_event(struct amd_sdw_manager *amd_manager)
>> +{
>> +	pm_request_resume(amd_manager->dev);
> is this needed?
>
> In the Intel case, the wakes do not necessarily come as in-band wakes,
> but they can also be notified by the PCI subsystem, so we do have to use
> pm_request_resume.
>
> In the AMD case, what happens if you don't do this? Doesn't the
> interrupt trigger a pm_runtime_resume already?
ACP PCI driver receives soundwire interrupt and soundwire manager
interrupt work queue will be scheduled.
In this work queue, wake interrupt status bit is checked.
As still soundwire manager in D3 state, it required to invoke
pm_request_resume(). Without pm_request_resume() call, pm_runtime_resume
won't be triggered in this scenario.
>
>> +	acp_reg_writel(0x00, amd_manager->acp_mmio + ACP_SW_WAKE_EN(amd_manager->instance));
>> +	acp_reg_writel(0x00, amd_manager->mmio + ACP_SW_STATE_CHANGE_STATUS_8TO11);
>> +}
> \