>On Tue, Jun 20, 2023 at 1:46 PM Shiji Yang <yangshiji66@xxxxxxxxxxx> wrote: >> >> The ralink_of_remap() function is repeated several times on SoC specific >> source files. They have the same structure, but just differ in compatible >> strings. In order to make commonly use of these codes, this patch >> introduces a newly designed mtmips_of_remap_node() function to match and >> remap all supported system controller and memory controller nodes. >> >> Build and run tested on MT7620 and MT7628. >> >> Signed-off-by: Shiji Yang <yangshiji66@xxxxxxxxxxx> >> --- >> arch/mips/ralink/common.h | 2 -- >> arch/mips/ralink/mt7620.c | 9 --------- >> arch/mips/ralink/mt7621.c | 9 --------- >> arch/mips/ralink/of.c | 42 ++++++++++++++++++++++++++++++++------- >> arch/mips/ralink/rt288x.c | 9 --------- >> arch/mips/ralink/rt305x.c | 9 --------- >> arch/mips/ralink/rt3883.c | 9 --------- >> 7 files changed, 35 insertions(+), 54 deletions(-) > >Awesome! More deletions than additions in arch folders is always a >good thing :-). > >Changes look good to me. Thanks for doing this! > >Reviewed-by: Sergio Paracuellos <sergio.paracuellos@xxxxxxxxx> > >Best regards, > Sergio Paracuellos Thank you for your review. After your new system control driver[1] is merged, with this patch, we can add more compatible IDs to the list. [1] https://lore.kernel.org/all/20230619040941.1340372-1-sergio.paracuellos@xxxxxxxxx/ Best regards, Shiji Yang
