27 comments:
File src/soc/mediatek/mt8192/include/soc/pmif.h:
Use tabs and align all of these comments.
Done
File src/soc/mediatek/mt8192/include/soc/pmif_spi.h:
Patch Set #28, Line 45: #define mtk_iocfg_tl ((struct mtk_iocfg_tl_regs *)IOCFG_TL_BASE)
Why not use a global variable just like mtk_pmicspi_mst?
Done
Patch Set #28, Line 51: #define mtk_modem_temp_share ((struct mtk_modem_temp_share_regs *)SPM_BASE)
Why not use a global variable just like mtk_pmicspi_mst?
remove it because our chip won't use modem feature.
No need for parentheses here.
Done
Use lowercase "x"?
Done
File src/soc/mediatek/mt8192/include/soc/pmif_spmi.h:
No need for parentheses.
Done
File src/soc/mediatek/mt8192/pmif.c:
Patch Set #28, Line 15: unsigned int
int
Done
Patch Set #28, Line 15: unsigned int
Use "long" to be consistent with the signature of stopwatch_init_usecs_expire().
Done
Patch Set #28, Line 16: unsigned int
u32
Done
Patch Set #28, Line 18: unsigned int
u32
Done
Patch Set #28, Line 31: unsigned int
If it's boolean, use "int". Otherwise, consider u32, u16 or u8. Same for other similar arguments.
Done
Patch Set #28, Line 46: if (write == 1)
if (write)
Done
No need for these.
Done
Patch Set #28, Line 55: if (write == 0)
if (!write)
Done
Patch Set #28, Line 74: *data = 0;
Is this for timeout case in pmif_send_cmd()?
yes, it is.
No need for these. Better to write […]
Done
No need for these. Better to write […]
Done
Patch Set #28, Line 175: if (inf == PMIF_SPMI)
Check "mstid < ARRAY_SIZE(pmif_spmi_arb)".
Done
Remove this blank line.
Done
File src/soc/mediatek/mt8192/pmif_spi.c:
void
Done
int
Done
Can we use int?
Done
Can we use int?
Done
Patch Set #28, Line 176: const u32 test_data[30] = {
Add a comment […]
Done
Patch Set #28, Line 206: size_t
int
Done
File src/soc/mediatek/mt8192/pmif_spmi.c:
What does this "b" mean?
Done
},
Done
To view, visit change 45398. To unsubscribe, or for help writing mail filters, visit settings.