Page MenuHomeFreeBSD

syscon: Introduce kobj and split out fdt bits
ClosedPublic

Authored by kevans on Dec 17 2017, 3:24 PM.
Tags
None
Referenced Files
F104206273: D13521.diff
Wed, Dec 4, 6:11 PM
Unknown Object (File)
Oct 29 2024, 11:30 AM
Unknown Object (File)
Oct 2 2024, 10:28 AM
Unknown Object (File)
Sep 20 2024, 3:48 PM
Unknown Object (File)
Sep 18 2024, 1:16 AM
Unknown Object (File)
Sep 5 2024, 9:07 PM
Unknown Object (File)
Sep 5 2024, 2:58 AM
Unknown Object (File)
Sep 4 2024, 6:55 PM
Subscribers

Details

Summary

Allow more flexibility by kobj'ifying syscon and splitting out fdt specific bits in preparation of a move to the extres framework. The generic fdt driver has been moved to syscon_generic.c and the fdt requirement has been removed from the syscon interface.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 13627

Event Timeline

Everything else looks OK for me.

sys/dev/syscon/syscon.c
90

I don't think that we needs/wants default methods for read/write/modify.
Its better to return error, if driver don't implement these.

227

And whats if single device register multiple syscons? I think that we can store ofw node to struct syscon and then use it for search. OF_device_from_xref() is not right approach I think.
see https://github.com/strejda/tegra/blob/master/sys/dev/extres/phy/phy.c#L93
and its usage.

Something like this, perhaps?

  • Remove default read/write/modify methods
  • Allow creation of syscon node from specific ofw node not necessarily belonging to provider

Re-add context... whoops.

This revision is now accepted and ready to land.Dec 23 2017, 9:20 AM
This revision was automatically updated to reflect the committed changes.