All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alexandre Belloni <alexandre.belloni@bootlin.com>
To: Jingbao Qiu <qiujingbao.dlmu@gmail.com>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
	robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org,
	conor+dt@kernel.org, chao.wei@sophgo.com,
	unicorn_wang@outlook.com, paul.walmsley@sifive.com,
	palmer@dabbelt.com, aou@eecs.berkeley.edu,
	linux-rtc@vger.kernel.org, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org, linux-riscv@lists.infradead.org,
	dlan@gentoo.org, inochiama@outlook.com
Subject: Re: [PATCH v6 3/3] riscv: dts: sophgo: add rtc dt node for CV1800
Date: Tue, 16 Jan 2024 17:53:56 +0100	[thread overview]
Message-ID: <20240116165356eaa221fe@mail.local> (raw)
In-Reply-To: <CAJRtX8Tkie+ykLv8L2EgBQcy9tVP5Yz-_J_eHE-9N9hjt+6gkg@mail.gmail.com>

On 17/01/2024 00:29:28+0800, Jingbao Qiu wrote:
> On Wed, Jan 17, 2024 at 12:03 AM Krzysztof Kozlowski
> <krzysztof.kozlowski@linaro.org> wrote:
> >
> > On 16/01/2024 16:51, Jingbao Qiu wrote:
> > >>> CV1800 is a RISCV based SOC that includes an RTC module. The RTC
> > >>> module has an OSC oscillator
> > >>
> > >>
> > >> I am not going to read pages of description. Please write concise replies.
> > >
> > > Thanks, What I mean is that this hardware includes two functions, RTC
> > > and POR. How should I describe their relationship?
> >
> > Your POR does not need to take any resources, so no need to describe any
> > relationship.
> >
> > ...
> >
> > >>> Your suggestion is, firstly, the por submodule does not have any
> > >>> resources, so it should be deleted.
> > >>
> > >> So where did you delete it? I still see it in this patch.
> > >
> > > Should I completely delete him? How can a por driver obtain device information?
> >
> > Delete completely.
> >
> > Device information? What is this? We already agreed you don't have any
> > resources for POR.
> >
> > ....
> >
> > >> Device is only one thing, not two.
> > >>
> > >>>                     reg = <0x5025000 0x2000>;
> > >>>                     interrupts = <17 IRQ_TYPE_LEVEL_HIGH>;
> > >>>                     clocks = <&osc>;
> > >>> };
> > >>> However, in reality, the POR submodule does not use IRQ and CLK.
> > >>> Please do not hesitate to teach. Thanks.
> > >>
> > >> I expect one device node. How many drivers you have does not matter: you
> > >> can instantiate 100 Linux devices in 100 Linux device drivers.
> > >
> > > I understand what you mean. A device node corresponds to multiple drivers.
> > > Should I completely delete the POR device tree node and add it when
> > > submitting the POR driver?
> >
> > ? I wrote it in previous messages and twice in this thread. Completely
> > delete. You do not add it back! Because if you ever intended to add it
> > back, it should be added since beginning. I don't understand what
> > submitting later would solve.
> >
> > > If that's the case, how can I explain that the rtc device tree node
> > > uses the syscon tag?
> > > How can I describe a POR device in DTS? POR is a submodule of RTC, and
> > > it also has corresponding drivers.
> >
> > I said, there is no need for POR in DTS, because you have nothing there.
> > Why do you insist on putting it on DTS?
> >
> > > It's just that his resources are only shared with RTC's Reg.
> >
> > What resources? Reg? That's not a separate resource.
> 
> I'm very sorry about this.
> But I found a binding file that only contains Reg and Compatible.
> 
> rtc@80920000 {
> compatible = "cirrus,ep9301-rtc";
> reg = <0x80920000 0x100>;
> };
> 
> Link: Documentation/devicetree/bindings/rtc/cirrus,ep9301-rtc.yaml
> 
> >
> > To summarize: Drop POR from DTS and never bring it back, unless you come
> > with some different arguments, which you did not say already.
> >
> 
> You are right, if there is no por device tree node, how can the por
> driver obtain the Reg?

I guess the question is why don't you register everything from the RTC
driver?


-- 
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv

WARNING: multiple messages have this Message-ID (diff)
From: Alexandre Belloni <alexandre.belloni@bootlin.com>
To: Jingbao Qiu <qiujingbao.dlmu@gmail.com>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
	robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org,
	conor+dt@kernel.org, chao.wei@sophgo.com,
	unicorn_wang@outlook.com, paul.walmsley@sifive.com,
	palmer@dabbelt.com, aou@eecs.berkeley.edu,
	linux-rtc@vger.kernel.org, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org, linux-riscv@lists.infradead.org,
	dlan@gentoo.org, inochiama@outlook.com
Subject: Re: [PATCH v6 3/3] riscv: dts: sophgo: add rtc dt node for CV1800
Date: Tue, 16 Jan 2024 17:53:56 +0100	[thread overview]
Message-ID: <20240116165356eaa221fe@mail.local> (raw)
In-Reply-To: <CAJRtX8Tkie+ykLv8L2EgBQcy9tVP5Yz-_J_eHE-9N9hjt+6gkg@mail.gmail.com>

On 17/01/2024 00:29:28+0800, Jingbao Qiu wrote:
> On Wed, Jan 17, 2024 at 12:03 AM Krzysztof Kozlowski
> <krzysztof.kozlowski@linaro.org> wrote:
> >
> > On 16/01/2024 16:51, Jingbao Qiu wrote:
> > >>> CV1800 is a RISCV based SOC that includes an RTC module. The RTC
> > >>> module has an OSC oscillator
> > >>
> > >>
> > >> I am not going to read pages of description. Please write concise replies.
> > >
> > > Thanks, What I mean is that this hardware includes two functions, RTC
> > > and POR. How should I describe their relationship?
> >
> > Your POR does not need to take any resources, so no need to describe any
> > relationship.
> >
> > ...
> >
> > >>> Your suggestion is, firstly, the por submodule does not have any
> > >>> resources, so it should be deleted.
> > >>
> > >> So where did you delete it? I still see it in this patch.
> > >
> > > Should I completely delete him? How can a por driver obtain device information?
> >
> > Delete completely.
> >
> > Device information? What is this? We already agreed you don't have any
> > resources for POR.
> >
> > ....
> >
> > >> Device is only one thing, not two.
> > >>
> > >>>                     reg = <0x5025000 0x2000>;
> > >>>                     interrupts = <17 IRQ_TYPE_LEVEL_HIGH>;
> > >>>                     clocks = <&osc>;
> > >>> };
> > >>> However, in reality, the POR submodule does not use IRQ and CLK.
> > >>> Please do not hesitate to teach. Thanks.
> > >>
> > >> I expect one device node. How many drivers you have does not matter: you
> > >> can instantiate 100 Linux devices in 100 Linux device drivers.
> > >
> > > I understand what you mean. A device node corresponds to multiple drivers.
> > > Should I completely delete the POR device tree node and add it when
> > > submitting the POR driver?
> >
> > ? I wrote it in previous messages and twice in this thread. Completely
> > delete. You do not add it back! Because if you ever intended to add it
> > back, it should be added since beginning. I don't understand what
> > submitting later would solve.
> >
> > > If that's the case, how can I explain that the rtc device tree node
> > > uses the syscon tag?
> > > How can I describe a POR device in DTS? POR is a submodule of RTC, and
> > > it also has corresponding drivers.
> >
> > I said, there is no need for POR in DTS, because you have nothing there.
> > Why do you insist on putting it on DTS?
> >
> > > It's just that his resources are only shared with RTC's Reg.
> >
> > What resources? Reg? That's not a separate resource.
> 
> I'm very sorry about this.
> But I found a binding file that only contains Reg and Compatible.
> 
> rtc@80920000 {
> compatible = "cirrus,ep9301-rtc";
> reg = <0x80920000 0x100>;
> };
> 
> Link: Documentation/devicetree/bindings/rtc/cirrus,ep9301-rtc.yaml
> 
> >
> > To summarize: Drop POR from DTS and never bring it back, unless you come
> > with some different arguments, which you did not say already.
> >
> 
> You are right, if there is no por device tree node, how can the por
> driver obtain the Reg?

I guess the question is why don't you register everything from the RTC
driver?


-- 
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

  reply	other threads:[~2024-01-16 16:54 UTC|newest]

Thread overview: 40+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-01-15 16:05 [PATCH v6 0/3] riscv: rtc: sophgo: add mfd and rtc support for CV1800 Jingbao Qiu
2024-01-15 16:05 ` Jingbao Qiu
2024-01-15 16:05 ` [PATCH v6 1/3] dt-bindings: rtc: sophgo: add RTC support for Sophgo CV1800 series SoC Jingbao Qiu
2024-01-15 16:05   ` Jingbao Qiu
2024-01-15 16:05 ` [PATCH v6 2/3] rtc: sophgo: add rtc support for Sophgo CV1800 SoC Jingbao Qiu
2024-01-15 16:05   ` Jingbao Qiu
2024-01-16  7:46   ` Krzysztof Kozlowski
2024-01-16  7:46     ` Krzysztof Kozlowski
2024-01-15 16:06 ` [PATCH v6 3/3] riscv: dts: sophgo: add rtc dt node for CV1800 Jingbao Qiu
2024-01-15 16:06   ` Jingbao Qiu
2024-01-16  7:44   ` Krzysztof Kozlowski
2024-01-16  7:44     ` Krzysztof Kozlowski
2024-01-16 14:41     ` Jingbao Qiu
2024-01-16 14:41       ` Jingbao Qiu
2024-01-16 15:25       ` Krzysztof Kozlowski
2024-01-16 15:25         ` Krzysztof Kozlowski
2024-01-16 15:51         ` Jingbao Qiu
2024-01-16 15:51           ` Jingbao Qiu
2024-01-16 16:03           ` Krzysztof Kozlowski
2024-01-16 16:03             ` Krzysztof Kozlowski
2024-01-16 16:29             ` Jingbao Qiu
2024-01-16 16:29               ` Jingbao Qiu
2024-01-16 16:53               ` Alexandre Belloni [this message]
2024-01-16 16:53                 ` Alexandre Belloni
2024-01-17  2:54                 ` Jingbao Qiu
2024-01-17  2:54                   ` Jingbao Qiu
2024-01-17  9:01                   ` Alexandre Belloni
2024-01-17  9:01                     ` Alexandre Belloni
2024-01-17 10:12                     ` Jingbao Qiu
2024-01-17 10:12                       ` Jingbao Qiu
2024-01-16 16:58               ` Krzysztof Kozlowski
2024-01-16 16:58                 ` Krzysztof Kozlowski
2024-01-17  3:24                 ` Jingbao Qiu
2024-01-17  3:24                   ` Jingbao Qiu
2024-01-17  7:28                   ` Krzysztof Kozlowski
2024-01-17  7:28                     ` Krzysztof Kozlowski
2024-01-17  7:39                     ` Jingbao Qiu
2024-01-17  7:39                       ` Jingbao Qiu
2024-01-17 12:59     ` Conor Dooley
2024-01-17 12:59       ` Conor Dooley

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=20240116165356eaa221fe@mail.local \
    --to=alexandre.belloni@bootlin.com \
    --cc=aou@eecs.berkeley.edu \
    --cc=chao.wei@sophgo.com \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=dlan@gentoo.org \
    --cc=inochiama@outlook.com \
    --cc=krzysztof.kozlowski+dt@linaro.org \
    --cc=krzysztof.kozlowski@linaro.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-riscv@lists.infradead.org \
    --cc=linux-rtc@vger.kernel.org \
    --cc=palmer@dabbelt.com \
    --cc=paul.walmsley@sifive.com \
    --cc=qiujingbao.dlmu@gmail.com \
    --cc=robh+dt@kernel.org \
    --cc=unicorn_wang@outlook.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.