Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
IronPort-SDR: Z7jN5Uj9tqL1GzLp4pIQI3wYaK3clwm1z8mH0FAGtQ06Sx5W2QDHB8GlfTQx9fLkpF007M7zfy
 yTkl8jLbHaWw==
IronPort-SDR: 2r4eqh7NX8XprghcOrgxODDadsgbYLHokaXJIF+0qxRFFRYI7mL/dfFa9Te8H2AOEDvtxPj+po
 kIWO0wuNZcew==
From:   "Winkler, Tomas" <tomas.winkler@intel.com>
To:     Greg Kroah-Hartman <gregkh@linuxfoundation.org>
CC:     "Usyskin, Alexander" <alexander.usyskin@intel.com>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "Grumbach, Emmanuel" <emmanuel.grumbach@intel.com>
Subject: RE: [char-misc-next 6/6] mei: bus: add client dma interface
Thread-Topic: [char-misc-next 6/6] mei: bus: add client dma interface
Thread-Index: AQHW/JZ6zfy6g0Lzl0WlwNIsjRgFAKpLE28AgAAknRA=
Date:   Sat, 6 Feb 2021 15:04:34 +0000
Message-ID: <2a9b40090acd41438fc6b1d2fb49f38d@intel.com>
References: <20210206144325.25682-1-tomas.winkler@intel.com>
 <20210206144325.25682-6-tomas.winkler@intel.com> <YB6sIi61X5p6Dq6y@kroah.com>
In-Reply-To: <YB6sIi61X5p6Dq6y@kroah.com>
Accept-Language: en-US
Content-Language: en-US
dlp-product: dlpe-windows
dlp-reaction: no-action
dlp-version: 11.5.1.3
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 8BIT
MIME-Version: 1.0
Precedence: bulk

> On Sat, Feb 06, 2021 at 04:43:25PM +0200, Tomas Winkler wrote:
> > From: Alexander Usyskin <alexander.usyskin@intel.com>
> >
> > Expose the client dma mapping via mei client bus interface.
> >
> > Signed-off-by: Alexander Usyskin <alexander.usyskin@intel.com>
> > Signed-off-by: Tomas Winkler <tomas.winkler@intel.com>
> > ---
> >  drivers/misc/mei/bus.c     | 46
> ++++++++++++++++++++++++++++++++++++++
> >  drivers/misc/mei/hw.h      |  5 +++++
> >  include/linux/mei_cl_bus.h |  3 +++
> >  3 files changed, 54 insertions(+)
> >
> > diff --git a/drivers/misc/mei/bus.c b/drivers/misc/mei/bus.c index
> > 34fb5e541fe5..d430710a5fe5 100644
> > --- a/drivers/misc/mei/bus.c
> > +++ b/drivers/misc/mei/bus.c
> > @@ -636,6 +636,52 @@ static void mei_cl_bus_vtag_free(struct
> mei_cl_device *cldev)
> >  	kfree(cl_vtag);
> >  }
> >
> > +void *mei_cldev_dma_map(struct mei_cl_device *cldev, u8 buffer_id,
> > +size_t size) {
> > +	struct mei_device *bus;
> > +	struct mei_cl *cl;
> > +	int ret;
> > +
> > +	if (!cldev || !buffer_id || !size)
> > +		return ERR_PTR(-EINVAL);
> > +
> > +	if (!IS_ALIGNED(size, MEI_FW_PAGE_SIZE)) {
> > +		dev_err(&cldev->dev, "Map size should be aligned to %lu\n",
> > +			MEI_FW_PAGE_SIZE);
> > +		return ERR_PTR(-EINVAL);
> > +	}
> > +
> > +	cl = cldev->cl;
> > +	bus = cldev->bus;
> > +
> > +	mutex_lock(&bus->device_lock);
> > +	ret = mei_cl_dma_alloc_and_map(cl, NULL, buffer_id, size);
> > +	mutex_unlock(&bus->device_lock);
> > +	if (ret)
> > +		return ERR_PTR(ret);
> > +	return cl->dma.vaddr;
> > +}
> > +EXPORT_SYMBOL_GPL(mei_cldev_dma_map);
> 
> Why are you exporting symbols without a user of them?
> 
> That's a sure way to get them removed by someone right after this lands in
> the tree :)
> 
> Please only add infrastructure for when you have a real user.
You are right,  the user is coming from wifi tree. We need merge before they do.
If you prefer we can merge that all from the wifi tree. 


Thanks
Tomas