Re: [U-Boot] Issue faced due to "serial: Remove DM_FLAG_PRE_RELOC flag in various drivers" patch

2019-01-07 Thread Simon Glass 
Hi,

On Mon, 7 Jan 2019 at 00:19, Priyanka Jain  wrote:
>
> Hello Simon,
>
>
>
> Creating a duplicate driver for enabling the flag will add a lot of redundant 
> code.

Please can you avoid top posting?

>
> Is there a way, I can add some conditional code to enable this flag for case 
> when pl01x_serial_platdata in the same driver.

Firstly, can you use device tree?

No you cannot add an arch-specific #ifdef here. Is the overhead a lot?

Regards,
Simon


>
>
>
> Regards
>
> Priyanka
>
> From: Simon Glass 
> Sent: Friday, January 4, 2019 3:00 AM
> To: Priyanka Jain 
> Cc: bmeng...@gmail.com; York Sun ; Vabhav Sharma 
> ; 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  wrote:
>
> Hello,
>
> I am working to add support for NXP LX2160ARDB board support" 
> "https://patchwork.ozlabs.org/patch/1004541/;
>
> 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
>
>
>
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] Issue faced due to "serial: Remove DM_FLAG_PRE_RELOC flag in various drivers" patch

2019-01-06 Thread Priyanka Jain 
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 
Sent: Friday, January 4, 2019 3:00 AM
To: Priyanka Jain 
Cc: bmeng...@gmail.com; York Sun ; Vabhav Sharma 
; 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 
mailto:priyanka.j...@nxp.com>> wrote:
Hello,

I am working to add support for NXP LX2160ARDB board support" 
"https://patchwork.ozlabs.org/patch/1004541/"

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



___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] Issue faced due to "serial: Remove DM_FLAG_PRE_RELOC flag in various drivers" patch

2019-01-03 Thread Simon Glass 
Hi Priyanka,

On Thu, 3 Jan 2019 at 03:31, Priyanka Jain  wrote:

> Hello,
>
> I am working to add support for NXP LX2160ARDB board support" "
> https://patchwork.ozlabs.org/patch/1004541/;
>
> 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
>
>
>
>
>
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] Issue faced due to "serial: Remove DM_FLAG_PRE_RELOC flag in various drivers" patch

2019-01-03 Thread Lukasz Majewski 
Hi Priyanka,

> Hello,
> 
> I am working to add support for NXP LX2160ARDB board support"
> "https://patchwork.ozlabs.org/patch/1004541/;
> 
> 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?

I'm not sure if this will help in your particular case, but I've got
similar issue with this flag.

Please see this patch and following discussion:
http://patchwork.ozlabs.org/patch/1019846/

> 
> Regards
> Priyanka
> 
> 
> 
> 
> ___
> U-Boot mailing list
> U-Boot@lists.denx.de
> https://lists.denx.de/listinfo/u-boot




Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,  Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lu...@denx.de


pgpKDxHkwMELG.pgp
Description: OpenPGP digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] Issue faced due to "serial: Remove DM_FLAG_PRE_RELOC flag in various drivers" patch

2019-01-03 Thread Priyanka Jain 
Hello,

I am working to add support for NXP LX2160ARDB board support" 
"https://patchwork.ozlabs.org/patch/1004541/;

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?

Regards
Priyanka




___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot