Date:   Mon, 13 Dec 2021 09:24:43 -0400
From:   Jason Gunthorpe <jgg@nvidia.com>
To:     Lu Baolu <baolu.lu@linux.intel.com>
Cc:     Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        Christoph Hellwig <hch@infradead.org>,
        Joerg Roedel <joro@8bytes.org>,
        Alex Williamson <alex.williamson@redhat.com>,
        Bjorn Helgaas <bhelgaas@google.com>,
        Kevin Tian <kevin.tian@intel.com>,
        Ashok Raj <ashok.raj@intel.com>, Will Deacon <will@kernel.org>,
        Robin Murphy <robin.murphy@arm.com>,
        Dan Williams <dan.j.williams@intel.com>, rafael@kernel.org,
        Diana Craciun <diana.craciun@oss.nxp.com>,
        Cornelia Huck <cohuck@redhat.com>,
        Eric Auger <eric.auger@redhat.com>,
        Liu Yi L <yi.l.liu@intel.com>,
        Jacob jun Pan <jacob.jun.pan@intel.com>,
        Chaitanya Kulkarni <kch@nvidia.com>,
        Stuart Yoder <stuyoder@gmail.com>,
        Laurentiu Tudor <laurentiu.tudor@nxp.com>,
        Thierry Reding <thierry.reding@gmail.com>,
        David Airlie <airlied@linux.ie>,
        Daniel Vetter <daniel@ffwll.ch>,
        Jonathan Hunter <jonathanh@nvidia.com>,
        Li Yang <leoyang.li@nxp.com>,
        Dmitry Osipenko <digetx@gmail.com>,
        iommu@lists.linux-foundation.org, linux-pci@vger.kernel.org,
        kvm@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v3 04/18] driver core: platform: Add driver dma ownership
 management
Message-ID: <20211213132443.GS6385@nvidia.com>
References: <20211206015903.88687-1-baolu.lu@linux.intel.com>
 <20211206015903.88687-5-baolu.lu@linux.intel.com>
 <Ya4f662Af+8kE2F/@infradead.org>
 <20211206150647.GE4670@nvidia.com>
 <56a63776-48ca-0d6e-c25c-016dc016e0d5@linux.intel.com>
 <20211207131627.GA6385@nvidia.com>
 <c170d215-6aef-ff21-8733-1bae4478e39c@linux.intel.com>
 <b42ffaee-bb96-6db4-8540-b399214f6881@linux.intel.com>
 <5d537286-2cb3-cf91-c0de-019355110aa1@linux.intel.com>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <5d537286-2cb3-cf91-c0de-019355110aa1@linux.intel.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?qgeNdbCgm6orJ4AJrkar+9GFRzzFIcHZRvfWQrQOBGA/fb8bF9BSTWkmAIld?=
 =?us-ascii?Q?wI9naXFedjpMlWjwrz4eQpqGdHFw9g9psfNswveyvHVmWfRelflXcSW4E0Xf?=
 =?us-ascii?Q?vo0ELz5+dJFIxzySgKA/4Ro97iZYglBPocc7rd10fwdWQEiHjRf+eLDFNKLz?=
 =?us-ascii?Q?MNYdsVuBEGMGq2JRSzUfm++hrPuU5ziDblMihITOhwhiNT1+U6wGpsZOtfeq?=
 =?us-ascii?Q?C1Z7ArUVrWNAfnVHXUE/Zu5/8JJu8ybhGXlMZe5TyrZdsiVoRRwTlyF2NwzM?=
 =?us-ascii?Q?cNq2zFtFGDMkwAJ+lptHV4P8MjGXdsHQLp3B4lSvk/2sMIgFi/NQlz4eEodV?=
 =?us-ascii?Q?eILMiIA5FhBiFO4LCCPaAdQlJfcr6GWHtuUgtStJA15MOoDaRWiUpx00EnNI?=
 =?us-ascii?Q?/N452gVYTGBv5UJfKJzjP3j/pMC1Qv3SsHOOTX5qzudOUuChFit/lGR0Y0s9?=
 =?us-ascii?Q?zaKXhRg+aQBeUoDggSnY0Q1FM+Og9/ng2FKkliaEMV1pIJxhzBYMcU8qrEGL?=
 =?us-ascii?Q?f4tQQVmkYsBMxx07qOr1lY/0FLgWo0yYJWaS+8PjXQxBDiCy9lnE2dgz+J1s?=
 =?us-ascii?Q?Nnm2KjAHMzlfXkbynak0UKVZTyqReXCjYqnFmcyRqfWjXpo9Z+YhBpoCMTd/?=
 =?us-ascii?Q?VEupV6GVmpdHT+1m6tjATi651NcmuhsiP3ZsLJXkEBNKBVn42L6qG8EZb4Xq?=
 =?us-ascii?Q?Xfb3IlCDGyJGaPojCcuaKekBzhX9BOZkWIwyq1TCW3iUDnCmC+h7N3i00yIG?=
 =?us-ascii?Q?W4uWRWaVKsYRCIBUTC3KzgpP8eVH0clbRxOQ/HRmkK/cZ5oMDBuK1K2EXaZw?=
 =?us-ascii?Q?KJntjIlr9jfu8MOug/oGdaqJ5RdGjwRIznAeP6LJheI7lsr+ejZ+3drF4+gT?=
 =?us-ascii?Q?+NMMrQVL8zIC5O6KJ2ottoZKxp3Gm6f9vDbbseE0Uto/yq9HVQXBO93qF43O?=
 =?us-ascii?Q?BVZVvkWXpI0ebsUhWS71hlQE6YPXyyS1lt7G+L6mK/Rqh0pkHL0SbQKvsEl8?=
 =?us-ascii?Q?HoYt7GPJO9rsTAPAEwUCG2SR825lYIHakvOn89peT+cf7mI7SsHJI0VsuBn3?=
 =?us-ascii?Q?ZTE8pOD04uQ45GzQiugtGfJHtKjlBcHtYN4bYaBkxQEdAmxycPiH1WqsNYlP?=
 =?us-ascii?Q?IIe9oT7Y6kI8fJXJlQQtSiEeMzvx5b1lwtWYGkDOt3Z1jZpwCslKoIM45dRt?=
 =?us-ascii?Q?P3HCt9cRtcsya0pEqDMCjxPWIBSa8MAuDq7Q/M5r9EtOHtZzaNmqS24M72Br?=
 =?us-ascii?Q?2vsAZbZxq/dpZjLhZgwBi8nmOmE0KceJUr/Uxj50LX3d3ksMegAD8umLq61o?=
 =?us-ascii?Q?rFdy6IkWfCQhYwifeJti3SIVTxQV0AZT+tpJqIQAXu9EzmbA7yBvbSUVv1NZ?=
 =?us-ascii?Q?86r7Ea3O/Qo972xrFlyraIieXJpT/5rHhfStMkPpMIg3nhhNX8TLW0vMEGLl?=
 =?us-ascii?Q?SW6GYJKzWGVWGOJLT7TZESevOmPF8xBYH7nC36fhhDmCCwL7J5y8bm/IVTYW?=
 =?us-ascii?Q?1fh4lNm9kXBJAtI9sIJXKrMUXvBNTnjUkU4fnRcTSkuGI53VtAigw8jdA9r/?=
 =?us-ascii?Q?x29dqwOXUAQBm2C8Vjc=3D?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 39e7bdac-ab4e-4103-96d2-08d9be3bedb3
X-MS-Exchange-CrossTenant-AuthSource: BL0PR12MB5506.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 Dec 2021 13:24:45.7297
 (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: ep7rAoZ8i6hspNKrFdCWrbo79reGyBgFp1ELpMtz2UDeQfJ5RwHEVyx8iM0jBa1q
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL0PR12MB5553
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Mon, Dec 13, 2021 at 08:50:05AM +0800, Lu Baolu wrote:

> > Does this work for you? Can I work towards this in the next version?
> 
> A kindly ping ... Is this heading the right direction? I need your
> advice to move ahead. :-)

I prefer this to all the duplicated code in the v3 series.. Given it
touches almost every bus type that implements the dma_configure it
seems appropriate.

Jason