回复: [PATCH v3 00/10] Enable USB and ENETC on i.MX943 EVK

[Alice Guo (OSS)]

> -----邮件原件-----
> 发件人: Tom Rini <trini at konsulko.com>
> 发送时间: 2025年10月22日 3:00
> 收件人: Fabio Estevam <festevam at gmail.com>
> 抄送: Alice Guo (OSS) <alice.guo at oss.nxp.com>; u-boot at lists.denx.de;
> dl-uboot-imx <uboot-imx at nxp.com>; Marek Vasut
> <marek.vasut at mailbox.org>; Marek Vasut <marex at denx.de>; Mattijs
> Korpershoek <mkorpershoek at kernel.org>; Patrice Chotard
> <patrice.chotard at foss.st.com>; Patrick Rudolph
> <patrick.rudolph at 9elements.com>; Alice Guo <alice.guo at nxp.com>; Stefano
> Babic <sbabic at nabladev.com>; Peng Fan <peng.fan at nxp.com>; Ye Li
> <ye.li at nxp.com>; Jacky Bai <ping.bai at nxp.com>; Jindong Yue
> <jindong.yue at nxp.com>; Marek Vasut <marek.vasut+renesas at mailbox.org>;
> Frank Li <frank.li at nxp.com>; Simon Glass <sjg at chromium.org>; Lukasz
> Majewski <lukma at denx.de>; Minkyu Kang <mk7.kang at samsung.com>; Sam
> Protsenko <semen.protsenko at linaro.org>; Minda Chen
> <minda.chen at starfivetech.com>; tharvey at gateworks.com; Paul Barker
> <paul.barker.ct at bp.renesas.com>; Roger Quadros <rogerq at kernel.org>; Joe
> Hershberger <joe.hershberger at ni.com>; Ramon Fried <rfried.dev at gmail.com>;
> Jerome Forissier <jerome.forissier at linaro.org>; Hanyuan Zhao
> <hanyuan-z at qq.com>; Ilias Apalodimas <ilias.apalodimas at linaro.org>; Weijie
> Gao <weijie.gao at mediatek.com>; Andrew Goodbody
> <andrew.goodbody at linaro.org>
> 主题: Re: [PATCH v3 00/10] Enable USB and ENETC on i.MX943 EVK
> 
> On Tue, Oct 21, 2025 at 03:41:54PM -0300, Fabio Estevam wrote:
> > Hi Alice,
> >
> > On Mon, Oct 13, 2025 at 3:59 AM <alice.guo at oss.nxp.com> wrote:
> > >
> > > Changes for v1:
> > >  - Adding USB2 and USB3 device tree nodes with required PHY and
> > >    regulator support.
> > >  - Enabling ENETC1 and ENETC2.
> > >  - Updating Kconfig and DTSI files to support these interfaces.
> > >
> > > Changes for v2:
> > >  - Resend patch series with updated To list.
> > >
> > > Changes for v3:
> > >  - Remove logically dead code in imx94_link_config().
> >
> > This series does not pass CI:
> >
> > https://source.denx.de/u-boot/custodians/u-boot-imx/-/jobs/1270141
> >
> > As I've mentioned multiple times, please run the series through CI on
> > your side first.
> >
> > This is becoming a pattern with the NXP folks' submissions, and this
> > needs to improve.
> 
> I agree and I want to link to
> https://docs.u-boot.org/en/latest/develop/ci_testing.html#azure-pipelines
> here as well because it will be better if everyone has their own Azure pipeline
> and doesn't get bottlenecked. We can also work out setting up contributor
> repositories in our gitlab if that would help, and run CI there.
> 
> --
Hi,

Thanks for the feedback. I think the patch set I sent may still receive comments from other reviewers, and it might take several rounds before it is accepted. My plan was to run CI testing on the later versions. I will be more mindful of this next time and consider setting up CI earlier in the process.

Thanks again,
Alice Guo