From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S966496AbeBOD7e (ORCPT ); Wed, 14 Feb 2018 22:59:34 -0500 Received: from mail-ot0-f194.google.com ([74.125.82.194]:33087 "EHLO mail-ot0-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752371AbeBOD7b (ORCPT ); Wed, 14 Feb 2018 22:59:31 -0500 X-Google-Smtp-Source: AH8x225Pp99FJi7rCsVMmmGoMX1plMkVMMEW3OtXwfprdQXMWkd5U8t8t6938ZMub/VIMWNrDsevaw== Date: Wed, 14 Feb 2018 19:59:25 -0800 In-Reply-To: References: <20180205235757.246758-1-brendanhiggins@google.com> <20180205235757.246758-3-brendanhiggins@google.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Subject: Re: [PATCH v9 2/3] arm: dts: add Nuvoton NPCM750 device tree To: Joel Stanley , Brendan Higgins , Arnd Bergmann , Olof Johansson CC: Rob Herring , Russell King , Mark Rutland , Tomer Maimon , Avi Fishman , julien.thierry@arm.com, Philippe Ombredanne , devicetree , OpenBMC Maillist , Linux Kernel Mailing List , Linux ARM From: Florian Fainelli Message-ID: Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from quoted-printable to 8bit by mail.home.local id w1F3xdpQ014267 On February 14, 2018 6:28:51 PM PST, Joel Stanley wrote: >On Tue, Feb 13, 2018 at 4:52 PM, Joel Stanley wrote: >> Hi Brendan, >> >> On Tue, Feb 6, 2018 at 10:27 AM, Brendan Higgins >> wrote: >>> Add a common device tree for all Nuvoton NPCM750 BMCs and a board >>> specific device tree for the NPCM750 (Poleg) evaluation board. >>> >>> Signed-off-by: Brendan Higgins >>> Reviewed-by: Tomer Maimon >>> Reviewed-by: Avi Fishman >>> Reviewed-by: Joel Stanley >>> Reviewed-by: Rob Herring >>> Tested-by: Tomer Maimon >>> Tested-by: Avi Fishman >> >> This looks well acked, reviewed and tested. How do you plan to have >> the ARM SoC maintainers merge your patches? > >Following up on an IRC conversation with Brendan: > >The process is normally to create a git branch based on on -rc1, apply >your patches, and send them to the ARM maintainers. You then push a >signed tag to a repository somewhere, send a pull request (a git >request-pull email, not a Github PR) and Arnd, Olof or one of the >other maintainers will pull your tree some time before the next merge >window opens. > >Arnd, do we have this documented somewhere for new maintainers to >follow? I would add a few things that we had to go through before for Broadcom SoCs: - send your pull requests to arm@kernel.org and copy Arnd, Olof and Kevin - you would want to get your PGP key signed by as many people as people as possible which should not be a problem if you are in an area with lots of kernel people like the Bay Area (which reminds me I should do that) - if you are going to be reasonably active every cycle consider getting a kernel.org account to host your tree (we are still not doing that...) - for future pull requests, you might want to break them into e.g: DTS, board/Kconfig, drivers, defconfig, maintainers file, and have as little dependencies between each branch to minimize merge conflicts - build test and run test your changes against at least one other platform, e.g: QEMU to check for multiplatform issues In case this is of any value, there is a script here that will automatically generate pull requests emails for you based on branches matching what was mentioned above, it will also take care of CC'ing the people involved in the different patches: https://github.com/ffainelli/misc-scripts/blob/master/gen-pull.pl It still requires you to create an appropriate tag for the pull requests, though I might semi-automate that in the future, at least spawn an editor and offer some guidance, based on commit messages as to what should be in the pull request email/tag. HTH -- Florian From mboxrd@z Thu Jan 1 00:00:00 1970 From: Florian Fainelli Subject: Re: [PATCH v9 2/3] arm: dts: add Nuvoton NPCM750 device tree Date: Wed, 14 Feb 2018 19:59:25 -0800 Message-ID: References: <20180205235757.246758-1-brendanhiggins@google.com> <20180205235757.246758-3-brendanhiggins@google.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Return-path: In-Reply-To: Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Joel Stanley , Brendan Higgins , Arnd Bergmann , Olof Johansson Cc: Rob Herring , Russell King , Mark Rutland , Tomer Maimon , Avi Fishman , julien.thierry-5wv7dgnIgG8@public.gmane.org, Philippe Ombredanne , devicetree , OpenBMC Maillist , Linux Kernel Mailing List , Linux ARM List-Id: devicetree@vger.kernel.org On February 14, 2018 6:28:51 PM PST, Joel Stanley wrot= e: >On Tue, Feb 13, 2018 at 4:52 PM, Joel Stanley wrote: >> Hi Brendan, >> >> On Tue, Feb 6, 2018 at 10:27 AM, Brendan Higgins >> wrote: >>> Add a common device tree for all Nuvoton NPCM750 BMCs and a board >>> specific device tree for the NPCM750 (Poleg) evaluation board=2E >>> >>> Signed-off-by: Brendan Higgins >>> Reviewed-by: Tomer Maimon >>> Reviewed-by: Avi Fishman >>> Reviewed-by: Joel Stanley >>> Reviewed-by: Rob Herring >>> Tested-by: Tomer Maimon >>> Tested-by: Avi Fishman >> >> This looks well acked, reviewed and tested=2E How do you plan to have >> the ARM SoC maintainers merge your patches? > >Following up on an IRC conversation with Brendan: > >The process is normally to create a git branch based on on -rc1, apply >your patches, and send them to the ARM maintainers=2E You then push a >signed tag to a repository somewhere, send a pull request (a git >request-pull email, not a Github PR) and Arnd, Olof or one of the >other maintainers will pull your tree some time before the next merge >window opens=2E > >Arnd, do we have this documented somewhere for new maintainers to >follow? I would add a few things that we had to go through before for Broadcom SoC= s: - send your pull requests to arm@kernel=2Eorg and copy Arnd, Olof and Kevi= n - you would want to get your PGP key signed by as many people as people as= possible which should not be a problem if you are in an area with lots of = kernel people like the Bay Area (which reminds me I should do that) - if you are going to be reasonably active every cycle consider getting a = kernel=2Eorg account to host your tree (we are still not doing that=2E=2E= =2E) - for future pull requests, you might want to break them into e=2Eg: DTS, = board/Kconfig, drivers, defconfig, maintainers file, and have as little dep= endencies between each branch to minimize merge conflicts - build test and run test your changes against at least one other platform= , e=2Eg: QEMU to check for multiplatform issues In case this is of any value, there is a script here that will automatical= ly generate pull requests emails for you based on branches matching what wa= s mentioned above, it will also take care of CC'ing the people involved in = the different patches: https://github=2Ecom/ffainelli/misc-scripts/blob/master/gen-pull=2Epl It still requires you to create an appropriate tag for the pull requests, = though I might semi-automate that in the future, at least spawn an editor a= nd offer some guidance, based on commit messages as to what should be in th= e pull request email/tag=2E HTH --=20 Florian -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 From: f.fainelli@gmail.com (Florian Fainelli) Date: Wed, 14 Feb 2018 19:59:25 -0800 Subject: [PATCH v9 2/3] arm: dts: add Nuvoton NPCM750 device tree In-Reply-To: References: <20180205235757.246758-1-brendanhiggins@google.com> <20180205235757.246758-3-brendanhiggins@google.com> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On February 14, 2018 6:28:51 PM PST, Joel Stanley wrote: >On Tue, Feb 13, 2018 at 4:52 PM, Joel Stanley wrote: >> Hi Brendan, >> >> On Tue, Feb 6, 2018 at 10:27 AM, Brendan Higgins >> wrote: >>> Add a common device tree for all Nuvoton NPCM750 BMCs and a board >>> specific device tree for the NPCM750 (Poleg) evaluation board. >>> >>> Signed-off-by: Brendan Higgins >>> Reviewed-by: Tomer Maimon >>> Reviewed-by: Avi Fishman >>> Reviewed-by: Joel Stanley >>> Reviewed-by: Rob Herring >>> Tested-by: Tomer Maimon >>> Tested-by: Avi Fishman >> >> This looks well acked, reviewed and tested. How do you plan to have >> the ARM SoC maintainers merge your patches? > >Following up on an IRC conversation with Brendan: > >The process is normally to create a git branch based on on -rc1, apply >your patches, and send them to the ARM maintainers. You then push a >signed tag to a repository somewhere, send a pull request (a git >request-pull email, not a Github PR) and Arnd, Olof or one of the >other maintainers will pull your tree some time before the next merge >window opens. > >Arnd, do we have this documented somewhere for new maintainers to >follow? I would add a few things that we had to go through before for Broadcom SoCs: - send your pull requests to arm at kernel.org and copy Arnd, Olof and Kevin - you would want to get your PGP key signed by as many people as people as possible which should not be a problem if you are in an area with lots of kernel people like the Bay Area (which reminds me I should do that) - if you are going to be reasonably active every cycle consider getting a kernel.org account to host your tree (we are still not doing that...) - for future pull requests, you might want to break them into e.g: DTS, board/Kconfig, drivers, defconfig, maintainers file, and have as little dependencies between each branch to minimize merge conflicts - build test and run test your changes against at least one other platform, e.g: QEMU to check for multiplatform issues In case this is of any value, there is a script here that will automatically generate pull requests emails for you based on branches matching what was mentioned above, it will also take care of CC'ing the people involved in the different patches: https://github.com/ffainelli/misc-scripts/blob/master/gen-pull.pl It still requires you to create an appropriate tag for the pull requests, though I might semi-automate that in the future, at least spawn an editor and offer some guidance, based on commit messages as to what should be in the pull request email/tag. HTH -- Florian