Page MenuHomeFreeBSD

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

Authored by on Jul 1 2018, 10:25 PM.



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

rS FreeBSD src repository
Lint Skipped
Unit Tests Skipped

Event Timeline

cosmetic things, added the Makefile line to build the rpi3 patch 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.