From: Krzysztof Kozlowski <krzk@kernel.org>
To: "Rob Herring (Arm)" <robh@kernel.org>,
Xiaoshun Xu <xiaoshun.xu@mediatek.corp-partner.google.com>
Cc: Xiaoshun Xu <xiaoshun.xu@mediatek.com>,
Vince-wl Liu <vince-wl.liu@mediatek.com>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
AngeloGioacchino Del Regno
<angelogioacchino.delregno@collabora.com>,
Project_Global_Chrome_Upstream_Group@mediatek.com,
linux-arm-kernel@lists.infradead.org,
Sirius Wang <sirius.wang@mediatek.com>,
devicetree@vger.kernel.org,
Matthias Brugger <matthias.bgg@gmail.com>,
linux-mediatek@lists.infradead.org,
Conor Dooley <conor+dt@kernel.org>,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/4] dt-bindings: soc: devapc: Add for support MT8189
Date: Fri, 12 Dec 2025 08:03:52 +0100 [thread overview]
Message-ID: <bf3dfa16-b6d5-44ad-bb0b-b1bf2d1c2b6c@kernel.org> (raw)
In-Reply-To: <176551375433.2514847.16610029422866924600.robh@kernel.org>
On 12/12/2025 05:29, Rob Herring (Arm) wrote:
>
> On Fri, 12 Dec 2025 11:13:42 +0800, Xiaoshun Xu wrote:
>>
>> External email : Please do not click links or open attachments until you have verified the sender or the content.
>>
>>
>> From: "xiaoshun.xu" <xiaoshun.xu@mediatek.com>
>>
>> Add compatible and vio-idx-num attribute of MT8189
>>
>> Signed-off-by: xiaoshun.xu <xiaoshun.xu@mediatek.com>
>> ---
>> Documentation/devicetree/bindings/soc/mediatek/devapc.yaml | 6 ++++++
>> 1 file changed, 6 insertions(+)
>>
>
> My bot found errors running 'make dt_binding_check' on your patch:
>
> yamllint warnings/errors:
>
> dtschema/dtc warnings/errors:
> /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/soc/mediatek/devapc.yaml: vio-idx-num: missing type definition
>
> doc reference errors (make refcheckdocs):
>
None of these patches here managed to pass spam checks on my inbox, so I
got only this report. Considering patch containing "External email"
clause in the commit msg, I would say posting was somehow non-standard
or just broken.
That's obviously a no-go and finally mediatek corp-partners need to act
on these poor postings (not an exception here).
Best regards,
Krzysztof
next prev parent reply other threads:[~2025-12-12 7:03 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-12-12 3:13 [PATCH 0/4] *** SUBJECT HERE *** Xiaoshun Xu
2025-12-12 3:13 ` [PATCH 1/4] dt-bindings: soc: devapc: Add for support MT8189 Xiaoshun Xu
2025-12-12 4:29 ` Rob Herring (Arm)
2025-12-12 7:03 ` Krzysztof Kozlowski [this message]
2025-12-12 3:13 ` [PATCH 2/4] soc: mediatek: mtk-devapc: Add support for MT8189 DEVAPC Xiaoshun Xu
2025-12-12 3:13 ` [PATCH 3/4] soc: mediatek: mtk-devapc: refine DEVAPC clock control Xiaoshun Xu
2025-12-12 3:13 ` [PATCH 4/4] soc: mediatek: mtk-devapc: refine DEVAPC irq handler Xiaoshun Xu
2025-12-12 11:30 ` [PATCH 0/4] *** SUBJECT HERE *** Xiaoshun Xu (徐晓顺)
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=bf3dfa16-b6d5-44ad-bb0b-b1bf2d1c2b6c@kernel.org \
--to=krzk@kernel.org \
--cc=Project_Global_Chrome_Upstream_Group@mediatek.com \
--cc=angelogioacchino.delregno@collabora.com \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=krzk+dt@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=matthias.bgg@gmail.com \
--cc=robh@kernel.org \
--cc=sirius.wang@mediatek.com \
--cc=vince-wl.liu@mediatek.com \
--cc=xiaoshun.xu@mediatek.com \
--cc=xiaoshun.xu@mediatek.corp-partner.google.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox