Re: [GIT PULL] ARM: SPEAr1310 PCIe updates

[Olof Johansson]

Hi,



On Tue, Jul 8, 2014 at 10:48 PM, Mohit KUMAR DCG <Mohit.KUMAR@xxxxxx> wrote:
> Hello Olof,
>
> Please pull PCIe updates for ST's SPEAr1310 SoC.
>
> Last version of posted V8 patches have been Acked-by respective maintainers.
> https://lkml.org/lkml/2014/4/15/260
>
> Apart from arch/arm/ changes, it also contains patches for drivers/{pci|phy}.
> Bjorn advised to get complete series via arm-soc tree.
> http://www.spinics.net/lists/linux-pci/msg30271.html
>
>  Regards
>  Mohit
>
> The following changes since commit 4c834452aad01531db949414f94f817a86348d59:
>   Linus Torvalds (1):
>         Linux 3.16-rc3
>
> are available in the git repository at:
>
>   https://git.kernel.org/pub/scm/linux/kernel/git/vireshk/linux.git spear/pcie-support-v8
>
> Mohit Kumar (1):
>       SPEAr13xx: defconfig: Update
>
> Pratyush Anand (8):
>       clk: SPEAr13XX: Fix pcie clock name
>       SPEAr13XX: Fix static mapping table
>       phy: SPEAr1310/40-miphy: Add binding information
>       SPEAr: misc: Add binding information
>       phy: SPEAr1310/40-miphy: Add phy driver for PCIe and SATA
>       SPEAr13XX: Add binding information for PCIe controller
>       SPEAr13XX: dts: Add PCIe node information
>       pcie: SPEAr13xx: Add designware wrapper support

There are so many things wrong with this pull request.

* Why is this not coming from the spear maintainer that knows how to
look at patches and deal with these things?
* Why have you separated out the PCIe node info, but not the phy DTS
updates in separate patches?
* Why are you sending a pull request for a repo that is not yours?
* Why is Viresh the git committer but hasn't signed off on the patches?

Are you sharing accounts on kernel.org? (Adding kernel.org admin on cc
just in case).

I'm not going to touch this pull request until the above has been
sorted out and explained.


-Olof
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/