Re: [PATCH] drivers/of/fdt.c: replace pr_warning by pr_warn

2015-07-26 Thread long.wanglong
On 2015/7/27 4:39, Rob Herring wrote: > On Fri, Jul 24, 2015 at 7:57 PM, long.wanglong > wrote: >> On 2015/7/25 3:00, Rob Herring wrote: >>> On Tue, Jul 21, 2015 at 3:01 AM, Wang Long wrote: Update the last pr_warning callsite in drivers/of. >>> >>> That is obvious in the diff, but can you

Re: [PATCH] drivers/of/fdt.c: replace pr_warning by pr_warn

2015-07-26 Thread Rob Herring
On Fri, Jul 24, 2015 at 7:57 PM, long.wanglong wrote: > On 2015/7/25 3:00, Rob Herring wrote: >> On Tue, Jul 21, 2015 at 3:01 AM, Wang Long wrote: >>> Update the last pr_warning callsite in drivers/of. >> >> That is obvious in the diff, but can you tell me why? > > Hi Rob, > > When we use script

Re: [PATCH] drivers/of/fdt.c: replace pr_warning by pr_warn

2015-07-24 Thread long.wanglong
On 2015/7/25 3:00, Rob Herring wrote: > On Tue, Jul 21, 2015 at 3:01 AM, Wang Long wrote: >> Update the last pr_warning callsite in drivers/of. > > That is obvious in the diff, but can you tell me why? Hi Rob, When we use script "./scripts/checkpatch.pl" to check a patch, using pr_warning will

Re: [PATCH] drivers/of/fdt.c: replace pr_warning by pr_warn

2015-07-24 Thread Rob Herring
On Tue, Jul 21, 2015 at 3:01 AM, Wang Long wrote: > Update the last pr_warning callsite in drivers/of. That is obvious in the diff, but can you tell me why? Rob > Signed-off-by: Wang Long > --- > drivers/of/fdt.c | 10 +- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --gi