On Thu, 2025-03-06 at 13:48 +0100, Krzysztof Kozlowski wrote:
> External email : Please do not click links or open attachments until
> you have verified the sender or the content.
>
>
> On 06/03/2025 13:45, Friday Yang (杨阳) wrote:
> > > > + const: mediatek,mt8188-smi-larb
> > > > + mediatek,larb-id:
> > > > + oneOf:
> > >
> > > Are you really sure that you need 'oneOf' here? :-)
> > >
> > > Regards,
> > > Angelo
> >
> > Yes, I have tested it. If I try to modify the 'examples'
> > like this. That is:
> > change the compatible to "mediatek,mt8188-smi-larb",
> > add 'mediatek,larb-id = <10>;'
> >
> > examples:
> > - |+
> > #include <dt-bindings/clock/mt8173-clk.h>
> > #includ
> > e <dt-bindings/power/mt8173-power.h>
> >
> > larb1: larb@16010000 {
> > compatible = "mediatek,mt8188-smi-larb";
> > reg = <0x16010000 0x1000>;
> > mediatek,smi = <&smi_common>;
> > mediatek,larb-id = <10>;
> > power-domains = <&scpsys MT8188_POWER_DOMAIN_VDEC>;
> > clocks = <&vdecsys CLK_VDEC_CKEN>,
> > <&vdecsys CLK_VDEC_LARB_CKEN>;
> > clock-names = "apb", "smi";
> > };
> >
> > The 'dt_binding_check' could give the following
> > errors:
> >
> > Documentation/devicetree/bindings/memory-controllers/mediatek,smi-
> > larb.example.dtb: larb@16010000: 'resets' is a required property
> > from schema $id:
> >
https://urldefense.com/v3/__http://devicetree.org/schemas/memory-controllers/mediatek,smi-larb.yaml*__;Iw!!CTRNKA9wMg0ARbw!kEwWhxyfjVtuHKBHazZGRaFdlmrU2bcIsiVDcsUDzEIManMw2XIG9RgOzq773vtmqlR9_sWZDFhU09SV$
> > Documentation/devicetree/bindings/memory-controllers/mediatek,smi-
> > larb.example.dtb: larb@16010000: 'reset-names' is a required
> > property
> > from schema $id:
> >
https://urldefense.com/v3/__http://devicetree.org/schemas/memory-controllers/mediatek,smi-larb.yaml*__;Iw!!CTRNKA9wMg0ARbw!kEwWhxyfjVtuHKBHazZGRaFdlmrU2bcIsiVDcsUDzEIManMw2XIG9RgOzq773vtmqlR9_sWZDFhU09SV$
> >
> > And this is what I want to achieve. On the MediaTek MT8188 SoC
> > platform, 'resets' and 'reset-names' are only required for SMI
> > LARBs
> > located in image, camera and ipe subsys. Others can be ignored. And
> > the
> > 'larb-id' of these SMI LARBs are shown in this array: [ 9, 10, 11,
> > 12,
> > 13, 16, 17, 18, 19, 20 ].
> >
> > Please feel free to let me know if you have any doubts.
>
> You did not really answer the question. Where is anything about oneOf
> in
> your reply?
>
> I am dropping this patchset from my queue.
>
In this SoC, we encountered power-off failures and SMI bus hang issues
during camera stress tests. SMI LARBs located in the image, IPE, and
camera subsystems need to implement a reset, while other LARBs do not
require it.
The 'mediatek,larb-id' for SMI LARBs in the image, IPE, and camera
subsystems are as follows:
- image subsystem: 9, 10, 11, 12, 16
- IPE subsystem: 13
- camera subsystem: 17, 18, 19, 20
Therefore, we believe that 'mediatek,larb-id' should be 'oneOf' the
values in the array. Could this answer the question, or is there
another property I should use in this situation?
> Best regards,
> Krzysztof
