On 12/6/20 9:36 PM, Sven Schuchmann wrote:
>> -----Ursprüngliche Nachricht-----
>> Von: Marc Kleine-Budde <mkl@xxxxxxxxxxxxxx>
>> Gesendet: Sonntag, 6. Dezember 2020 21:15
>>>> As it's now working in general, Sven can test from setup under Linux.
>>>
>>> if I try to load under linux with:
>>> sudo dtoverlay mcp251xfd spi0-0 interrupt=25
>>>
>>> I get:
>>> [ 36.154548] CAN device driver interface
>>> [ 36.158644] mcp251xfd spi0.0: No Oscillator (clock) defined.
>>
>> The clock is missing. Let's see if it is added to the DT.
>>
>> Do a diff of the DT before and after applying the overlay.
>>
>> | sudo dtc -I fs /proc/device-tree -o before
>> | sudo dtoverlay mcp251xfd spi0-0 interrupt=25
>> | sudo dtc -I fs /proc/device-tree -o after
>> | diff -u before after
>>
>> Send the output of the diff.
> Here is the diff:
> --- before 2020-12-06 20:31:40.430361119 +0000
> +++ after 2020-12-06 20:32:35.939656243 +0000
> @@ -35,6 +35,13 @@
> clock-frequency = < 0x337f980 >;
> };
>
> + mcp251xfd-spi0-0-osc {
> + compatible = "fixed-clock";
> + #clock-cells = < 0x00 >;
> + phandle = < 0xde >;
^^^^^^^^
> + clock-frequency = < 0x2625a00 >;
> + };
> +
> clk-usb {
> compatible = "fixed-clock";
> #clock-cells = < 0x00 >;
> @@ -1029,6 +1036,12 @@
> brcm,function = < 0x04 >;
> };
>
> + mcp251xfd_spi0_0_pins {
> + brcm,pins = < 0x19 >;
> + phandle = < 0xdd >;
> + brcm,function = < 0x00 >;
> + };
> +
> uart1_pins {
> brcm,pins;
> phandle = < 0x15 >;
> @@ -1546,6 +1559,18 @@
> dmas = < 0x0b 0x06 0x0b 0x07 >;
> pinctrl-names = "default";
>
> + mcp251xfd@0 {
> + compatible = "microchip,mcp251xfd";
> + clocks = < 0xde >;
^^^^^^^^
That looks good so far.
> + interrupt-parent = < 0x0f >;
> + interrupts = < 0x19 0x08 >;
> + phandle = < 0xdf >;
> + reg = < 0x00 >;
> + pinctrl-0 = < 0xdd >;
> + spi-max-frequency = < 0x1312d00 >;
> + pinctrl-names = "default";
> + };
> +
> spidev@1 {
> compatible = "spidev";
> #address-cells = < 0x01 >;
> @@ -1557,6 +1582,7 @@
>
> spidev@0 {
> compatible = "spidev";
> + status = "disabled";
> #address-cells = < 0x01 >;
> #size-cells = < 0x00 >;
> phandle = < 0xa6 >;
Is the clock detcted and loaded by the clock framework?
Can you create a diff for "/sys/kernel/debug/clk/clk_summary" before and after
loading the overlay?
regards,
Marc
--
Pengutronix e.K. | Marc Kleine-Budde |
Embedded Linux | https://www.pengutronix.de |
Vertretung West/Dortmund | Phone: +49-231-2826-924 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
Attachment:
signature.asc
Description: OpenPGP digital signature
