Re: [PATCH] MIPS: octeon: Add support for the UBNT E200 board

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On Mon, Nov 24, 2014 at 09:25:26AM -0800, David Daney wrote:
> On 11/24/2014 05:51 AM, James Cowgill wrote:
> >From: Markos Chandras <markos.chandras@xxxxxxxxxx>
> >
> >Add support for the UBNT E200 board (EdgeRouter/EdgeRouter Pro 8 port).
> >
> >Signed-off-by: Markos Chandras <markos.chandras@xxxxxxxxxx>
> >Signed-off-by: James Cowgill <James.Cowgill@xxxxxxxxxx>
> 
> NACK.
> 
> As far as I know, these boards have a boot loader that supplies a correct
> device tree, there should be no need to hack up the kernel like this.
> 
> As far as I know, Andreas is running a kernel.org kernel on these boards
> without anything like this.

It gets called from Octeon Ethernet driver through cvmx_helper_link_get()
frequently so the console gets spammed about unknown board, and probably
also the link status is bogus as a result.

I guess all those board helper functions should be factored
out and Octeon Ethernet should use DT only. I think we need TODO
file under staging/octeon.

A.





[Index of Archives]     [Linux MIPS Home]     [LKML Archive]     [Linux ARM Kernel]     [Linux ARM]     [Linux]     [Git]     [Yosemite News]     [Linux SCSI]     [Linux Hams]

  Powered by Linux