From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Sun, 7 Aug 2016 09:31:56 -0400 Subject: [U-Boot] [PATCH 0/2] rockchip: Initial RK3368 and GeekBox support In-Reply-To: <14672ec5-d381-74e8-d693-96e821b80aad@suse.de> References: <1468803967-29233-1-git-send-email-afaerber@suse.de> <14672ec5-d381-74e8-d693-96e821b80aad@suse.de> Message-ID: <20160807133156.GL9942@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Sat, Aug 06, 2016 at 06:05:29PM +0200, Andreas F?rber 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 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 > >> Cc: Kever Yang > >> Cc: Heiko St?bner > >> > >> 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...? > > > 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. In the U-Boot community, we are not in the habit of cc'ing everyone with a change in a given pull request. Is there a tool the kernel folks use here that makes this easy? And the rule of thumb that I use, and I try and get everyone else to use as well is that a patch should be out for a week before it gets picked up and merged as that should give everyone time to review, comment and test. Did that not happen with the patches Simon picked up? Thanks! -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: Digital signature URL: