All of lore.kernel.org
 help / color / mirror / Atom feed
From: <Conor.Dooley@microchip.com>
To: <krzysztof.kozlowski@linaro.org>, <mail@conchuod.ie>,
	<airlied@linux.ie>,  <daniel@ffwll.ch>, <robh+dt@kernel.org>,
	<krzysztof.kozlowski+dt@linaro.org>,  <thierry.reding@gmail.com>,
	<sam@ravnborg.org>, <Eugeniy.Paltsev@synopsys.com>,
	<vkoul@kernel.org>, <fancer.lancer@gmail.com>,
	<daniel.lezcano@linaro.org>, <palmer@dabbelt.com>,
	 <palmer@rivosinc.com>
Cc: niklas.cassel@wdc.com, aou@eecs.berkeley.edu,
	devicetree@vger.kernel.org, masahiroy@kernel.org,
	linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org,
	geert@linux-m68k.org, paul.walmsley@sifive.com,
	dmaengine@vger.kernel.org, linux-riscv@lists.infradead.org,
	dillon.minfei@gmail.com, damien.lemoal@opensource.wdc.com
Subject: Re: [PATCH v5 04/13] dt-bindings: memory-controllers: add canaan k210 sram controller
Date: Tue, 16 Aug 2022 09:31:23 +0000	[thread overview]
Message-ID: <d1c1e2bd-39cb-4927-ac0b-22583ac7a420@microchip.com> (raw)
In-Reply-To: <d6acd450-a3bc-d5bd-eda9-9664e6beadf9@linaro.org>

On 16/08/2022 10:27, Krzysztof Kozlowski wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> 
> On 06/07/2022 00:52, Conor Dooley wrote:
>> From: Conor Dooley <conor.dooley@microchip.com>
>>
>> The k210 U-Boot port has been using the clocks defined in the
>> devicetree to bring up the board's SRAM, but this violates the
>> dt-schema. As such, move the clocks to a dedicated node with
>> the same compatible string & document it.
>>
>> Signed-off-by: Conor Dooley <conor.dooley@microchip.com>
>> ---
> 
> Does not apply to my tree. Please rebase and resend.

Palmer took it with the rest of the canaan stuff as per:
https://lore.kernel.org/all/eeed43cf-7bd6-9d77-9e1f-e018a236a058@linaro.org/

It is now in v6.0-rc1:
commit 727b05e46cffd74adca96ca13e57352339875586
Author: Conor Dooley <conor.dooley@microchip.com>
Date:   Tue Jul 5 22:52:05 2022 +0100

     dt-bindings: memory-controllers: add canaan k210 sram controller
     
     The k210 U-Boot port has been using the clocks defined in the
     devicetree to bring up the board's SRAM, but this violates the
     dt-schema. As such, move the clocks to a dedicated node with
     the same compatible string & document it.
     
     Signed-off-by: Conor Dooley <conor.dooley@microchip.com>
     Reviewed-by: Rob Herring <robh@kernel.org>
     Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
     Link: https://lore.kernel.org/r/20220705215213.1802496-5-mail@conchuod.ie
     Signed-off-by: Palmer Dabbelt <palmer@rivosinc.com>


WARNING: multiple messages have this Message-ID (diff)
From: <Conor.Dooley@microchip.com>
To: <krzysztof.kozlowski@linaro.org>, <mail@conchuod.ie>,
	<airlied@linux.ie>, <daniel@ffwll.ch>, <robh+dt@kernel.org>,
	<krzysztof.kozlowski+dt@linaro.org>, <thierry.reding@gmail.com>,
	<sam@ravnborg.org>, <Eugeniy.Paltsev@synopsys.com>,
	<vkoul@kernel.org>, <fancer.lancer@gmail.com>,
	<daniel.lezcano@linaro.org>, <palmer@dabbelt.com>,
	<palmer@rivosinc.com>
Cc: <paul.walmsley@sifive.com>, <aou@eecs.berkeley.edu>,
	<masahiroy@kernel.org>, <damien.lemoal@opensource.wdc.com>,
	<geert@linux-m68k.org>, <niklas.cassel@wdc.com>,
	<dillon.minfei@gmail.com>, <dri-devel@lists.freedesktop.org>,
	<devicetree@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
	<dmaengine@vger.kernel.org>, <linux-riscv@lists.infradead.org>
Subject: Re: [PATCH v5 04/13] dt-bindings: memory-controllers: add canaan k210 sram controller
Date: Tue, 16 Aug 2022 09:31:23 +0000	[thread overview]
Message-ID: <d1c1e2bd-39cb-4927-ac0b-22583ac7a420@microchip.com> (raw)
In-Reply-To: <d6acd450-a3bc-d5bd-eda9-9664e6beadf9@linaro.org>

On 16/08/2022 10:27, Krzysztof Kozlowski wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> 
> On 06/07/2022 00:52, Conor Dooley wrote:
>> From: Conor Dooley <conor.dooley@microchip.com>
>>
>> The k210 U-Boot port has been using the clocks defined in the
>> devicetree to bring up the board's SRAM, but this violates the
>> dt-schema. As such, move the clocks to a dedicated node with
>> the same compatible string & document it.
>>
>> Signed-off-by: Conor Dooley <conor.dooley@microchip.com>
>> ---
> 
> Does not apply to my tree. Please rebase and resend.

Palmer took it with the rest of the canaan stuff as per:
https://lore.kernel.org/all/eeed43cf-7bd6-9d77-9e1f-e018a236a058@linaro.org/

It is now in v6.0-rc1:
commit 727b05e46cffd74adca96ca13e57352339875586
Author: Conor Dooley <conor.dooley@microchip.com>
Date:   Tue Jul 5 22:52:05 2022 +0100

     dt-bindings: memory-controllers: add canaan k210 sram controller
     
     The k210 U-Boot port has been using the clocks defined in the
     devicetree to bring up the board's SRAM, but this violates the
     dt-schema. As such, move the clocks to a dedicated node with
     the same compatible string & document it.
     
     Signed-off-by: Conor Dooley <conor.dooley@microchip.com>
     Reviewed-by: Rob Herring <robh@kernel.org>
     Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
     Link: https://lore.kernel.org/r/20220705215213.1802496-5-mail@conchuod.ie
     Signed-off-by: Palmer Dabbelt <palmer@rivosinc.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: <Conor.Dooley@microchip.com>
To: <krzysztof.kozlowski@linaro.org>, <mail@conchuod.ie>,
	<airlied@linux.ie>, <daniel@ffwll.ch>, <robh+dt@kernel.org>,
	<krzysztof.kozlowski+dt@linaro.org>, <thierry.reding@gmail.com>,
	<sam@ravnborg.org>, <Eugeniy.Paltsev@synopsys.com>,
	<vkoul@kernel.org>, <fancer.lancer@gmail.com>,
	<daniel.lezcano@linaro.org>, <palmer@dabbelt.com>,
	<palmer@rivosinc.com>
Cc: <paul.walmsley@sifive.com>, <aou@eecs.berkeley.edu>,
	<masahiroy@kernel.org>, <damien.lemoal@opensource.wdc.com>,
	<geert@linux-m68k.org>, <niklas.cassel@wdc.com>,
	<dillon.minfei@gmail.com>, <dri-devel@lists.freedesktop.org>,
	<devicetree@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
	<dmaengine@vger.kernel.org>, <linux-riscv@lists.infradead.org>
Subject: Re: [PATCH v5 04/13] dt-bindings: memory-controllers: add canaan k210 sram controller
Date: Tue, 16 Aug 2022 09:31:23 +0000	[thread overview]
Message-ID: <d1c1e2bd-39cb-4927-ac0b-22583ac7a420@microchip.com> (raw)
In-Reply-To: <d6acd450-a3bc-d5bd-eda9-9664e6beadf9@linaro.org>

On 16/08/2022 10:27, Krzysztof Kozlowski wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> 
> On 06/07/2022 00:52, Conor Dooley wrote:
>> From: Conor Dooley <conor.dooley@microchip.com>
>>
>> The k210 U-Boot port has been using the clocks defined in the
>> devicetree to bring up the board's SRAM, but this violates the
>> dt-schema. As such, move the clocks to a dedicated node with
>> the same compatible string & document it.
>>
>> Signed-off-by: Conor Dooley <conor.dooley@microchip.com>
>> ---
> 
> Does not apply to my tree. Please rebase and resend.

Palmer took it with the rest of the canaan stuff as per:
https://lore.kernel.org/all/eeed43cf-7bd6-9d77-9e1f-e018a236a058@linaro.org/

It is now in v6.0-rc1:
commit 727b05e46cffd74adca96ca13e57352339875586
Author: Conor Dooley <conor.dooley@microchip.com>
Date:   Tue Jul 5 22:52:05 2022 +0100

     dt-bindings: memory-controllers: add canaan k210 sram controller
     
     The k210 U-Boot port has been using the clocks defined in the
     devicetree to bring up the board's SRAM, but this violates the
     dt-schema. As such, move the clocks to a dedicated node with
     the same compatible string & document it.
     
     Signed-off-by: Conor Dooley <conor.dooley@microchip.com>
     Reviewed-by: Rob Herring <robh@kernel.org>
     Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
     Link: https://lore.kernel.org/r/20220705215213.1802496-5-mail@conchuod.ie
     Signed-off-by: Palmer Dabbelt <palmer@rivosinc.com>


  reply	other threads:[~2022-08-16  9:31 UTC|newest]

Thread overview: 108+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-07-05 21:52 [PATCH v5 00/13] Canaan devicetree fixes Conor Dooley
2022-07-05 21:52 ` Conor Dooley
2022-07-05 21:52 ` Conor Dooley
2022-07-05 21:52 ` [PATCH v5 01/13] dt-bindings: display: convert ilitek,ili9341.txt to dt-schema Conor Dooley
2022-07-05 21:52   ` [PATCH v5 01/13] dt-bindings: display: convert ilitek, ili9341.txt " Conor Dooley
2022-07-05 21:52   ` [PATCH v5 01/13] dt-bindings: display: convert ilitek,ili9341.txt " Conor Dooley
2022-07-05 21:52 ` [PATCH v5 02/13] dt-bindings: display: ili9341: document canaan kd233's lcd Conor Dooley
2022-07-05 21:52   ` Conor Dooley
2022-07-05 21:52   ` Conor Dooley
2022-07-12  7:56   ` Conor.Dooley
2022-07-12  7:56     ` Conor.Dooley
2022-07-12  7:56     ` Conor.Dooley
2022-07-05 21:52 ` [PATCH v5 03/13] dt-bindings: dma: dw-axi-dmac: extend the number of interrupts Conor Dooley
2022-07-05 21:52   ` Conor Dooley
2022-07-05 21:52   ` Conor Dooley
2022-07-06  5:45   ` Vinod Koul
2022-07-06  5:45     ` Vinod Koul
2022-07-06  5:45     ` Vinod Koul
2022-07-18 15:12   ` Ben Dooks
2022-07-18 15:12     ` Ben Dooks
2022-07-18 15:12     ` Ben Dooks
2022-07-18 15:23     ` Conor.Dooley
2022-07-18 15:23       ` Conor.Dooley
2022-07-18 15:23       ` Conor.Dooley
2022-07-05 21:52 ` [PATCH v5 04/13] dt-bindings: memory-controllers: add canaan k210 sram controller Conor Dooley
2022-07-05 21:52   ` Conor Dooley
2022-07-05 21:52   ` Conor Dooley
2022-07-06 14:48   ` Rob Herring
2022-07-06 14:48     ` Rob Herring
2022-07-06 14:48     ` Rob Herring
2022-07-10 19:39   ` Conor.Dooley
2022-07-10 19:39     ` Conor.Dooley
2022-07-10 19:39     ` Conor.Dooley
2022-07-10 23:21     ` Damien Le Moal
2022-07-10 23:21       ` Damien Le Moal
2022-07-10 23:21       ` Damien Le Moal
2022-07-12 15:54       ` Conor.Dooley
2022-07-12 15:54         ` Conor.Dooley
2022-07-12 15:54         ` Conor.Dooley
2022-07-12 15:59         ` Krzysztof Kozlowski
2022-07-12 15:59           ` Krzysztof Kozlowski
2022-07-12 15:59           ` Krzysztof Kozlowski
2022-07-12 16:04           ` Conor.Dooley
2022-07-12 16:04             ` Conor.Dooley
2022-07-12 16:04             ` Conor.Dooley
2022-08-16  9:27   ` Krzysztof Kozlowski
2022-08-16  9:27     ` Krzysztof Kozlowski
2022-08-16  9:27     ` Krzysztof Kozlowski
2022-08-16  9:31     ` Conor.Dooley [this message]
2022-08-16  9:31       ` Conor.Dooley
2022-08-16  9:31       ` Conor.Dooley
2022-07-05 21:52 ` [PATCH v5 05/13] riscv: dts: canaan: fix the k210's memory node Conor Dooley
2022-07-05 21:52   ` Conor Dooley
2022-07-05 21:52   ` Conor Dooley
2022-07-05 21:52 ` [PATCH v5 06/13] riscv: dts: canaan: fix the k210's timer nodes Conor Dooley
2022-07-05 21:52   ` Conor Dooley
2022-07-05 21:52   ` Conor Dooley
2022-07-05 21:52 ` [PATCH v5 07/13] riscv: dts: canaan: fix mmc node names Conor Dooley
2022-07-05 21:52   ` Conor Dooley
2022-07-05 21:52   ` Conor Dooley
2022-07-05 21:52 ` [PATCH v5 08/13] riscv: dts: canaan: fix kd233 display spi frequency Conor Dooley
2022-07-05 21:52   ` Conor Dooley
2022-07-05 21:52   ` Conor Dooley
2022-07-05 21:52 ` [PATCH v5 09/13] riscv: dts: canaan: use custom compatible for k210 i2s Conor Dooley
2022-07-05 21:52   ` Conor Dooley
2022-07-05 21:52   ` Conor Dooley
2022-07-05 21:52 ` [PATCH v5 10/13] riscv: dts: canaan: remove spi-max-frequency from controllers Conor Dooley
2022-07-05 21:52   ` Conor Dooley
2022-07-05 21:52   ` Conor Dooley
2022-07-05 21:52 ` [PATCH v5 11/13] riscv: dts: canaan: fix bus {ranges,reg} warnings Conor Dooley
2022-07-05 21:52   ` Conor Dooley
2022-07-05 21:52   ` Conor Dooley
2022-07-05 21:52 ` [PATCH v5 12/13] riscv: dts: canaan: add specific compatible for kd233's LCD Conor Dooley
2022-07-05 21:52   ` Conor Dooley
2022-07-05 21:52   ` Conor Dooley
2022-07-05 21:52 ` [PATCH v5 13/13] riscv: dts: canaan: build all devicetress if SOC_CANAAN Conor Dooley
2022-07-05 21:52   ` Conor Dooley
2022-07-05 21:52   ` Conor Dooley
2022-07-06  8:03 ` [PATCH v5 00/13] Canaan devicetree fixes Geert Uytterhoeven
2022-07-06  8:03   ` Geert Uytterhoeven
2022-07-06  8:03   ` Geert Uytterhoeven
2022-07-06  8:21   ` Conor.Dooley
2022-07-06  8:21     ` Conor.Dooley
2022-07-06  8:21     ` Conor.Dooley
2022-07-06 11:55   ` Damien Le Moal
2022-07-06 11:55     ` Damien Le Moal
2022-07-06 11:55     ` Damien Le Moal
2022-07-06 12:01     ` Conor.Dooley
2022-07-06 12:01       ` Conor.Dooley
2022-07-06 12:01       ` Conor.Dooley
2022-07-06  8:09 ` Geert Uytterhoeven
2022-07-06  8:09   ` Geert Uytterhoeven
2022-07-06  8:09   ` Geert Uytterhoeven
2022-07-14 22:04 ` Palmer Dabbelt
2022-07-14 22:04   ` Palmer Dabbelt
2022-07-14 22:04   ` Palmer Dabbelt
2022-07-14 22:11   ` Conor.Dooley
2022-07-14 22:11     ` Conor.Dooley
2022-07-14 22:11     ` Conor.Dooley
2022-08-05 17:51     ` Conor.Dooley
2022-08-05 17:51       ` Conor.Dooley
2022-08-05 17:51       ` Conor.Dooley
2022-08-10 22:01       ` Palmer Dabbelt
2022-08-10 22:01         ` Palmer Dabbelt
2022-08-10 22:01         ` Palmer Dabbelt
2022-08-11  6:26         ` Conor.Dooley
2022-08-11  6:26           ` Conor.Dooley
2022-08-11  6:26           ` 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=d1c1e2bd-39cb-4927-ac0b-22583ac7a420@microchip.com \
    --to=conor.dooley@microchip.com \
    --cc=Eugeniy.Paltsev@synopsys.com \
    --cc=airlied@linux.ie \
    --cc=aou@eecs.berkeley.edu \
    --cc=damien.lemoal@opensource.wdc.com \
    --cc=daniel.lezcano@linaro.org \
    --cc=daniel@ffwll.ch \
    --cc=devicetree@vger.kernel.org \
    --cc=dillon.minfei@gmail.com \
    --cc=dmaengine@vger.kernel.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=fancer.lancer@gmail.com \
    --cc=geert@linux-m68k.org \
    --cc=krzysztof.kozlowski+dt@linaro.org \
    --cc=krzysztof.kozlowski@linaro.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-riscv@lists.infradead.org \
    --cc=mail@conchuod.ie \
    --cc=masahiroy@kernel.org \
    --cc=niklas.cassel@wdc.com \
    --cc=palmer@dabbelt.com \
    --cc=palmer@rivosinc.com \
    --cc=paul.walmsley@sifive.com \
    --cc=robh+dt@kernel.org \
    --cc=sam@ravnborg.org \
    --cc=thierry.reding@gmail.com \
    --cc=vkoul@kernel.org \
    /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.