> -----Original Message----- > From: Lucas Stach [mailto:l.stach@xxxxxxxxxxxxxx] > Sent: Tuesday, September 23, 2014 5:19 PM > To: Zhu Richard-R65037 > Cc: linux-pci-owner@xxxxxxxxxxxxxxx; linux-pci@xxxxxxxxxxxxxxx; Guo Shawn- > R65073; festevam@xxxxxxxxx; tharvey@xxxxxxxxxxxxx > Subject: Re: [PATCH v2]PCI: imx6: enable pcie on imx6sx sdb and imx6qdl > sabreauto. > > Hi Richard, > > Am Dienstag, den 23.09.2014, 12:11 +0800 schrieb Richard Zhu: > > Hi Tim: > > After I changed the wait clocks stabilize delay after pcie_ref_en is > > set in this patch-set. > > Can you help to make a double check whether it's ok or not at your side? > > > > Thanks in advanced. > > > I'll go through this series today. Please give me some time to properly > comment on every patch before posting a new version of the series. > Thanks. > Ok no problem, thanks a lot for your kindly help. Best Regards Richard Zhu > > Main changes since the v1: > > 1. Regarding to Lucas' comments, seperated the enalbe pcie on imx6qdl > > sabreauto patch. > > 2. Add the description why the wait clock stabilize delay should be > > run after pcie_ref_en is set. > > 3. Return 0 directly in suspend call back. > > > > Main changes since the RFC: > > Thanks for quick review from Lucas. > > 1. seperate the smashed patch-set. > > 2. remove the "power-on-gpio". > > 3. add/update the pcie-supply of the dts and binding. > > 4. > > > > [PATCH v2 1/5] PCI: imx6: enable pcie on imx6qdl sabreauto [PATCH v2 > > 2/5] PCI: imx6: wait the clocks to stabilize after ref_en [PATCH v2 > > 3/5] PCI: imx6: update dts and binding for imx6sx pcie [PATCH v2 4/5] > > PCI: imx6: add imx6sx pcie related gpr bits [PATCH v2 5/5] PCI: imx6: > > add imx6sx pcie support > > -- > Pengutronix e.K. | Lucas Stach | > Industrial Linux Solutions | http://www.pengutronix.de/ | ��.n��������+%������w��{.n�����{���"�)��jg��������ݢj����G�������j:+v���w�m������w�������h�����٥