Message ID | 20180326051639.16555-1-joel@jms.id.au |
---|---|
State | Superseded, archived |
Headers | show |
Series | [linux,dev-4.13] clk: aspeed: Mark bclk (PCIe) as critical | expand |
On Mon, 26 Mar 2018, at 15:46, Joel Stanley wrote: > This is used by the host to talk to the BMC's PCIe slave device. The BMC > is not involved, but the clock needs to be enabled so the host can use > the device. > > Signed-off-by: Joel Stanley <joel@jms.id.au> Acked-by: Andrew Jeffery <andrew@aj.id.au> > --- > drivers/clk/clk-aspeed.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/clk/clk-aspeed.c b/drivers/clk/clk-aspeed.c > index 5af80b4e673e..5be88f0a0678 100644 > --- a/drivers/clk/clk-aspeed.c > +++ b/drivers/clk/clk-aspeed.c > @@ -97,7 +97,7 @@ static const struct aspeed_gate_data aspeed_gates[] = > { > [ASPEED_CLK_GATE_GCLK] = { 1, 7, "gclk-gate", NULL, 0 }, /* 2D engine */ > [ASPEED_CLK_GATE_MCLK] = { 2, -1, "mclk-gate", "mpll", > CLK_IS_CRITICAL }, /* SDRAM */ > [ASPEED_CLK_GATE_VCLK] = { 3, 6, "vclk-gate", NULL, 0 }, /* Video > Capture */ > - [ASPEED_CLK_GATE_BCLK] = { 4, 10, "bclk-gate", "bclk", 0 }, /* PCIe/PCI > */ > + [ASPEED_CLK_GATE_BCLK] = { 4, 10, "bclk-gate", "bclk", > CLK_IS_CRITICAL }, /* PCIe/PCI */ > [ASPEED_CLK_GATE_DCLK] = { 5, -1, "dclk-gate", NULL, 0 }, /* DAC */ > [ASPEED_CLK_GATE_REFCLK] = { 6, -1, "refclk-gate", "clkin", > CLK_IS_CRITICAL }, > [ASPEED_CLK_GATE_USBPORT2CLK] = { 7, 3, "usb-port2-gate", NULL, 0 }, /* > USB2.0 Host port 2 */ > -- > 2.15.1 >
Tested-by: Lei YU <mine260309@gmail.com> With the patch, bclk is enabled (by default). However, the PCI/VGA device is probed by skiboot but failed to train in petiboot: [ 55.446446203,5] PHB#0005:02:00.0 [PCID] 1a03 2000 R:41 C:030000 ( vga) ... [ 72.096462749,3] PHB#0005[0:5]: PHB Freeze/Fence detected ! ... The full log is posted at https://github.com/openbmc/openbmc/files/1850909/log_romulus_host_with_bmc_4.13.txt On Mon, Mar 26, 2018 at 4:07 PM, Andrew Jeffery <andrew@aj.id.au> wrote: > On Mon, 26 Mar 2018, at 15:46, Joel Stanley wrote: >> This is used by the host to talk to the BMC's PCIe slave device. The BMC >> is not involved, but the clock needs to be enabled so the host can use >> the device. >> >> Signed-off-by: Joel Stanley <joel@jms.id.au> > > Acked-by: Andrew Jeffery <andrew@aj.id.au> > >> --- >> drivers/clk/clk-aspeed.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/clk/clk-aspeed.c b/drivers/clk/clk-aspeed.c >> index 5af80b4e673e..5be88f0a0678 100644 >> --- a/drivers/clk/clk-aspeed.c >> +++ b/drivers/clk/clk-aspeed.c >> @@ -97,7 +97,7 @@ static const struct aspeed_gate_data aspeed_gates[] = >> { >> [ASPEED_CLK_GATE_GCLK] = { 1, 7, "gclk-gate", NULL, 0 }, /* 2D engine */ >> [ASPEED_CLK_GATE_MCLK] = { 2, -1, "mclk-gate", "mpll", >> CLK_IS_CRITICAL }, /* SDRAM */ >> [ASPEED_CLK_GATE_VCLK] = { 3, 6, "vclk-gate", NULL, 0 }, /* Video >> Capture */ >> - [ASPEED_CLK_GATE_BCLK] = { 4, 10, "bclk-gate", "bclk", 0 }, /* PCIe/PCI >> */ >> + [ASPEED_CLK_GATE_BCLK] = { 4, 10, "bclk-gate", "bclk", >> CLK_IS_CRITICAL }, /* PCIe/PCI */ >> [ASPEED_CLK_GATE_DCLK] = { 5, -1, "dclk-gate", NULL, 0 }, /* DAC */ >> [ASPEED_CLK_GATE_REFCLK] = { 6, -1, "refclk-gate", "clkin", >> CLK_IS_CRITICAL }, >> [ASPEED_CLK_GATE_USBPORT2CLK] = { 7, 3, "usb-port2-gate", NULL, 0 }, /* >> USB2.0 Host port 2 */ >> -- >> 2.15.1 >>
diff --git a/drivers/clk/clk-aspeed.c b/drivers/clk/clk-aspeed.c index 5af80b4e673e..5be88f0a0678 100644 --- a/drivers/clk/clk-aspeed.c +++ b/drivers/clk/clk-aspeed.c @@ -97,7 +97,7 @@ static const struct aspeed_gate_data aspeed_gates[] = { [ASPEED_CLK_GATE_GCLK] = { 1, 7, "gclk-gate", NULL, 0 }, /* 2D engine */ [ASPEED_CLK_GATE_MCLK] = { 2, -1, "mclk-gate", "mpll", CLK_IS_CRITICAL }, /* SDRAM */ [ASPEED_CLK_GATE_VCLK] = { 3, 6, "vclk-gate", NULL, 0 }, /* Video Capture */ - [ASPEED_CLK_GATE_BCLK] = { 4, 10, "bclk-gate", "bclk", 0 }, /* PCIe/PCI */ + [ASPEED_CLK_GATE_BCLK] = { 4, 10, "bclk-gate", "bclk", CLK_IS_CRITICAL }, /* PCIe/PCI */ [ASPEED_CLK_GATE_DCLK] = { 5, -1, "dclk-gate", NULL, 0 }, /* DAC */ [ASPEED_CLK_GATE_REFCLK] = { 6, -1, "refclk-gate", "clkin", CLK_IS_CRITICAL }, [ASPEED_CLK_GATE_USBPORT2CLK] = { 7, 3, "usb-port2-gate", NULL, 0 }, /* USB2.0 Host port 2 */
This is used by the host to talk to the BMC's PCIe slave device. The BMC is not involved, but the clock needs to be enabled so the host can use the device. Signed-off-by: Joel Stanley <joel@jms.id.au> --- drivers/clk/clk-aspeed.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)