On Wed, Nov 20, 2013 at 11:27:22AM +0100, Marek Vasut wrote: > We must clear this IMX6Q_GPR13_SATA_MPLL_CLK_EN bit on i.MX6Q, otherwise > Linux will fail to find the attached drive on some boards. > > This entire fix was: > Reported-by: Eric Nelson <eric.nelson@xxxxxxxxxxxxxxxxxxx> > > Signed-off-by: Marek Vasut <marex@xxxxxxx> > Cc: Shawn Guo <shawn.guo@xxxxxxxxxx> > Cc: Richard Zhu <r65037@xxxxxxxxxxxxx> > Cc: Tejun Heo <tj@xxxxxxxxxx> > Cc: Linux-IDE <linux-ide@xxxxxxxxxxxxxxx> > --- > drivers/ata/ahci_imx.c | 1 + > 1 file changed, 1 insertion(+) > > RESEND: Fix the Cc here. > > diff --git a/drivers/ata/ahci_imx.c b/drivers/ata/ahci_imx.c > index ae2d73f..478921b 100644 > --- a/drivers/ata/ahci_imx.c > +++ b/drivers/ata/ahci_imx.c > @@ -125,6 +125,7 @@ static int imx6q_sata_init(struct device *dev, void __iomem *mmio) > | IMX6Q_GPR13_SATA_TX_BOOST_MASK > | IMX6Q_GPR13_SATA_TX_LVL_MASK > | IMX6Q_GPR13_SATA_TX_EDGE_RATE > + | IMX6Q_GPR13_SATA_MPLL_CLK_EN > , IMX6Q_GPR13_SATA_RX_EQ_VAL_3_0_DB > | IMX6Q_GPR13_SATA_RX_LOS_LVL_SATA2M > | IMX6Q_GPR13_SATA_RX_DPLL_MODE_2P_4F Not your fault but when things stretch over multiples lines, the convention is to put the operators and commas at the end of each line not the other way around. I'd appreciate if you add a trivial patch to update that too. Thanks. -- tejun -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html