6 comments:
Can more common code be used?
On architecure, mt8192 is different from mt8183 or mt8173, so no common code can use now.
Taking (copied(?)) from where?
mt8192 is a new design, is not copied from mt8183 or mt8173.
File src/soc/mediatek/mt8192/include/soc/dramc_pi_api.h:
Patch Set #6, Line 128: DQS_8PH_DEGREE_45,
Sort it?
Ack
Patch Set #6, Line 157: u8 chn;
Why not unsigned int? The architecture native type has to be used anyway?
The size of structure member should be 1 or 2 or 4 bytes.
Eg: mode register as above "struct mr_values".
Another way, u8 is enough form channel info.
For portable, if unsigned int, maybe we can't assure the size by different compiler.
Patch Set #6, Line 158: u8 rank;
Ditto?
As above.
Patch Set #6, Line 159: u8 fsp;
Please add a comment, what *fsp* is.
Ack
To view, visit change 44699. To unsubscribe, or for help writing mail filters, visit settings.