mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Paul Cercueil <paul@crapouillou.net>
To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Cc: "H. Nikolaus Schaller" <hns@goldelico.com>,
	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 13/18] dt-bindings: fix jz4780-nemc issue as reported by dtbscheck
Date: Sat, 09 Apr 2022 13:55:00 +0100	[thread overview]
Message-ID: <O7P2AR.D8D4QSYF9P8E@crapouillou.net> (raw)
In-Reply-To: <2e1495ab-2773-a3bb-0ad2-4af607cccc11@linaro.org>



Le sam., avril 9 2022 at 14:47:23 +0200, Krzysztof Kozlowski 
<krzysztof.kozlowski@linaro.org> a écrit :
> On 09/04/2022 14:37, Paul Cercueil wrote:
>>>  The true question is whether you need simple-mfd. Isn't the binding
>>>  (and
>>>  the driver) expected to instantiate its children?
>> 
>>  I can explain that one. There is the EFUSE controller located inside
>>  the nemc's memory area, and the two are pretty much unrelated, hence
>>  the "simple-mfd" compatible string.
> 
> I saw the efuse children and that's why I asked who is expected to
> populate them. You said that simple-mfd is required for this, I say 
> no.
> It should work without simple-mfd...
> 
> I am kind of repeating myself but I really do not see the need of
> simple-mfd in the bindings.

Well, it is a "simple MFD", so I don't see why we can't use the 
"simple-mfd" compatible. Why would we not want to use it?

Besides, if the nemc driver is responsible for populating the efuse 
device, that means the nemc driver must be enabled for the efuse to 
work, which is nonsense, the two IP blocks being unrelated.

-Paul



  reply	other threads:[~2022-04-09 12:55 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
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 [this message]
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=O7P2AR.D8D4QSYF9P8E@crapouillou.net \
    --to=paul@crapouillou.net \
    --cc=devicetree@vger.kernel.org \
    --cc=hns@goldelico.com \
    --cc=krzysztof.kozlowski@linaro.org \
    --cc=letux-kernel@openphoenux.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mips@vger.kernel.org \
    --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

all inboxes | Powered by JetHome®