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 12:27 -0800, Eric Anholt wrote:
> Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx> 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?
_______________________________________________
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