On Tue, Feb 02, 2016 at 10:10:49PM +0100, Arnd Bergmann wrote:
> On Tuesday 02 February 2016 21:47:39 Linus Walleij wrote:
> > } else {
> > -#ifdef CONFIG_ARCH_VERSATILE
> > - fb->off_ienb = CLCD_PL111_IENB;
> > - fb->off_cntl = CLCD_PL111_CNTL;
> > -#else
> > - fb->off_ienb = CLCD_PL110_IENB;
> > - fb->off_cntl = CLCD_PL110_CNTL;
> > -#endif
> > + if (of_machine_is_compatible("arm,versatile-ab") ||
> > + of_machine_is_compatible("arm,versatile-pb")) {
> > + fb->off_ienb = CLCD_PL111_IENB;
> > + fb->off_cntl = CLCD_PL111_CNTL;
> > + } else {
> > + fb->off_ienb = CLCD_PL110_IENB;
> > + fb->off_cntl = CLCD_PL110_CNTL;
> > + }
> > }
> >
>
> Could that be done based on the AMBA device ID instead?
Unfortunately not. It's a mistake made on one Versatile board
which reverses the registers. There's nothing to distinguish it
in the primecell itself.
--
RMK's Patch system: http://www.arm.linux.org.uk/developer/patches/
FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up
according to speedtest.net.
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html