Messages in this thread | | | Date | Mon, 19 Jan 2015 09:58:37 +0800 | From | Greg KH <> | Subject | Re: [PATCH 1/2] staging: andriod: ion: Add ion_phys_get_by_id function |
| |
On Mon, Jan 19, 2015 at 01:14:49AM +0000, Tang, Feng wrote: > Hi Greg, > > Thanks for your review. > > > -----Original Message----- > > From: Greg KH [mailto:gregkh@linuxfoundation.org] > > Sent: Sunday, January 18, 2015 8:04 AM > > To: Tang, Feng > > Cc: One Thousand Gnomes; John Stultz; Colin Cross; Heesub Shin; Mitchel > > Humpherys; linux-kernel@vger.kernel.org > > Subject: Re: [PATCH 1/2] staging: andriod: ion: Add ion_phys_get_by_id > > function > > > > > > > > > > On Sun, 4 Jan 2015 15:11:51 +0800 > > > > Feng Tang <feng.tang@intel.com> wrote: > > > > > > > > > There is user case that user only knows the ion_handle idand > > > > > needs to get the physical addr and len. So add this wrapper to > > > > > meet this requirement. > > > > > > > > What is the in-tree upstream user of this new function ? I don't see one > > > > in your patches ? > > > > > > That's true. We are working on enabling a new platform which heavily > > > uses ION and some driver needs to use this API, so I guess there is some > > > other plaforms may have the similar requirements. If you think this patch > > > should be merged after there is an in-tree user, I'm OK with it :) > > > > > > I assume that driver's author will start to push his drivers in batch soon. > > > > We need to see those patches first, we can't add apis for no in-tree > > user, otherwise they will just get ripped out. > I see. I'll repost this patch after our "user" driver get merged. > > How about the 2/2 patch, which improve the profiling info through > debugfs? > [PATCH 2/2] staging: android: ion: Add pss info for each ion_client
I don't see it in my queue anywhere, sorry, feel free to resend it.
thanks,
greg k-h
| |