+Tom for comment
Hi Andreas,
On 6 August 2016 at 10:05, Andreas Färber afaerber@suse.de wrote:
Hi Simon,
Am 06.08.2016 um 06:30 schrieb Simon Glass:
Hi Andreas,
On 17 July 2016 at 19:06, Andreas Färber afaerber@suse.de wrote:
Hi,
This series adds initial support for RK3368 SoC and GeekBox. For more details see the commit message.
Will need to be rebased onto Heiko's cleanups and Kever's RK3399 series.
Regards, Andreas
Cc: Simon Glass sjg@chromium.org Cc: Kever Yang kever.yang@rock-chips.com Cc: Heiko Stübner heiko@sntech.de
Andreas Färber (2): dts: Import rk3368-geekbox.dts ARM64: rockchip: Add initial support for RK3368 based GeekBox
Are you planning to respin these patches?
Eventually...?
If it is soon then I can get this in for the upcoming release. Otherwise I'll need to wait. Also can you please add a README with a bit more detail on how to test it? Your cover letter suggests that it requires shorting two pins (which pins?) after flashing U-Boot. Is there another way for me to try it?
I'd like to get them applied soon.
And I'd like to get my work recognized! However, despite our previous IRC chat, I had to find out _while_ replying to the rk3399 mails that you had once again not just applied all patches (twenty minutes after ack'ing them on a Saturday) but already sent a pull on Tuesday my nighttime that I was not CC'ed on and that Tom has merged the night after. So it feels like I'm wasting my time here and consequently I stopped my review and rebase.
Not only that it's too late for the patches themselves, but Wolfgang also publishes statistics of Reviewed-bys, which we can't add to commits after merging. If you as maintainer don't give people who spent time providing serious review comments sufficient chance to add their Reviewed-by - and a weekend of absence is certainly something to cope with since you said you were away end of that week as well - it becomes a distorted, non-telling statistic. So I am kindly suggesting that Wolfgang drops the Reviewed-by statistics if maintainers don't care for them and instead we just all do ugly fix-ups so that we can at least get recognition in the Signed-off-by statistics.
While we're only talking about 4 tags here, it's a matter of principle and respect.
What do you mean by respect?
I normally expect a follow-up review to come through fairly quickly. The last review I saw from you was 10 days ago. I tend to want to move things forward fairly quickly in general for Rockchip, but particularly at this point where the merge window is closed and pending patches really need to get in for testing.
So a few questions: - How long do you expect a patch to sit on the list before I pick it up? - Similarly for a follow-up v2, etc. patch? - How long between my Ack and applying? - Anything else you are looking for?
I seldom copy people on the pull requests - is that something we should be doing?
I'm happy to adjust the rules for Rockchip if it will encourage more participation. This is very new in mainline so there is a lot to do.
Thanks, Andreas
Regards, Simon