RE: [PATCH] arm64: dts: exynos: Added the ethernet pin configuration

["Yashwant Varur" <yashwant.v@samsung.com>]

Hi Krzysztof

> -----Original Message-----
> From: Krzysztof Kozlowski <krzk@kernel.org>
> Sent: Tuesday, April 29, 2025 6:37 PM
> To: Yashwant Varur <yashwant.v@samsung.com>; robh@kernel.org;
> krzk+dt@kernel.org; conor+dt@kernel.org; alim.akhtar@samsung.com;
> devicetree@vger.kernel.org; linux-arm-kernel@lists.infradead.org; linux-
> samsung-soc@vger.kernel.org; linux-kernel@vger.kernel.org
> Cc: cs0617.lee@samsung.com; g.naidu@samsung.com;
> niyas.ahmed@samsung.com
> Subject: Re: [PATCH] arm64: dts: exynos: Added the ethernet pin configuration
> 
> On 29/04/2025 11:29, Yashwant Varur wrote:
> >
> > Please follow DTS coding style carefully. This applies to all commits you try to
> send from your downstream/vendor code.
> 
> 
> hm?
> 
Got the issue, coding style says
Node and property names can use only the following characters:
Lowercase characters: [a-z]
Digits: [0-9]
Dash: -
I was using underscore for node name.
> >>
> > Sure, thanks
> >
> > What is more important, I don't really understand why you are doing
> > this
> > - there is no user of these entries - and commit msg does not help here.
> >>
> > Understood, in v2 will add the Ethernet node as well.
> 
> I don't understand what is your reply here and what is quote. Use standard email
> style, not some mySingle or Outlook output. I suggest reading typical guidelines
> how to use email based workflows (kernel also has one).
> 

Looks like mailer was not configured properly, hope this time it is fine.
I am working on upstreaming Ethernet driver for exynosauto (which has a variant of STMMAC controller)
What I wanted to reply previously was, will add pin control, along with Ethernet node(which will consume the pincontrol) and
I realize that first dt-binding should go for this IP.
My plan is to send Ethernet driver along with Ethernet dtsi node as well.

> 
> Best regards,
> Krzysztof