Wednesday, March 2, 2022

[fedora-arm] Re: Change in kernel PPS GPIO handling?

Hi Peter,

Am 02.03.22 um 14:37 schrieb Peter Robinson:
> Hi Stefan,
>
>> Am 02.03.22 um 10:51 schrieb Peter Robinson:
>>>> I have an RPi4 running Fedora 35. It hadn't been updated in a while, so
>>>> I applied updates today. When I boot to the kernel 5.16.11-200, I lose
>>>> the PPS device from my GPS hat. Boot back to 5.14.16-301.fc35 and it
>>>> works.
>>>>
>>>> I'm booting with EFI firmware, and with a device tree config.txt that
>>> All our firmware are EFI, that's the only way we support booting, is
>>> it the default U-Boot based one or are you using the edk2 one?
>>>
>>>> has "dtoverlay=pps-gpio,gpiopin=4" in it. I removed the /boot/dtb
>>>> symlink and set /etc/u-boot.conf to not re-add it.
>>>>
>>>> When I boot 5.16, I see /proc/device-tree, and it has
>>>> /proc/device-tree/pps@4 in it (and the contents look correct), but
>>>> loading the pps-gpio kernel module just gives:
>>>>
>>>> pps-gpio: probe of pps@4 failed with error -22
>>> Any chance you can tell us which kernel it started with, 5.14.x to
>>> 5.16.x is a big window to debug and looking at kernel logs for
>>> drivers/pps there's been no changes in that space in the 5.15+ kernels
>>> at all.
>>>
>>> I wonder if it's a change/regression in the firmware overlay <->
>>> kernel interface.
>> do you use the DTB file from 5.14.x or 5.16x?
> I'm guessing this is fixed in the RPi firmware DTs with their rebase to 5.15.x?
>
> I've not pushed newer firmware/DT to stable Fedora of late as we've
> had a bunch of regressions on various RPi devices, currently upstream
> firmware on Fedora doesn't boot on any 32 bit RPi device for example.
Please never ever use the DTS from the Raspberry Pi tree in combination
with the mainline kernel. Please use the DTS from mainline tree. Any
other combination is not tested and has unexpected results.
>
>> I guess this is related to this commit:
>>
>> https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?h=linux-5.16.y&id=266423e60ea1b953fcc0cd97f3dad85857e434d1
>>
>> Unfortunately this is a change which requires this DTS fix:
>>
>> https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?h=linux-5.16.y&id=c8013355ead68dce152cf426686f8a5f80d88b40
>>
>> So DTB and kernel must "match".
>>
>>> _______________________________________________
>>> arm mailing list -- arm@lists.fedoraproject.org
>>> To unsubscribe send an email to arm-leave@lists.fedoraproject.org
>>> Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
>>> List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
>>> List Archives: https://lists.fedoraproject.org/archives/list/arm@lists.fedoraproject.org
>>> Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure
_______________________________________________
arm mailing list -- arm@lists.fedoraproject.org
To unsubscribe send an email to arm-leave@lists.fedoraproject.org
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/arm@lists.fedoraproject.org
Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure

No comments:

Post a Comment