Re: [PATCH 0/6] SolidRun Hummingboard/Cubox-i DT updates

On Sat, Aug 23, 2014 at 10:10:58AM +0100, Russell King - ARM Linux wrote:

Here's a set of patches updating the Cubox-i/Hummingboard DT files.

The first two are fairly important fixes, so should at least go in
for -rc kernels.

The SPDIF patch is something which could go in -rc as well as it's a
cosmetic patch, changing the name displayed in (eg) pulseaudio for the
SPDIF output from the developer-centric "imx-spdif" to "Integrated SPDIF".

I will sent the first 3 for -rc.  But I second Iain's comment that the
first patch should have a better commit log to explain why it's an
important fix.

Patch 4 updates the Hummingboard support as SolidRun now have a Dual
version of this platform, which needs imx6q.dtsi instead of imx6dl.dtsi,
and also has mSATA support.

Though it's mostly just a DTS split, the diff stat looks too scary to
be a material of -rc.

Patch 5 adds the mSATA support for the Dual version of the Hummingboard.

Patch 6 is the first patch which breaks something on the initial version
of the Hummingboard, aka Carrier1, specifically the IrDA receiver.  In
hindsight, we should not have used the name "Hummingboard" for the
Carrier1.  The Carrier1 is obsolete, and people are being encouraged to
dispose of the C1 hardware.

Out of these, I think it would be worth getting the first five patches
in for 3.17 (even though we're in -rc), but patch 6 should certainly
wait for the next merge window.

Patches 4 ~ 6 are not fixes in any way, so I prefer to queue them for
the next merge window.

Shawn

On Mon, Aug 25, 2014 at 02:29:32PM +0800, Shawn Guo wrote:

On Sat, Aug 23, 2014 at 10:10:58AM +0100, Russell King - ARM Linux wrote:

quoted

Here's a set of patches updating the Cubox-i/Hummingboard DT files.

The first two are fairly important fixes, so should at least go in
for -rc kernels.

The SPDIF patch is something which could go in -rc as well as it's a
cosmetic patch, changing the name displayed in (eg) pulseaudio for the
SPDIF output from the developer-centric "imx-spdif" to "Integrated SPDIF".

I will sent the first 3 for -rc.  But I second Iain's comment that the
first patch should have a better commit log to explain why it's an
important fix.

I still have no answer for that, so I suggest dropping it until Rabeeh
can write a better commit log.

-- 
FTTC broadband for 0.8mile line: currently at 9.5Mbps down 400kbps up
according to speedtest.net.