| Message ID | 20220706165406.117349-1-tmaimon77@gmail.com |
|---|---|
| Headers | show |
| Series | Introduce Nuvoton Arbel NPCM8XX BMC SoC | expand |
On Fri, Jul 8, 2022 at 8:50 AM Joel Stanley <joel@jms.id.au> wrote: > > Hi Arnd, > > On Wed, 6 Jul 2022 at 16:54, Tomer Maimon <tmaimon77@gmail.com> wrote: > > > > This patchset adds initial support for the Nuvoton > > Arbel NPCM8XX Board Management controller (BMC) SoC family. > > Are you happy with a cross tree new soc branch for this series? If so > I can put them in a branch and get some build coverage before sending > them out. > > (I notice the clock and reset changes are waiting on acks still) > Hi Joel, Yes, I think we should merge the parts that are reviewed already, but I'd leave out the clk (4/16) and reset (9/16) patches in this case. Krzysztof has reviewed the binding changes, so I don't mind having the DT nodes added in the soc tree even if the two drivers are still missing. I would do the same thing for the sunplus SP7021 platform that Qin Jian has been posting, as this is also waiting for a final Ack on the clk driver. I would put both of the new platforms into a single branch in the SoC tree, separate from the usual dt/drivers/soc/defconfig branches. I was already planning to pick npcm8xx up myself, but if you can do a pull request, that would be even better. Arnd
On Fri, Jul 8, 2022 at 3:45 PM Arnd Bergmann <arnd@arndb.de> wrote: > On Fri, Jul 8, 2022 at 8:50 AM Joel Stanley <joel@jms.id.au> wrote: > > Are you happy with a cross tree new soc branch for this series? If so > > I can put them in a branch and get some build coverage before sending > > them out. > > > > (I notice the clock and reset changes are waiting on acks still) > > Yes, I think we should merge the parts that are reviewed already, but I'd > leave out the clk (4/16) and reset (9/16) patches in this case. > > Krzysztof has reviewed the binding changes, so I don't mind having > the DT nodes added in the soc tree even if the two drivers are still > missing. > > I would do the same thing for the sunplus SP7021 platform that > Qin Jian has been posting, as this is also waiting for a final Ack > on the clk driver. > > I would put both of the new platforms into a single branch in the > SoC tree, separate from the usual dt/drivers/soc/defconfig > branches. I was already planning to pick npcm8xx up myself, > but if you can do a pull request, that would be even better. I see there is now a v9 of the series, let me know if I should apply some or all of those, or if I should wait for a pull request from you. I've just merged the Sunplus sp7021 support leaving out the clock driver in order to make progress on that one, and we can do the same thing here if there is still ongoing discussion about some of the drivers. It would be nice to not wait too long though, as we are already past -rc7. Arnd
Hi Arnd, Appreciate you taking care of this! Are these questions direct to me or Joel? On our side we will be happy that you will start to merge Arbel NPCM8XX without the clock driver, hopefully the NPCM8XX clock driver will ACK soon and maybe you can merge it as well. What do you say Joel? Thanks a lot Tomer On Mon, 18 Jul 2022 at 14:28, Arnd Bergmann <arnd@arndb.de> wrote: > > On Fri, Jul 8, 2022 at 3:45 PM Arnd Bergmann <arnd@arndb.de> wrote: > > On Fri, Jul 8, 2022 at 8:50 AM Joel Stanley <joel@jms.id.au> wrote: > > > Are you happy with a cross tree new soc branch for this series? If so > > > I can put them in a branch and get some build coverage before sending > > > them out. > > > > > > (I notice the clock and reset changes are waiting on acks still) > > > > Yes, I think we should merge the parts that are reviewed already, but I'd > > leave out the clk (4/16) and reset (9/16) patches in this case. > > > > Krzysztof has reviewed the binding changes, so I don't mind having > > the DT nodes added in the soc tree even if the two drivers are still > > missing. > > > > I would do the same thing for the sunplus SP7021 platform that > > Qin Jian has been posting, as this is also waiting for a final Ack > > on the clk driver. > > > > I would put both of the new platforms into a single branch in the > > SoC tree, separate from the usual dt/drivers/soc/defconfig > > branches. I was already planning to pick npcm8xx up myself, > > but if you can do a pull request, that would be even better. > > I see there is now a v9 of the series, let me know if I should > apply some or all of those, or if I should wait for a pull request > from you. > > I've just merged the Sunplus sp7021 support leaving out the > clock driver in order to make progress on that one, and > we can do the same thing here if there is still ongoing discussion > about some of the drivers. It would be nice to not wait too long > though, as we are already past -rc7. > > Arnd
On Mon, Jul 18, 2022 at 3:12 PM Tomer Maimon <tmaimon77@gmail.com> wrote: > > Hi Arnd, > > Appreciate you taking care of this! > > Are these questions direct to me or Joel? > > On our side we will be happy that you will start to merge Arbel > NPCM8XX without the clock driver, hopefully the NPCM8XX clock driver > will ACK soon and maybe you can merge it as well. > > What do you say Joel? As discussed off-list, I have now merged the series into the soc tree as part of the arm/newsoc branch directly, with the exception of the clk driver. No need to resend the patches I merged for future versions, if any changes are required before the merge window, please send them directly to soc@kernel.org. After that, it's best to continue working with Joel so he can merge and forward future patches. Regarding the clk driver, please make sure this applies cleanly on top of what I just merged. This can be applied to either the soc tree at the moment, or the common-clk tree in the future if it misses the merge window. Arnd