Thread (64 messages) 64 messages, 8 authors, 2022-06-16

Re: [PATCH v2 12/20] dt-bindings: reset: npcm: Add support for NPCM8XX

From: Tomer Maimon <tmaimon77@gmail.com>
Date: 2022-06-16 13:25:09
Also in: linux-clk, linux-devicetree, linux-serial, linux-watchdog, lkml

Hi Krzysztof,

On Wed, 15 Jun 2022 at 20:03, Krzysztof Kozlowski
[off-list ref] wrote:
On 13/06/2022 02:25, Tomer Maimon wrote:
quoted
Hi Krzysztof,

Thanks for your clarification.

We can remove the dt-binding file and use numbers in the DTS,
appreciate if you can answer few additional questions:
1. Do you suggest adding all NPCM reset values to the NPCM reset
document or the reset values should describe in the module
documentation that uses it?
What is "NPCM reset document"? Are these reset values anyhow different
than interrupts or pins?
No, they represent the same values.
quoted
2. Some of the NPCM7XX document modules describe the reset value they
use from the dt-binding for example:
https://github.com/torvalds/linux/blob/master/Documentation/devicetree/bindings/iio/adc/nuvoton%2Cnpcm750-adc.yaml#L61
This is NPCM750
quoted
If we remove the NPCM8XX dt-binding file should we describe the
NPCM8XX values in the NPCM-ADC document file?
What is NPCM-ADC document file? What do you want to describe there?
Again - how is it different than interrupts?
It is not different from the interrupts.
I will remove the dt-binding reset include file, the reset property
will use numbers and not macro's.


Best regards,
Krzysztof
Best regards,

Tomer

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help