Re: [PATCH v1 3/5] phy: pci serdes phy driver for keystone

2014-06-02 Thread Murali Karicheri
On 6/2/2014 2:16 AM, ABRAHAM, KISHON VIJAY wrote: Hi, On Thursday 15 May 2014 09:31 PM, Murali Karicheri wrote: This phy driver is used by keystone PCI driver. The hw vendor that provides the phy hw published only registers and their values. So this driver uses these hard coded values to

Re: [PATCH v1 3/5] phy: pci serdes phy driver for keystone

2014-06-02 Thread Jingoo Han
On Monday, June 02, 2014 3:17 PM, Kishon Vijay Abraham I wrote: > On Thursday 15 May 2014 09:31 PM, Murali Karicheri wrote: > > This phy driver is used by keystone PCI driver. The hw vendor that > > provides the phy hw published only registers and their values. So > > this driver uses these hard

Re: [PATCH v1 3/5] phy: pci serdes phy driver for keystone

2014-06-02 Thread Kishon Vijay Abraham I
Hi, On Thursday 15 May 2014 09:31 PM, Murali Karicheri wrote: > This phy driver is used by keystone PCI driver. The hw vendor that > provides the phy hw published only registers and their values. So > this driver uses these hard coded values to initialize the phy. > > CC: Grant Likely > CC: Rob

Re: [PATCH v1 3/5] phy: pci serdes phy driver for keystone

2014-06-02 Thread Kishon Vijay Abraham I
Hi, On Thursday 15 May 2014 09:31 PM, Murali Karicheri wrote: This phy driver is used by keystone PCI driver. The hw vendor that provides the phy hw published only registers and their values. So this driver uses these hard coded values to initialize the phy. CC: Grant Likely

Re: [PATCH v1 3/5] phy: pci serdes phy driver for keystone

2014-06-02 Thread Jingoo Han
On Monday, June 02, 2014 3:17 PM, Kishon Vijay Abraham I wrote: On Thursday 15 May 2014 09:31 PM, Murali Karicheri wrote: This phy driver is used by keystone PCI driver. The hw vendor that provides the phy hw published only registers and their values. So this driver uses these hard coded

Re: [PATCH v1 3/5] phy: pci serdes phy driver for keystone

2014-06-02 Thread Murali Karicheri
On 6/2/2014 2:16 AM, ABRAHAM, KISHON VIJAY wrote: Hi, On Thursday 15 May 2014 09:31 PM, Murali Karicheri wrote: This phy driver is used by keystone PCI driver. The hw vendor that provides the phy hw published only registers and their values. So this driver uses these hard coded values to

Re: [PATCH v1 3/5] phy: pci serdes phy driver for keystone

2014-05-27 Thread Arnd Bergmann
On Tuesday 27 May 2014 12:46:54 Murali Karicheri wrote: > I have checked the register set of the following drivers under > drivers/phy from the > master branch (v3.15-rc6) using the registers set available with us > internally and > I can't find a match. > > phy-exynos5250-sata.c

Re: [PATCH v1 3/5] phy: pci serdes phy driver for keystone

2014-05-27 Thread Murali Karicheri
On 5/23/2014 3:23 PM, Arnd Bergmann wrote: On Friday 23 May 2014 13:14:00 Murali Karicheri wrote: On 5/15/2014 12:14 PM, Arnd Bergmann wrote: On Thursday 15 May 2014 12:01:30 Murali Karicheri wrote: +static struct serdes_config ks_100mhz_pcie_5gbps_serdes[] = { + {0x, 0x0800,

Re: [PATCH v1 3/5] phy: pci serdes phy driver for keystone

2014-05-27 Thread Murali Karicheri
On 5/23/2014 3:23 PM, Arnd Bergmann wrote: On Friday 23 May 2014 13:14:00 Murali Karicheri wrote: On 5/15/2014 12:14 PM, Arnd Bergmann wrote: On Thursday 15 May 2014 12:01:30 Murali Karicheri wrote: +static struct serdes_config ks_100mhz_pcie_5gbps_serdes[] = { + {0x, 0x0800,

Re: [PATCH v1 3/5] phy: pci serdes phy driver for keystone

2014-05-27 Thread Arnd Bergmann
On Tuesday 27 May 2014 12:46:54 Murali Karicheri wrote: I have checked the register set of the following drivers under drivers/phy from the master branch (v3.15-rc6) using the registers set available with us internally and I can't find a match. phy-exynos5250-sata.c

Re: [PATCH v1 3/5] phy: pci serdes phy driver for keystone

2014-05-23 Thread Arnd Bergmann
On Friday 23 May 2014 13:14:00 Murali Karicheri wrote: > On 5/15/2014 12:14 PM, Arnd Bergmann wrote: > > On Thursday 15 May 2014 12:01:30 Murali Karicheri wrote: > >> +static struct serdes_config ks_100mhz_pcie_5gbps_serdes[] = { > >> + {0x, 0x0800, 0xff00}, > >> + {0x0060,

Re: [PATCH v1 3/5] phy: pci serdes phy driver for keystone

2014-05-23 Thread Murali Karicheri
On 5/15/2014 12:14 PM, Arnd Bergmann wrote: On Thursday 15 May 2014 12:01:30 Murali Karicheri wrote: +static struct serdes_config ks_100mhz_pcie_5gbps_serdes[] = { + {0x, 0x0800, 0xff00}, + {0x0060, 0x00041c5c, 0x00ff}, + {0x0064, 0x0343c700, 0xff00}, +

Re: [PATCH v1 3/5] phy: pci serdes phy driver for keystone

2014-05-23 Thread Murali Karicheri
On 5/15/2014 12:14 PM, Arnd Bergmann wrote: On Thursday 15 May 2014 12:01:30 Murali Karicheri wrote: +static struct serdes_config ks_100mhz_pcie_5gbps_serdes[] = { + {0x, 0x0800, 0xff00}, + {0x0060, 0x00041c5c, 0x00ff}, + {0x0064, 0x0343c700, 0xff00}, +

Re: [PATCH v1 3/5] phy: pci serdes phy driver for keystone

2014-05-23 Thread Arnd Bergmann
On Friday 23 May 2014 13:14:00 Murali Karicheri wrote: On 5/15/2014 12:14 PM, Arnd Bergmann wrote: On Thursday 15 May 2014 12:01:30 Murali Karicheri wrote: +static struct serdes_config ks_100mhz_pcie_5gbps_serdes[] = { + {0x, 0x0800, 0xff00}, + {0x0060, 0x00041c5c,

Re: [PATCH v1 3/5] phy: pci serdes phy driver for keystone

2014-05-15 Thread Arnd Bergmann
On Thursday 15 May 2014 12:01:30 Murali Karicheri wrote: > +static struct serdes_config ks_100mhz_pcie_5gbps_serdes[] = { > + {0x, 0x0800, 0xff00}, > + {0x0060, 0x00041c5c, 0x00ff}, > + {0x0064, 0x0343c700, 0xff00}, > + {0x006c, 0x0012, 0x00ff},

[PATCH v1 3/5] phy: pci serdes phy driver for keystone

2014-05-15 Thread Murali Karicheri
This phy driver is used by keystone PCI driver. The hw vendor that provides the phy hw published only registers and their values. So this driver uses these hard coded values to initialize the phy. CC: Grant Likely CC: Rob Herring CC: Mohit Kumar CC: Jingoo Han CC: Bjorn Helgaas

[PATCH v1 3/5] phy: pci serdes phy driver for keystone

2014-05-15 Thread Murali Karicheri
This phy driver is used by keystone PCI driver. The hw vendor that provides the phy hw published only registers and their values. So this driver uses these hard coded values to initialize the phy. CC: Grant Likely grant.lik...@linaro.org CC: Rob Herring robh...@kernel.org CC: Mohit Kumar

Re: [PATCH v1 3/5] phy: pci serdes phy driver for keystone

2014-05-15 Thread Arnd Bergmann
On Thursday 15 May 2014 12:01:30 Murali Karicheri wrote: +static struct serdes_config ks_100mhz_pcie_5gbps_serdes[] = { + {0x, 0x0800, 0xff00}, + {0x0060, 0x00041c5c, 0x00ff}, + {0x0064, 0x0343c700, 0xff00}, + {0x006c, 0x0012, 0x00ff}, +