Re: [PATCH 1/2] ARM: dts: BCM5301X: Explicitly disable unused switch CPU ports

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



From: Florian Fainelli <f.fainelli@xxxxxxxxx>

On Fri, 13 Oct 2023 12:33:13 +0200, Rafał Miłecki <zajec5@xxxxxxxxx> wrote:
> From: Rafał Miłecki <rafal@xxxxxxxxxx>
> 
> When redescribing ports I assumed that missing "label" (like "cpu")
> means switch port isn't used. That was incorrect and I realized my
> change made Linux always use the first (5) CPU port (there are 3 of
> them).
> 
> While above should technically be possible it often isn't correct:
> 1. Non-default switch ports are often connected to Ethernet interfaces
>    not fully covered by vendor setup (they may miss MACs)
> 2. On some devices non-default ports require specifying fixed link
> 
> This fixes network connectivity for some devices. It was reported &
> tested for Netgear R8000. It also affects Linksys EA9200 with its
> downstream DTS.
> 
> Fixes: ba4aebce23b2 ("ARM: dts: BCM5301X: Describe switch ports in the main DTS")
> Signed-off-by: Rafał Miłecki <rafal@xxxxxxxxxx>
> ---

Applied to https://github.com/Broadcom/stblinux/commits/devicetree/next, thanks!
--
Florian

Attachment: smime.p7s
Description: S/MIME Cryptographic Signature


[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux