Page MenuHomeFreeBSD

correct spigen-rpi2.dtso, add spigen-rpi3.dtso
ClosedPublic

Authored by bobf_mrp3.com on Jul 1 2018, 10:25 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Mar 24, 3:28 AM
Unknown Object (File)
Fri, Mar 22, 11:22 AM
Unknown Object (File)
Sun, Mar 17, 2:23 AM
Unknown Object (File)
Wed, Mar 6, 11:28 PM
Unknown Object (File)
Feb 15 2024, 11:08 PM
Unknown Object (File)
Feb 15 2024, 11:08 PM
Unknown Object (File)
Feb 15 2024, 11:08 PM
Unknown Object (File)
Feb 15 2024, 11:08 PM

Details

Summary

spigen-rpi2.dtso had an entry for CS=2 that actually does not work, so it has been removed. it now looks exactly like spigen-rpi-b.dtso except for the compatibility string.

Also adding spigen-rpi3.dtso with a compatibility string for the RPi 3.

Test Plan

verify operation on all 3 platforms, RPi 1 model , RPi 2, RPi 3

(verified on RPi 2 by copying the new as-built overlay and booting on slightly earlier kernel)

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

cosmetic things, added the Makefile line to build the rpi3 patch

bobf_mrp3.com removed 1 blocking reviewer(s): manu.

Looks good to me. It would be good to get this into -12 release

We tested the spigen-rpi3.dtso overlay and it seems to work fine.

is it too late to commit this to 12?

I forgot to mention, someone suggested modifying everything so that only a single overlay is used for all 3 RPi versions. It would require that the compat strings in the single overlay include all 3 of the driver specs. In the past when I have attempted multiple compat strings, the overlay wouldn't load. It is possible I was doing something else wrong, however. Not sure if this should become a blocking issue or not.

This revision was not accepted when it landed; it landed in state Needs Review.Jan 16 2019, 1:08 AM
This revision was automatically updated to reflect the committed changes.