forked from mirrors/linux
		
	leds: ktd2692: Add missing of_node_put
The refcount of device_node increases after of_node_get() is called. So, a break out of the loop requires of_node_put(). This patch adds missing of_node_put() when loop breaks. Signed-off-by: Ingi Kim <ingi2.kim@samsung.com> Acked-by: Rob Herring <robh@kernel.org> Signed-off-by: Jacek Anaszewski <j.anaszewski@samsung.com>
This commit is contained in:
		
							parent
							
								
									f16703360d
								
							
						
					
					
						commit
						77e7915b15
					
				
					 1 changed files with 6 additions and 3 deletions
				
			
		|  | @ -306,21 +306,24 @@ static int ktd2692_parse_dt(struct ktd2692_context *led, struct device *dev, | |||
| 				   &cfg->movie_max_microamp); | ||||
| 	if (ret) { | ||||
| 		dev_err(dev, "failed to parse led-max-microamp\n"); | ||||
| 		return ret; | ||||
| 		goto err_parse_dt; | ||||
| 	} | ||||
| 
 | ||||
| 	ret = of_property_read_u32(child_node, "flash-max-microamp", | ||||
| 				   &cfg->flash_max_microamp); | ||||
| 	if (ret) { | ||||
| 		dev_err(dev, "failed to parse flash-max-microamp\n"); | ||||
| 		return ret; | ||||
| 		goto err_parse_dt; | ||||
| 	} | ||||
| 
 | ||||
| 	ret = of_property_read_u32(child_node, "flash-max-timeout-us", | ||||
| 				   &cfg->flash_max_timeout); | ||||
| 	if (ret) | ||||
| 	if (ret) { | ||||
| 		dev_err(dev, "failed to parse flash-max-timeout-us\n"); | ||||
| 		goto err_parse_dt; | ||||
| 	} | ||||
| 
 | ||||
| err_parse_dt: | ||||
| 	of_node_put(child_node); | ||||
| 	return ret; | ||||
| } | ||||
|  |  | |||
		Loading…
	
		Reference in a new issue
	
	 Ingi Kim
						Ingi Kim