On 3/14/2023 3:55 AM, Konrad Dybcio wrote: > > On 13.03.2023 22:25, Melody Olvera wrote: >> >> On 3/8/2023 2:23 AM, Konrad Dybcio wrote: >>> On 7.03.2023 00:17, Melody Olvera wrote: >>>> Add nodes for IPCC, MPSS, and AOSS drivers. Also update >>>> the scm node to include its interconnect. >>> Quite a bit of stuff in a single commit, this could be >>> separated into: >>> >>> - scm icc >>> - aoss+ipcc >>> - smp2p+mpss >> Hmm ok. Will split this patch into a few patches. >> >>>> Signed-off-by: Melody Olvera <quic_molvera@xxxxxxxxxxx> >>>> --- > [...] > >>>> + <0x0 0x4180000 0x0 0x1000>; >>> No reg-names? >> No; we don't use reg-names in the driver. Lmk if we should be. >> > qcom_q6v5_mss.c / qcom_q6v5_wcss.c get the 'rmb' region with > [...]_byname and I think it'd scale better if we did the same here, > as one day there may be a weird SoC that'd have an "XYZ" region, > different to "base" and "rmb", which we would need to handle.. somehow.. Yeah that's sensible. Will update the driver and this entry. Thanks, Melody > > Konrad