[PATCH v2 01/46] dm: syscon: Fix calling dev_dbg with an uninitialized device

Sean Anderson seanga2 at gmail.com
Wed Sep 16 14:51:56 CEST 2020


On 9/16/20 8:44 AM, Patrick DELAUNAY wrote:
> Hi Sean,
> 
>> From: U-Boot <u-boot-bounces at lists.denx.de> On Behalf Of Sean Anderson
>> Sent: mardi 15 septembre 2020 16:45
>>
>> We can't use dev_dbg here because we haven't bound to the device yet. Use
>> log_debug instead.
>>
>> Signed-off-by: Sean Anderson <seanga2 at gmail.com>
>> ---
>>
>> Changes in v2:
>> - New
>>
>>  drivers/core/syscon-uclass.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/core/syscon-uclass.c b/drivers/core/syscon-uclass.c index
>> b5cd763b6b..453933f59e 100644
>> --- a/drivers/core/syscon-uclass.c
>> +++ b/drivers/core/syscon-uclass.c
>> @@ -66,7 +66,7 @@ static int syscon_probe_by_ofnode(ofnode node, struct
>> udevice **devp)
>>
>>  	/* found node with "syscon" compatible, not bounded to SYSCON
>> UCLASS */
>>  	if (!ofnode_device_is_compatible(node, "syscon")) {
>> -		dev_dbg(dev, "invalid compatible for syscon device\n");
>> +		log_debug("invalid compatible for syscon device\n");
>>  		return -EINVAL;
>>  	}
>>
>> --
>> 2.28.0
> 
> As log API is used, the category could be indicated at the beginning of the files...
> before ' #include <log.h>'
> 
> + #define LOG_CATEGORY UCLASS_SYSCON
> +
> #include <common.h>
> #include <log.h>

Sure, I will include that in v3.

> Anyway
> 
> Reviewed-by: Patrick Delaunay <patrick.delaunay at st.com>
> 
> Thanks
> 
> Patrick
> 



More information about the U-Boot mailing list