Re: [maemo-leste] [PATCH] arm: Remove nokia_rx51 board

[Tom Rini <trini@konsulko.com>]

[-- Attachment #1: Type: text/plain, Size: 2308 bytes --]

On Tue, Jun 15, 2021 at 08:40:30AM +0300, Ivaylo Dimitrov wrote:
> 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.

Please post your WIP patches, thanks.

-- 
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]