radxa-pkg / radxa-overlays

Additional device tree overlays to support different hardwares on Radxa products
GNU General Public License v2.0
39 stars 28 forks source link

SPI1 could not be found on RSETUP menu for overlays #270

Open sriram-rs opened 6 months ago

sriram-rs commented 6 months ago

I am using ROCKPIS running 6.1.68-1-stable. Thanks to @RadxaYuntian .for the rc image

I am using rsetup to configure the overlays.

I can see that the SPI2 is provisioned to be enabled and disabled. But could not see SPI1 on the menu. How do I get SPI1 to be configured and working....

RadxaYuntian commented 6 months ago

@RadxaMitchell disabled them in #222 since the upstream kernel does not support the pinmux used by SPI1.

Please see if you can add the missing pinctrl data. For example I have done this for ZERO.

sriram-rs commented 6 months ago

@RadxaYuntian and @RadxaMitchell Is it a standard thing to use SPI2 with 6.1.68 ? I don't want to make this a custom change or patch the image to make the change, build it. Prefer to use the image as is.

Please advise.

RadxaYuntian commented 6 months ago

The overlay was disabled back then due to missing kernel support. I have tasked Mitchell to investigate if we can add this feature to the current kernel. If that was possible we will publish an updated kernel package, and you can simply update your system to receive this feature.

I'll let you know the result of our investigation.

RadxaMitchell commented 6 months ago

@sriram-rs Since the upstream 6.1.68 kernel may be missing the configuration of SPI1 related pins, I will try to add these configurations in the next few days, and I will notify you as soon as they are available.

sriram-rs commented 6 months ago

The overlay was disabled back then due to missing kernel support. I have tasked Mitchell to investigate if we can add this feature to the current kernel. If that was possible we will publish an updated kernel package, and you can simply update your system to receive this feature.

I'll let you know the result of our investigation.

Thanks very much @RadxaYuntian. !!

sriram-rs commented 6 months ago

@sriram-rs Since the upstream 6.1.68 kernel may be missing the configuration of SPI1 related pins, I will try to add these configurations in the next few days, and I will notify you as soon as they are available.

@RadxaMitchell Thanks. That would be great!!