Re: [U-Boot] ad-hoc config error

2016-09-23 Thread york sun
On 09/23/2016 11:23 AM, Tom Rini wrote: > On Fri, Sep 23, 2016 at 04:42:24PM +, york sun wrote: >> Tom, >> >> Following this thread, I have different macros to resolve. Please take a >> look at this patch http://patchwork.ozlabs.org/patch/660787/. >> >> It adds CONFIG_SYS_DCSR_RCPM_ADDR and CON

Re: [U-Boot] ad-hoc config error

2016-09-23 Thread Tom Rini
On Fri, Sep 23, 2016 at 04:42:24PM +, york sun wrote: > Tom, > > Following this thread, I have different macros to resolve. Please take a > look at this patch http://patchwork.ozlabs.org/patch/660787/. > > It adds CONFIG_SYS_DCSR_RCPM_ADDR and CONFIG_SYS_GIC_ADDR. This kind of > macros are

Re: [U-Boot] ad-hoc config error

2016-09-23 Thread york sun
Tom, Following this thread, I have different macros to resolve. Please take a look at this patch http://patchwork.ozlabs.org/patch/660787/. It adds CONFIG_SYS_DCSR_RCPM_ADDR and CONFIG_SYS_GIC_ADDR. This kind of macros are used across platforms to define the base addresses. All existing macros

Re: [U-Boot] ad-hoc config error

2016-09-21 Thread york sun
On 09/21/2016 08:45 AM, Tom Rini wrote: > On Wed, Sep 21, 2016 at 03:22:59PM +, york sun wrote: >> On 09/20/2016 03:30 PM, Tom Rini wrote: >>> On Tue, Sep 20, 2016 at 09:40:00PM +, york sun wrote: On 09/20/2016 02:36 PM, Tom Rini wrote: > On Tue, Sep 20, 2016 at 09:22:09PM +, y

Re: [U-Boot] ad-hoc config error

2016-09-21 Thread york sun
On 09/21/2016 12:01 PM, Tom Rini wrote: > On Wed, Sep 21, 2016 at 04:07:49PM +, york sun wrote: >> On 09/21/2016 08:45 AM, Tom Rini wrote: >>> On Wed, Sep 21, 2016 at 03:22:59PM +, york sun wrote: On 09/20/2016 03:30 PM, Tom Rini wrote: > On Tue, Sep 20, 2016 at 09:40:00PM +, y

Re: [U-Boot] ad-hoc config error

2016-09-21 Thread Tom Rini
On Wed, Sep 21, 2016 at 04:07:49PM +, york sun wrote: > On 09/21/2016 08:45 AM, Tom Rini wrote: > > On Wed, Sep 21, 2016 at 03:22:59PM +, york sun wrote: > >> On 09/20/2016 03:30 PM, Tom Rini wrote: > >>> On Tue, Sep 20, 2016 at 09:40:00PM +, york sun wrote: > On 09/20/2016 02:36 P

Re: [U-Boot] ad-hoc config error

2016-09-21 Thread Tom Rini
On Wed, Sep 21, 2016 at 03:22:59PM +, york sun wrote: > On 09/20/2016 03:30 PM, Tom Rini wrote: > > On Tue, Sep 20, 2016 at 09:40:00PM +, york sun wrote: > >> On 09/20/2016 02:36 PM, Tom Rini wrote: > >>> On Tue, Sep 20, 2016 at 09:22:09PM +, york sun wrote: > >>> > Tom and Simon,

Re: [U-Boot] ad-hoc config error

2016-09-21 Thread york sun
On 09/20/2016 03:30 PM, Tom Rini wrote: > On Tue, Sep 20, 2016 at 09:40:00PM +, york sun wrote: >> On 09/20/2016 02:36 PM, Tom Rini wrote: >>> On Tue, Sep 20, 2016 at 09:22:09PM +, york sun wrote: >>> Tom and Simon, After commit 371244cb19f9804711dd66e4281ff7979915fd2e, all m

[U-Boot] ad-hoc config error

2016-09-20 Thread york sun
Tom and Simon, After commit 371244cb19f9804711dd66e4281ff7979915fd2e, all merges with new macros defined will have the compiling error. How shall we fix it? Some macros can be added to Kconfig. But some are for local use, better than magic numbers. Adding them to the white-list doesn't sound ri

Re: [U-Boot] ad-hoc config error

2016-09-20 Thread Tom Rini
On Tue, Sep 20, 2016 at 09:40:00PM +, york sun wrote: > On 09/20/2016 02:36 PM, Tom Rini wrote: > > On Tue, Sep 20, 2016 at 09:22:09PM +, york sun wrote: > > > >> Tom and Simon, > >> > >> After commit 371244cb19f9804711dd66e4281ff7979915fd2e, all merges with > >> new macros defined will hav

Re: [U-Boot] ad-hoc config error

2016-09-20 Thread york sun
On 09/20/2016 02:36 PM, Tom Rini wrote: > On Tue, Sep 20, 2016 at 09:22:09PM +, york sun wrote: > >> Tom and Simon, >> >> After commit 371244cb19f9804711dd66e4281ff7979915fd2e, all merges with >> new macros defined will have the compiling error. How shall we fix it? >> Some macros can be added

Re: [U-Boot] ad-hoc config error

2016-09-20 Thread Tom Rini
On Tue, Sep 20, 2016 at 09:22:09PM +, york sun wrote: > Tom and Simon, > > After commit 371244cb19f9804711dd66e4281ff7979915fd2e, all merges with > new macros defined will have the compiling error. How shall we fix it? > Some macros can be added to Kconfig. But some are for local use, bette