From mboxrd@z Thu Jan 1 00:00:00 1970 From: olof@lixom.net (Olof Johansson) Date: Fri, 5 Oct 2018 10:33:31 -0700 Subject: [GIT PULL] Allwinner H3/H5 changes for 4.20, bis In-Reply-To: References: <20181004193630.lbxyiecochf5cdfq@flea> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Fri, Oct 5, 2018 at 8:43 AM Arnd Bergmann wrote: > > On Thu, Oct 4, 2018 at 9:36 PM Maxime Ripard wrote: > > > > Hi Arnd, Olof, > > > > Here is a second attempt at the previous PR for the H3 and H5 changes, > > hopefully with the right SoB this time. It replaces the previous one. > > Hi Maxime, > > I'm not completely sure about this, we generally try not to rebase the > branches, and the previous version is now deep in the git history of the > next/dt branch. > > Obviously having a missing signoff is also bad, but I think keeping > the old version of your branch is better in this case. Merging the > new branch on top of next/dt would give us a proper signoff chain on > each patch, but would not remove the patches with the missing > signoff. Instead we'd have lots of duplicate commits, which is probably > worse. > > Olof, do you have any other ideas? We now have the history documented on the mailing list, so we can find the history if needed, and the missing signoff was by one of several co-maintainers so it's not ideal but also not catastrophic. We should probably get better at tooling for this instead (a pull request linter using Stephen's script or somesuch). -Olof