* [PATCH v1 0/2] Add 1024x768 timing for AST2600
@ 2022-03-04 6:31 Tommy Haung
2022-03-04 6:31 ` [PATCH v1 1/2] drm/aspeed: Add gfx flags and clock selection " Tommy Haung
2022-03-04 6:31 ` [PATCH v1 2/2] drm/aspeed: Add 1024x768 mode " Tommy Haung
0 siblings, 2 replies; 8+ messages in thread
From: Tommy Haung @ 2022-03-04 6:31 UTC (permalink / raw)
To: joel, airlied, daniel, robh+dt, andrew, linux-aspeed, dri-devel,
devicetree, linux-arm-kernel, linux-kernel
Cc: BMC-SW
v1:
Add 1024x768@70Hz for AST2600 soc display timing selection.
Add gfx flag for future usage.
Testing steps:
1. Add below config to turn VT and LOGO on.
CONFIG_TTY=y
CONFIG_VT=y
CONFIG_CONSOLE_TRANSLATIONS=y
CONFIG_VT_CONSOLE=y
CONFIG_VT_CONSOLE_SLEEP=y
CONFIG_HW_CONSOLE=y
CONFIG_VT_HW_CONSOLE_BINDING=y
CONFIG_UNIX98_PTYS=y
CONFIG_LDISC_AUTOLOAD=y
CONFIG_DEVMEM=y
CONFIG_DUMMY_CONSOLE=y
CONFIG_FRAMEBUFFER_CONSOLE=y
CONFIG_FRAMEBUFFER_CONSOLE_DETECT_PRIMARY=y
CONFIG_LOGO=y
CONFIG_LOGO_LINUX_CLUT224=y
2. The Linux logo will be shown on the screen, when the BMC boot in Linux.
3. Check the display mode is 1024x768@70Hz at AST2600.
4. Check the display mode is 800x600@60Hz at AST2500.
Tommy Haung (2):
drm/aspeed: Add gfx flags and clock selection for AST2600
drm/aspeed: Add 1024x768 mode for AST2600
drivers/gpu/drm/aspeed/aspeed_gfx.h | 15 ++++++++++
drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c | 35 ++++++++++++++++++++++++
drivers/gpu/drm/aspeed/aspeed_gfx_drv.c | 20 ++++++++++++--
drivers/gpu/drm/aspeed/aspeed_gfx_out.c | 14 +++++++++-
4 files changed, 81 insertions(+), 3 deletions(-)
--
2.17.1
^ permalink raw reply [flat|nested] 8+ messages in thread* [PATCH v1 1/2] drm/aspeed: Add gfx flags and clock selection for AST2600 2022-03-04 6:31 [PATCH v1 0/2] Add 1024x768 timing for AST2600 Tommy Haung @ 2022-03-04 6:31 ` Tommy Haung 2022-03-04 6:31 ` [PATCH v1 2/2] drm/aspeed: Add 1024x768 mode " Tommy Haung 1 sibling, 0 replies; 8+ messages in thread From: Tommy Haung @ 2022-03-04 6:31 UTC (permalink / raw) To: joel, airlied, daniel, robh+dt, andrew, linux-aspeed, dri-devel, devicetree, linux-arm-kernel, linux-kernel Cc: BMC-SW Add clock selection code for AST2600. At AST2600 user could select more than one dispaly timing. Add gfx flags for future usage. Signed-off-by: Tommy Haung <tommy_huang@aspeedtech.com> --- drivers/gpu/drm/aspeed/aspeed_gfx.h | 11 +++++++++++ drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c | 14 ++++++++++++++ drivers/gpu/drm/aspeed/aspeed_gfx_drv.c | 4 ++++ 3 files changed, 29 insertions(+) diff --git a/drivers/gpu/drm/aspeed/aspeed_gfx.h b/drivers/gpu/drm/aspeed/aspeed_gfx.h index 4e6a442c3886..eb4c267cde5e 100644 --- a/drivers/gpu/drm/aspeed/aspeed_gfx.h +++ b/drivers/gpu/drm/aspeed/aspeed_gfx.h @@ -16,6 +16,7 @@ struct aspeed_gfx { u32 vga_scratch_reg; u32 throd_val; u32 scan_line_max; + u32 flags; struct drm_simple_display_pipe pipe; struct drm_connector connector; @@ -106,3 +107,13 @@ int aspeed_gfx_create_output(struct drm_device *drm); /* CRT_THROD */ #define CRT_THROD_LOW(x) (x) #define CRT_THROD_HIGH(x) ((x) << 8) + +/* SCU control */ +#define SCU_G6_CLK_COURCE 0x300 + +/* GFX FLAGS */ +#define CLK_MASK BIT(0) +#define CLK_G6 BIT(0) + +#define G6_CLK_MASK (BIT(8) | BIT(9) | BIT(10)) +#define G6_USB_40_CLK BIT(9) diff --git a/drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c b/drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c index 827e62c1daba..a24fab22eac4 100644 --- a/drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c +++ b/drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c @@ -77,6 +77,18 @@ static void aspeed_gfx_disable_controller(struct aspeed_gfx *priv) regmap_update_bits(priv->scu, priv->dac_reg, BIT(16), 0); } +static void aspeed_gfx_set_clk(struct aspeed_gfx *priv) +{ + switch (priv->flags & CLK_MASK) { + case CLK_G6: + regmap_update_bits(priv->scu, SCU_G6_CLK_COURCE, G6_CLK_MASK, 0x0); + regmap_update_bits(priv->scu, SCU_G6_CLK_COURCE, G6_CLK_MASK, G6_USB_40_CLK); + break; + default: + break; + } +} + static void aspeed_gfx_crtc_mode_set_nofb(struct aspeed_gfx *priv) { struct drm_display_mode *m = &priv->pipe.crtc.state->adjusted_mode; @@ -87,6 +99,8 @@ static void aspeed_gfx_crtc_mode_set_nofb(struct aspeed_gfx *priv) if (err) return; + aspeed_gfx_set_clk(priv); + #if 0 /* TODO: we have only been able to test with the 40MHz USB clock. The * clock is fixed, so we cannot adjust it here. */ diff --git a/drivers/gpu/drm/aspeed/aspeed_gfx_drv.c b/drivers/gpu/drm/aspeed/aspeed_gfx_drv.c index d10246b1d1c2..af56ffdccc65 100644 --- a/drivers/gpu/drm/aspeed/aspeed_gfx_drv.c +++ b/drivers/gpu/drm/aspeed/aspeed_gfx_drv.c @@ -64,6 +64,7 @@ struct aspeed_gfx_config { u32 vga_scratch_reg; /* VGA scratch register in SCU */ u32 throd_val; /* Default Threshold Seting */ u32 scan_line_max; /* Max memory size of one scan line */ + u32 gfx_flags; /* Flags for gfx chip caps */ }; static const struct aspeed_gfx_config ast2400_config = { @@ -72,6 +73,7 @@ static const struct aspeed_gfx_config ast2400_config = { .vga_scratch_reg = 0x50, .throd_val = CRT_THROD_LOW(0x1e) | CRT_THROD_HIGH(0x12), .scan_line_max = 64, + .gfx_flags = 0, }; static const struct aspeed_gfx_config ast2500_config = { @@ -80,6 +82,7 @@ static const struct aspeed_gfx_config ast2500_config = { .vga_scratch_reg = 0x50, .throd_val = CRT_THROD_LOW(0x24) | CRT_THROD_HIGH(0x3c), .scan_line_max = 128, + .gfx_flags = 0, }; static const struct aspeed_gfx_config ast2600_config = { @@ -88,6 +91,7 @@ static const struct aspeed_gfx_config ast2600_config = { .vga_scratch_reg = 0x50, .throd_val = CRT_THROD_LOW(0x50) | CRT_THROD_HIGH(0x70), .scan_line_max = 128, + .gfx_flags = CLK_G6, }; static const struct of_device_id aspeed_gfx_match[] = { -- 2.17.1 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH v1 2/2] drm/aspeed: Add 1024x768 mode for AST2600 2022-03-04 6:31 [PATCH v1 0/2] Add 1024x768 timing for AST2600 Tommy Haung 2022-03-04 6:31 ` [PATCH v1 1/2] drm/aspeed: Add gfx flags and clock selection " Tommy Haung @ 2022-03-04 6:31 ` Tommy Haung 2022-04-26 3:27 ` Joel Stanley 2022-04-26 7:47 ` Joel Stanley 1 sibling, 2 replies; 8+ messages in thread From: Tommy Haung @ 2022-03-04 6:31 UTC (permalink / raw) To: joel, airlied, daniel, robh+dt, andrew, linux-aspeed, dri-devel, devicetree, linux-arm-kernel, linux-kernel Cc: BMC-SW Update the aspeed_gfx_set_clk with display width. At AST2600, the display clock could be coming from HPLL clock / 16 = 75MHz. It would fit 1024x768@70Hz. Another chip will still keep 800x600. Signed-off-by: Tommy Haung <tommy_huang@aspeedtech.com> --- drivers/gpu/drm/aspeed/aspeed_gfx.h | 12 ++++++---- drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c | 29 ++++++++++++++++++++---- drivers/gpu/drm/aspeed/aspeed_gfx_drv.c | 16 +++++++++++-- drivers/gpu/drm/aspeed/aspeed_gfx_out.c | 14 +++++++++++- 4 files changed, 60 insertions(+), 11 deletions(-) diff --git a/drivers/gpu/drm/aspeed/aspeed_gfx.h b/drivers/gpu/drm/aspeed/aspeed_gfx.h index eb4c267cde5e..c7aefee0657a 100644 --- a/drivers/gpu/drm/aspeed/aspeed_gfx.h +++ b/drivers/gpu/drm/aspeed/aspeed_gfx.h @@ -109,11 +109,15 @@ int aspeed_gfx_create_output(struct drm_device *drm); #define CRT_THROD_HIGH(x) ((x) << 8) /* SCU control */ -#define SCU_G6_CLK_COURCE 0x300 +#define G6_CLK_SOURCE 0x300 +#define G6_CLK_SOURCE_MASK (BIT(8) | BIT(9) | BIT(10)) +#define G6_CLK_SOURCE_HPLL (BIT(8) | BIT(9) | BIT(10)) +#define G6_CLK_SOURCE_USB BIT(9) +#define G6_CLK_SEL3 0x308 +#define G6_CLK_DIV_MASK 0x3F000 +#define G6_CLK_DIV_16 (BIT(16)|BIT(15)|BIT(13)|BIT(12)) +#define G6_USB_40_CLK BIT(9) /* GFX FLAGS */ #define CLK_MASK BIT(0) #define CLK_G6 BIT(0) - -#define G6_CLK_MASK (BIT(8) | BIT(9) | BIT(10)) -#define G6_USB_40_CLK BIT(9) diff --git a/drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c b/drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c index a24fab22eac4..5829be9c7c67 100644 --- a/drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c +++ b/drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c @@ -23,6 +23,28 @@ drm_pipe_to_aspeed_gfx(struct drm_simple_display_pipe *pipe) return container_of(pipe, struct aspeed_gfx, pipe); } +static void aspeed_gfx_set_clock_source(struct aspeed_gfx *priv, int mode_width) +{ + regmap_update_bits(priv->scu, G6_CLK_SOURCE, G6_CLK_SOURCE_MASK, 0x0); + regmap_update_bits(priv->scu, G6_CLK_SEL3, G6_CLK_DIV_MASK, 0x0); + + switch (mode_width) { + case 1024: + /* hpll div 16 = 75Mhz */ + regmap_update_bits(priv->scu, G6_CLK_SOURCE, + G6_CLK_SOURCE_MASK, G6_CLK_SOURCE_HPLL); + regmap_update_bits(priv->scu, G6_CLK_SEL3, + G6_CLK_DIV_MASK, G6_CLK_DIV_16); + break; + case 800: + default: + /* usb 40Mhz */ + regmap_update_bits(priv->scu, G6_CLK_SOURCE, + G6_CLK_SOURCE_MASK, G6_CLK_SOURCE_USB); + break; + } +} + static int aspeed_gfx_set_pixel_fmt(struct aspeed_gfx *priv, u32 *bpp) { struct drm_crtc *crtc = &priv->pipe.crtc; @@ -77,12 +99,11 @@ static void aspeed_gfx_disable_controller(struct aspeed_gfx *priv) regmap_update_bits(priv->scu, priv->dac_reg, BIT(16), 0); } -static void aspeed_gfx_set_clk(struct aspeed_gfx *priv) +static void aspeed_gfx_set_clk(struct aspeed_gfx *priv, int mode_width) { switch (priv->flags & CLK_MASK) { case CLK_G6: - regmap_update_bits(priv->scu, SCU_G6_CLK_COURCE, G6_CLK_MASK, 0x0); - regmap_update_bits(priv->scu, SCU_G6_CLK_COURCE, G6_CLK_MASK, G6_USB_40_CLK); + aspeed_gfx_set_clock_source(priv, mode_width); break; default: break; @@ -99,7 +120,7 @@ static void aspeed_gfx_crtc_mode_set_nofb(struct aspeed_gfx *priv) if (err) return; - aspeed_gfx_set_clk(priv); + aspeed_gfx_set_clk(priv, m->hdisplay); #if 0 /* TODO: we have only been able to test with the 40MHz USB clock. The diff --git a/drivers/gpu/drm/aspeed/aspeed_gfx_drv.c b/drivers/gpu/drm/aspeed/aspeed_gfx_drv.c index af56ffdccc65..e1a814aebc2d 100644 --- a/drivers/gpu/drm/aspeed/aspeed_gfx_drv.c +++ b/drivers/gpu/drm/aspeed/aspeed_gfx_drv.c @@ -110,6 +110,7 @@ static const struct drm_mode_config_funcs aspeed_gfx_mode_config_funcs = { static int aspeed_gfx_setup_mode_config(struct drm_device *drm) { + struct aspeed_gfx *priv = to_aspeed_gfx(drm); int ret; ret = drmm_mode_config_init(drm); @@ -118,8 +119,18 @@ static int aspeed_gfx_setup_mode_config(struct drm_device *drm) drm->mode_config.min_width = 0; drm->mode_config.min_height = 0; - drm->mode_config.max_width = 800; - drm->mode_config.max_height = 600; + + switch (priv->flags & CLK_MASK) { + case CLK_G6: + drm->mode_config.max_width = 1024; + drm->mode_config.max_height = 768; + break; + default: + drm->mode_config.max_width = 800; + drm->mode_config.max_height = 600; + break; + } + drm->mode_config.funcs = &aspeed_gfx_mode_config_funcs; return ret; @@ -167,6 +178,7 @@ static int aspeed_gfx_load(struct drm_device *drm) priv->vga_scratch_reg = config->vga_scratch_reg; priv->throd_val = config->throd_val; priv->scan_line_max = config->scan_line_max; + priv->flags = config->gfx_flags; priv->scu = syscon_regmap_lookup_by_phandle(np, "syscon"); if (IS_ERR(priv->scu)) { diff --git a/drivers/gpu/drm/aspeed/aspeed_gfx_out.c b/drivers/gpu/drm/aspeed/aspeed_gfx_out.c index 6759cb88415a..5d5e04f15c59 100644 --- a/drivers/gpu/drm/aspeed/aspeed_gfx_out.c +++ b/drivers/gpu/drm/aspeed/aspeed_gfx_out.c @@ -10,7 +10,19 @@ static int aspeed_gfx_get_modes(struct drm_connector *connector) { - return drm_add_modes_noedid(connector, 800, 600); + struct aspeed_gfx *priv = container_of(connector, struct aspeed_gfx, connector); + int mode_count = 0; + + switch (priv->flags & CLK_MASK) { + case CLK_G6: + mode_count = drm_add_modes_noedid(connector, 1024, 768); + break; + default: + mode_count = drm_add_modes_noedid(connector, 800, 600); + break; + } + + return mode_count; } static const struct -- 2.17.1 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH v1 2/2] drm/aspeed: Add 1024x768 mode for AST2600 2022-03-04 6:31 ` [PATCH v1 2/2] drm/aspeed: Add 1024x768 mode " Tommy Haung @ 2022-04-26 3:27 ` Joel Stanley 2022-04-26 5:20 ` Tommy Huang 2022-04-26 7:47 ` Joel Stanley 1 sibling, 1 reply; 8+ messages in thread From: Joel Stanley @ 2022-04-26 3:27 UTC (permalink / raw) To: Tommy Haung Cc: David Airlie, Daniel Vetter, Rob Herring, Andrew Jeffery, linux-aspeed, open list:DRM DRIVERS, devicetree, Linux ARM, Linux Kernel Mailing List, BMC-SW On Fri, 4 Mar 2022 at 06:32, Tommy Haung <tommy_huang@aspeedtech.com> wrote: > > Update the aspeed_gfx_set_clk with display width. > At AST2600, the display clock could be coming from > HPLL clock / 16 = 75MHz. It would fit 1024x768@70Hz. > Another chip will still keep 800x600. > > Signed-off-by: Tommy Haung <tommy_huang@aspeedtech.com> > --- > drivers/gpu/drm/aspeed/aspeed_gfx.h | 12 ++++++---- > drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c | 29 ++++++++++++++++++++---- > drivers/gpu/drm/aspeed/aspeed_gfx_drv.c | 16 +++++++++++-- > drivers/gpu/drm/aspeed/aspeed_gfx_out.c | 14 +++++++++++- > 4 files changed, 60 insertions(+), 11 deletions(-) > > diff --git a/drivers/gpu/drm/aspeed/aspeed_gfx.h b/drivers/gpu/drm/aspeed/aspeed_gfx.h > index eb4c267cde5e..c7aefee0657a 100644 > --- a/drivers/gpu/drm/aspeed/aspeed_gfx.h > +++ b/drivers/gpu/drm/aspeed/aspeed_gfx.h > @@ -109,11 +109,15 @@ int aspeed_gfx_create_output(struct drm_device *drm); > #define CRT_THROD_HIGH(x) ((x) << 8) > > /* SCU control */ > -#define SCU_G6_CLK_COURCE 0x300 > +#define G6_CLK_SOURCE 0x300 > +#define G6_CLK_SOURCE_MASK (BIT(8) | BIT(9) | BIT(10)) > +#define G6_CLK_SOURCE_HPLL (BIT(8) | BIT(9) | BIT(10)) > +#define G6_CLK_SOURCE_USB BIT(9) > +#define G6_CLK_SEL3 0x308 > +#define G6_CLK_DIV_MASK 0x3F000 > +#define G6_CLK_DIV_16 (BIT(16)|BIT(15)|BIT(13)|BIT(12)) > +#define G6_USB_40_CLK BIT(9) > > /* GFX FLAGS */ > #define CLK_MASK BIT(0) > #define CLK_G6 BIT(0) > - > -#define G6_CLK_MASK (BIT(8) | BIT(9) | BIT(10)) > -#define G6_USB_40_CLK BIT(9) > diff --git a/drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c b/drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c > index a24fab22eac4..5829be9c7c67 100644 > --- a/drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c > +++ b/drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c > @@ -23,6 +23,28 @@ drm_pipe_to_aspeed_gfx(struct drm_simple_display_pipe *pipe) > return container_of(pipe, struct aspeed_gfx, pipe); > } > > +static void aspeed_gfx_set_clock_source(struct aspeed_gfx *priv, int mode_width) > +{ > + regmap_update_bits(priv->scu, G6_CLK_SOURCE, G6_CLK_SOURCE_MASK, 0x0); > + regmap_update_bits(priv->scu, G6_CLK_SEL3, G6_CLK_DIV_MASK, 0x0); > + > + switch (mode_width) { > + case 1024: > + /* hpll div 16 = 75Mhz */ > + regmap_update_bits(priv->scu, G6_CLK_SOURCE, > + G6_CLK_SOURCE_MASK, G6_CLK_SOURCE_HPLL); > + regmap_update_bits(priv->scu, G6_CLK_SEL3, > + G6_CLK_DIV_MASK, G6_CLK_DIV_16); > + break; > + case 800: > + default: > + /* usb 40Mhz */ > + regmap_update_bits(priv->scu, G6_CLK_SOURCE, > + G6_CLK_SOURCE_MASK, G6_CLK_SOURCE_USB); > + break; > + } I'm not familiar with this area, but I think this belongs in the clock driver. We want to be able to call clk_set_rate() from the drm driver and have the clock driver update the correct bits in the SCU. Instead of specialising the 2600 vs others, could clk_set_rate() fail on the others, and cause the driver to stay at 800x600. If the set succeeds it can then run at the higher resolution. If this is not how the APIs work, we could instead have a clock_rate in struct aspeed_gfx and each platform can define its expected clock rate. It would then need a corresponding resolution. Please take a look at other drivers and see what they do. > +} > + > static int aspeed_gfx_set_pixel_fmt(struct aspeed_gfx *priv, u32 *bpp) > { > struct drm_crtc *crtc = &priv->pipe.crtc; > @@ -77,12 +99,11 @@ static void aspeed_gfx_disable_controller(struct aspeed_gfx *priv) > regmap_update_bits(priv->scu, priv->dac_reg, BIT(16), 0); > } > > -static void aspeed_gfx_set_clk(struct aspeed_gfx *priv) > +static void aspeed_gfx_set_clk(struct aspeed_gfx *priv, int mode_width) > { > switch (priv->flags & CLK_MASK) { > case CLK_G6: > - regmap_update_bits(priv->scu, SCU_G6_CLK_COURCE, G6_CLK_MASK, 0x0); > - regmap_update_bits(priv->scu, SCU_G6_CLK_COURCE, G6_CLK_MASK, G6_USB_40_CLK); > + aspeed_gfx_set_clock_source(priv, mode_width); > break; > default: > break; > @@ -99,7 +120,7 @@ static void aspeed_gfx_crtc_mode_set_nofb(struct aspeed_gfx *priv) > if (err) > return; > > - aspeed_gfx_set_clk(priv); > + aspeed_gfx_set_clk(priv, m->hdisplay); > > #if 0 > /* TODO: we have only been able to test with the 40MHz USB clock. The > diff --git a/drivers/gpu/drm/aspeed/aspeed_gfx_drv.c b/drivers/gpu/drm/aspeed/aspeed_gfx_drv.c > index af56ffdccc65..e1a814aebc2d 100644 > --- a/drivers/gpu/drm/aspeed/aspeed_gfx_drv.c > +++ b/drivers/gpu/drm/aspeed/aspeed_gfx_drv.c > @@ -110,6 +110,7 @@ static const struct drm_mode_config_funcs aspeed_gfx_mode_config_funcs = { > > static int aspeed_gfx_setup_mode_config(struct drm_device *drm) > { > + struct aspeed_gfx *priv = to_aspeed_gfx(drm); > int ret; > > ret = drmm_mode_config_init(drm); > @@ -118,8 +119,18 @@ static int aspeed_gfx_setup_mode_config(struct drm_device *drm) > > drm->mode_config.min_width = 0; > drm->mode_config.min_height = 0; > - drm->mode_config.max_width = 800; > - drm->mode_config.max_height = 600; > + > + switch (priv->flags & CLK_MASK) { > + case CLK_G6: > + drm->mode_config.max_width = 1024; > + drm->mode_config.max_height = 768; > + break; > + default: > + drm->mode_config.max_width = 800; > + drm->mode_config.max_height = 600; > + break; > + } > + > drm->mode_config.funcs = &aspeed_gfx_mode_config_funcs; > > return ret; > @@ -167,6 +178,7 @@ static int aspeed_gfx_load(struct drm_device *drm) > priv->vga_scratch_reg = config->vga_scratch_reg; > priv->throd_val = config->throd_val; > priv->scan_line_max = config->scan_line_max; > + priv->flags = config->gfx_flags; > > priv->scu = syscon_regmap_lookup_by_phandle(np, "syscon"); > if (IS_ERR(priv->scu)) { > diff --git a/drivers/gpu/drm/aspeed/aspeed_gfx_out.c b/drivers/gpu/drm/aspeed/aspeed_gfx_out.c > index 6759cb88415a..5d5e04f15c59 100644 > --- a/drivers/gpu/drm/aspeed/aspeed_gfx_out.c > +++ b/drivers/gpu/drm/aspeed/aspeed_gfx_out.c > @@ -10,7 +10,19 @@ > > static int aspeed_gfx_get_modes(struct drm_connector *connector) > { > - return drm_add_modes_noedid(connector, 800, 600); > + struct aspeed_gfx *priv = container_of(connector, struct aspeed_gfx, connector); > + int mode_count = 0; > + > + switch (priv->flags & CLK_MASK) { > + case CLK_G6: > + mode_count = drm_add_modes_noedid(connector, 1024, 768); > + break; > + default: > + mode_count = drm_add_modes_noedid(connector, 800, 600); > + break; > + } > + > + return mode_count; > } > > static const struct > -- > 2.17.1 > ^ permalink raw reply [flat|nested] 8+ messages in thread
* RE: [PATCH v1 2/2] drm/aspeed: Add 1024x768 mode for AST2600 2022-04-26 3:27 ` Joel Stanley @ 2022-04-26 5:20 ` Tommy Huang 0 siblings, 0 replies; 8+ messages in thread From: Tommy Huang @ 2022-04-26 5:20 UTC (permalink / raw) To: Joel Stanley Cc: David Airlie, Daniel Vetter, Rob Herring, Andrew Jeffery, linux-aspeed, open list:DRM DRIVERS, devicetree, Linux ARM, Linux Kernel Mailing List, BMC-SW Hi Joel, > -----Original Message----- > From: Joel Stanley <joel@jms.id.au> > Sent: Tuesday, April 26, 2022 11:27 AM > To: Tommy Huang <tommy_huang@aspeedtech.com> > Cc: David Airlie <airlied@linux.ie>; Daniel Vetter <daniel@ffwll.ch>; Rob > Herring <robh+dt@kernel.org>; Andrew Jeffery <andrew@aj.id.au>; > linux-aspeed <linux-aspeed@lists.ozlabs.org>; open list:DRM DRIVERS > <dri-devel@lists.freedesktop.org>; devicetree <devicetree@vger.kernel.org>; > Linux ARM <linux-arm-kernel@lists.infradead.org>; Linux Kernel Mailing List > <linux-kernel@vger.kernel.org>; BMC-SW <BMC-SW@aspeedtech.com> > Subject: Re: [PATCH v1 2/2] drm/aspeed: Add 1024x768 mode for AST2600 > > On Fri, 4 Mar 2022 at 06:32, Tommy Haung <tommy_huang@aspeedtech.com> > wrote: > > > > Update the aspeed_gfx_set_clk with display width. > > At AST2600, the display clock could be coming from HPLL clock / 16 = > > 75MHz. It would fit 1024x768@70Hz. > > Another chip will still keep 800x600. > > > > Signed-off-by: Tommy Haung <tommy_huang@aspeedtech.com> > > --- > > drivers/gpu/drm/aspeed/aspeed_gfx.h | 12 ++++++---- > > drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c | 29 > > ++++++++++++++++++++---- drivers/gpu/drm/aspeed/aspeed_gfx_drv.c | > > 16 +++++++++++-- drivers/gpu/drm/aspeed/aspeed_gfx_out.c | 14 > > +++++++++++- > > 4 files changed, 60 insertions(+), 11 deletions(-) > > > > diff --git a/drivers/gpu/drm/aspeed/aspeed_gfx.h > > b/drivers/gpu/drm/aspeed/aspeed_gfx.h > > index eb4c267cde5e..c7aefee0657a 100644 > > --- a/drivers/gpu/drm/aspeed/aspeed_gfx.h > > +++ b/drivers/gpu/drm/aspeed/aspeed_gfx.h > > @@ -109,11 +109,15 @@ int aspeed_gfx_create_output(struct drm_device > *drm); > > #define CRT_THROD_HIGH(x) ((x) << 8) > > > > /* SCU control */ > > -#define SCU_G6_CLK_COURCE 0x300 > > +#define G6_CLK_SOURCE 0x300 > > +#define G6_CLK_SOURCE_MASK (BIT(8) | BIT(9) | BIT(10)) > > +#define G6_CLK_SOURCE_HPLL (BIT(8) | BIT(9) | BIT(10)) > > +#define G6_CLK_SOURCE_USB BIT(9) > > +#define G6_CLK_SEL3 0x308 > > +#define G6_CLK_DIV_MASK 0x3F000 > > +#define G6_CLK_DIV_16 > (BIT(16)|BIT(15)|BIT(13)|BIT(12)) > > +#define G6_USB_40_CLK BIT(9) > > > > /* GFX FLAGS */ > > #define CLK_MASK BIT(0) > > #define CLK_G6 BIT(0) > > - > > -#define G6_CLK_MASK (BIT(8) | BIT(9) | BIT(10)) > > -#define G6_USB_40_CLK BIT(9) > > diff --git a/drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c > > b/drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c > > index a24fab22eac4..5829be9c7c67 100644 > > --- a/drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c > > +++ b/drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c > > @@ -23,6 +23,28 @@ drm_pipe_to_aspeed_gfx(struct > drm_simple_display_pipe *pipe) > > return container_of(pipe, struct aspeed_gfx, pipe); } > > > > +static void aspeed_gfx_set_clock_source(struct aspeed_gfx *priv, int > > +mode_width) { > > + regmap_update_bits(priv->scu, G6_CLK_SOURCE, > G6_CLK_SOURCE_MASK, 0x0); > > + regmap_update_bits(priv->scu, G6_CLK_SEL3, G6_CLK_DIV_MASK, > > +0x0); > > + > > + switch (mode_width) { > > + case 1024: > > + /* hpll div 16 = 75Mhz */ > > + regmap_update_bits(priv->scu, G6_CLK_SOURCE, > > + G6_CLK_SOURCE_MASK, G6_CLK_SOURCE_HPLL); > > + regmap_update_bits(priv->scu, G6_CLK_SEL3, > > + G6_CLK_DIV_MASK, G6_CLK_DIV_16); > > + break; > > + case 800: > > + default: > > + /* usb 40Mhz */ > > + regmap_update_bits(priv->scu, G6_CLK_SOURCE, > > + G6_CLK_SOURCE_MASK, G6_CLK_SOURCE_USB); > > + break; > > + } > > I'm not familiar with this area, but I think this belongs in the clock driver. > > We want to be able to call clk_set_rate() from the drm driver and have the > clock driver update the correct bits in the SCU. > > Instead of specialising the 2600 vs others, could clk_set_rate() fail on the > others, and cause the driver to stay at 800x600. If the set succeeds it can then > run at the higher resolution. If this is not how the APIs work, we could instead > have a clock_rate in struct aspeed_gfx and each platform can define its > expected clock rate. It would then need a corresponding resolution. > > Please take a look at other drivers and see what they do. Thanks for your common. The pixel clock selection is very limit on soc display. Until now, there are just two resolutions (800x600@60 and 1024x768@70)could be selected at ast2600. And there is just 800x600@60 could be selected at ast2500 and ast2400. Although I also want to have clk_set_rate() function to select flexible pixel clock with a programable pll. We might define support clock rate in every device structure for this situation. > > > > > +} > > + > > static int aspeed_gfx_set_pixel_fmt(struct aspeed_gfx *priv, u32 > > *bpp) { > > struct drm_crtc *crtc = &priv->pipe.crtc; @@ -77,12 +99,11 @@ > > static void aspeed_gfx_disable_controller(struct aspeed_gfx *priv) > > regmap_update_bits(priv->scu, priv->dac_reg, BIT(16), 0); } > > > > -static void aspeed_gfx_set_clk(struct aspeed_gfx *priv) > > +static void aspeed_gfx_set_clk(struct aspeed_gfx *priv, int > > +mode_width) > > { > > switch (priv->flags & CLK_MASK) { > > case CLK_G6: > > - regmap_update_bits(priv->scu, SCU_G6_CLK_COURCE, > G6_CLK_MASK, 0x0); > > - regmap_update_bits(priv->scu, SCU_G6_CLK_COURCE, > G6_CLK_MASK, G6_USB_40_CLK); > > + aspeed_gfx_set_clock_source(priv, mode_width); > > break; > > default: > > break; > > @@ -99,7 +120,7 @@ static void aspeed_gfx_crtc_mode_set_nofb(struct > aspeed_gfx *priv) > > if (err) > > return; > > > > - aspeed_gfx_set_clk(priv); > > + aspeed_gfx_set_clk(priv, m->hdisplay); > > > > #if 0 > > /* TODO: we have only been able to test with the 40MHz USB > > clock. The diff --git a/drivers/gpu/drm/aspeed/aspeed_gfx_drv.c > > b/drivers/gpu/drm/aspeed/aspeed_gfx_drv.c > > index af56ffdccc65..e1a814aebc2d 100644 > > --- a/drivers/gpu/drm/aspeed/aspeed_gfx_drv.c > > +++ b/drivers/gpu/drm/aspeed/aspeed_gfx_drv.c > > @@ -110,6 +110,7 @@ static const struct drm_mode_config_funcs > > aspeed_gfx_mode_config_funcs = { > > > > static int aspeed_gfx_setup_mode_config(struct drm_device *drm) { > > + struct aspeed_gfx *priv = to_aspeed_gfx(drm); > > int ret; > > > > ret = drmm_mode_config_init(drm); @@ -118,8 +119,18 @@ > static > > int aspeed_gfx_setup_mode_config(struct drm_device *drm) > > > > drm->mode_config.min_width = 0; > > drm->mode_config.min_height = 0; > > - drm->mode_config.max_width = 800; > > - drm->mode_config.max_height = 600; > > + > > + switch (priv->flags & CLK_MASK) { > > + case CLK_G6: > > + drm->mode_config.max_width = 1024; > > + drm->mode_config.max_height = 768; > > + break; > > + default: > > + drm->mode_config.max_width = 800; > > + drm->mode_config.max_height = 600; > > + break; > > + } > > + > > drm->mode_config.funcs = &aspeed_gfx_mode_config_funcs; > > > > return ret; > > @@ -167,6 +178,7 @@ static int aspeed_gfx_load(struct drm_device *drm) > > priv->vga_scratch_reg = config->vga_scratch_reg; > > priv->throd_val = config->throd_val; > > priv->scan_line_max = config->scan_line_max; > > + priv->flags = config->gfx_flags; > > > > priv->scu = syscon_regmap_lookup_by_phandle(np, "syscon"); > > if (IS_ERR(priv->scu)) { > > diff --git a/drivers/gpu/drm/aspeed/aspeed_gfx_out.c > > b/drivers/gpu/drm/aspeed/aspeed_gfx_out.c > > index 6759cb88415a..5d5e04f15c59 100644 > > --- a/drivers/gpu/drm/aspeed/aspeed_gfx_out.c > > +++ b/drivers/gpu/drm/aspeed/aspeed_gfx_out.c > > @@ -10,7 +10,19 @@ > > > > static int aspeed_gfx_get_modes(struct drm_connector *connector) { > > - return drm_add_modes_noedid(connector, 800, 600); > > + struct aspeed_gfx *priv = container_of(connector, struct > aspeed_gfx, connector); > > + int mode_count = 0; > > + > > + switch (priv->flags & CLK_MASK) { > > + case CLK_G6: > > + mode_count = drm_add_modes_noedid(connector, 1024, > 768); > > + break; > > + default: > > + mode_count = drm_add_modes_noedid(connector, 800, > 600); > > + break; > > + } > > + > > + return mode_count; > > } > > > > static const struct > > -- > > 2.17.1 > > ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v1 2/2] drm/aspeed: Add 1024x768 mode for AST2600 2022-03-04 6:31 ` [PATCH v1 2/2] drm/aspeed: Add 1024x768 mode " Tommy Haung 2022-04-26 3:27 ` Joel Stanley @ 2022-04-26 7:47 ` Joel Stanley 2022-04-26 8:28 ` Tommy Huang 1 sibling, 1 reply; 8+ messages in thread From: Joel Stanley @ 2022-04-26 7:47 UTC (permalink / raw) To: Tommy Haung Cc: David Airlie, Daniel Vetter, Rob Herring, Andrew Jeffery, linux-aspeed, open list:DRM DRIVERS, devicetree, Linux ARM, Linux Kernel Mailing List, BMC-SW On Fri, 4 Mar 2022 at 06:32, Tommy Haung <tommy_huang@aspeedtech.com> wrote: > > Update the aspeed_gfx_set_clk with display width. > At AST2600, the display clock could be coming from > HPLL clock / 16 = 75MHz. It would fit 1024x768@70Hz. > Another chip will still keep 800x600. > > Signed-off-by: Tommy Haung <tommy_huang@aspeedtech.com> > --- > drivers/gpu/drm/aspeed/aspeed_gfx.h | 12 ++++++---- > drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c | 29 ++++++++++++++++++++---- > drivers/gpu/drm/aspeed/aspeed_gfx_drv.c | 16 +++++++++++-- > drivers/gpu/drm/aspeed/aspeed_gfx_out.c | 14 +++++++++++- > 4 files changed, 60 insertions(+), 11 deletions(-) > > diff --git a/drivers/gpu/drm/aspeed/aspeed_gfx.h b/drivers/gpu/drm/aspeed/aspeed_gfx.h > index eb4c267cde5e..c7aefee0657a 100644 > --- a/drivers/gpu/drm/aspeed/aspeed_gfx.h > +++ b/drivers/gpu/drm/aspeed/aspeed_gfx.h > @@ -109,11 +109,15 @@ int aspeed_gfx_create_output(struct drm_device *drm); > #define CRT_THROD_HIGH(x) ((x) << 8) > > /* SCU control */ > -#define SCU_G6_CLK_COURCE 0x300 > +#define G6_CLK_SOURCE 0x300 > +#define G6_CLK_SOURCE_MASK (BIT(8) | BIT(9) | BIT(10)) > +#define G6_CLK_SOURCE_HPLL (BIT(8) | BIT(9) | BIT(10)) > +#define G6_CLK_SOURCE_USB BIT(9) > +#define G6_CLK_SEL3 0x308 > +#define G6_CLK_DIV_MASK 0x3F000 This register is defined in the data sheet as: 17:12 SOC Display clock selection when source is from DisplayPort PHY That doesn't match with what the code is doing. Can you clarify the register definition? > +#define G6_CLK_DIV_16 (BIT(16)|BIT(15)|BIT(13)|BIT(12)) > +#define G6_USB_40_CLK BIT(9) > > /* GFX FLAGS */ > #define CLK_MASK BIT(0) > #define CLK_G6 BIT(0) > - > -#define G6_CLK_MASK (BIT(8) | BIT(9) | BIT(10)) > -#define G6_USB_40_CLK BIT(9) > diff --git a/drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c b/drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c > index a24fab22eac4..5829be9c7c67 100644 > --- a/drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c > +++ b/drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c > @@ -23,6 +23,28 @@ drm_pipe_to_aspeed_gfx(struct drm_simple_display_pipe *pipe) > return container_of(pipe, struct aspeed_gfx, pipe); > } > > +static void aspeed_gfx_set_clock_source(struct aspeed_gfx *priv, int mode_width) > +{ > + regmap_update_bits(priv->scu, G6_CLK_SOURCE, G6_CLK_SOURCE_MASK, 0x0); > + regmap_update_bits(priv->scu, G6_CLK_SEL3, G6_CLK_DIV_MASK, 0x0); > + > + switch (mode_width) { > + case 1024: > + /* hpll div 16 = 75Mhz */ > + regmap_update_bits(priv->scu, G6_CLK_SOURCE, > + G6_CLK_SOURCE_MASK, G6_CLK_SOURCE_HPLL); > + regmap_update_bits(priv->scu, G6_CLK_SEL3, > + G6_CLK_DIV_MASK, G6_CLK_DIV_16); > + break; > + case 800: > + default: > + /* usb 40Mhz */ > + regmap_update_bits(priv->scu, G6_CLK_SOURCE, > + G6_CLK_SOURCE_MASK, G6_CLK_SOURCE_USB); > + break; > + } > +} > + > static int aspeed_gfx_set_pixel_fmt(struct aspeed_gfx *priv, u32 *bpp) > { > struct drm_crtc *crtc = &priv->pipe.crtc; > @@ -77,12 +99,11 @@ static void aspeed_gfx_disable_controller(struct aspeed_gfx *priv) > regmap_update_bits(priv->scu, priv->dac_reg, BIT(16), 0); > } > > -static void aspeed_gfx_set_clk(struct aspeed_gfx *priv) > +static void aspeed_gfx_set_clk(struct aspeed_gfx *priv, int mode_width) > { > switch (priv->flags & CLK_MASK) { > case CLK_G6: > - regmap_update_bits(priv->scu, SCU_G6_CLK_COURCE, G6_CLK_MASK, 0x0); > - regmap_update_bits(priv->scu, SCU_G6_CLK_COURCE, G6_CLK_MASK, G6_USB_40_CLK); > + aspeed_gfx_set_clock_source(priv, mode_width); > break; > default: > break; > @@ -99,7 +120,7 @@ static void aspeed_gfx_crtc_mode_set_nofb(struct aspeed_gfx *priv) > if (err) > return; > > - aspeed_gfx_set_clk(priv); > + aspeed_gfx_set_clk(priv, m->hdisplay); > > #if 0 > /* TODO: we have only been able to test with the 40MHz USB clock. The > diff --git a/drivers/gpu/drm/aspeed/aspeed_gfx_drv.c b/drivers/gpu/drm/aspeed/aspeed_gfx_drv.c > index af56ffdccc65..e1a814aebc2d 100644 > --- a/drivers/gpu/drm/aspeed/aspeed_gfx_drv.c > +++ b/drivers/gpu/drm/aspeed/aspeed_gfx_drv.c > @@ -110,6 +110,7 @@ static const struct drm_mode_config_funcs aspeed_gfx_mode_config_funcs = { > > static int aspeed_gfx_setup_mode_config(struct drm_device *drm) > { > + struct aspeed_gfx *priv = to_aspeed_gfx(drm); > int ret; > > ret = drmm_mode_config_init(drm); > @@ -118,8 +119,18 @@ static int aspeed_gfx_setup_mode_config(struct drm_device *drm) > > drm->mode_config.min_width = 0; > drm->mode_config.min_height = 0; > - drm->mode_config.max_width = 800; > - drm->mode_config.max_height = 600; > + > + switch (priv->flags & CLK_MASK) { > + case CLK_G6: > + drm->mode_config.max_width = 1024; > + drm->mode_config.max_height = 768; > + break; > + default: > + drm->mode_config.max_width = 800; > + drm->mode_config.max_height = 600; > + break; > + } > + > drm->mode_config.funcs = &aspeed_gfx_mode_config_funcs; > > return ret; > @@ -167,6 +178,7 @@ static int aspeed_gfx_load(struct drm_device *drm) > priv->vga_scratch_reg = config->vga_scratch_reg; > priv->throd_val = config->throd_val; > priv->scan_line_max = config->scan_line_max; > + priv->flags = config->gfx_flags; > > priv->scu = syscon_regmap_lookup_by_phandle(np, "syscon"); > if (IS_ERR(priv->scu)) { > diff --git a/drivers/gpu/drm/aspeed/aspeed_gfx_out.c b/drivers/gpu/drm/aspeed/aspeed_gfx_out.c > index 6759cb88415a..5d5e04f15c59 100644 > --- a/drivers/gpu/drm/aspeed/aspeed_gfx_out.c > +++ b/drivers/gpu/drm/aspeed/aspeed_gfx_out.c > @@ -10,7 +10,19 @@ > > static int aspeed_gfx_get_modes(struct drm_connector *connector) > { > - return drm_add_modes_noedid(connector, 800, 600); > + struct aspeed_gfx *priv = container_of(connector, struct aspeed_gfx, connector); > + int mode_count = 0; > + > + switch (priv->flags & CLK_MASK) { > + case CLK_G6: > + mode_count = drm_add_modes_noedid(connector, 1024, 768); > + break; > + default: > + mode_count = drm_add_modes_noedid(connector, 800, 600); > + break; > + } > + > + return mode_count; > } > > static const struct > -- > 2.17.1 > ^ permalink raw reply [flat|nested] 8+ messages in thread
* RE: [PATCH v1 2/2] drm/aspeed: Add 1024x768 mode for AST2600 2022-04-26 7:47 ` Joel Stanley @ 2022-04-26 8:28 ` Tommy Huang 2022-06-07 1:30 ` Tommy Huang 0 siblings, 1 reply; 8+ messages in thread From: Tommy Huang @ 2022-04-26 8:28 UTC (permalink / raw) To: Joel Stanley Cc: David Airlie, Daniel Vetter, Rob Herring, Andrew Jeffery, linux-aspeed, open list:DRM DRIVERS, devicetree, Linux ARM, Linux Kernel Mailing List, BMC-SW > -----Original Message----- > From: Joel Stanley <joel@jms.id.au> > Sent: Tuesday, April 26, 2022 3:48 PM > To: Tommy Huang <tommy_huang@aspeedtech.com> > Cc: David Airlie <airlied@linux.ie>; Daniel Vetter <daniel@ffwll.ch>; Rob > Herring <robh+dt@kernel.org>; Andrew Jeffery <andrew@aj.id.au>; > linux-aspeed <linux-aspeed@lists.ozlabs.org>; open list:DRM DRIVERS > <dri-devel@lists.freedesktop.org>; devicetree <devicetree@vger.kernel.org>; > Linux ARM <linux-arm-kernel@lists.infradead.org>; Linux Kernel Mailing List > <linux-kernel@vger.kernel.org>; BMC-SW <BMC-SW@aspeedtech.com> > Subject: Re: [PATCH v1 2/2] drm/aspeed: Add 1024x768 mode for AST2600 > > On Fri, 4 Mar 2022 at 06:32, Tommy Haung <tommy_huang@aspeedtech.com> > wrote: > > > > Update the aspeed_gfx_set_clk with display width. > > At AST2600, the display clock could be coming from HPLL clock / 16 = > > 75MHz. It would fit 1024x768@70Hz. > > Another chip will still keep 800x600. > > > > Signed-off-by: Tommy Haung <tommy_huang@aspeedtech.com> > > --- > > drivers/gpu/drm/aspeed/aspeed_gfx.h | 12 ++++++---- > > drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c | 29 > > ++++++++++++++++++++---- drivers/gpu/drm/aspeed/aspeed_gfx_drv.c | > > 16 +++++++++++-- drivers/gpu/drm/aspeed/aspeed_gfx_out.c | 14 > > +++++++++++- > > 4 files changed, 60 insertions(+), 11 deletions(-) > > > > diff --git a/drivers/gpu/drm/aspeed/aspeed_gfx.h > > b/drivers/gpu/drm/aspeed/aspeed_gfx.h > > index eb4c267cde5e..c7aefee0657a 100644 > > --- a/drivers/gpu/drm/aspeed/aspeed_gfx.h > > +++ b/drivers/gpu/drm/aspeed/aspeed_gfx.h > > @@ -109,11 +109,15 @@ int aspeed_gfx_create_output(struct drm_device > *drm); > > #define CRT_THROD_HIGH(x) ((x) << 8) > > > > /* SCU control */ > > -#define SCU_G6_CLK_COURCE 0x300 > > +#define G6_CLK_SOURCE 0x300 > > +#define G6_CLK_SOURCE_MASK (BIT(8) | BIT(9) | BIT(10)) > > +#define G6_CLK_SOURCE_HPLL (BIT(8) | BIT(9) | BIT(10)) > > +#define G6_CLK_SOURCE_USB BIT(9) > > +#define G6_CLK_SEL3 0x308 > > +#define G6_CLK_DIV_MASK 0x3F000 > > This register is defined in the data sheet as: > > 17:12 SOC Display clock selection when source is from DisplayPort PHY > > That doesn't match with what the code is doing. Can you clarify the register > definition? OK. I will clarify it and response it to you. > > > +#define G6_CLK_DIV_16 > (BIT(16)|BIT(15)|BIT(13)|BIT(12)) > > +#define G6_USB_40_CLK BIT(9) > > > > /* GFX FLAGS */ > > #define CLK_MASK BIT(0) > > #define CLK_G6 BIT(0) > > - > > -#define G6_CLK_MASK (BIT(8) | BIT(9) | BIT(10)) > > -#define G6_USB_40_CLK BIT(9) > > diff --git a/drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c > > b/drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c > > index a24fab22eac4..5829be9c7c67 100644 > > --- a/drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c > > +++ b/drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c > > @@ -23,6 +23,28 @@ drm_pipe_to_aspeed_gfx(struct > drm_simple_display_pipe *pipe) > > return container_of(pipe, struct aspeed_gfx, pipe); } > > > > +static void aspeed_gfx_set_clock_source(struct aspeed_gfx *priv, int > > +mode_width) { > > + regmap_update_bits(priv->scu, G6_CLK_SOURCE, > G6_CLK_SOURCE_MASK, 0x0); > > + regmap_update_bits(priv->scu, G6_CLK_SEL3, G6_CLK_DIV_MASK, > > +0x0); > > + > > + switch (mode_width) { > > + case 1024: > > + /* hpll div 16 = 75Mhz */ > > + regmap_update_bits(priv->scu, G6_CLK_SOURCE, > > + G6_CLK_SOURCE_MASK, G6_CLK_SOURCE_HPLL); > > + regmap_update_bits(priv->scu, G6_CLK_SEL3, > > + G6_CLK_DIV_MASK, G6_CLK_DIV_16); > > + break; > > + case 800: > > + default: > > + /* usb 40Mhz */ > > + regmap_update_bits(priv->scu, G6_CLK_SOURCE, > > + G6_CLK_SOURCE_MASK, G6_CLK_SOURCE_USB); > > + break; > > + } > > +} > > + > > static int aspeed_gfx_set_pixel_fmt(struct aspeed_gfx *priv, u32 > > *bpp) { > > struct drm_crtc *crtc = &priv->pipe.crtc; @@ -77,12 +99,11 @@ > > static void aspeed_gfx_disable_controller(struct aspeed_gfx *priv) > > regmap_update_bits(priv->scu, priv->dac_reg, BIT(16), 0); } > > > > -static void aspeed_gfx_set_clk(struct aspeed_gfx *priv) > > +static void aspeed_gfx_set_clk(struct aspeed_gfx *priv, int > > +mode_width) > > { > > switch (priv->flags & CLK_MASK) { > > case CLK_G6: > > - regmap_update_bits(priv->scu, SCU_G6_CLK_COURCE, > G6_CLK_MASK, 0x0); > > - regmap_update_bits(priv->scu, SCU_G6_CLK_COURCE, > G6_CLK_MASK, G6_USB_40_CLK); > > + aspeed_gfx_set_clock_source(priv, mode_width); > > break; > > default: > > break; > > @@ -99,7 +120,7 @@ static void aspeed_gfx_crtc_mode_set_nofb(struct > aspeed_gfx *priv) > > if (err) > > return; > > > > - aspeed_gfx_set_clk(priv); > > + aspeed_gfx_set_clk(priv, m->hdisplay); > > > > #if 0 > > /* TODO: we have only been able to test with the 40MHz USB > > clock. The diff --git a/drivers/gpu/drm/aspeed/aspeed_gfx_drv.c > > b/drivers/gpu/drm/aspeed/aspeed_gfx_drv.c > > index af56ffdccc65..e1a814aebc2d 100644 > > --- a/drivers/gpu/drm/aspeed/aspeed_gfx_drv.c > > +++ b/drivers/gpu/drm/aspeed/aspeed_gfx_drv.c > > @@ -110,6 +110,7 @@ static const struct drm_mode_config_funcs > > aspeed_gfx_mode_config_funcs = { > > > > static int aspeed_gfx_setup_mode_config(struct drm_device *drm) { > > + struct aspeed_gfx *priv = to_aspeed_gfx(drm); > > int ret; > > > > ret = drmm_mode_config_init(drm); @@ -118,8 +119,18 @@ > static > > int aspeed_gfx_setup_mode_config(struct drm_device *drm) > > > > drm->mode_config.min_width = 0; > > drm->mode_config.min_height = 0; > > - drm->mode_config.max_width = 800; > > - drm->mode_config.max_height = 600; > > + > > + switch (priv->flags & CLK_MASK) { > > + case CLK_G6: > > + drm->mode_config.max_width = 1024; > > + drm->mode_config.max_height = 768; > > + break; > > + default: > > + drm->mode_config.max_width = 800; > > + drm->mode_config.max_height = 600; > > + break; > > + } > > + > > drm->mode_config.funcs = &aspeed_gfx_mode_config_funcs; > > > > return ret; > > @@ -167,6 +178,7 @@ static int aspeed_gfx_load(struct drm_device *drm) > > priv->vga_scratch_reg = config->vga_scratch_reg; > > priv->throd_val = config->throd_val; > > priv->scan_line_max = config->scan_line_max; > > + priv->flags = config->gfx_flags; > > > > priv->scu = syscon_regmap_lookup_by_phandle(np, "syscon"); > > if (IS_ERR(priv->scu)) { > > diff --git a/drivers/gpu/drm/aspeed/aspeed_gfx_out.c > > b/drivers/gpu/drm/aspeed/aspeed_gfx_out.c > > index 6759cb88415a..5d5e04f15c59 100644 > > --- a/drivers/gpu/drm/aspeed/aspeed_gfx_out.c > > +++ b/drivers/gpu/drm/aspeed/aspeed_gfx_out.c > > @@ -10,7 +10,19 @@ > > > > static int aspeed_gfx_get_modes(struct drm_connector *connector) { > > - return drm_add_modes_noedid(connector, 800, 600); > > + struct aspeed_gfx *priv = container_of(connector, struct > aspeed_gfx, connector); > > + int mode_count = 0; > > + > > + switch (priv->flags & CLK_MASK) { > > + case CLK_G6: > > + mode_count = drm_add_modes_noedid(connector, 1024, > 768); > > + break; > > + default: > > + mode_count = drm_add_modes_noedid(connector, 800, > 600); > > + break; > > + } > > + > > + return mode_count; > > } > > > > static const struct > > -- > > 2.17.1 > > ^ permalink raw reply [flat|nested] 8+ messages in thread
* RE: [PATCH v1 2/2] drm/aspeed: Add 1024x768 mode for AST2600 2022-04-26 8:28 ` Tommy Huang @ 2022-06-07 1:30 ` Tommy Huang 0 siblings, 0 replies; 8+ messages in thread From: Tommy Huang @ 2022-06-07 1:30 UTC (permalink / raw) To: Joel Stanley Cc: David Airlie, Daniel Vetter, Rob Herring, Andrew Jeffery, linux-aspeed, open list:DRM DRIVERS, devicetree, Linux ARM, Linux Kernel Mailing List, BMC-SW Hi Joel, We have released related datasheet and update these registers description. SCU300: Clock Selection Register ... 10:8 Soc display clock selection 000b: D-PLL 001b: Reserved 010b: 40MHz from USB 2.0 port1 PHY 011b: GPIOC6 100b: Reserved 101b: E-PLL divided by SCU308[17:12] 110b: M-PLL divided by SCU308[17:12] 111b: H-PLL divided by SCU308[17:12] SCU308: Clock Selection Register Set 3 17:12 Soc display clock divider selection 000000b: div 1 011011b: div 16 Thanks, By Tommy > -----Original Message----- > From: Tommy Huang > Sent: Tuesday, April 26, 2022 4:28 PM > To: Joel Stanley <joel@jms.id.au> > Cc: David Airlie <airlied@linux.ie>; Daniel Vetter <daniel@ffwll.ch>; Rob > Herring <robh+dt@kernel.org>; Andrew Jeffery <andrew@aj.id.au>; > linux-aspeed <linux-aspeed@lists.ozlabs.org>; open list:DRM DRIVERS > <dri-devel@lists.freedesktop.org>; devicetree <devicetree@vger.kernel.org>; > Linux ARM <linux-arm-kernel@lists.infradead.org>; Linux Kernel Mailing List > <linux-kernel@vger.kernel.org>; BMC-SW <BMC-SW@aspeedtech.com> > Subject: RE: [PATCH v1 2/2] drm/aspeed: Add 1024x768 mode for AST2600 > > > > > -----Original Message----- > > From: Joel Stanley <joel@jms.id.au> > > Sent: Tuesday, April 26, 2022 3:48 PM > > To: Tommy Huang <tommy_huang@aspeedtech.com> > > Cc: David Airlie <airlied@linux.ie>; Daniel Vetter <daniel@ffwll.ch>; > > Rob Herring <robh+dt@kernel.org>; Andrew Jeffery <andrew@aj.id.au>; > > linux-aspeed <linux-aspeed@lists.ozlabs.org>; open list:DRM DRIVERS > > <dri-devel@lists.freedesktop.org>; devicetree > > <devicetree@vger.kernel.org>; Linux ARM > > <linux-arm-kernel@lists.infradead.org>; Linux Kernel Mailing List > > <linux-kernel@vger.kernel.org>; BMC-SW <BMC-SW@aspeedtech.com> > > Subject: Re: [PATCH v1 2/2] drm/aspeed: Add 1024x768 mode for AST2600 > > > > On Fri, 4 Mar 2022 at 06:32, Tommy Haung > <tommy_huang@aspeedtech.com> > > wrote: > > > > > > Update the aspeed_gfx_set_clk with display width. > > > At AST2600, the display clock could be coming from HPLL clock / 16 = > > > 75MHz. It would fit 1024x768@70Hz. > > > Another chip will still keep 800x600. > > > > > > Signed-off-by: Tommy Haung <tommy_huang@aspeedtech.com> > > > --- > > > drivers/gpu/drm/aspeed/aspeed_gfx.h | 12 ++++++---- > > > drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c | 29 > > > ++++++++++++++++++++---- drivers/gpu/drm/aspeed/aspeed_gfx_drv.c > | > > > 16 +++++++++++-- drivers/gpu/drm/aspeed/aspeed_gfx_out.c | 14 > > > +++++++++++- > > > 4 files changed, 60 insertions(+), 11 deletions(-) > > > > > > diff --git a/drivers/gpu/drm/aspeed/aspeed_gfx.h > > > b/drivers/gpu/drm/aspeed/aspeed_gfx.h > > > index eb4c267cde5e..c7aefee0657a 100644 > > > --- a/drivers/gpu/drm/aspeed/aspeed_gfx.h > > > +++ b/drivers/gpu/drm/aspeed/aspeed_gfx.h > > > @@ -109,11 +109,15 @@ int aspeed_gfx_create_output(struct drm_device > > *drm); > > > #define CRT_THROD_HIGH(x) ((x) << 8) > > > > > > /* SCU control */ > > > -#define SCU_G6_CLK_COURCE 0x300 > > > +#define G6_CLK_SOURCE 0x300 > > > +#define G6_CLK_SOURCE_MASK (BIT(8) | BIT(9) | BIT(10)) > > > +#define G6_CLK_SOURCE_HPLL (BIT(8) | BIT(9) | BIT(10)) > > > +#define G6_CLK_SOURCE_USB BIT(9) > > > +#define G6_CLK_SEL3 0x308 > > > +#define G6_CLK_DIV_MASK 0x3F000 > > > > This register is defined in the data sheet as: > > > > 17:12 SOC Display clock selection when source is from DisplayPort PHY > > > > That doesn't match with what the code is doing. Can you clarify the > > register definition? > > OK. I will clarify it and response it to you. > > > > > > +#define G6_CLK_DIV_16 > > (BIT(16)|BIT(15)|BIT(13)|BIT(12)) > > > +#define G6_USB_40_CLK BIT(9) > > > > > > /* GFX FLAGS */ > > > #define CLK_MASK BIT(0) > > > #define CLK_G6 BIT(0) > > > - > > > -#define G6_CLK_MASK (BIT(8) | BIT(9) | BIT(10)) > > > -#define G6_USB_40_CLK BIT(9) > > > diff --git a/drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c > > > b/drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c > > > index a24fab22eac4..5829be9c7c67 100644 > > > --- a/drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c > > > +++ b/drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c > > > @@ -23,6 +23,28 @@ drm_pipe_to_aspeed_gfx(struct > > drm_simple_display_pipe *pipe) > > > return container_of(pipe, struct aspeed_gfx, pipe); } > > > > > > +static void aspeed_gfx_set_clock_source(struct aspeed_gfx *priv, > > > +int > > > +mode_width) { > > > + regmap_update_bits(priv->scu, G6_CLK_SOURCE, > > G6_CLK_SOURCE_MASK, 0x0); > > > + regmap_update_bits(priv->scu, G6_CLK_SEL3, > G6_CLK_DIV_MASK, > > > +0x0); > > > + > > > + switch (mode_width) { > > > + case 1024: > > > + /* hpll div 16 = 75Mhz */ > > > + regmap_update_bits(priv->scu, G6_CLK_SOURCE, > > > + G6_CLK_SOURCE_MASK, G6_CLK_SOURCE_HPLL); > > > + regmap_update_bits(priv->scu, G6_CLK_SEL3, > > > + G6_CLK_DIV_MASK, G6_CLK_DIV_16); > > > + break; > > > + case 800: > > > + default: > > > + /* usb 40Mhz */ > > > + regmap_update_bits(priv->scu, G6_CLK_SOURCE, > > > + G6_CLK_SOURCE_MASK, G6_CLK_SOURCE_USB); > > > + break; > > > + } > > > +} > > > + > > > static int aspeed_gfx_set_pixel_fmt(struct aspeed_gfx *priv, u32 > > > *bpp) { > > > struct drm_crtc *crtc = &priv->pipe.crtc; @@ -77,12 +99,11 > > > @@ static void aspeed_gfx_disable_controller(struct aspeed_gfx *priv) > > > regmap_update_bits(priv->scu, priv->dac_reg, BIT(16), 0); } > > > > > > -static void aspeed_gfx_set_clk(struct aspeed_gfx *priv) > > > +static void aspeed_gfx_set_clk(struct aspeed_gfx *priv, int > > > +mode_width) > > > { > > > switch (priv->flags & CLK_MASK) { > > > case CLK_G6: > > > - regmap_update_bits(priv->scu, SCU_G6_CLK_COURCE, > > G6_CLK_MASK, 0x0); > > > - regmap_update_bits(priv->scu, SCU_G6_CLK_COURCE, > > G6_CLK_MASK, G6_USB_40_CLK); > > > + aspeed_gfx_set_clock_source(priv, mode_width); > > > break; > > > default: > > > break; > > > @@ -99,7 +120,7 @@ static void aspeed_gfx_crtc_mode_set_nofb(struct > > aspeed_gfx *priv) > > > if (err) > > > return; > > > > > > - aspeed_gfx_set_clk(priv); > > > + aspeed_gfx_set_clk(priv, m->hdisplay); > > > > > > #if 0 > > > /* TODO: we have only been able to test with the 40MHz USB > > > clock. The diff --git a/drivers/gpu/drm/aspeed/aspeed_gfx_drv.c > > > b/drivers/gpu/drm/aspeed/aspeed_gfx_drv.c > > > index af56ffdccc65..e1a814aebc2d 100644 > > > --- a/drivers/gpu/drm/aspeed/aspeed_gfx_drv.c > > > +++ b/drivers/gpu/drm/aspeed/aspeed_gfx_drv.c > > > @@ -110,6 +110,7 @@ static const struct drm_mode_config_funcs > > > aspeed_gfx_mode_config_funcs = { > > > > > > static int aspeed_gfx_setup_mode_config(struct drm_device *drm) { > > > + struct aspeed_gfx *priv = to_aspeed_gfx(drm); > > > int ret; > > > > > > ret = drmm_mode_config_init(drm); @@ -118,8 +119,18 @@ > > static > > > int aspeed_gfx_setup_mode_config(struct drm_device *drm) > > > > > > drm->mode_config.min_width = 0; > > > drm->mode_config.min_height = 0; > > > - drm->mode_config.max_width = 800; > > > - drm->mode_config.max_height = 600; > > > + > > > + switch (priv->flags & CLK_MASK) { > > > + case CLK_G6: > > > + drm->mode_config.max_width = 1024; > > > + drm->mode_config.max_height = 768; > > > + break; > > > + default: > > > + drm->mode_config.max_width = 800; > > > + drm->mode_config.max_height = 600; > > > + break; > > > + } > > > + > > > drm->mode_config.funcs = &aspeed_gfx_mode_config_funcs; > > > > > > return ret; > > > @@ -167,6 +178,7 @@ static int aspeed_gfx_load(struct drm_device *drm) > > > priv->vga_scratch_reg = config->vga_scratch_reg; > > > priv->throd_val = config->throd_val; > > > priv->scan_line_max = config->scan_line_max; > > > + priv->flags = config->gfx_flags; > > > > > > priv->scu = syscon_regmap_lookup_by_phandle(np, "syscon"); > > > if (IS_ERR(priv->scu)) { > > > diff --git a/drivers/gpu/drm/aspeed/aspeed_gfx_out.c > > > b/drivers/gpu/drm/aspeed/aspeed_gfx_out.c > > > index 6759cb88415a..5d5e04f15c59 100644 > > > --- a/drivers/gpu/drm/aspeed/aspeed_gfx_out.c > > > +++ b/drivers/gpu/drm/aspeed/aspeed_gfx_out.c > > > @@ -10,7 +10,19 @@ > > > > > > static int aspeed_gfx_get_modes(struct drm_connector *connector) { > > > - return drm_add_modes_noedid(connector, 800, 600); > > > + struct aspeed_gfx *priv = container_of(connector, struct > > aspeed_gfx, connector); > > > + int mode_count = 0; > > > + > > > + switch (priv->flags & CLK_MASK) { > > > + case CLK_G6: > > > + mode_count = drm_add_modes_noedid(connector, > 1024, > > 768); > > > + break; > > > + default: > > > + mode_count = drm_add_modes_noedid(connector, > 800, > > 600); > > > + break; > > > + } > > > + > > > + return mode_count; > > > } > > > > > > static const struct > > > -- > > > 2.17.1 > > > ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2022-06-07 1:30 UTC | newest] Thread overview: 8+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2022-03-04 6:31 [PATCH v1 0/2] Add 1024x768 timing for AST2600 Tommy Haung 2022-03-04 6:31 ` [PATCH v1 1/2] drm/aspeed: Add gfx flags and clock selection " Tommy Haung 2022-03-04 6:31 ` [PATCH v1 2/2] drm/aspeed: Add 1024x768 mode " Tommy Haung 2022-04-26 3:27 ` Joel Stanley 2022-04-26 5:20 ` Tommy Huang 2022-04-26 7:47 ` Joel Stanley 2022-04-26 8:28 ` Tommy Huang 2022-06-07 1:30 ` Tommy Huang
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).