Add mdp3 mutex compatible for mt8186 SoC.
Signed-off-by: Allen-KH Cheng <allen-kh.cheng@mediatek.com>
Signed-off-by: Xiandong Wang <xiandong.wang@mediatek.com>
---
.../devicetree/bindings/soc/mediatek/mediatek,mutex.yaml | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/devicetree/bindings/soc/mediatek/mediatek,mutex.yaml b/Documentation/devicetree/bindings/soc/mediatek/mediatek,mutex.yaml
index 627dcc3e8b32..234fa5dc07c2 100644
--- a/Documentation/devicetree/bindings/soc/mediatek/mediatek,mutex.yaml
+++ b/Documentation/devicetree/bindings/soc/mediatek/mediatek,mutex.yaml
@@ -30,6 +30,7 @@ properties:
- mediatek,mt8173-disp-mutex
- mediatek,mt8183-disp-mutex
- mediatek,mt8186-disp-mutex
+ - mediatek,mt8186-mdp3-mutex
- mediatek,mt8192-disp-mutex
- mediatek,mt8195-disp-mutex
--
2.18.0
Il 05/07/22 14:26, Allen-KH Cheng ha scritto: > Add mdp3 mutex compatible for mt8186 SoC. > > Signed-off-by: Allen-KH Cheng <allen-kh.cheng@mediatek.com> > Signed-off-by: Xiandong Wang <xiandong.wang@mediatek.com> Please drop this commit. Adding a mdp3-mutex compatible is not needed here. > --- > .../devicetree/bindings/soc/mediatek/mediatek,mutex.yaml | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/Documentation/devicetree/bindings/soc/mediatek/mediatek,mutex.yaml b/Documentation/devicetree/bindings/soc/mediatek/mediatek,mutex.yaml > index 627dcc3e8b32..234fa5dc07c2 100644 > --- a/Documentation/devicetree/bindings/soc/mediatek/mediatek,mutex.yaml > +++ b/Documentation/devicetree/bindings/soc/mediatek/mediatek,mutex.yaml > @@ -30,6 +30,7 @@ properties: > - mediatek,mt8173-disp-mutex > - mediatek,mt8183-disp-mutex > - mediatek,mt8186-disp-mutex > + - mediatek,mt8186-mdp3-mutex > - mediatek,mt8192-disp-mutex > - mediatek,mt8195-disp-mutex >
On 07/07/2022 10:52, AngeloGioacchino Del Regno wrote: > Il 05/07/22 14:26, Allen-KH Cheng ha scritto: >> Add mdp3 mutex compatible for mt8186 SoC. >> >> Signed-off-by: Allen-KH Cheng <allen-kh.cheng@mediatek.com> >> Signed-off-by: Xiandong Wang <xiandong.wang@mediatek.com> > > > Please drop this commit. Adding a mdp3-mutex compatible is not needed here. > Thanks for checking. We probably would need a fallback compatible. We can only know from the HW engineers that can confirm if the IP block is the same as the disp mutex or a different one. I'll drop both patches for now until things got clear. Regards, Matthias >> --- >> .../devicetree/bindings/soc/mediatek/mediatek,mutex.yaml | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git >> a/Documentation/devicetree/bindings/soc/mediatek/mediatek,mutex.yaml >> b/Documentation/devicetree/bindings/soc/mediatek/mediatek,mutex.yaml >> index 627dcc3e8b32..234fa5dc07c2 100644 >> --- a/Documentation/devicetree/bindings/soc/mediatek/mediatek,mutex.yaml >> +++ b/Documentation/devicetree/bindings/soc/mediatek/mediatek,mutex.yaml >> @@ -30,6 +30,7 @@ properties: >> - mediatek,mt8173-disp-mutex >> - mediatek,mt8183-disp-mutex >> - mediatek,mt8186-disp-mutex >> + - mediatek,mt8186-mdp3-mutex >> - mediatek,mt8192-disp-mutex >> - mediatek,mt8195-disp-mutex > > >
Il 07/07/22 12:41, Matthias Brugger ha scritto: > > > On 07/07/2022 10:52, AngeloGioacchino Del Regno wrote: >> Il 05/07/22 14:26, Allen-KH Cheng ha scritto: >>> Add mdp3 mutex compatible for mt8186 SoC. >>> >>> Signed-off-by: Allen-KH Cheng <allen-kh.cheng@mediatek.com> >>> Signed-off-by: Xiandong Wang <xiandong.wang@mediatek.com> >> >> >> Please drop this commit. Adding a mdp3-mutex compatible is not needed here. >> > > Thanks for checking. We probably would need a fallback compatible. We can only know > from the HW engineers that can confirm if the IP block is the same as the disp > mutex or a different one. > > I'll drop both patches for now until things got clear. > They're located in a different iospace from each other, but either the platform data needs to *not be* joined together, or if they're together, I would not like having two different compatible strings for essentially the same thing. I would at this point prefer dropping '-disp' from 'mediatek,mt8186-disp-mutex' so that we would be able to declare two 'mediatek,mt8186-mutex' in devicetree... ...or simply have two mediatek,mt8186-disp-mutex (although logically incorrect?). Cheers, Angelo > Regards, > Matthias > >>> --- >>> .../devicetree/bindings/soc/mediatek/mediatek,mutex.yaml | 1 + >>> 1 file changed, 1 insertion(+) >>> >>> diff --git a/Documentation/devicetree/bindings/soc/mediatek/mediatek,mutex.yaml >>> b/Documentation/devicetree/bindings/soc/mediatek/mediatek,mutex.yaml >>> index 627dcc3e8b32..234fa5dc07c2 100644 >>> --- a/Documentation/devicetree/bindings/soc/mediatek/mediatek,mutex.yaml >>> +++ b/Documentation/devicetree/bindings/soc/mediatek/mediatek,mutex.yaml >>> @@ -30,6 +30,7 @@ properties: >>> - mediatek,mt8173-disp-mutex >>> - mediatek,mt8183-disp-mutex >>> - mediatek,mt8186-disp-mutex >>> + - mediatek,mt8186-mdp3-mutex >>> - mediatek,mt8192-disp-mutex >>> - mediatek,mt8195-disp-mutex >> >> >>
© 2016 - 2024 Red Hat, Inc.