svn commit: r328257 - in head/sys: arm/broadcom/bcm2835 dts/arm modules

[Oleksandr Tymoshenko]

Poul-Henning Kamp (phk at phk.freebsd.dk) wrote:
> --------
> In message <20180130025048.GA76676 at bluezbox.com>, Oleksandr Tymoshenko writes:
> 
> >> You seem to have skipped the bit about "without reboot" ?
> >
> >No I haven't. config.txt is parsed by VC firmware on boot time
> >and pwm.dtbo is applied even before boot loader kicks in. When
> >kernel starts pwm node has pinctl information ready and status
> >property is set to "okay". So if you download next available image,
> >flash and boot it and run kldload bcm2835_pwm it will give you
> >working PWM device out of the box without reboot.
> 
> But only if you add pwm.dtbo to config.txt and reboot, because
> I see nothing indicating that we will put it there in the
> release images we (may start to) produce ?

No. We're going to add them in our release image. That's what
I meant by "working with Glen and Brad". Glen as release
engineer built all those scripts that prepare SoC images.
He works closely with FreeBSD/ARM community and although
I can't speak for him I hope he won't object to the change
that introduces new feature and doesn't violate POLA.
We already install two overlays that are required to get
reasonable boot-time defaults. 

PI3 release image config is here:
http://svn.freebsd.org/base/head/release/arm64/RPI3.conf

Adding pwm.dtbo is one-line change. Adding line to config.txt
is one more line.

RaspBSD is volunteer project by Brad Davis so it's completely
up to him whether to enable PWM by default or not. He knows his
userbase best. All one can do is to make a case and present it
to Brad.

-- 
gonzo