mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Ben Dooks <ben.dooks@codethink.co.uk>
To: Chen Wang <unicornxw@gmail.com>,
	aou@eecs.berkeley.edu, chao.wei@sophgo.com, conor@kernel.org,
	devicetree@vger.kernel.org, emil.renner.berthing@canonical.com,
	guoren@kernel.org, jszhang@kernel.org,
	krzysztof.kozlowski+dt@linaro.org, linux-kernel@vger.kernel.org,
	linux-riscv@lists.infradead.org, palmer@dabbelt.com,
	paul.walmsley@sifive.com, robh+dt@kernel.org,
	xiaoguang.xing@sophgo.com
Cc: Chen Wang <wangchen20@iscas.ac.cn>
Subject: Re: [PATCH v2 08/11] serial: 8250_dw: Add Sophgo SG2042 support
Date: Fri, 22 Sep 2023 10:41:04 +0100	[thread overview]
Message-ID: <bcdc4623-a37e-455b-8f10-c539e2b04554@codethink.co.uk> (raw)
In-Reply-To: <888d57a2d5e62affb8e29e0098402e428facd969.1695189879.git.wangchen20@iscas.ac.cn>

On 20/09/2023 07:40, Chen Wang wrote:
> From: Emil Renner Berthing <emil.renner.berthing@canonical.com>
> 
> Add quirk to skip setting the input clock rate for the uarts on the
> Sophgo SG2042 SoC similar to the StarFive JH7100.

I'd love an actual explanation of why this is necessary here.

> 
> Signed-off-by: Emil Renner Berthing <emil.renner.berthing@canonical.com>
> Signed-off-by: Chen Wang <wangchen20@iscas.ac.cn>
> ---
>   drivers/tty/serial/8250/8250_dw.c | 5 +++--
>   1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/tty/serial/8250/8250_dw.c b/drivers/tty/serial/8250/8250_dw.c
> index f4cafca1a7da..6c344877a07f 100644
> --- a/drivers/tty/serial/8250/8250_dw.c
> +++ b/drivers/tty/serial/8250/8250_dw.c
> @@ -770,7 +770,7 @@ static const struct dw8250_platform_data dw8250_renesas_rzn1_data = {
>   	.quirks = DW_UART_QUIRK_IS_DMA_FC,
>   };
>   
> -static const struct dw8250_platform_data dw8250_starfive_jh7100_data = {
> +static const struct dw8250_platform_data dw8250_skip_set_rate_data = {
>   	.usr_reg = DW_UART_USR,
>   	.quirks = DW_UART_QUIRK_SKIP_SET_RATE,
>   };
> @@ -780,7 +780,8 @@ static const struct of_device_id dw8250_of_match[] = {
>   	{ .compatible = "cavium,octeon-3860-uart", .data = &dw8250_octeon_3860_data },
>   	{ .compatible = "marvell,armada-38x-uart", .data = &dw8250_armada_38x_data },
>   	{ .compatible = "renesas,rzn1-uart", .data = &dw8250_renesas_rzn1_data },
> -	{ .compatible = "starfive,jh7100-uart", .data = &dw8250_starfive_jh7100_data },
> +	{ .compatible = "sophgo,sg2042-uart", .data = &dw8250_skip_set_rate_data },
> +	{ .compatible = "starfive,jh7100-uart", .data = &dw8250_skip_set_rate_data },
>   	{ /* Sentinel */ }
>   };
>   MODULE_DEVICE_TABLE(of, dw8250_of_match);

-- 
Ben Dooks				http://www.codethink.co.uk/
Senior Engineer				Codethink - Providing Genius

https://www.codethink.co.uk/privacy.html


  parent reply	other threads:[~2023-09-22  9:41 UTC|newest]

Thread overview: 88+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-09-20  6:33 [PATCH v2 00/11] Add Milk-V Pioneer RISC-V board support Chen Wang
2023-09-20  6:34 ` [PATCH v2 01/11] riscv: Add SOPHGO SOC family Kconfig support Chen Wang
2023-09-20  7:34   ` Guo Ren
2023-09-20  8:21   ` Conor Dooley
2023-09-20  6:37 ` [PATCH v2 02/11] dt-bindings: vendor-prefixes: add milkv/sophgo Chen Wang
2023-09-20  7:38   ` Guo Ren
2023-09-20  8:22   ` Conor Dooley
2023-09-20  9:14     ` Chen Wang
2023-09-20  6:37 ` [PATCH v2 03/11] dt-bindings: riscv: add sophgo sg2042 bindings Chen Wang
2023-09-20  7:43   ` Guo Ren
2023-09-20  8:28   ` Conor Dooley
2023-09-21 10:21     ` Chen Wang
2023-09-21 12:18       ` Conor Dooley
2023-09-21 13:40         ` Chen Wang
2023-09-21 13:51         ` Chen Wang
2023-09-21 14:00           ` Conor Dooley
2023-09-22  1:48     ` Chen Wang
2023-09-20 11:55   ` Krzysztof Kozlowski
2023-09-20 12:03     ` 汪辰
2023-09-21  0:48       ` Jisheng Zhang
2023-09-20  6:38 ` [PATCH v2 04/11] dt-bindings: riscv: Add T-HEAD C920 compatibles Chen Wang
2023-09-20  7:44   ` Guo Ren
2023-09-20  8:37   ` Conor Dooley
2023-09-20  6:39 ` [PATCH v2 05/11] dt-bindings: interrupt-controller: Add SOPHGO's SG2042 PLIC Chen Wang
2023-09-20  7:45   ` Guo Ren
2023-09-20  8:57   ` Conor Dooley
2023-09-20  6:39 ` [PATCH v2 06/11] dt-bindings: timer: Add Sophgo sg2042 clint Chen Wang
2023-09-20  8:12   ` Guo Ren
2023-09-20  8:50   ` Conor Dooley
2023-09-20  9:08     ` Inochi Amaoto
2023-09-20  9:53       ` Conor Dooley
2023-09-20 11:24         ` Inochi Amaoto
2023-09-20 13:03           ` Conor Dooley
2023-09-21  0:43             ` Inochi Amaoto
2023-09-21  8:05               ` Conor Dooley
2023-09-21  8:18                 ` Inochi Amaoto
2023-09-21  8:52                   ` Conor Dooley
2023-09-21  9:44                     ` Inochi Amaoto
2023-09-20 11:57   ` Krzysztof Kozlowski
2023-09-20 12:15     ` Inochi Amaoto
2023-09-20 12:30       ` Krzysztof Kozlowski
2023-09-20 12:40         ` Inochi Amaoto
2023-09-20 12:58           ` Conor Dooley
2023-09-20 13:09             ` Krzysztof Kozlowski
2023-09-20 14:38             ` Anup Patel
2023-09-20 14:51               ` Conor Dooley
2023-09-20 22:20                 ` Inochi Amaoto
2023-09-22  5:16                 ` Inochi Amaoto
2023-09-22  7:43                   ` Conor Dooley
2023-09-22  8:18                     ` Inochi Amaoto
2023-09-20  6:39 ` [PATCH v2 07/11] dt-bindings: serial: snps-dw-apb-uart: Add Sophgo SG2042 uarts Chen Wang
2023-09-20  7:51   ` Guo Ren
2023-09-20  8:37   ` Conor Dooley
2023-09-20  6:40 ` [PATCH v2 08/11] serial: 8250_dw: Add Sophgo SG2042 support Chen Wang
2023-09-20  7:53   ` Guo Ren
2023-09-20  8:05     ` Chen Wang
2023-09-20  8:08       ` Guo Ren
2023-09-22  9:41   ` Ben Dooks [this message]
2023-09-22 10:40     ` Emil Renner Berthing
2023-09-22 11:39       ` Chen Wang
2023-09-26  7:38       ` Chen Wang
2023-09-20  6:40 ` [PATCH v2 09/11] riscv: dts: add initial SOPHGO SG2042 SoC device tree Chen Wang
2023-09-20  8:04   ` Guo Ren
2023-09-20  8:57   ` Conor Dooley
2023-09-20  9:07     ` [PATCH v2 06/11] dt-bindings: timer: Add Sophgo sg2042 clint Inochi Amaoto
2023-09-21  9:56     ` [PATCH v2 09/11] riscv: dts: add initial SOPHGO SG2042 SoC device tree Chen Wang
2023-09-21 10:15       ` Conor Dooley
2023-09-21 10:27         ` Chen Wang
2023-09-21 12:06           ` Conor Dooley
2023-09-20 11:32   ` Emil Renner Berthing
2023-09-20 12:09     ` 汪辰
2023-09-20 12:32       ` Emil Renner Berthing
2023-09-20 12:37         ` 汪辰
2023-09-20 15:19   ` Palmer Dabbelt
2023-09-20 15:31     ` Conor Dooley
2023-09-20  6:40 ` [PATCH v2 10/11] riscv: dts: sophgo: add Milk-V Pioneer board " Chen Wang
2023-09-20  8:05   ` Guo Ren
2023-09-20  8:16   ` Conor Dooley
2023-09-20 11:59   ` Krzysztof Kozlowski
2023-09-20  6:41 ` [PATCH v2 11/11] riscv: defconfig: enable SOPHGO SoC Chen Wang
2023-09-20  8:06   ` Guo Ren
2023-09-20  8:58   ` Conor Dooley
2023-09-20 10:01 ` [PATCH v2 00/11] Add Milk-V Pioneer RISC-V board support Conor Dooley
2023-09-22 10:24   ` Chen Wang
2023-09-22 10:50     ` Conor Dooley
2023-09-22 11:28       ` Chen Wang
2023-09-20 15:22 ` Palmer Dabbelt
2023-09-26 10:29   ` Chen Wang

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=bcdc4623-a37e-455b-8f10-c539e2b04554@codethink.co.uk \
    --to=ben.dooks@codethink.co.uk \
    --cc=aou@eecs.berkeley.edu \
    --cc=chao.wei@sophgo.com \
    --cc=conor@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=emil.renner.berthing@canonical.com \
    --cc=guoren@kernel.org \
    --cc=jszhang@kernel.org \
    --cc=krzysztof.kozlowski+dt@linaro.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-riscv@lists.infradead.org \
    --cc=palmer@dabbelt.com \
    --cc=paul.walmsley@sifive.com \
    --cc=robh+dt@kernel.org \
    --cc=unicornxw@gmail.com \
    --cc=wangchen20@iscas.ac.cn \
    --cc=xiaoguang.xing@sophgo.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 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®