lkml.org 
[lkml]   [2014]   [May]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    SubjectRe: [PATCH v1 3/5] phy: pci serdes phy driver for keystone
    Date
    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


    \
     
     \ /
      Last update: 2014-05-16 04:21    [W:2.281 / U:0.212 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site