Messages in this thread | | | From | Moudy Ho (何宗原) <> | Subject | Re: [PATCH v7 02/16] dt-bindings: media: mediatek: mdp3: merge the indentical RDMA under display | Date | Wed, 18 Oct 2023 03:06:55 +0000 |
| |
On Fri, 2023-10-13 at 08:46 +0200, Krzysztof Kozlowski wrote: >
Hi Krzysztof,
Thank you for assisting with the review.
> External email : Please do not click links or open attachments until > you have verified the sender or the content. > On 12/10/2023 10:40, Moudy Ho wrote: > > > > > +allOf: > > + - if: > > + properties: > > + compatible: > > + contains: > > + const: mediatek,mt8183-mdp3-rdma > > + > > + then: > > + properties: > > + clocks: > > + items: > > + - description: RDMA clock > > + - description: RSZ clock (shared SRAM with RDMA) > > + > > + mboxes: > > + items: > > + - description: used for 1st data pipe from RDMA > > + - description: used for 2nd data pipe from RDMA > > interrupts: > false >
As Angelo provided additional clarification in [15/16], explaining that certain conditions in [2/16] and [3/16] were intentionally omitted due to the need to integrate the same IP with different operations. Apologies for any inconvenience this has caused you.
> > + > > + required: > > + - mboxes > > + - mediatek,gce-events > > + > > + - if: > > + properties: > > + compatible: > > + contains: > > + const: mediatek,mt8195-vdo1-rdma > > + > > + then: > > + properties: > > + clocks: > > + items: > > + - description: RDMA clock > > mboxes: false > mediatek,gce-events: false > > I am not so sure it is actually "simpler" to merge these. They are > quite > different. You will end up with unmanageable allOf with a lot of > branches (which supposedly you want to remove). > >
Upon examining the minor hardware changes in MDP for MT8183 and MT8195 RDMA ([3/16]), it appears that branching cannot be avoided. However, consolidating these changes has the additional advantage of addressing Rob's concerns from v4. Perhaps we can consider the current changes as a form of progress.
Sincerely, Moudy
> > + > > additionalProperties: false > > > > examples: > > Best regards, > Krzysztof >
| |