From: kernel test robot <lkp@intel.com>
To: Daniel Golle <daniel@makrotopia.org>,
Hauke Mehrtens <hauke@hauke-m.de>, Andrew Lunn <andrew@lunn.ch>,
Vladimir Oltean <olteanv@gmail.com>,
"David S. Miller" <davem@davemloft.net>,
Eric Dumazet <edumazet@google.com>,
Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>,
Philipp Zabel <p.zabel@pengutronix.de>,
Russell King <linux@armlinux.org.uk>,
linux-kernel@vger.kernel.org
Cc: oe-kbuild-all@lists.linux.dev, netdev@vger.kernel.org,
Andreas Schirm <andreas.schirm@siemens.com>,
Lukas Stockmann <lukas.stockmann@siemens.com>,
Alexander Sverdlin <alexander.sverdlin@siemens.com>,
Peter Christen <peter.christen@siemens.com>,
Avinash Jayaraman <ajayaraman@maxlinear.com>,
Bing tao Xu <bxu@maxlinear.com>, Liang Xu <lxu@maxlinear.com>,
Juraj Povazanec <jpovazanec@maxlinear.com>,
"Fanni (Fang-Yi) Chan" <fchan@maxlinear.com>,
"Benny (Ying-Tsan) Weng" <yweng@maxlinear.com>,
"Livia M. Rosu" <lrosu@maxlinear.com>,
John Crispin <john@phrozen.org>
Subject: Re: [PATCH net-next v2 7/8] net: dsa: lantiq_gswip: store switch API version in priv
Date: Wed, 20 Aug 2025 02:54:39 +0800 [thread overview]
Message-ID: <202508200254.WPms5O1T-lkp@intel.com> (raw)
In-Reply-To: <56f6b06e22b3b6dbfb45ef04a43bac13a8cb02e5.1755564606.git.daniel@makrotopia.org>
Hi Daniel,
kernel test robot noticed the following build warnings:
[auto build test WARNING on net-next/main]
url: https://github.com/intel-lab-lkp/linux/commits/Daniel-Golle/net-dsa-lantiq_gswip-deduplicate-dsa_switch_ops/20250819-094007
base: net-next/main
patch link: https://lore.kernel.org/r/56f6b06e22b3b6dbfb45ef04a43bac13a8cb02e5.1755564606.git.daniel%40makrotopia.org
patch subject: [PATCH net-next v2 7/8] net: dsa: lantiq_gswip: store switch API version in priv
config: m68k-randconfig-r113-20250819 (https://download.01.org/0day-ci/archive/20250820/202508200254.WPms5O1T-lkp@intel.com/config)
compiler: m68k-linux-gcc (GCC) 8.5.0
reproduce: (https://download.01.org/0day-ci/archive/20250820/202508200254.WPms5O1T-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202508200254.WPms5O1T-lkp@intel.com/
sparse warnings: (new ones prefixed by >>)
>> drivers/net/dsa/lantiq_gswip.c:1940:25: sparse: sparse: cast to restricted __le16
>> drivers/net/dsa/lantiq_gswip.c:1940:25: sparse: sparse: cast to restricted __le16
>> drivers/net/dsa/lantiq_gswip.c:1940:25: sparse: sparse: cast to restricted __le16
>> drivers/net/dsa/lantiq_gswip.c:1940:25: sparse: sparse: cast to restricted __le16
vim +1940 drivers/net/dsa/lantiq_gswip.c
1869
1870 static int gswip_probe(struct platform_device *pdev)
1871 {
1872 struct device_node *np, *gphy_fw_np;
1873 struct device *dev = &pdev->dev;
1874 struct gswip_priv *priv;
1875 int err;
1876 int i;
1877 u32 version;
1878
1879 priv = devm_kzalloc(dev, sizeof(*priv), GFP_KERNEL);
1880 if (!priv)
1881 return -ENOMEM;
1882
1883 priv->gswip = devm_platform_ioremap_resource(pdev, 0);
1884 if (IS_ERR(priv->gswip))
1885 return PTR_ERR(priv->gswip);
1886
1887 priv->mdio = devm_platform_ioremap_resource(pdev, 1);
1888 if (IS_ERR(priv->mdio))
1889 return PTR_ERR(priv->mdio);
1890
1891 priv->mii = devm_platform_ioremap_resource(pdev, 2);
1892 if (IS_ERR(priv->mii))
1893 return PTR_ERR(priv->mii);
1894
1895 priv->hw_info = of_device_get_match_data(dev);
1896 if (!priv->hw_info)
1897 return -EINVAL;
1898
1899 priv->ds = devm_kzalloc(dev, sizeof(*priv->ds), GFP_KERNEL);
1900 if (!priv->ds)
1901 return -ENOMEM;
1902
1903 priv->ds->dev = dev;
1904 priv->ds->num_ports = priv->hw_info->max_ports;
1905 priv->ds->priv = priv;
1906 priv->ds->ops = &gswip_switch_ops;
1907 priv->ds->phylink_mac_ops = &gswip_phylink_mac_ops;
1908 priv->dev = dev;
1909 mutex_init(&priv->pce_table_lock);
1910 version = gswip_switch_r(priv, GSWIP_VERSION);
1911
1912 np = dev->of_node;
1913 switch (version) {
1914 case GSWIP_VERSION_2_0:
1915 case GSWIP_VERSION_2_1:
1916 if (!of_device_is_compatible(np, "lantiq,xrx200-gswip"))
1917 return -EINVAL;
1918 break;
1919 case GSWIP_VERSION_2_2:
1920 case GSWIP_VERSION_2_2_ETC:
1921 if (!of_device_is_compatible(np, "lantiq,xrx300-gswip") &&
1922 !of_device_is_compatible(np, "lantiq,xrx330-gswip"))
1923 return -EINVAL;
1924 break;
1925 default:
1926 return dev_err_probe(dev, -ENOENT,
1927 "unknown GSWIP version: 0x%x\n", version);
1928 }
1929
1930 /* bring up the mdio bus */
1931 gphy_fw_np = of_get_compatible_child(dev->of_node, "lantiq,gphy-fw");
1932 if (gphy_fw_np) {
1933 err = gswip_gphy_fw_list(priv, gphy_fw_np, version);
1934 of_node_put(gphy_fw_np);
1935 if (err)
1936 return dev_err_probe(dev, err,
1937 "gphy fw probe failed\n");
1938 }
1939
> 1940 priv->version = le16_to_cpu(version);
1941
1942 /* bring up the mdio bus */
1943 err = gswip_mdio(priv);
1944 if (err) {
1945 dev_err_probe(dev, err, "mdio probe failed\n");
1946 goto gphy_fw_remove;
1947 }
1948
1949 err = dsa_register_switch(priv->ds);
1950 if (err) {
1951 dev_err_probe(dev, err, "dsa switch registration failed\n");
1952 goto gphy_fw_remove;
1953 }
1954
1955 err = gswip_validate_cpu_port(priv->ds);
1956 if (err)
1957 goto disable_switch;
1958
1959 platform_set_drvdata(pdev, priv);
1960
1961 dev_info(dev, "probed GSWIP version %lx mod %lx\n",
1962 (version & GSWIP_VERSION_REV_MASK) >> GSWIP_VERSION_REV_SHIFT,
1963 (version & GSWIP_VERSION_MOD_MASK) >> GSWIP_VERSION_MOD_SHIFT);
1964 return 0;
1965
1966 disable_switch:
1967 gswip_mdio_mask(priv, GSWIP_MDIO_GLOB_ENABLE, 0, GSWIP_MDIO_GLOB);
1968 dsa_unregister_switch(priv->ds);
1969 gphy_fw_remove:
1970 for (i = 0; i < priv->num_gphy_fw; i++)
1971 gswip_gphy_fw_remove(priv, &priv->gphy_fw[i]);
1972 return err;
1973 }
1974
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
next prev parent reply other threads:[~2025-08-19 18:55 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-08-19 1:31 [PATCH net-next v2 0/8] net: dsa: lantiq_gswip: prepare for supporting new features Daniel Golle
2025-08-19 1:31 ` [PATCH net-next v2 1/8] net: dsa: lantiq_gswip: deduplicate dsa_switch_ops Daniel Golle
2025-08-19 1:32 ` [PATCH net-next v2 2/8] net: dsa: lantiq_gswip: prepare for more CPU port options Daniel Golle
2025-08-19 1:33 ` [PATCH net-next v2 3/8] net: dsa: lantiq_gswip: move definitions to header Daniel Golle
2025-08-19 10:50 ` Vladimir Oltean
2025-08-19 11:11 ` Daniel Golle
2025-08-19 12:11 ` Vladimir Oltean
2025-08-19 1:33 ` [PATCH net-next v2 4/8] net: dsa: lantiq_gswip: introduce bitmap for MII ports Daniel Golle
2025-08-19 1:33 ` [PATCH net-next v2 5/8] net: dsa: lantiq_gswip: load model-specific microcode Daniel Golle
2025-08-19 1:33 ` [PATCH net-next v2 6/8] net: dsa: lantiq_gswip: make DSA tag protocol model-specific Daniel Golle
2025-08-19 1:34 ` [PATCH net-next v2 7/8] net: dsa: lantiq_gswip: store switch API version in priv Daniel Golle
2025-08-19 18:54 ` kernel test robot [this message]
2025-08-19 1:34 ` [PATCH net-next v2 8/8] net: dsa: lantiq_gswip: add support for SWAPI version 2.3 Daniel Golle
2025-08-19 10:53 ` [PATCH net-next v2 0/8] net: dsa: lantiq_gswip: prepare for supporting new features Vladimir Oltean
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202508200254.WPms5O1T-lkp@intel.com \
--to=lkp@intel.com \
--cc=ajayaraman@maxlinear.com \
--cc=alexander.sverdlin@siemens.com \
--cc=andreas.schirm@siemens.com \
--cc=andrew@lunn.ch \
--cc=bxu@maxlinear.com \
--cc=daniel@makrotopia.org \
--cc=davem@davemloft.net \
--cc=edumazet@google.com \
--cc=fchan@maxlinear.com \
--cc=hauke@hauke-m.de \
--cc=john@phrozen.org \
--cc=jpovazanec@maxlinear.com \
--cc=kuba@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@armlinux.org.uk \
--cc=lrosu@maxlinear.com \
--cc=lukas.stockmann@siemens.com \
--cc=lxu@maxlinear.com \
--cc=netdev@vger.kernel.org \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=olteanv@gmail.com \
--cc=p.zabel@pengutronix.de \
--cc=pabeni@redhat.com \
--cc=peter.christen@siemens.com \
--cc=yweng@maxlinear.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.