From: "H. Nikolaus Schaller" <hns@goldelico.com>
To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Cc: Paul Cercueil <paul@crapouillou.net>,
Rob Herring <robh+dt@kernel.org>,
Thomas Bogendoerfer <tsbogend@alpha.franken.de>,
linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
linux-mips@vger.kernel.org, letux-kernel@openphoenux.org
Subject: Re: [PATCH 07/18] MIPS: DTS: jz4780: fix otg node as reported by dtbscheck
Date: Sat, 9 Apr 2022 15:37:51 +0200 [thread overview]
Message-ID: <51EEBAB0-849A-4AA5-80E0-B9FAC8FC5E14@goldelico.com> (raw)
In-Reply-To: <2961d892-609c-c0bf-e9c1-c54306f608c7@linaro.org>
> Am 09.04.2022 um 15:23 schrieb Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>:
>
> On 09/04/2022 15:18, H. Nikolaus Schaller wrote:
> hould have a specific compatible.
>>>> Please mention why it does not.
>>>
>>> Agreed. The "snps,dwc2" should be a fallback string, otherwise there is no way to uniquely identify the JZ4780 implementation of the IP.
>>
>> Well, there is no specifc implementation and driver for it. So no need to uniquely identify it.
>
> Specific implementation and driver are not arguments here. This does not
> matter. It's really unrelated argument.
The argumentation is in reverse: if there is no need for a specialized driver or implementation,
why is there is a need to define a specialization.
Your argument was:
"there is no way to uniquely identify the JZ4780 implementation of the IP"
My question is:
"what do we need that for?"
> Bindings are not about implementation in Linux. Implementation can
> change, so bindings should also?
No. Implementations should be agnostic.
next prev parent reply other threads:[~2022-04-09 13:38 UTC|newest]
Thread overview: 83+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-04-08 18:37 [PATCH 00/18] MIPS: DTS: fix some findings by "make ci20_defconfig dt_binding_check dtbs_check" H. Nikolaus Schaller
2022-04-08 18:37 ` [PATCH 01/18] MIPS: DTS: jz4780: remove cpu clock-names as reported by dtbscheck H. Nikolaus Schaller
2022-04-09 11:07 ` Krzysztof Kozlowski
2022-04-09 13:02 ` H. Nikolaus Schaller
2022-04-09 13:10 ` Krzysztof Kozlowski
2022-04-08 18:37 ` [PATCH 02/18] MIPS: DTS: jz4780: fix cgu " H. Nikolaus Schaller
2022-04-09 11:07 ` Krzysztof Kozlowski
2022-04-08 18:37 ` [PATCH 03/18] MIPS: DTS: jz4780: fix tcu timer " H. Nikolaus Schaller
2022-04-09 11:11 ` Krzysztof Kozlowski
2022-04-09 12:24 ` Paul Cercueil
2022-04-09 12:38 ` Krzysztof Kozlowski
2022-04-09 13:12 ` H. Nikolaus Schaller
2022-04-09 13:03 ` H. Nikolaus Schaller
2022-04-09 13:11 ` Krzysztof Kozlowski
2022-04-09 13:18 ` H. Nikolaus Schaller
2022-04-09 13:22 ` Krzysztof Kozlowski
2022-04-09 13:30 ` H. Nikolaus Schaller
2022-04-08 18:37 ` [PATCH 04/18] MIPS: DTS: jz4780: fix ost " H. Nikolaus Schaller
2022-04-09 11:12 ` Krzysztof Kozlowski
2022-04-08 18:37 ` [PATCH 05/18] MIPS: DTS: jz4780: fix pinctrl " H. Nikolaus Schaller
2022-04-09 11:13 ` Krzysztof Kozlowski
2022-04-09 13:04 ` H. Nikolaus Schaller
2022-04-09 13:13 ` Krzysztof Kozlowski
2022-04-09 13:22 ` H. Nikolaus Schaller
2022-04-09 13:24 ` Krzysztof Kozlowski
2022-04-09 13:41 ` H. Nikolaus Schaller
2022-04-09 13:46 ` Krzysztof Kozlowski
2022-04-09 13:57 ` H. Nikolaus Schaller
2022-04-09 14:00 ` Krzysztof Kozlowski
2022-04-09 14:06 ` H. Nikolaus Schaller
2022-04-08 18:37 ` [PATCH 06/18] MIPS: DTS: jz4780: fix rtc node " H. Nikolaus Schaller
2022-04-09 11:14 ` Krzysztof Kozlowski
2022-04-09 13:04 ` H. Nikolaus Schaller
2022-04-08 18:37 ` [PATCH 07/18] MIPS: DTS: jz4780: fix otg " H. Nikolaus Schaller
2022-04-09 11:15 ` Krzysztof Kozlowski
2022-04-09 12:27 ` Paul Cercueil
2022-04-09 13:18 ` H. Nikolaus Schaller
2022-04-09 13:23 ` Krzysztof Kozlowski
2022-04-09 13:37 ` H. Nikolaus Schaller [this message]
2022-04-09 13:43 ` Paul Cercueil
2022-04-09 13:05 ` H. Nikolaus Schaller
2022-04-09 13:15 ` Krzysztof Kozlowski
2022-04-09 13:32 ` H. Nikolaus Schaller
2022-04-09 13:44 ` Krzysztof Kozlowski
2022-04-09 13:53 ` H. Nikolaus Schaller
2022-04-10 16:32 ` Zhou Yanjie
2022-04-10 19:13 ` H. Nikolaus Schaller
2022-04-12 9:49 ` Zhou Yanjie
2022-04-12 11:15 ` H. Nikolaus Schaller
2022-04-08 18:37 ` [PATCH 08/18] MIPS: DTS: jz4780: fix lcd controllers " H. Nikolaus Schaller
2022-04-09 11:17 ` Krzysztof Kozlowski
2022-04-08 18:37 ` [PATCH 09/18] MIPS: DTS: jz4780: fix dma-controller " H. Nikolaus Schaller
2022-04-09 11:17 ` Krzysztof Kozlowski
2022-04-08 18:37 ` [PATCH 10/18] MIPS: DTS: jz4780: fix uart dmas " H. Nikolaus Schaller
2022-04-09 11:18 ` Krzysztof Kozlowski
2022-04-09 13:07 ` H. Nikolaus Schaller
2022-04-09 13:16 ` Krzysztof Kozlowski
2022-04-09 13:26 ` H. Nikolaus Schaller
2022-04-09 13:28 ` Krzysztof Kozlowski
2022-04-09 13:46 ` H. Nikolaus Schaller
2022-04-08 18:37 ` [PATCH 11/18] MIPS: DTS: jz4780: fix i2c " H. Nikolaus Schaller
2022-04-08 18:37 ` [PATCH 12/18] MIPS: DTS: jz4780: fix nemc memory controller " H. Nikolaus Schaller
2022-04-09 11:18 ` Krzysztof Kozlowski
2022-04-08 18:37 ` [PATCH 13/18] dt-bindings: fix jz4780-nemc issue " H. Nikolaus Schaller
2022-04-09 11:26 ` Krzysztof Kozlowski
2022-04-09 12:37 ` Paul Cercueil
2022-04-09 12:47 ` Krzysztof Kozlowski
2022-04-09 12:55 ` Paul Cercueil
2022-04-09 13:01 ` Krzysztof Kozlowski
2022-04-09 13:09 ` H. Nikolaus Schaller
2022-04-09 13:18 ` Krzysztof Kozlowski
2022-04-10 15:35 ` Krzysztof Kozlowski
2022-04-10 19:13 ` H. Nikolaus Schaller
2022-04-10 15:27 ` Rob Herring
2022-04-08 18:37 ` [PATCH 14/18] MIPS: DTS: CI20: add missing model " H. Nikolaus Schaller
2022-04-08 18:37 ` [PATCH 15/18] MIPS: DTS: CI20: fix fixed regulators " H. Nikolaus Schaller
2022-04-09 11:20 ` Krzysztof Kozlowski
2022-04-08 18:37 ` [PATCH 16/18] MIPS: DTS: CI20: fix /memory " H. Nikolaus Schaller
2022-04-08 18:38 ` [PATCH 17/18] MIPS: DTS: CI20: fix wifi " H. Nikolaus Schaller
2022-04-09 11:21 ` Krzysztof Kozlowski
2022-04-08 18:38 ` [PATCH 18/18] MIPS: DTS: CI20: fix bluetooth " H. Nikolaus Schaller
2022-04-09 11:22 ` Krzysztof Kozlowski
2022-04-09 13:10 ` H. Nikolaus Schaller
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=51EEBAB0-849A-4AA5-80E0-B9FAC8FC5E14@goldelico.com \
--to=hns@goldelico.com \
--cc=devicetree@vger.kernel.org \
--cc=krzysztof.kozlowski@linaro.org \
--cc=letux-kernel@openphoenux.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mips@vger.kernel.org \
--cc=paul@crapouillou.net \
--cc=robh+dt@kernel.org \
--cc=tsbogend@alpha.franken.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).