Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
MIME-Version: 1.0
Message-ID: <trinity-85ae3f9c-38f9-4442-98d3-bdc01279c7a8-1631193592256@3c-app-gmx-bs01>
From:   Lino Sanfilippo <LinoSanfilippo@gmx.de>
To:     Vladimir Oltean <olteanv@gmail.com>
Cc:     p.rosenberger@kunbus.com, woojung.huh@microchip.com,
        UNGLinuxDriver@microchip.com, andrew@lunn.ch,
        vivien.didelot@gmail.com, f.fainelli@gmail.com,
        davem@davemloft.net, kuba@kernel.org, netdev@vger.kernel.org,
        linux-kernel@vger.kernel.org
Subject: Aw: Re: [PATCH 0/3] Fix for KSZ DSA switch shutdown
Content-Type: text/plain; charset=UTF-8
Date:   Thu, 9 Sep 2021 15:19:52 +0200
Importance: normal
Sensitivity: Normal
In-Reply-To: <20210909125606.giiqvil56jse4bjk@skbuf>
References: <20210909095324.12978-1-LinoSanfilippo@gmx.de>
 <20210909101451.jhfk45gitpxzblap@skbuf>
 <81c1a19f-c5dc-ab4a-76ff-59704ea95849@gmx.de>
 <20210909114248.aijujvl7xypkh7qe@skbuf>
 <20210909125606.giiqvil56jse4bjk@skbuf>
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

Hi Vladimir, Andrew,

sorry for the late response.

> Gesendet: Donnerstag, 09. September 2021 um 14:56 Uhr
> Von: "Vladimir Oltean" <olteanv@gmail.com>
> An: "Lino Sanfilippo" <LinoSanfilippo@gmx.de>
> Cc: p.rosenberger@kunbus.com, woojung.huh@microchip.com, UNGLinuxDriver@=
microchip.com, andrew@lunn.ch, vivien.didelot@gmail.com, f.fainelli@gmail.=
com, davem@davemloft.net, kuba@kernel.org, netdev@vger.kernel.org, linux-k=
ernel@vger.kernel.org
> Betreff: Re: [PATCH 0/3] Fix for KSZ DSA switch shutdown
>
> On Thu, Sep 09, 2021 at 02:42:48PM +0300, Vladimir Oltean wrote:
> > I feel that something is missing in your system. Is the device link
> > created? Is it deleted before going into effect on shutdown?
>
> So in case my questions were confusing, you can check the presence of
> the device links via sysfs.
>
> On my board, eno2 is the top-level DSA master, there is a switch which
> is PCIe PF 0000:00:00.5 which is its consumer:
>
> ls -la /sys/class/net/eno2/device/consumer\:pci\:0000\:00\:00.5
> lrwxrwxrwx    1 root     root             0 Jan  1 00:00 /sys/class/net/=
eno2/device/consumer:pci:0000:00:00.5 -> ../../../../../virtual/devlink/pc=
i:0000:00:00.2--pci:0000:00:00.5
>
> In turn, that switch is a DSA master on two ports for SPI-attached switc=
hes:
>
> ls -la /sys/class/net/swp0/device/consumer\:spi\:spi2.*
> lrwxrwxrwx    1 root     root             0 Jan  1 00:04 /sys/class/net/=
swp0/device/consumer:spi:spi2.0 -> ../../../../../virtual/devlink/pci:0000=
:00:00.5--spi:spi2.0
> lrwxrwxrwx    1 root     root             0 Jan  1 00:04 /sys/class/net/=
swp0/device/consumer:spi:spi2.1 -> ../../../../../virtual/devlink/pci:0000=
:00:00.5--spi:spi2.1
>
> Do you see similar things on your 5.10 kernel?

For the master device is see

lrwxrwxrwx 1 root root 0 Sep  9 14:10 /sys/class/net/eth0/device/consumer:=
spi:spi3.0 -> ../../../virtual/devlink/platform:fd580000.ethernet--spi:spi=
3.0


>
> Please note that I don't think that particular patch with device links
> was backported to v5.10, at least I don't see it when I run:


> git tag --contains  07b90056cb15f
>
> So how did it reach your tree?
>

The kernel I use is the Raspberry Pi 5.10 kernel. The commit number in thi=
s kernel is d0b97c8cd63e37e6d4dc9fefd6381b09f6c31a67

Andrew: the switch is not on a hat, the device tree part I use is:


       spi@7e204c00 {
            cs-gpios =3D <0x0000000f 0x00000010 0x00000001 0x0000000f 0x00=
000012 0x00000001>;
            pinctrl-0 =3D <0x000000e5 0x000000e6>;
            pinctrl-names =3D "default";
            compatible =3D "brcm,bcm2835-spi";
            reg =3D <0x7e204c00 0x00000200>;
            interrupts =3D <0x00000000 0x00000076 0x00000004>;
            clocks =3D <0x00000007 0x00000014>;
            #address-cells =3D <0x00000001>;
            #size-cells =3D <0x00000000>;
            status =3D "okay";
            phandle =3D <0x000000be>;
            tpm@1 {
                phandle =3D <0x000000ed>;
                status =3D "okay";
                interrupts =3D <0x0000000a 0x00000008>;
                #interrupt-cells =3D <0x00000002>;
                interrupt-parent =3D <0x0000000f>;
                spi-max-frequency =3D <0x000f4240>;
                reg =3D <0x00000001>;
                pinctrl-0 =3D <0x000000e7>;
                pinctrl-names =3D "default";
                compatible =3D "infineon,slb9670";
            };
            ksz9897@0 {
                phandle =3D <0x000000ec>;
                status =3D "okay";
                reset-gpios =3D <0x000000e1 0x0000000d 0x00000001>;
                spi-cpol;
                spi-cpha;
                spi-max-frequency =3D <0x01f78a40>;
                reg =3D <0x00000000>;
                compatible =3D "microchip,ksz9897";
                ports {
                    #size-cells =3D <0x00000000>;
                    #address-cells =3D <0x00000001>;
                    port@2 {
                        label =3D "piright";
                        reg =3D <0x00000002>;
                    };
                    port@1 {
                        label =3D "pileft";
                        reg =3D <0x00000001>;
                    };
                    port@0 {
                        ethernet =3D <0x000000d7>;
                        label =3D "cpu";
                        reg =3D <0x00000000>;
                    };
                };
            };

Regards,
Lino