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 Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux