linux-kernel - RE: [PATCH v2 2/2] phy: renesas: r8a779f0-ether-serdes: add new step added to latest datasheet

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID:
 <TY4PR01MB14282C38CF2E371F52DB8EF088264A@TY4PR01MB14282.jpnprd01.prod.outlook.com>
Date: Tue, 27 May 2025 11:32:36 +0000
From: Michael Dege <michael.dege@...esas.com>
To: kernel test robot <lkp@...el.com>, Vinod Koul <vkoul@...nel.org>, Kishon
 Vijay Abraham I <kishon@...nel.org>, Yoshihiro Shimoda
	<yoshihiro.shimoda.uh@...esas.com>
CC: "oe-kbuild-all@...ts.linux.dev" <oe-kbuild-all@...ts.linux.dev>,
	Uwe Kleine-König <u.kleine-koenig@...libre.com>,
	"linux-phy@...ts.infradead.org" <linux-phy@...ts.infradead.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH v2 2/2] phy: renesas: r8a779f0-ether-serdes: add new step
 added to latest datasheet

Hello,

> -----Original Message-----
> From: kernel test robot <lkp@...el.com>
> Sent: Tuesday, May 27, 2025 11:41 AM
> To: Michael Dege <michael.dege@...esas.com>; Vinod Koul <vkoul@...nel.org>; Kishon Vijay Abraham I
> <kishon@...nel.org>; Yoshihiro Shimoda <yoshihiro.shimoda.uh@...esas.com>
> Cc: oe-kbuild-all@...ts.linux.dev; Michael Dege <michael.dege@...esas.com>; Uwe Kleine-König
> <u.kleine-koenig@...libre.com>; linux-phy@...ts.infradead.org; linux-kernel@...r.kernel.org
> Subject: Re: [PATCH v2 2/2] phy: renesas: r8a779f0-ether-serdes: add new step added to latest
> datasheet
>
> Hi Michael,
>
> kernel test robot noticed the following build errors:
>
> [auto build test ERROR on a5806cd506af5a7c19bcd596e4708b5c464bfd21]
>
> url:    https://github.com/intel-lab-
> lkp%2Flinux%2Fcommits%2FMichael-Dege%2Fphy-renesas-r8a779f0-ether-serdes-add-USXGMII-mode%2F20250527-
> 150048&data=05%7C02%7Cmichael.dege%40renesas.com%7C8302e9fad5af4162a89e08dd9d02baac%7C53d82571da1947e4
> 9cb4625a166a4a2a%7C0%7C0%7C638839357252567523%7CUnknown%7CTWFpbGZsb3d8eyJFbXB0eU1hcGkiOnRydWUsIlYiOiIw
> LjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdata=%2FImajCm4YD6H2309lF5m
> zmCObCS6%2FpCZMotTI57bteo%3D&reserved=0
> base:   a5806cd506af5a7c19bcd596e4708b5c464bfd21
> patch link:
> https://lore.kernel.org/r/20250527-
> renesas-serdes-update-v2-2-
> ef17c71cd94c%2540renesas.com&data=05%7C02%7Cmichael.dege%40renesas.com%7C8302e9fad5af4162a89e08dd9d02b
> aac%7C53d82571da1947e49cb4625a166a4a2a%7C0%7C0%7C638839357252584964%7CUnknown%7CTWFpbGZsb3d8eyJFbXB0eU
> 1hcGkiOnRydWUsIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdata=
> JzM8KY23Z1AzRSwou%2BkMTRWBkvC6hOaDEDENTTH6GEQ%3D&reserved=0
> patch subject: [PATCH v2 2/2] phy: renesas: r8a779f0-ether-serdes: add new step added to latest
> datasheet
> config: arc-randconfig-001-20250527
> (https://download.01.org/0day-
> ci%2Farchive%2F20250527%2F202505271711.WM48TTTI-
> lkp%40intel.com%2Fconfig&data=05%7C02%7Cmichael.dege%40renesas.com%7C8302e9fad5af4162a89e08dd9d02baac%
> 7C53d82571da1947e49cb4625a166a4a2a%7C0%7C0%7C638839357252595528%7CUnknown%7CTWFpbGZsb3d8eyJFbXB0eU1hcG
> kiOnRydWUsIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdata=8JtB
> v%2BFmcJng5BiSSjnOhSodkQ2dtBH8kQnVQdXBhmo%3D&reserved=0)
> compiler: arc-linux-gcc (GCC) 10.5.0
> reproduce (this is a W=1 build):
> (https://download.01.org/0day-
> ci%2Farchive%2F20250527%2F202505271711.WM48TTTI-
> lkp%40intel.com%2Freproduce&data=05%7C02%7Cmichael.dege%40renesas.com%7C8302e9fad5af4162a89e08dd9d02ba
> ac%7C53d82571da1947e49cb4625a166a4a2a%7C0%7C0%7C638839357252606183%7CUnknown%7CTWFpbGZsb3d8eyJFbXB0eU1
> hcGkiOnRydWUsIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdata=b
> tqrLQmBkVgPbAQ64H1ls6R9%2BQn0WTfNWUVrTApuDjY%3D&reserved=0)
>
> 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@...el.com>
> | Closes:
> | https://lore/
> | .kernel.org%2Foe-kbuild-all%2F202505271711.WM48TTTI-lkp%40intel.com%2F
> | &data=05%7C02%7Cmichael.dege%40renesas.com%7C8302e9fad5af4162a89e08dd9
> | d02baac%7C53d82571da1947e49cb4625a166a4a2a%7C0%7C0%7C63883935725261673
> | 0%7CUnknown%7CTWFpbGZsb3d8eyJFbXB0eU1hcGkiOnRydWUsIlYiOiIwLjAuMDAwMCIs
> | IlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdata=ja
> | r%2FB6U8ENcw71CzP2jduv2gEQrSkOUj%2Fnhf6jcPv5A%3D&reserved=0
>
> All errors (new ones prefixed by >>):

I will fix the issue and send around a new version of the patch set.

>
>    drivers/phy/renesas/r8a779f0-ether-serdes.c: In function 'r8a779f0_eth_serdes_read32':
> >> drivers/phy/renesas/r8a779f0-ether-serdes.c:54:25: error: 'RENESAS_ETH_SERDES_BANK_SELECT'
> undeclared (first use in this function); did you mean 'R8A779F0_ETH_SERDES_BANK_SELECT'?
>       54 |  iowrite32(bank, addr + RENESAS_ETH_SERDES_BANK_SELECT);
>          |                         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>          |                         R8A779F0_ETH_SERDES_BANK_SELECT
>    drivers/phy/renesas/r8a779f0-ether-serdes.c:54:25: note: each undeclared identifier is reported
> only once for each function it appears in
>
>
> vim +54 drivers/phy/renesas/r8a779f0-ether-serdes.c
>
>     51
>     52        static u32 r8a779f0_eth_serdes_read32(void __iomem *addr, u32 offs,  u32 bank)
>     53        {
>   > 54                iowrite32(bank, addr + RENESAS_ETH_SERDES_BANK_SELECT);
>     55
>     56                return ioread32(addr + offs);
>     57        }
>     58
>

Thank you and best regards,

Michael
> --
> 0-DAY CI Kernel Test Service
> https://github.com/intel/lkp-
> tests%2Fwiki&data=05%7C02%7Cmichael.dege%40renesas.com%7C8302e9fad5af4162a89e08dd9d02baac%7C53d82571da
> 1947e49cb4625a166a4a2a%7C0%7C0%7C638839357252626902%7CUnknown%7CTWFpbGZsb3d8eyJFbXB0eU1hcGkiOnRydWUsIl
> YiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdata=s8LGQlWaPw4C3ihZ
> 9X6WiQaZWuxKLPlqd0sGv7T8uw8%3D&reserved=0
________________________________

Renesas Electronics Europe GmbH
Registered Office: Arcadiastrasse 10
DE-40472 Duesseldorf
Commercial Registry: Duesseldorf, HRB 3708
Managing Director: Carsten Jauch
VAT-No.: DE 14978647
Tax-ID-No: 105/5839/1793

Legal Disclaimer: This e-mail communication (and any attachment/s) is confidential and contains proprietary information, some or all of which may be legally privileged. It is intended solely for the use of the individual or entity to which it is addressed. Access to this email by anyone else is unauthorized. If you are not the intended recipient, any disclosure, copying, distribution or any action taken or omitted to be taken in reliance on it, is prohibited and may be unlawful.