Re: [PATCH 6/7] [media] marvell-ccic: MMP_CAMERA never worked

From: Jonathan Corbet
Date: Wed Jan 28 2015 - 20:11:17 EST


On Wed, 28 Jan 2015 22:17:46 +0100
Arnd Bergmann <arnd@xxxxxxxx> wrote:

> The mmp ccic driver uses a platform_data structure that has never
> existed in an upstream kernel and always fails to build:

This driver most assuredly did work on XO 1.75 machines, and the
platform_data structure does exist; it's the stuff added by Libin
afterward that apparently broke things. Strange that it only came out now,
though, nearly two years later. Libin, any thoughts on this?

Meanwhile, it is clearly broken, and I don't have an immediate fix, so,

Acked-by: Jonathan Corbet <corbet@xxxxxxx>

(Though I would like a different patch subject, since the current one is
wrong).

I think the right thing to do, alas, might be to back out all of Libin's
changes. They clearly are not being used by anybody and do not seem to be
getting any further development attention. It would like to keep this
driver working for XO systems for a while yet...but I won't get to it
right away.

Thanks,

jon
--
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/