linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
To: "H. Nikolaus Schaller" <hns@goldelico.com>
Cc: Rob Herring <robh+dt@kernel.org>,
	Paul Cercueil <paul@crapouillou.net>,
	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:15:32 +0200	[thread overview]
Message-ID: <535e3eab-a28e-46f3-2a7e-f1ffd1913470@linaro.org> (raw)
In-Reply-To: <822182F3-5429-4731-9FA1-8F18C5D95DEC@goldelico.com>

On 09/04/2022 15:05, H. Nikolaus Schaller wrote:
>>
>> This looks wrong, the block usually should have a specific compatible.
>> Please mention why it does not.
> 
> Well, I did not even have that idea that it could need an explanation.
> 
> There is no "ingenic,jz4780-otg" and none is needed here to make it work.

Make it work in what terms? We talk about hardware description, right?

> 
> Therefore the generic "snps,dwc2" is sufficient.

No, you are mixing now driver behavior (is sufficient) with hardware
description. Most of licensed blocks require the specific compatible to
differentiate it.


Best regards,
Krzysztof

  reply	other threads:[~2022-04-09 13:15 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
2022-04-09 13:43             ` Paul Cercueil
2022-04-09 13:05     ` H. Nikolaus Schaller
2022-04-09 13:15       ` Krzysztof Kozlowski [this message]
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=535e3eab-a28e-46f3-2a7e-f1ffd1913470@linaro.org \
    --to=krzysztof.kozlowski@linaro.org \
    --cc=devicetree@vger.kernel.org \
    --cc=hns@goldelico.com \
    --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).