Hi Arnd, On Tue, 14 Sept 2021 at 22:54, Arnd Bergmann <arnd@xxxxxxxx> wrote: > Sounds good to me. > > One process issue though: These emails were sent 'cc:soc@xxxxxxxxxx', so > they end up in patchwork, but I'm guessing they were not actually meant > for us to apply from there. I'm happy to take merge these patches (or > a future version of the series) directly from patchwork if that's what you > both had in mind, but for clarity please either > > a) drop soc@xxxxxxxxxx from Cc for patches that are meant for review > and will be part of a pull request. > > or > > b) put soc@xxxxxxxxxx in the 'to:' field after they have been reviewed > and you want them to get merged. Noted for the future. Until now we haven't done pull requests so having the patches go into the soc patchwork made sense but from now on we'll send pull requests when needed to you and Olof for the ARM specific bits. Which I don't think there will be much aside from dts stuff. > Or even better, add an explicit sentence below the '---' line asking > for the patches to be merged directly into the soc tree when this > is what you want. Noted. > FWIW, I'd suggest merging the MAINTAINERS file change as a > bugfix for 5.15, it's generally better to do these as early as possible > to make sure any patches reach all the right people. That makes sense. Would it be possible for you to merge that single commit for me? Seems like that would be easier than me creating a fixes branch for a single commit, sending you a PR, finding out I still don't actually know how to do PR properly.. :) If it's a hassle I'll do the PR though. Cheers, Daniel
