On Thursday 15 May 2014 12:01:30 Murali Karicheri wrote: > +static struct serdes_config ks_100mhz_pcie_5gbps_serdes[] = { > + {0x0000, 0x00000800, 0x0000ff00}, > + {0x0060, 0x00041c5c, 0x00ffffff}, > + {0x0064, 0x0343c700, 0xffffff00}, > + {0x006c, 0x00000012, 0x000000ff}, > + {0x0068, 0x00070000, 0x00ff0000}, > + {0x0078, 0x0000c000, 0x0000ff00}, It looks like the PHY is generic, but the configuration above is PCI specific. If this is true, you should have #phy-cells=<1> and document the possible modes, adding a lookup table here to pick the configuration based on the argument. It's fine to just implement pcie-5ghz initially, but the binding should list all the modes that the PHY can support. Also, please list the exact name of the phy if you can find it out. You mention that you don't know the register descriptions, but you should at least be able to let us know what phy this is, in case some other SoC reuses the same thing. > +static int ks_phy_init(struct phy *phy) > +{ > + struct serdes_config *p; > + struct phy_keystone *ks_phy = phy_get_drvdata(phy); > + > + int i; > + > + for (i = 0, p = &ks_100mhz_pcie_5gbps_serdes[0]; > + i < ARRAY_SIZE(ks_100mhz_pcie_5gbps_serdes); > + i++, p++) { > + reg_rmw((ks_phy->base + p->reg), p->val, p->mask); > + reg_dump((ks_phy->base + p->reg), p->mask); > + } > + udelay(2000); This should probably be msleep(2); Arnd -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html