Hello Simon,
Creating a duplicate driver for enabling the flag will add a lot of redundant code. Is there a way, I can add some conditional code to enable this flag for case when pl01x_serial_platdata in the same driver.
Regards Priyanka
From: Simon Glass sjg@chromium.org Sent: Friday, January 4, 2019 3:00 AM To: Priyanka Jain priyanka.jain@nxp.com Cc: bmeng.cn@gmail.com; York Sun york.sun@nxp.com; Vabhav Sharma vabhav.sharma@nxp.com; u-boot@lists.denx.de Subject: Re: Issue faced due to "serial: Remove DM_FLAG_PRE_RELOC flag in various drivers" patch
Hi Priyanka,
On Thu, 3 Jan 2019 at 03:31, Priyanka Jain <priyanka.jain@nxp.commailto:priyanka.jain@nxp.com> wrote: Hello,
I am working to add support for NXP LX2160ARDB board support" "https://patchwork.ozlabs.org/patch/1004541/https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatchwork.ozlabs.org%2Fpatch%2F1004541%2F&data=02%7C01%7Cpriyanka.jain%40nxp.com%7C3bb67c5c25b9469f486a08d671c2a526%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C636821478141571674&sdata=N5gMRlinhUzEIrg5Tz5Dk%2BrYkn2lUN2PnmRchhrZ9ys%3D&reserved=0"
For serial driver, we are using PL01X_SERIAL driver with DM_SERIAL enabled Since we need to support various clock frequencies, we have defined pl01x_serial_platdata in board file So it's a combination of CONFIG_DM_SERIAL + pl01x_serial_platdata
Also we have requirement to use serial driver before relocation as well.
Now with the patch " serial: Remove DM_FLAG_PRE_RELOC flag in various drivers" the flag "DM_FLAG_PRE_RELOC" is no longer getting set resulting in UART driver not being available before relocation.
I tried adding "u-boot,dm-pre-reloc" in uart node, but it doesn't helped. [May be because uart nodes are set to disable to use 'pl01x_serial_platdata' method]
Can you please provide some suggestions on this issue. How to use "DM_SERIAL + pl01x_serial_platdata" feature in pre_reloc phase?
This really should be using device tree - how come your board cannot use that?
But it seems that removing the flag has in fact made it impossible to use platdata before relocation? That seems bad. One option is to create your own driver when that flag set.
Regards, Simon
Regards Priyanka