Subject: [PATCH 2/2] PCI: imx6: Fix the clock for PCIe

From: [email-masked] (Marek Vasut)
Date: Fri, 11 Oct 2013 13:55:04 +0200

Dear Jingoo Han,

> On Friday, October 11, 2013 11:13 AM, Marek Vasut wrote:
> > The clk #205 are lvds_sel, not lvds_gate , so fix this. Moreover,
> > the PCIe needs lvds_gate for successful operation, so make the PCIe
> > driver enable (set as output) the lvds_gate .
> > 
> > Signed-off-by: Marek Vasut <[email-masked]>
> > Cc: Bjorn Helgaas <[email-masked]>
> > Cc: Frank Li <[email-masked]>
> > Cc: Richard Zhu <[email-masked]>
> > Cc: Sascha Hauer <[email-masked]>
> > Cc: Sean Cross <[email-masked]>
> > Cc: Shawn Guo <[email-masked]>
> > Cc: Tim Harvey <[email-masked]>
> > Cc: Yinghai Lu <[email-masked]>
> > ---
> > 
> >  arch/arm/boot/dts/imx6qdl.dtsi |  6 +++---
> >  drivers/pci/host/pci-imx6.c    | 19 ++++++++++++++++++-
> >  2 files changed, 21 insertions(+), 4 deletions(-)
> 
> Hi Marek Vasut,
> 
> How about splitting this patch into two patches such as
> arch part and driver part?
> 
> It is because driver part was merged into PCI tree by Bjorn Helgaas,[1]
> and arch part was merged into i.MX tree by Shawn Guo.[2]
> Thus, your patch can make the conflict issue.
> 
> For example,
>   [PATCH 1/2] ARM: dts: imx6qdl: Fix the clock for PCIe
>   [PATCH 2/2] PCI: imx6: Fix the clock for PCIe
> 
> Then, 1st patch can be merged to i.MX tree, and 2nd patch can be merged
> to PCI tree.
> 
> If I am wrong, please let me know kindly. :-)

By all means, you're right. Is the patch addressing the issue correctly (if we 
ignore that it's not split)? Shawn?

Best regards,
Marek Vasut