linux-kernel - Re: [V1,01/12] dt-bindings: mediatek: Add mediatek, mt8196-jpgdec compatible

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <595e8af841fae48fd306fd92574d181b75ea843a.camel@mediatek.com>
Date: Mon, 7 Apr 2025 09:39:08 +0000
From: Kyrie Wu (吴晗) <Kyrie.Wu@...iatek.com>
To: "matthias.bgg@...il.com" <matthias.bgg@...il.com>, "mchehab@...nel.org"
	<mchehab@...nel.org>, "tzungbi@...omium.org" <tzungbi@...omium.org>,
	"krzk@...nel.org" <krzk@...nel.org>, "robh+dt@...nel.org"
	<robh+dt@...nel.org>, "hverkuil-cisco@...all.nl" <hverkuil-cisco@...all.nl>
CC: "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"linux-mediatek@...ts.infradead.org" <linux-mediatek@...ts.infradead.org>,
	"linux-media@...r.kernel.org" <linux-media@...r.kernel.org>,
	"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
	"kyrie.wu@...iatek.corp-partner.google.com"
	<kyrie.wu@...iatek.corp-partner.google.com>,
	Bin Liu (刘彬) <bin.liu@...iatek.com>,
	"conor+dt@...nel.org" <conor+dt@...nel.org>, "robh@...nel.org"
	<robh@...nel.org>, "linux-arm-kernel@...ts.infradead.org"
	<linux-arm-kernel@...ts.infradead.org>, "krzk+dt@...nel.org"
	<krzk+dt@...nel.org>, AngeloGioacchino Del Regno
	<angelogioacchino.delregno@...labora.com>
Subject: Re: [V1,01/12] dt-bindings: mediatek: Add mediatek, mt8196-jpgdec
 compatible

On Mon, 2025-04-07 at 09:19 +0200, Krzysztof Kozlowski wrote:
> External email : Please do not click links or open attachments until
> you have verified the sender or the content.
> 
> 
> On 07/04/2025 09:13, Kyrie Wu (吴晗) wrote:
> > On Thu, 2025-01-09 at 14:54 +0100, Krzysztof Kozlowski wrote:
> 
> ...
> 
> 
> > > > 
> > > >  properties:
> > > >    compatible:
> > > > -    const: mediatek,mt8195-jpgdec
> > > > +    enum:
> > > > +      - mediatek,mt8195-jpgdec
> > > > +      - mediatek,mt8196-jpgdec
> > > 
> > > Devices look compatible, so express it with fallback.
> > > 
> > > Best regards,
> > > Krzysztof
> > 
> > Dear Krzysztof,
> > 
> > The MT8195 and MT8196 are both multi-hardware chips, I renamed the
> > file
> > to treat it as a common file, if I don't do that it will need to
> > recreate a file with similar contents. Is this acceptable?
> 
> No, do not create own rules. I do not see how this is related to my
> comment at all.
> 
> You responded three months after my review, so entire context
> including
> actual mails in my inbox are gone.
> 
> If you disagree with feedback, respond in timely manner. I feel
> discussing something after three months like wasting my time.
> 
> 
> > If the above description is acceptable, I will apply your request
> > in
> > the next version.
> 
> Best regards,
> Krzysztof

Dear Krzysztof,
 
I'm truly sorry for what I have done. I feel very ashamed and self-
blame after I tried to stand in your shoes. 
 
You contributed a lot of time to help me review my patch, you could
have spent it with your family or your friends, and I failed you in
your selfless kindness. I have not only wasted your precious time but
also disrespected your contribution by ignoring your reply,
undoubtedly.

This was never my intention, and I‘m committed to improving my
communication and collaboration within the community. I value your
expertise and feedback, and I will ensure that I respond promptly and
thoughtfully to any future reviews. (I have improved my email
notifications, and I will share this experience within my team to
remind all developers.)
 
Thank you for your understanding and patience. I hope to rebuild your
trust and continue contributing positively to the community.
 
Sincerely,
Kyrie