Message ID | 20240705-bspimx8m-3180-v2-1-dc1a7e1689be@phytec.de (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | [v2] arm64: dts: freescale: imx8mp-phycore: Add no-eth overlay | expand |
Hi Benjamin, On Fri, Jul 5, 2024 at 9:14 AM Benjamin Hahn <B.Hahn@phytec.de> wrote: > > Add a devicetree overlay to disable ethernet for boards where it is not > populated. > > Signed-off-by: Benjamin Hahn <B.Hahn@phytec.de> > --- > Changes in v2: > - Remove the compatible from the overlay Reviewed-by: Fabio Estevam <festevam@gmail.com>
On 05/07/2024 14:14, Benjamin Hahn wrote: > Add a devicetree overlay to disable ethernet for boards where it is not > populated. > > Signed-off-by: Benjamin Hahn <B.Hahn@phytec.de> > --- > Changes in v2: That's some old kernel you are developing on, right? <form letter> Please use scripts/get_maintainers.pl to get a list of necessary people and lists to CC (and consider --no-git-fallback argument). It might happen, that command when run on an older kernel, gives you outdated entries. Therefore please be sure you base your patches on recent Linux kernel. Tools like b4 or scripts/get_maintainer.pl provide you proper list of people, so fix your workflow. Tools might also fail if you work on some ancient tree (don't, instead use mainline) or work on fork of kernel (don't, instead use mainline). Just use b4 and everything should be fine, although remember about `b4 prep --auto-to-cc` if you added new patches to the patchset. </form letter> Best regards, Krzysztof
On 08.07.24 12:46, Krzysztof Kozlowski wrote: > On 05/07/2024 14:14, Benjamin Hahn wrote: >> Add a devicetree overlay to disable ethernet for boards where it is not >> populated. >> >> Signed-off-by: Benjamin Hahn <B.Hahn@phytec.de> >> --- >> Changes in v2: > That's some old kernel you are developing on, right? You are correct. I was still on Kernel version 6.8. I rebased to master and sent a v3. Regards, Benjamin > > <form letter> > Please use scripts/get_maintainers.pl to get a list of necessary people > and lists to CC (and consider --no-git-fallback argument). It might > happen, that command when run on an older kernel, gives you outdated > entries. Therefore please be sure you base your patches on recent Linux > kernel. > > Tools like b4 or scripts/get_maintainer.pl provide you proper list of > people, so fix your workflow. Tools might also fail if you work on some > ancient tree (don't, instead use mainline) or work on fork of kernel > (don't, instead use mainline). Just use b4 and everything should be > fine, although remember about `b4 prep --auto-to-cc` if you added new > patches to the patchset. > </form letter> > > Best regards, > Krzysztof >
diff --git a/arch/arm64/boot/dts/freescale/Makefile b/arch/arm64/boot/dts/freescale/Makefile index 2cb0212b63c6..16e06e6f055c 100644 --- a/arch/arm64/boot/dts/freescale/Makefile +++ b/arch/arm64/boot/dts/freescale/Makefile @@ -164,6 +164,8 @@ dtb-$(CONFIG_ARCH_MXC) += imx8mp-evk.dtb dtb-$(CONFIG_ARCH_MXC) += imx8mp-icore-mx8mp-edimm2.2.dtb dtb-$(CONFIG_ARCH_MXC) += imx8mp-msc-sm2s-ep1.dtb dtb-$(CONFIG_ARCH_MXC) += imx8mp-phyboard-pollux-rdk.dtb +imx8mp-phyboard-pollux-rdk-no-eth-dtbs += imx8mp-phyboard-pollux-rdk.dtb imx8mp-phycore-no-eth.dtbo +dtb-$(CONFIG_ARCH_MXC) += imx8mp-phyboard-pollux-rdk-no-eth.dtb dtb-$(CONFIG_ARCH_MXC) += imx8mp-skov-revb-hdmi.dtb dtb-$(CONFIG_ARCH_MXC) += imx8mp-skov-revb-lt6.dtb dtb-$(CONFIG_ARCH_MXC) += imx8mp-skov-revb-mi1010ait-1cp1.dtb diff --git a/arch/arm64/boot/dts/freescale/imx8mp-phycore-no-eth.dtso b/arch/arm64/boot/dts/freescale/imx8mp-phycore-no-eth.dtso new file mode 100644 index 000000000000..5f0278bf61ee --- /dev/null +++ b/arch/arm64/boot/dts/freescale/imx8mp-phycore-no-eth.dtso @@ -0,0 +1,16 @@ +// SPDX-License-Identifier: GPL-2.0 +/* + * Copyright (C) 2024 PHYTEC Messtechnik GmbH + * Author: Cem Tenruh <c.tenruh@phytec.de> + */ + +/dts-v1/; +/plugin/; + +ðphy1 { + status = "disabled"; +}; + +&fec { + status = "disabled"; +};
Add a devicetree overlay to disable ethernet for boards where it is not populated. Signed-off-by: Benjamin Hahn <B.Hahn@phytec.de> --- Changes in v2: - Remove the compatible from the overlay - Link to v1: https://lore.kernel.org/r/20240703-bspimx8m-3180-v1-1-e43e035a9b2a@phytec.de --- arch/arm64/boot/dts/freescale/Makefile | 2 ++ arch/arm64/boot/dts/freescale/imx8mp-phycore-no-eth.dtso | 16 ++++++++++++++++ 2 files changed, 18 insertions(+) --- base-commit: e8f897f4afef0031fe618a8e94127a0934896aba change-id: 20240617-bspimx8m-3180-47bed184a854 Best regards,