On Fri, Oct 10, 2014 at 02:04:34PM +0400, Sergei Shtylyov wrote: > Hello. > > On 10/10/2014 12:45 PM, Yoshihiro Kaneko wrote: > > >From: Koji Matsuoka <koji.matsuoka.xm@xxxxxxxxxxx> > > >This workaround must be applied to R8A7790(H2) ES1 > > >Signed-off-by: Koji Matsuoka <koji.matsuoka.xm@xxxxxxxxxxx> > >Signed-off-by: Yoshihiro Kaneko <ykaneko0929@xxxxxxxxx> For the record, this patch has been superseded by "ata: sata_rcar: Disable DIPM mode for r8a7790 ES1". > >--- > > >This patch is against for-next branch of Tejun's libata repo. > > > drivers/ata/sata_rcar.c | 25 +++++++++++++++++++++++-- > > 1 file changed, 23 insertions(+), 2 deletions(-) > > > >diff --git a/drivers/ata/sata_rcar.c b/drivers/ata/sata_rcar.c > >index 61eb6d7..0ba5974 100644 > >--- a/drivers/ata/sata_rcar.c > >+++ b/drivers/ata/sata_rcar.c > >@@ -143,6 +143,11 @@ > > #define RCAR_GEN2_PHY_CTL5_DC BIT(1) /* DC connection */ > > #define RCAR_GEN2_PHY_CTL5_TR BIT(2) /* Termination Resistor */ > > > >+/* Product Register */ > >+#define PRODUCT_REGISTER 0xFF000044 > >+#define PRODUCT_CUT_MASK 0x00007FF0 > >+#define PRODUCT_H2_BIT (0x45 << 8) > > Why _BIT? And why not 0x00004500? > > >@@ -751,18 +756,30 @@ done: > > return IRQ_RETVAL(handled); > > } > > > >-static void sata_rcar_setup_port(struct ata_host *host) > >+static int sata_rcar_setup_port(struct ata_host *host) > > { > > struct ata_port *ap = host->ports[0]; > > struct ata_ioports *ioaddr = &ap->ioaddr; > > struct sata_rcar_priv *priv = host->private_data; > > void __iomem *base = priv->base; > >+ void __iomem *product_reg; > > > > ap->ops = &sata_rcar_port_ops; > > ap->pio_mask = ATA_PIO4; > > ap->udma_mask = ATA_UDMA6; > > ap->flags |= ATA_FLAG_SATA; > > > >+ product_reg = ioremap_nocache(PRODUCT_REGISTER, 0x04); > > Don't we need some platform function to read that register instead? It's > not specific to SATA, AFAICT. > > >+ if (!product_reg) { > >+ dev_warn(host->dev, "ioremap fail\n"); > > s/fail/failed/. > > >+ return -ENOMEM; > >+ } > >+ /* Add the workaround of DIPM mode disabling in R-Car H2 ES1.x.*/ > > Please add space before */ > > [...] > > WBR, Sergei > > -- > To unsubscribe from this list: send the line "unsubscribe linux-sh" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- 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