Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.37 as permitted sender) client-ip=23.128.96.37;
Message-ID: <6127727d-cb4e-7729-5216-83c93fbb22ee@amd.com>
Date:   Tue, 26 Sep 2023 15:18:58 +0530
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Thunderbird/102.15.1
Subject: Re: [PATCH v4] cdx: add MSI support for CDX bus
To:     gregkh@linuxfoundation.org, maz@kernel.org, tglx@linutronix.de,
        jgg@ziepe.ca, linux-kernel@vger.kernel.org
Cc:     git@amd.com, harpreet.anand@amd.com,
        pieter.jansen-van-vuuren@amd.com, nikhil.agarwal@amd.com,
        michal.simek@amd.com, abhijit.gangurde@amd.com,
        srivatsa@csail.mit.edu
References: <20230911135259.14046-1-nipun.gupta@amd.com>
Content-Language: en-US
From:   "Gupta, Nipun" <nipun.gupta@amd.com>
In-Reply-To: <20230911135259.14046-1-nipun.gupta@amd.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?RFN4TjZjZlBLMkNaNWtRK0JFcUtJUHhZNzdPOERDYUxxMGx0bVI1K3UvQWp1?=
 =?utf-8?B?elRVSXdXTUUzMjNKWnhLZTRWcG11dWVGajRnWFFiQm5pMUdic1RkZUhjRlln?=
 =?utf-8?B?TUp4aERkZERENi9zdGFNR0piSWFFSXk4Mjd3RkhVM0h6d2lVUzlsdlhnNXI5?=
 =?utf-8?B?N3VpOHBTYjA4ZTM0aEJhQVUvQ29vNmFpV1JPMmhJeWNCaGVHV3FTa0VKVldn?=
 =?utf-8?B?bXFUb0xxMzJObVY5YkpWQmxPQkVndDUxK0JaT2F6c05ObmNlc1VBOE14QnU2?=
 =?utf-8?B?OTlwMDZBRWErdytlclVuWWJmekk4Rml3d1o4K1VHaGxkR1UrcG4zb2FmRFVU?=
 =?utf-8?B?T0lOWHQySTAwMkNvY3JmRzhJWVBET1FKekhZZ0NxdXA4S0xDMmVMRUVSM0J5?=
 =?utf-8?B?K1gzRDhwcXBlZ2Q0MjgycTVTeEl0R0g1SFpFQ0h6SktxZytha29sMTV1Um40?=
 =?utf-8?B?VW9tSnY4Rjl1M3d2VnZNcm96UWh5aktDa2tDTTJFMEMwZDFjQ0RaUDVJQi9J?=
 =?utf-8?B?d3Q2b3RsV0dGWEFqSU42cnM2Ty9iMkc0R2lOcDAyTWQ4cDY2UVhVdW9wVUUw?=
 =?utf-8?B?TlpMa2JqNE9RMGtPRU5vaVdWbkg1U0xiWmpVMUtZNlhGekxudjMyaGFTRUg5?=
 =?utf-8?B?V3pZeUY1R1NYOWtlMG45Y3lNbGRuTllseGErQWhZdU1jdnpMUHFQTk1aaUZj?=
 =?utf-8?B?V0U0empPVktzWTE1djI0aTJ6RURRWUV5elVUN0dyWkRPN3FvUjh5R2huYlJx?=
 =?utf-8?B?Y2xzY2NqKzFOQjdtMWRqcHRVWEVrNHF4SXdoeWF0NlZwY05FTHQ3ZWVVdExj?=
 =?utf-8?B?VjFJMlpJbGR4eUpFQlRuUGRnN3kyejJxNEN2a2w2OFlPWG1yNDBKNzVyVlhs?=
 =?utf-8?B?SGdMelRwVVdMRWdtMHl4SzdDbGRHVlFwODZ2THh3SmUzaUVncVhVbnFva29G?=
 =?utf-8?B?TVNpQXRZZm0wVUZUYzVzR0NubmxTclBlRnJXS01zMmRRSmlhL0JhVWJBYytp?=
 =?utf-8?B?RlhSNU5LMlVORit6YXZoaisrK2pmbjQ3aVJNZ3BmSUJwaGdTTGd3THN5YXB6?=
 =?utf-8?B?OXBZR3JuYjhlZld2ZEtybzMwd0FWeUFZdlNURHNaSTB6d2xhd2V6M29xVUhn?=
 =?utf-8?B?TEZJeEJsZFg5c3dpcmNPc2RYTkx4MTc0SXFJSlV0d0tISEg2b1NsNTdjL2cv?=
 =?utf-8?B?UkdyS0tGcXRla2RnSnhrOFQ1V0ZoQyt2bnJIYjc2Y0tkTXVVSEJzMDhhUG1w?=
 =?utf-8?B?cDZqVkFXbEZqODB4aHk5WXJ1YkFjQzJMakI0VU1xamlSd0lKdVpMZ0hiL3JF?=
 =?utf-8?B?NitjN2FGTVZUOVlKNUxZWUo4R0h1bjBncFI4N0ZaSWE5R0RLZFFrSzlCVjVX?=
 =?utf-8?B?bU1LeWJXUjRFbm1jQXNiQ096dUt2VTBwWjUvTDFPR3ZtbWxJRmdZUUJJTjRj?=
 =?utf-8?B?U3p2dlhQVWVqVDRIYnpQMkdmdWVNOFpBdnZSc2hDV1ZaRVRLWGl2WWJVb24v?=
 =?utf-8?B?UG5EeG4wVmFiL21acTNRNi9WVEwvTEttRDIvc2c2Z0xIZGE3cytXZjJqV2ZH?=
 =?utf-8?B?RWZTTk5SUTVUMHR2d29sVWR4U3FNcTZtYnZWVUhXc1dWUm4xRnhqb2JRNUZS?=
 =?utf-8?B?ckZmRVpuSlRJcXhlMXRnYXlCWmZCK3JRY0hIekJxamRGc2VKb3I0MVY0Snll?=
 =?utf-8?B?MWFMc3RaUktGSGxuWG83Z0FwY25EaHhJcWxVRDJTS1pjcHNTWmRWY2RTRWl5?=
 =?utf-8?B?aHd5WXRKTG5FVEQrZW55S09FWjJDUlJxRkJwWVpaMVZtYUJQbVU3aml2OXAy?=
 =?utf-8?B?Nng5cGJtOXRQUmROU3dPcldzNkpYbFBMeEZ4dXdpY2w3OTV0aXJRdnR4dklF?=
 =?utf-8?B?MHRFbldEUmFDNDI1cG01ZjhXdjVPaS81eGpZZFByMnR1V3ZuaVNISW40c2Zv?=
 =?utf-8?B?bXdONE1oQUJualRVTzJuZjFPTkNNSWVMbkFzbGxvdG9GQ0w3dE5oTHJ3NlVq?=
 =?utf-8?B?ZDRvbFZjMllxQ3ZicnNNd2FzWXhlczlXbFBhOGk4VzhSODEwNTJJSy9MRDl2?=
 =?utf-8?B?L2prMTYyYmIrUHRrejR4TWs2b1dkeGVxMHJiS3l3clJIWGNMV0lpOWNmRDAv?=
 =?utf-8?Q?6LfPkHDwOX7lZRcrx4NCAO7fo?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 1103902e-e281-4027-5bb4-08dbbe75d6ca
X-MS-Exchange-CrossTenant-AuthSource: CH3PR12MB8308.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 26 Sep 2023 09:49:13.0789
 (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: XExrUM5E5XmNvYUVqfLYMP4ITUqALex4FVSy/PtiyAFQobnmb/tybooBs9Lr9Wdf
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH7PR12MB5735
X-Spam-Status: No, score=-2.6 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FORGED_SPF_HELO,
        NICE_REPLY_A,RCVD_IN_DNSWL_BLOCKED,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,
        SPF_NONE autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (snail.vger.email [0.0.0.0]); Tue, 26 Sep 2023 02:49:22 -0700 (PDT)

Hi Greg,

On 9/11/2023 7:22 PM, Nipun Gupta wrote:
> Add CDX-MSI domain per CDX controller with gic-its domain as
> a parent, to support MSI for CDX devices. CDX devices allocate
> MSIs from the CDX domain. Also, introduce APIs to alloc and free
> IRQs for CDX domain.
> 
> In CDX subsystem firmware is a controller for all devices and
> their configuration. CDX bus controller sends all the write_msi_msg
> commands to firmware running on RPU and the firmware interfaces with
> actual devices to pass this information to devices
> 
> Since, CDX controller is the only way to communicate with the Firmware
> for MSI write info, CDX domain per controller required in contrast to
> having a CDX domain per device.
> 
> Co-developed-by: Nikhil Agarwal <nikhil.agarwal@amd.com>
> Signed-off-by: Nikhil Agarwal <nikhil.agarwal@amd.com>
> Co-developed-by: Abhijit Gangurde <abhijit.gangurde@amd.com>
> Signed-off-by: Abhijit Gangurde <abhijit.gangurde@amd.com>
> Signed-off-by: Nipun Gupta <nipun.gupta@amd.com>
> Reviewed-by: Pieter Jansen van Vuuren <pieter.jansen-van-vuuren@amd.com>
> Tested-by: Nikhil Agarwal <nikhil.agarwal@amd.com>

Please help in making progress on this patch. Is there anything that 
needs to be updated in this patch for CDX bus?

Thanks,
Nipun