[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <963915a3-819f-46f7-88eb-f3420d9660d5@quicinc.com>
Date: Tue, 4 Mar 2025 15:28:32 +0530
From: Ayushi Makhija <quic_amakhija@...cinc.com>
To: Krzysztof Kozlowski <krzk@...nel.org>, <linux-arm-msm@...r.kernel.org>,
<dri-devel@...ts.freedesktop.org>, <freedreno@...ts.freedesktop.org>,
<devicetree@...r.kernel.org>, <linux-kernel@...r.kernel.org>
CC: <robdclark@...il.com>, <dmitry.baryshkov@...aro.org>, <sean@...rly.run>,
<marijn.suijten@...ainline.org>, <andersson@...nel.org>,
<robh@...nel.org>, <robh+dt@...nel.org>, <krzk+dt@...nel.org>,
<konradybcio@...nel.org>, <conor+dt@...nel.org>,
<andrzej.hajda@...el.com>, <neil.armstrong@...aro.org>,
<rfoss@...nel.org>, <Laurent.pinchart@...asonboard.com>,
<jonas@...boo.se>, <jernej.skrabec@...il.com>,
<quic_abhinavk@...cinc.com>, <quic_rajeevny@...cinc.com>,
<quic_vproddut@...cinc.com>, <quic_jesszhan@...cinc.com>
Subject: Re: [PATCH 07/11] arm64: dts: qcom: sa8775p-ride: add anx7625 DSI to
DP bridge nodes
On 2/26/2025 2:05 PM, Krzysztof Kozlowski wrote:
> On Tue, Feb 25, 2025 at 02:31:05PM +0100, Krzysztof Kozlowski wrote:
>> On 25/02/2025 13:18, Ayushi Makhija wrote:
>>> + pinctrl-0 = <&dsi0_int_pin>,
>>> + <&dsi0_cbl_det_pin>,
>>> + <&dsi1_int_pin>,
>>> + <&dsi1_cbl_det_pin>;
>>> + pinctrl-names = "default";
>>> +
>>> + dsi0_int_pin: gpio2_cfg {
>>
>>
>> No underscores, see DTS coding style.
>
> And as Rob's bot pointed out: insufficient testing. :(
>
> Please be 100% sure everything is tested before you post new version.
> You shouldn't use reviewers for the job of tools, that's quite waste of
> our time.
>
> Best regards,
> Krzysztof
Hi Krzysztof,
Thanks, for the review.
Sorry for the mistake. I will post the clean patch in my next patchset.
Thank,
Ayushi
Powered by blists - more mailing lists