Hi, > -----Original Message----- > From: Christophe Leroy <christophe.le...@csgroup.eu> > Sent: 27 May 2022 08:23 > To: Christian Zigotzky <chzigot...@xenosoft.de>; rob.herr...@calxeda.com; > Prabhakar Mahadev Lad <prabhakar.mahadev-lad...@bp.renesas.com> > Cc: Darren Stevens <dar...@stevens-zone.net>; linuxppc-dev <linuxppc- > d...@lists.ozlabs.org>; mad skateman <madskate...@gmail.com>; R.T.Dickinson > <r...@xtra.co.nz>; Christian Zigotzky <i...@xenosoft.de> > Subject: Re: [FSL P50x0] Keyboard and mouse don't work anymore after the > devicetree updates for 5.19 > > Hi > > Le 26/05/2022 à 19:42, Christian Zigotzky a écrit : > > Hello, > > > > My keyboard and mouse don't work anymore with my Cyrus+ board with a > > FSL > > P50x0 PowerPC SoC [1] after the devicetree updates for 5.19 [2]. > > After reverting the devicetree updates, my keyboard and mouse work > > without any problems. > > I figured out that the issue is in the patch for the file platform.c > > [3]. I created a patch for reverting the problematic code. (see > > attachment) > > After reverting the changes with the attached patch, the keyboard and > > mouse work again. > > Please check your changes in the file platform.c [3]. > > > > Thanks, > > Christian > > > > [1] > > https://jpn01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fwiki. > > amiga.org%2Findex.php%3Ftitle%3DX5000&data=05%7C01%7Cprabhakar.mah > > adev-lad.rj%40bp.renesas.com%7C4e9c08d1e3874a34bd4208da3fb1c007%7C53d8 > > 2571da1947e49cb4625a166a4a2a%7C0%7C0%7C637892329912063922%7CUnknown%7C > > TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVC > > I6Mn0%3D%7C3000%7C%7C%7C&sdata=fSABvBDi%2FYlqU1eydQB6%2F4BzxXkqRM0 > > Ln9hdInyTp6w%3D&reserved=0 > > [2] > > https://jpn01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgit. > > kernel.org%2Fpub%2Fscm%2Flinux%2Fkernel%2Fgit%2Ftorvalds%2Flinux.git%2 > > Fcommit%2F%3Fid%3D86c87bea6b42100c67418af690919c44de6ede6e&data=05 > > %7C01%7Cprabhakar.mahadev-lad.rj%40bp.renesas.com%7C4e9c08d1e3874a34bd > > 4208da3fb1c007%7C53d82571da1947e49cb4625a166a4a2a%7C0%7C0%7C6378923299 > > 12063922%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiL > > CJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=ENkjlza0J7xFiI > > aPUwMBxHBIkXJNkT%2BLTZ3xuPz%2B10Q%3D&reserved=0 > > > > [3] > > https://jpn01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgit. > > kernel.org%2Fpub%2Fscm%2Flinux%2Fkernel%2Fgit%2Ftorvalds%2Flinux.git%2 > > Fdiff%2Fdrivers%2Fof%2Fplatform.c%3Fid%3D86c87bea6b42100c67418af690919 > > c44de6ede6e&data=05%7C01%7Cprabhakar.mahadev-lad.rj%40bp.renesas.c > > om%7C4e9c08d1e3874a34bd4208da3fb1c007%7C53d82571da1947e49cb4625a166a4a > > 2a%7C0%7C0%7C637892329912063922%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLj > > AwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&a > > mp;sdata=yEJUK%2BGK2dzWARC5rfhsSSFSwD%2BLZm8aNNHqQhPYP7Y%3D&reserv > > ed=0 > > > Based on your patch I would say the culprit commit is > https://jpn01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.c > om%2Ftorvalds%2Flinux%2Fcommit%2Fa1a2b7125e1079cfcc13a116aa3af3df2f9e002b& > amp;data=05%7C01%7Cprabhakar.mahadev- > lad.rj%40bp.renesas.com%7C4e9c08d1e3874a34bd4208da3fb1c007%7C53d82571da194 > 7e49cb4625a166a4a2a%7C0%7C0%7C637892329912063922%7CUnknown%7CTWFpbGZsb3d8e > yJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7 > C%7C%7C&sdata=ONR1CiaSID6q4%2Fo%2BI6MlPA4ij89BJphQRpEu5tQxvYQ%3D&r > eserved=0 > > commit a1a2b7125e1079cfcc13a116aa3af3df2f9e002b > Author: Lad Prabhakar <prabhakar.mahadev-lad...@bp.renesas.com> > Date: Wed Mar 16 20:06:33 2022 +0000 > > of/platform: Drop static setup of IRQ resource from DT core > > Now that all the DT drivers have switched to platform_get_irq() we > can now > safely drop the static setup of IRQ resource from DT core code. > > With the above change hierarchical setup of irq domains is no longer > bypassed and thus allowing hierarchical interrupt domains to describe > interrupts using "interrupts" DT property. > > Signed-off-by: Lad Prabhakar <prabhakar.mahadev- > lad...@bp.renesas.com> > Acked-by: Marc Zyngier <m...@kernel.org> > Tested-by: Marc Zyngier <m...@kernel.org> > Signed-off-by: Rob Herring <r...@kernel.org> > Link: > https://jpn01.safelinks.protection.outlook.com/?url=https%3A%2F%2Flore.ker > nel.org%2Fr%2F20220316200633.28974-1-prabhakar.mahadev- > lad.rj%40bp.renesas.com&data=05%7C01%7Cprabhakar.mahadev- > lad.rj%40bp.renesas.com%7C4e9c08d1e3874a34bd4208da3fb1c007%7C53d82571da194 > 7e49cb4625a166a4a2a%7C0%7C0%7C637892329912063922%7CUnknown%7CTWFpbGZsb3d8e > yJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7 > C%7C%7C&sdata=ri76vfLpmxe7vFDAlsBjyrSSkuTMz0ydftu3XObLGLA%3D&reser > ved=0 > Looks like the driver which you are using has not been converted to use platform_get_irq(), could you please check that.
Cheers, Prabhakar
