Page MenuHomeFreeBSD
Differential D34110

qcom_mdio: add initial IPQ4018 MDIO support
ClosedPublic
Actions

Authored by adrian on Jan 30 2022, 9:48 PM.
Tags
None
Referenced Files
F168687032: D34110.id102362.diff
Sat, Aug 29, 3:22 PM
F168656646: D34110.id.diff
Sat, Aug 29, 11:29 AM
F168652025: D34110.id102304.diff
Sat, Aug 29, 10:59 AM
F168643311: D34110.id102137.diff
Sat, Aug 29, 10:04 AM
F168643201: D34110.id102204.diff
Sat, Aug 29, 10:03 AM
F168640450: D34110.id102207.diff
Sat, Aug 29, 9:40 AM
F168627627: D34110.diff
Sat, Aug 29, 8:05 AM
Unknown Object (File)
Sat, Aug 22, 5:26 PM
View All Files
Subscribers
andrew
imp

Details

Reviewers
manu
imp
Group Reviewers
network
Commits
rG29332c0dcee1: qcom_mdio: add initial IPQ4018 MDIO support
Summary

This adds support for the IPQ4018/IPQ4019 MDIO bus. This is used to
talk to external PHYs and switches. (There's an internal switch
in the IPQ4018/IPQ4019 as well, but it's accessible via MMIO/AXI.)

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 44237
Build 41125: arc lint + arc unit

Event Timeline

adrian created this revision.Jan 30 2022, 9:48 PM
Herald added subscribers: andrew, imp. · View Herald TranscriptJan 30 2022, 9:48 PM
adrian requested review of this revision.Jan 30 2022, 9:48 PM
Harbormaster completed remote builds in B44180: Diff 102137.Jan 30 2022, 9:48 PM
adrian added a child revision: D34111: dts: add IPQ4018/IPQ4019 ethernet MAC and ethernet switch definitions.Jan 30 2022, 9:49 PM
adrian mentioned this in D34096: qcom: add initial IPQ4018 MDIO support and etherswitch support.Jan 30 2022, 9:49 PM
adrian added reviewers: manu, imp.
adrian added a reviewer: network.
manu added inline comments.Jan 31 2022, 8:16 AM
sys/dev/qcom_mdio/qcom_mdio_ipq4018.c
295

1/ Does it needs to be that early ? INTERRUPT is really early in the boot, we have BUS_PASS_SUPPORTDEV that usually work for driver like that.
2/ Do you really need to be on simplebus and ofwbus ? In the DTS it's under /soc which is simple-bus compatible so I don't think you need the ofwbus declaration.

manu added a comment.Jan 31 2022, 8:17 AM

mdio and ofw* part looks good, only two questions for me.

adrian updated this revision to Diff 102203.Feb 1 2022, 5:35 AM

Update dates

Harbormaster completed remote builds in B44212: Diff 102203.Feb 1 2022, 5:35 AM
adrian updated this revision to Diff 102204.Feb 1 2022, 5:35 AM

update dates

Harbormaster completed remote builds in B44213: Diff 102204.Feb 1 2022, 5:35 AM
adrian marked an inline comment as done.Feb 1 2022, 5:42 AM
adrian added inline comments.
sys/dev/qcom_mdio/qcom_mdio_debug.h
35

....... whoops, i should fix that

adrian updated this revision to Diff 102207.Feb 1 2022, 5:56 AM

Fix debug

Harbormaster completed remote builds in B44216: Diff 102207.Feb 1 2022, 5:56 AM
adrian updated this revision to Diff 102242.Feb 2 2022, 5:15 AM

address more things

Harbormaster completed remote builds in B44237: Diff 102242.Feb 2 2022, 5:15 AM
manu accepted this revision.Feb 2 2022, 7:35 AM
This revision is now accepted and ready to land.Feb 2 2022, 7:35 AM
adrian updated this revision to Diff 102304.Feb 3 2022, 5:19 AM

update id

This revision now requires review to proceed.Feb 3 2022, 5:19 AM
Harbormaster completed remote builds in B44264: Diff 102304.Feb 3 2022, 5:19 AM
manu accepted this revision.Feb 3 2022, 9:20 AM
This revision is now accepted and ready to land.Feb 3 2022, 9:20 AM
Closed by commit rG29332c0dcee1: qcom_mdio: add initial IPQ4018 MDIO support (authored by adrian). · Explain WhyFeb 4 2022, 5:29 AM
This revision was automatically updated to reflect the committed changes.
adrian added a commit: rG29332c0dcee1: qcom_mdio: add initial IPQ4018 MDIO support.

Revision Contents
Changeset List

PathSize
sys/
arm/
conf/
6 lines
qualcomm/
2 lines
dev/
qcom_mdio/
42 lines
299 lines
47 lines
71 lines

Diff 102242

View Options

sys/arm/conf/std.qca

Loading...
View Options

sys/arm/qualcomm/std.ipq4018

Loading...
View Options

sys/dev/qcom_mdio/qcom_mdio_debug.h

Loading...
View Options

sys/dev/qcom_mdio/qcom_mdio_ipq4018.c

Loading...
View Options

sys/dev/qcom_mdio/qcom_mdio_ipq4018_reg.h

Loading...
View Options

sys/dev/qcom_mdio/qcom_mdio_ipq4018_var.h

Loading...