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
F88264155: D16088.diff
Sat, Jul 13, 6:39 AM
Unknown Object (File)
Mon, Jul 1, 4:45 PM
Unknown Object (File)
Fri, Jun 28, 11:53 PM
Unknown Object (File)
Fri, Jun 28, 2:42 PM
Unknown Object (File)
Thu, Jun 27, 3:24 AM
Unknown Object (File)
Sat, Jun 22, 12:27 AM
Unknown Object (File)
Wed, Jun 19, 12:24 PM
Unknown Object (File)
Fri, Jun 14, 10:43 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 Not Applicable
Unit
Tests Not Applicable

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.