Re: [PATCH V2 0/6] iio: mxs-lradc: Clean up and add datasheet names

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

 



On Sunday, July 19, 2015 at 10:49:47 PM, Dan Carpenter wrote:
> On Sun, Jul 19, 2015 at 11:33:43PM +0300, Dan Carpenter wrote:
> > I didn't necessarily read the thread that way, but the thread was
> > long...
> 
> I read the thread as saying that you sometimes get:
> 
> Reviewed-by: Some Person <email@xxxxxxxxxxxxxx>

Hm, I'm trying to use this email as a single point-of-presence, but
maybe I should start separating reviews I do in my free time and
send my R-B and A-B from my private address ?

> With no other comments and what does that even mean because you don't
> know who that person is and if they are good at reviewing.  There isn't
> a solution to that.  But since we know Marek then that doesn't really
> apply here.  Also people were worried that companies were going to start
> counting Reviewed-by tags so we should discourage that.

Uh, OK, I wasn't aware of this.

> Anyway, I don't see Reviewed-by tags as a reward, only as added
> responsibility.

Yeah, that's a good point. If you review code and it turns out to be crap,
then the value of your R-B and A-B decreases.

Best regards,
Marek Vasut
_______________________________________________
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