Re: [PATCH 2/2] max9286: Add MAX9286 driver

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

 



Hi Kieran,

Thanks for your feedback.

On 2019-08-09 09:09:43 +0100, Kieran Bingham wrote:
> Hi Niklas,
> 
> This should be at least v5.

I don't agree ;-) This is a "new" series where multiple streams are not 
supported and there are no external dependencies. To indicate this I 
reset the version. I don't feel strongly about this next submission can 
remedy this if you do feel strongly about it.

> 
> Did you take the last v4 and work from there?
> I have made changes since the last posting. Did you get an update from
> my branches?
> 
> What changes have you made to this posting compared to whichever
> patch-base you have taken to start from?

I took my latest known good state and diffed it with all gmsl branches i 
could find picked what seamed most recent. Then I removed multiplexed 
stream support, fixed a few todos in error paths to clean up notifiers 
and unified naming of the private data structure.

-- 
Regards,
Niklas Söderlund



[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux