Hi,
On 22.05.21 г. 0:36 ч., Pali Rohár wrote:
On Friday 21 May 2021 10:44:18 Tom Rini wrote:
On Wed, May 19, 2021 at 11:52:03AM -0400, Tom Rini wrote:
On Wed, May 19, 2021 at 03:27:48PM +0200, Pali Rohár wrote:
On Tuesday 18 May 2021 21:26:40 Tom Rini wrote:
This board has not been converted to CONFIG_DM_USB by the deadline.
Remove it.
I'm very disappointed that you want to remove Nokia N900 from U-Boot.
I was waiting waiting half of year because other developers did not
react to issues which were introduced and neither to patches which I
sent (+ trying to remind open issues). And also I was waiting another
half of year until other N900 related patches were merged. So the whole
slowdown was not caused by me, why it is taking so long.
Now there is still one N900 DM related patch waiting for review. I'm
converting code step by step.
So the ball is not on my side.
So, what patch(es) need to be applied to get DM_USB enabled? Thanks.
I don't see any open patches from you that look related to enabling
DM_USB on the platform. If you want to disable USB on the platform for
now instead, that's fine too.
I tried to migrate the latest master to DM_USB, but unfortunately the
results are pretty much sad - adding OF_CONTROL (which is a prerequisite
to have DM_USB IIUC) and OF_BOARD (so binary to be compiled), adds ~100k
to the size of the u-boot binary, so it becomes 370284 bytes. Given that
we have less than 256k of storage space for the u-boot, the produced
binary cannot be used on n900 the same way current (no DM_USB) binary is
used.
As I see it, there are not much options left - u-boot on N900 is not
SPL, so we can't use OF_PLATDATA, which in turn always links libfdt.
Also, if I read the code (usb-uclass.c) correctly, enabling DM_USB
requires the board to be converted to DT and this is way bigger change.
Please advice on how to proceed.
I wanted to look at next DM step -- USB -- after current DM step
watchdog is finished. But patch for watchdog is still waiting for
review:
http://patchwork.ozlabs.org/project/uboot/patch/20210309201915.16586-1-p...@kernel.org/
Due to big chance of diff conflicts between these board patches, I do
not want to have more than one part of code in "in progress" state.
Regards,
Ivo
