Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Mon, 29 Aug 2022 12:31:45 -0300
From:   Jason Gunthorpe <jgg@nvidia.com>
To:     "Gupta, Nipun" <Nipun.Gupta@amd.com>
Cc:     Robin Murphy <robin.murphy@arm.com>,
        Saravana Kannan <saravanak@google.com>,
        Greg KH <gregkh@linuxfoundation.org>,
        "robh+dt@kernel.org" <robh+dt@kernel.org>,
        "krzysztof.kozlowski+dt@linaro.org" 
        <krzysztof.kozlowski+dt@linaro.org>,
        "rafael@kernel.org" <rafael@kernel.org>,
        "eric.auger@redhat.com" <eric.auger@redhat.com>,
        "alex.williamson@redhat.com" <alex.williamson@redhat.com>,
        "cohuck@redhat.com" <cohuck@redhat.com>,
        "Gupta, Puneet (DCG-ENG)" <puneet.gupta@amd.com>,
        "song.bao.hua@hisilicon.com" <song.bao.hua@hisilicon.com>,
        "mchehab+huawei@kernel.org" <mchehab+huawei@kernel.org>,
        "maz@kernel.org" <maz@kernel.org>,
        "f.fainelli@gmail.com" <f.fainelli@gmail.com>,
        "jeffrey.l.hugo@gmail.com" <jeffrey.l.hugo@gmail.com>,
        "Michael.Srba@seznam.cz" <Michael.Srba@seznam.cz>,
        "mani@kernel.org" <mani@kernel.org>,
        "yishaih@nvidia.com" <yishaih@nvidia.com>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "kvm@vger.kernel.org" <kvm@vger.kernel.org>,
        "okaya@kernel.org" <okaya@kernel.org>,
        "Anand, Harpreet" <harpreet.anand@amd.com>,
        "Agarwal, Nikhil" <nikhil.agarwal@amd.com>,
        "Simek, Michal" <michal.simek@amd.com>,
        "git (AMD-Xilinx)" <git@amd.com>
Subject: Re: [RFC PATCH v2 2/6] bus/cdx: add the cdx bus driver
Message-ID: <Ywzb4RmbgbnQYTIl@nvidia.com>
References: <20220817150542.483291-3-nipun.gupta@amd.com>
 <Yv0KHROjESUI59Pd@kroah.com>
 <DM6PR12MB3082D966CFC0FA1C2148D8FAE8719@DM6PR12MB3082.namprd12.prod.outlook.com>
 <YwOEv6107RfU5p+H@kroah.com>
 <DM6PR12MB3082B4BDD39632264E7532B8E8739@DM6PR12MB3082.namprd12.prod.outlook.com>
 <YwYVhJCSAuYcgj1/@kroah.com>
 <20220824233122.GA4068@nvidia.com>
 <CAGETcx846Pomh_DUToncbaOivHMhHrdt-MTVYqkfLUKvM8b=6w@mail.gmail.com>
 <a6ca5a5a-8424-c953-6f76-c9212db88485@arm.com>
 <DM6PR12MB30824C5129A7251C589F1461E8769@DM6PR12MB3082.namprd12.prod.outlook.com>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <DM6PR12MB30824C5129A7251C589F1461E8769@DM6PR12MB3082.namprd12.prod.outlook.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?9bkRGbJPxgfjbMDcdNxAuOPUxsRM0UhIRk0g01UaQb6nfbvvMhTHEJqsyHqO?=
 =?us-ascii?Q?h6Hw4wykvDLkoWrxYCFBaBaR7WjkCkOoGfhVlhowwYZd7BltNqObYOpHUeXC?=
 =?us-ascii?Q?ljAMhQg6Abf/CtcnwEyfpnx/FGmURVhe0J1WZ/bQ0A8cFcDP4RsvNjkTkAb9?=
 =?us-ascii?Q?QHzjkSZ6tJfVVWYct6vaUW7+kWJnU07nNPB35NaWgpDMHX+EttekyYUD4oMm?=
 =?us-ascii?Q?BTG+WhIupHr+57IKH1TeazSSEiLGJLTKGVdh2j5pHU6qiIeNsr0vBChZczqa?=
 =?us-ascii?Q?xBE6dKN9SGeI5oU06hWt7gXLipCL3aYOxA9RDWzCeFniKfaIrd4CqXrnVoq5?=
 =?us-ascii?Q?odANncBcAJNmP7wChwH/MukZPQVnipTRelVDZTJK6UqtG50w9ZVjuI43sF89?=
 =?us-ascii?Q?UFGdpB6g0LeAczKmbcxEhWFieptOkXTADPxSrI0CAH03TNZzFYNSwWGw4UII?=
 =?us-ascii?Q?GoLkTfSuNHTf0ssINVc7WZf09pxCyL5qiQvxVJY5BC3bkYwSSoEZsDL7Wuk5?=
 =?us-ascii?Q?3stFH2Vc/pFXixBb0Je7yfrXkpxMOGWDjaCt1RSn6mckKCtwVI7dr5z3wc0y?=
 =?us-ascii?Q?LOnvqC9ZvUkh16SFK7LbI/vcuW3BjKTLoa12KLylwSzuG8ugIS35OqrFsRqq?=
 =?us-ascii?Q?WEnQ0cI2PoMyw0dRiTr8ZSPjo6moky+ykru715Y+RrdLhLeOdRnCJmxpC8GI?=
 =?us-ascii?Q?y6KLj9ICxxTPLNzCO/+mX4dt7BSi+z6Oo119lnRIeTv+0sxPmFdaZecVOjHv?=
 =?us-ascii?Q?+J/ewURqAYJzllWBgp1MzKiuAQzaXmkSI5WnlxXZrjJWf61ZLbSEJbNqgPfp?=
 =?us-ascii?Q?LAXXE13TNNlhF9n6UhlZLtTXmosDrSiogMmDyrFtp0WrrDE2NfstnrMNCXTK?=
 =?us-ascii?Q?c0VImDxJ6CGs8k8WvnfW67/4VFZgoWAielrDnVPl/i7I5yAmPTbiFBKn8YDY?=
 =?us-ascii?Q?GZCqQrkri6Z6bAD50mIPpcxOk8oFxsWuXoGFKulzNG2xlC0/Dcs8nBeEs3EX?=
 =?us-ascii?Q?tl8yrc849gA9EKko5brKBH7NCyEAaIMDQ/CNPKK+EzBnHEAGhmNKhs1TSKP1?=
 =?us-ascii?Q?ogo0xr8owLaG7yKnsPC1ZTdbYiqUoFm+5TKMGt7BlwE6QsbRPcGZ0XCf3RL5?=
 =?us-ascii?Q?HNCzIUigspt94tXdHmXV8//u38Rfm+/C7g7OGxa1ZNiQmbJZv7rGZG9xGE+L?=
 =?us-ascii?Q?YB9ecNWHVLKeltFftOt+TH+OpZ0RMJkWCbgSp9faPCpLonjNiMoSh+K+2q17?=
 =?us-ascii?Q?Ad3bfteBCf6XSlCJxhblYRkjygZXB5oluDwzuS3fQdcouR3UM85tNPKmRnlZ?=
 =?us-ascii?Q?BIWe9ptcen2RnfDjab/fiis4NiMLlByrkEe9VoTymTGyFniYpBjGjqD3SMjM?=
 =?us-ascii?Q?zedy1R/gP0oN7ak4yf5UdE40/xK+L4mueFxc+cQLxzRwYRdonh5UH7lt/yFD?=
 =?us-ascii?Q?YZ+ltAh5/J6Wb8AEo42Pmn6l1hjZgVfQCb4rcQ0rYg9+Z8uDnqNXNVHujvu7?=
 =?us-ascii?Q?MeR8pmZYG5+t+WJcag1aiKEcgzaZF5zWz5rAn+RI6aMAxnwcx9saRtvH9AV3?=
 =?us-ascii?Q?a8I4IM4pKJbdCV4m8rl78AO/MrlvEJrPIF4mLgMl?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 6844f3c6-8fb7-49fe-35fe-08da89d3952c
X-MS-Exchange-CrossTenant-AuthSource: MN2PR12MB4192.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 29 Aug 2022 15:31:46.2965
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: ItY5JfFvpl7l6i5tvWvD06aeZxtiDbDKGm2bgPP5nl11vn+4RZpoIECYjqMFXWbJ
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY4PR12MB1750
X-Spam-Status: No, score=-1.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FORGED_SPF_HELO,
        RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_NONE,
        T_SCC_BODY_TEXT_LINE autolearn=no 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

On Mon, Aug 29, 2022 at 04:49:02AM +0000, Gupta, Nipun wrote:

> Devices are created in FPFGA with a CDX wrapper, and CDX controller(firmware)
> reads that CDX wrapper to find out new devices. Host driver then interacts with
> firmware to find newly discovered devices. This bus aligns with PCI infrastructure.
> It happens to be an embedded interface as opposed to off-chip connection.

Why do you need an FW in all of this?

And why do you need DT at all?

It is still not clear

Jason