Thread (29 messages) 29 messages, 6 authors, 2018-07-20

[PATCH 5/5] net: add MTD support to eth_platform_get_mac_address()

From: andrew@lunn.ch (Andrew Lunn)
Date: 2018-07-20 14:16:25
Also in: linux-omap, lkml, netdev

On Fri, Jul 20, 2018 at 10:47:51AM +0530, Sekhar Nori wrote:
On Thursday 19 July 2018 09:05 PM, Bartosz Golaszewski wrote:
quoted
2018-07-19 17:27 GMT+02:00 Andrew Lunn [off-list ref]:
quoted
quoted
Unfortunately: this would effectively block me from improving the
support for older davinci boards.
Is there something blocking you from converting the board to device
tree? This is something i did with a lot of the Marvell boards a few
years ago. For a while, we had both DT and board setup files. After a
couple of cycles, we killed off the setup files.

       Andrew
Actually some board are supported both in DT and board files
(da850-evm) right now, but Sekhar wants to keep the support via board
files in the kernel so that's a no go.
Its not that I want it that way, but we cannot get rid of board files
till DT has equivalent support.

The bigger issue is not on DA850, but on the 5 older DaVinci SoCs which
do not support device-tree based boot today.
The nice thing about board files is they keep any ugly code near to
where it is needed. The proposal here is to put some 'temporary' code
in the net core. And it is assumed at some point somebody will write
nvmem over MTD, which can be used to replace this temporary code, but
that in itself needs an ugly list of special cases when using board
files?

I would prefer somebody just did the work to convert these 5 boards to
DT, with a clean design of how nvmem over MTD would work. Having
converted a number of Marvell boards to DT, i have an idea of the
effort required. If most of the drivers already support DT, it can be
done quickly. So the big job here is probably nvmem over MTD.

     Andrew
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help