Re: [PATCH V2 0/3] Add vchi_queue_kernel_message and vchi_queue_user_message

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 2017-01-25 at 14:33 +0100, Greg KH wrote:
> 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.
> 
> thanks,
> 
> greg k-h

Again, that's not my decision.  I can go either way.  But if the
decision was made to delete it, I think the 4.10 version should also be
deleted because the same logic applies.  Nothing uses it 4.10 either
and 4.10 is still an RC.




_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel




[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux