am335x_ehrpwm.c remove sysctl interface
ClosedPublic
Actions

Authored by oskar.holmlund_ohdata.se on Apr 2 2021, 9:11 AM.

Details

Reviewers

manu
freebsd-rj_obsigna.com

Group Reviewers

ARM

Commits

rGdbaf4b653976: arm: ti am335x ehrpwm remove sysctl interface

Summary

To minimize the maintenance time of this driver when new features are added I want to remove the legacy sysctl interface.

Upcoming new feature are invert the polarity of the pulse see D29137

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

oskar.holmlund_ohdata.se created this revision.Apr 2 2021, 9:11 AM

Herald added a subscriber: andrew. · View Herald TranscriptApr 2 2021, 9:11 AM

oskar.holmlund_ohdata.se requested review of this revision.Apr 2 2021, 9:11 AM

oskar.holmlund_ohdata.se mentioned this in D29547: am335x_ehrpwm.c add support for flags (PWM_POLARITY_INVERTED).Apr 2 2021, 9:24 AM

I'll leave this for a few days in case someone have an objection.

This revision is now accepted and ready to land.Apr 2 2021, 1:19 PM

Note: I am a first time reviewer, and I hope that I do not something wrong.

I use BeagleBone Blacks in some of my projects, and I am fine with the above changes. I don’t use the deprecated sysctl interface, anyway, it shouldn’t be too difficult to switch to pwmbus.

Closed by commit rGdbaf4b653976: arm: ti am335x ehrpwm remove sysctl interface (authored by oh). · Explain WhyJun 12 2021, 6:36 AM

This revision was automatically updated to reflect the committed changes.

oskar.holmlund_ohdata.se added a commit: rGdbaf4b653976: arm: ti am335x ehrpwm remove sysctl interface.

Revision Contents
Changeset List

Path

Size

sys/

arm/

ti/

am335x/

am335x_ehrpwm.c

224 lines

Diff 90793

View Options

sys/arm/ti/am335x/am335x_ehrpwm.c

am335x_ehrpwm.c remove sysctl interfaceClosedPublicActions