<div dir="ltr"><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, 16 Feb 2026 at 20:03, Krzysztof Kozlowski <<a href="mailto:krzk@kernel.org" target="_blank">krzk@kernel.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Mon, Feb 16, 2026 at 04:59:18PM +0200, Tomer Maimon wrote:<br>
> On Mon, 16 Feb 2026 at 16:48, Krzysztof Kozlowski <<a href="mailto:krzk@kernel.org" target="_blank">krzk@kernel.org</a>> wrote:<br>
> > >>>>> +  syscon:<br>
> > >>>><br>
> > >>>> First iteration. See "How to Get Your DT Schema Bindings Accepted in<br>
> > >>>> Less Than 10 Iterations"<br>
> > >>>><br>
> > >>> Thanks, it was very helpful.<br>
> > >>> the syscon property is already found in the WD node<br>
> > >>> in nuvoton-common-npcm8xx.dtsi file, what should I do:<br>
> > >><br>
> > >> How is that file related to this binding?<br>
> > >><br>
> > >> Either you document existing ABI or you add new (for new device). Commit<br>
> > >> msg MUST be explicit about it and provide the reasons. If wrong (e.g.<br>
> > >> discouraged) ABI was already used then it depends how and when it got<br>
> > >> into the kernel, e.g. if someone bypassed DT completely just to get it<br>
> > >> inside.<br>
> > >><br>
> > > The syscon property is already used in the upstream NPCM8xx DTSI watchdog<br>
> > > node, so I will document it as existing ABI and mark it deprecated. I<br>
> > will<br>
> ><br>
> > And how it is used? I cannot find its usage, so I do not agree on<br>
> > documenting it. Property should be removed or at least provide the<br>
> > justification/impact of removal, if you need it to stay.<br>
> ><br>
> > Understood. The syscon phandle is used by the watchdog driver to read and<br>
<br>
You messed up quotes.<br>
<br>
Can you point me to the line? I REALLY want to be sure that we are not<br>
wasting each other time, e.g. me looking at wrong code or you telling me<br>
some bollocks from downstream.<br></blockquote><div style="font-size:14px;line-height:20px"><font face="arial, sans-serif">The <code style="">syscon</code> property is not used in the current upstream <code style="">npcm_wdt</code> driver.<br></font></div><div><font face="arial, sans-serif"><span style="font-size:14px">It is used </span><strong style="font-size:14px">in this patch set</strong><span style="font-size:14px">, which introduces the function </span><code style="font-size:14px">npcm_get_reset_status()</code><span style="font-size:14px">. Inside that function, the code calls:</span></font></div><div><span style="color:rgb(0,0,0)"><font face="arial, sans-serif">gcr_regmap = syscon_regmap_lookup_by_phandle(dev->of_node, "syscon");</font></span></div><div><div style="font-size:14px;line-height:20px"><font face="arial, sans-serif">to obtain the GCR regmap, read the reset‑status registers, clear them, and report the reset cause via the watchdog <code>bootstatus</code>.</font></div></div><div style="font-size:14px;line-height:20px"><div style="line-height:20px"><font face="arial, sans-serif">This usage is shown here:<br>
<a href="https://www.spinics.net/lists/kernel/msg6045622.html">https://www.spinics.net/lists/kernel/msg6045622.html</a></font></div><div style="line-height:20px"><div style="line-height:20px"><font face="arial, sans-serif">In addition, the watchdog maintainer confirmed that the reset cause can be exposed without the deprecated reset‑type DT properties:<br>
<a href="https://www.spinics.net/lists/kernel/msg6052106.html">https://www.spinics.net/lists/kernel/msg6052106.html</a></font></div><div style="line-height:20px"><div style="line-height:20px"><p style=""><font face="arial, sans-serif">Given this, should I remove the unused <code style="">syscon</code> phandle from the NPCM8xx DTSI first, since no upstream driver currently uses it and it is therefore not part of the ABI?<br>
In this patch set, I am adding the first upstream usage of the GCR regmap in the watchdog driver, so I plan to introduce the new, preferred <code style="">nuvoton,sysgcr</code> property in the binding and use it in the DTSI, without keeping the old <code style="">syscon</code> name unless you think backward compatibility is needed.</font></p>
<p style=""><font face="arial, sans-serif">Please let me know if you prefer that I update the DTSI first or include the DTSI and binding changes in the same series.</font></p></div></div></div></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> clear the GCR reset‑status registers and then report the reset cause<br>
> through the watchdog bootstatus bits.<br>
> Therefore, this property should appear in the binding only because the<br>
> watchdog driver actually uses it — which I am implementing in this patch<br>
> set.<br>
> I will document it accordingly, and also add the new nuvoton,sysgcr phandle<br>
> as the preferred name.<br>
<br>
Best regards,<br>
Krzysztof<br>
<br></blockquote><div><br></div><div>Best regards,</div><div><br></div><div>Tomer </div></div></div>
</div>