Re: [PATCH] iio: max5481: Add support for Maxim digital potentiometers

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

 



On 07/01/17 12:28, Slawomir Stepien wrote:
> On Jan 03, 2017 11:37, Matthew Weber wrote:
>> Thanks for the review.  At this time we aren't in a position to spend
>> time on updates but just wanted to share what we pulled together for
>> others to start from.  I'll mark this as rejected.
> 
> In that case I would like to continue in resubmitting this driver. 
> I do not have the chip on me, but I will try to do my best to analyze it on my
> signal analyzer ;)
Excellent.
> 
> Any hints for me regarding resubmitting drivers created by someone else
> (copyright, authors, etc issues)?
> 
Basically you keep the From and initial sign off as per the original submission then
add your own sign off afterwards.  If the changes you make are significant then it is
fair enough to add your own copyright notice (maintaining any already in place as well).

Easiest thing is to apply the original patch then just make changes using git commit --amend
as that will keep everything correct.

Looking forward to updated driver.  I'm could take this into drivers/staging/iio but
as you are working on it there is probably little point and it would be cleaner to take
it directly into drivers/iio.

Thanks,

Jonathan
--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux