Thread (22 messages) 22 messages, 3 authors, 2017-01-30

Re: [PATCH v5 5/8] clocksource/drivers/rockchip_timer: split bc_timer into rk_timer and rk_clock_event_device

From: Heiko Stuebner <heiko@sntech.de>
Date: 2017-01-29 20:31:36
Also in: linux-arm-kernel, linux-rockchip, lkml

Hi Alexander,

Am Dienstag, 24. Januar 2017, 18:16:20 CET schrieb Alexander Kochetkov:
quoted
24 янв. 2017 г., в 18:02, Heiko Stübner [off-list ref] написал(а):

Please don't add Reviewed-by tags without explicit mention of them by
reviewers. (Also it's spelled wrong).

I haven't looked that deeply into the driver itself and the changes made
to
feel comfortable with a "Reviewed-by".
Than I posted another linux code I was asked to add such tags (and others)
to add credits to people.

Sorry for that.
no problem. In general people reviewing patches will respond to the patch with 
tags they are comfortable with being added. And a Reviewed-by is a pretty 
strong tag, indicating that some thought went into reviewing a specific patch, 
which I hadn't done with my cursory glance at the time :-)

Heiko

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help