Date:   Mon, 6 Dec 2021 11:04:15 -0400
From:   Jason Gunthorpe <jgg@nvidia.com>
To:     Christoph Hellwig <hch@infradead.org>
Cc:     Stuart Yoder <stuyoder@gmail.com>, rafael@kernel.org,
        David Airlie <airlied@linux.ie>, linux-pci@vger.kernel.org,
        Thierry Reding <thierry.reding@gmail.com>,
        Diana Craciun <diana.craciun@oss.nxp.com>,
        Dmitry Osipenko <digetx@gmail.com>,
        Will Deacon <will@kernel.org>, Ashok Raj <ashok.raj@intel.com>,
        Jonathan Hunter <jonathanh@nvidia.com>,
        Kevin Tian <kevin.tian@intel.com>,
        Chaitanya Kulkarni <kch@nvidia.com>,
        Alex Williamson <alex.williamson@redhat.com>,
        kvm@vger.kernel.org, Bjorn Helgaas <bhelgaas@google.com>,
        Dan Williams <dan.j.williams@intel.com>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        Cornelia Huck <cohuck@redhat.com>,
        linux-kernel@vger.kernel.org, Li Yang <leoyang.li@nxp.com>,
        iommu@lists.linux-foundation.org,
        Jacob jun Pan <jacob.jun.pan@intel.com>,
        Daniel Vetter <daniel@ffwll.ch>,
        Robin Murphy <robin.murphy@arm.com>
Subject: Re: [PATCH v3 03/18] driver core: platform: Rename
 platform_dma_configure()
Message-ID: <20211206150415.GD4670@nvidia.com>
References: <20211206015903.88687-1-baolu.lu@linux.intel.com>
 <20211206015903.88687-4-baolu.lu@linux.intel.com>
 <Ya3BYxrgkNK3kbGI@kroah.com>
 <Ya4abbx5M31LYd3N@infradead.org>
 <20211206144535.GB4670@nvidia.com>
 <Ya4ikRpenoQPXfML@infradead.org>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <Ya4ikRpenoQPXfML@infradead.org>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?wz981IQHcmE3qe40G09+HOF1+ReCjSdPgYuszb7xFAbxIG/xf1xnEfLdKo/G?=
 =?us-ascii?Q?ebbLgZJv3ZVReGa+SMZ1uNmlDHokb6qQ982Yf9eg6jmASlDKTJGKcN9huK5z?=
 =?us-ascii?Q?yYxk2Pazb4kD9p9QyGAjP59sTpa3OzbRMTar0VNEIAkPLtetHHx+/jSidyj0?=
 =?us-ascii?Q?h5gozntb9nrwY6mq+e3Rk149RnK94xcC/5WGOwBSdiO5mlq7lBJMfYMbcaMU?=
 =?us-ascii?Q?VrIcE55qziydWFSVSJYA0reaagXD2l33X1njIAv002ocML0y2FG5fF6lvYuv?=
 =?us-ascii?Q?ES/W1Hzm+y3uiZ4vim0KeNX36ihgfI6j4HhqXiNz72bGGpdBOM/iQiIzEsh6?=
 =?us-ascii?Q?r8/l4zckMLN/7f4r64WBIPoTu8pVAld6uF3Lifcjhb++75GqmVS4qkVfPa8n?=
 =?us-ascii?Q?YE9iu2NHsYVXAuDjfVUHg2YshwISyK/cTJvulaSSuThSVsJnwYFoTV7XhOBw?=
 =?us-ascii?Q?LOGpi2UkvoawhtvhSaBxWT+q9RVn6/N9FEgowE+C3rut4P9EskBxWTMnWlcM?=
 =?us-ascii?Q?lvGdmXd56lMbWt0Vakj+1Qu7TR9aoqNJBqUfhTNCdAYv9Zf2WvOkyJcD/BUo?=
 =?us-ascii?Q?L6gQLyc+ywKs5yxPSwzwTSq4TuH3p3eLKPjjXjw21xSm54GIliFoCrFunYUs?=
 =?us-ascii?Q?BKMlKBmqVz3StyNS+vfvaUjtHBoea9pO3i7lBTYI7tIN7WnLaJYa3ViCbfrE?=
 =?us-ascii?Q?Rr+z2PnSudYBvx3wpyyDTBSFZhVW336HLgyL+NCR/pUUUtntAwQ3ZX8SUQba?=
 =?us-ascii?Q?/KhevkRNmR4f4ZCvxmy4AzaYj06F2GU8KMqyoJuE0fvcKDRGTa5ueh+Zdfwf?=
 =?us-ascii?Q?IrkPJ2COoifDYAdCX+HrzObnA9b28b7SPlbAJD3q8MH+w266Q3rI+13pY7ll?=
 =?us-ascii?Q?rSk8R5C8QGWNo6N266OAp5sA7A2N6WRyrhaMxfomLrgsInKFdT6dJ2mOIKxm?=
 =?us-ascii?Q?ddV4HMmdH6kBNsf3hssQsxqOID9khSoah7vEZHCdWGPoEdgHbyHbR2IuhGKN?=
 =?us-ascii?Q?Yjo9UHiFia7885S8r4K5wElFEOGN2yTkIA+9zFUCwy/e7mUJunu4Pp67fNjF?=
 =?us-ascii?Q?/BsA8E/JQ8puxHvqZoWOAu5kHC9+0d8MS89/ue0RFrVjAfFx0Ts4VnbxGEuR?=
 =?us-ascii?Q?zhKkJ3OsMEXV7UCKgB1MDiEQVqpL27zE2M35O/Hi3JsJRWLdxp+shw/J9Uop?=
 =?us-ascii?Q?Lpmizkh7KqthS1mt8xNkk/pMfbxBCUFtNuWjZVzmG4qtVsm1m09Mwqt+8XyY?=
 =?us-ascii?Q?GgjgoU5MMt0fC634nKf9xa2KO55dNg5mSjfBVWnG8ELhPsw4jomJsCPu16Nv?=
 =?us-ascii?Q?G/WtINOWtVCHoncce5ca03q9U/PYZxV1r+eJQtb9xUDdrQH221a6lmyxUmT7?=
 =?us-ascii?Q?U2+G0nQ3tcOwov2mlq8uqrpL91eJGGHwyOiE3rrjnOBVR8pf1DF4C+sfeWin?=
 =?us-ascii?Q?5vqPEe41B4BKzdTvT1qeewn0HOwHhvdldh9wkkFIjrS6uT02DxaZO/bxRt9N?=
 =?us-ascii?Q?yVWXndzfTWldI9PRCGaCFEAWAKG4xPfqAK2raxDRGql8iqORG/rnPiQyFFJm?=
 =?us-ascii?Q?2lKtW1OBpF1RDOnKffc=3D?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: b3f94b43-9e98-4466-2753-08d9b8c9ac37
X-MS-Exchange-CrossTenant-AuthSource: BL0PR12MB5506.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 06 Dec 2021 15:04:16.9368
 (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: zkaCSUCTJ14osS4oFFs0ls5R3q0vnwLB8QXdSITAVigzr0nSq/S1k6s3CVkkAqaT
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL0PR12MB5539
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Mon, Dec 06, 2021 at 06:47:45AM -0800, Christoph Hellwig wrote:
> On Mon, Dec 06, 2021 at 10:45:35AM -0400, Jason Gunthorpe via iommu wrote:
> > IIRC the only thing this function does is touch ACPI and OF stuff?
> > Isn't that firmware?
> > 
> > AFAICT amba uses this because AMBA devices might be linked to DT
> > descriptions?
> 
> But DT descriptions aren't firmware.  They are usually either passed onb
> the bootloader or in some deeply embedded setups embedded into the
> kernel image.

Pedenatically yes, but do you know of a common word to refer to both
OF and ACPI that is better than firmware? :)

AFAICT we already use firwmare for this in a few places, eg
fwnode_handle and so on.

Jason