Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1642378

Re: [PATCH 1/2] arm64: marvell: dts: fill MachiatoBin board description

From Russell King - ARM Linux <linux@armlinux.org.uk>
Newsgroups linux.kernel
Subject Re: [PATCH 1/2] arm64: marvell: dts: fill MachiatoBin board description
Date 2017-05-16 12:00 +0200
Message-ID <tHHqO-3Wv-13@gated-at.bofh.it> (permalink)
References <tHxB8-6e6-5@gated-at.bofh.it> <tHxB8-6e6-15@gated-at.bofh.it> <tHxB8-6e6-13@gated-at.bofh.it> <tHGXM-3Nc-21@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Tue, May 16, 2017 at 11:27:19AM +0200, Marcin Wojtas wrote:
> I do not intend to interfere your work at all. I took a look on your
> branch, and my patch does not seem to be really colliding with it.

I can't see how you can say that when the branch contains support for
SDHCI and ethernet.  It obviously will collide, since it conflicts with
the changes I have.

> The board is starting to get really popular and a lot has been happening
> around it recently - missing bits like 'chosen' node or the interfaces
> is pretty annoying.

Given that features like SDHCI and basic ethernet support have only just
been merged during the merge window, how about giving those who are
supporting the platform some time to organise their trees and get patches
out there, rather than cutting across those who have put considerable
effort into the platform already, or working with those who have.

The whole Armada 8k support is all very new, and there's still lots of
fundamental bits that are missing - pinmux and gpio are the two biggest
ones.

I've already put effort into cleaning up the mvebu pinmux code (already
merged) so that we can cleanly merge the pinmux support, but both of
these are a sticking point with free-electrons - they have a view on
how it should be represented in DT which does not fit with the current
orion-gpio usage, nor with the "system controller" being in drivers/clk.

The code which I have in my tree is correct for the Armada 8k hardware
(which has some weirdness about which gpios on each CP110 appear to the
external world - some are used for inter-CP110 communication and must
not be exposed) so any additional work should be based on the code in
my tree.

I have no solution for the sticking point at present, and this is an
area that's worth additional effort to resolve.

-- 
RMK's Patch system: http://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up
according to speedtest.net.

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

Re: [PATCH 1/2] arm64: marvell: dts: fill MachiatoBin board  description Russell King - ARM Linux <linux@armlinux.org.uk> - 2017-05-16 01:30 +0200
  Re: [PATCH 1/2] arm64: marvell: dts: fill MachiatoBin board description Marcin Wojtas <mw@semihalf.com> - 2017-05-16 11:30 +0200
    Re: [PATCH 1/2] arm64: marvell: dts: fill MachiatoBin board  description Thomas Petazzoni <thomas.petazzoni@free-electrons.com> - 2017-05-16 12:00 +0200
      Re: [PATCH 1/2] arm64: marvell: dts: fill MachiatoBin board  description Russell King - ARM Linux <linux@armlinux.org.uk> - 2017-05-16 12:10 +0200
        Re: [PATCH 1/2] arm64: marvell: dts: fill MachiatoBin board  description Thomas Petazzoni <thomas.petazzoni@free-electrons.com> - 2017-05-16 14:00 +0200
          Re: [PATCH 1/2] arm64: marvell: dts: fill MachiatoBin board  description Russell King - ARM Linux <linux@armlinux.org.uk> - 2017-05-16 14:20 +0200
            Re: [PATCH 1/2] arm64: marvell: dts: fill MachiatoBin board description Marcin Wojtas <mw@semihalf.com> - 2017-05-16 14:50 +0200
              Re: [PATCH 1/2] arm64: marvell: dts: fill MachiatoBin board  description Russell King - ARM Linux <linux@armlinux.org.uk> - 2017-05-16 15:50 +0200
    Re: [PATCH 1/2] arm64: marvell: dts: fill MachiatoBin board  description Russell King - ARM Linux <linux@armlinux.org.uk> - 2017-05-16 12:00 +0200
    Re: [PATCH 1/2] arm64: marvell: dts: fill MachiatoBin board  description Russell King - ARM Linux <linux@armlinux.org.uk> - 2017-05-16 13:20 +0200
      Re: [PATCH 1/2] arm64: marvell: dts: fill MachiatoBin board description Marcin Wojtas <mw@semihalf.com> - 2017-05-16 13:40 +0200

csiph-web