[PATCH V2 0/3] Add vchi_queue_kernel_message and vchi_queue_user_message

Michael Zoran mzoran at crowfest.net
Wed Jan 25 12:44:51 PST 2017


On Wed, 2017-01-25 at 12:27 -0800, Eric Anholt wrote:
> Greg KH <gregkh at linuxfoundation.org> writes:
> 
> > On Wed, Jan 25, 2017 at 05:27:14AM -0800, Michael Zoran wrote:
> > > On Wed, 2017-01-25 at 16:14 +0300, Dan Carpenter wrote:
> > > > On Wed, Jan 25, 2017 at 04:33:46AM -0800, Michael Zoran wrote:
> > > > > Just for clarity, when you mean staging code? Are you talking
> > > > > about
> > > > > the
> > > > > entire vc04_services driver or are we talking about the
> > > > > vchi_msg_queue
> > > > > function?  
> > > > > 
> > > > > At this point, vc04_services isn't used by anything in the
> > > > > tree at
> > > > > all.
> > > > >  So if you want all the unused code deleted, well that sounds
> > > > > to me
> > > > > like you are talking about the whole driver.
> > > > 
> > > > Yep.  Let's either merge the rest of the code or delete the
> > > > driver.
> > > > What we have now doesn't work.
> > > > 
> > > 
> > > That's Greg and Eric's decision.  
> > > 
> > > The vc04_services was already checked into staging when I was
> > > asked to
> > > submit some ARM64 patches to remove some compiler warnings. My
> > > goal has
> > > always been simply to get ARM64 to work, and I can probably
> > > accomplish
> > > that by submitting changes to the RPI specific tree.
> > > 
> > > I'm just saying that IF the decision was made to continue
> > > upstreaming
> > > these drivers, then I'm willing to continue helping by submitting
> > > improvements.
> > 
> > Either they should be merged, or these apis and structures will be
> > deleted from the in-kernel code, it can't stay as-is.
> 
> FWIW, I've just I've started on the camera driver.

Eric, if you're going to submit the camera driver can you PLEASE,
PLEASE use a modern text editor to fix most of the formating issues out
of the gate?



More information about the linux-rpi-kernel mailing list