On 22.07.2020 16:44, cristian.bir...@microchip.com wrote:
> From: Claudiu Beznea <claudiu.bez...@microchip.com>
>
> Instead of trying to match every possible compatible use
> of_find_matching_node_and_match() and pass the compatible array.
>
> Signed-off-by: Claudiu Beznea <claudiu.bez...@microchip.com>
> ---
> drivers/usb/gadget/udc/atmel_usba_udc.c | 25 ++++++++++++++++++-------
> 1 file changed, 18 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/usb/gadget/udc/atmel_usba_udc.c
> b/drivers/usb/gadget/udc/atmel_usba_udc.c
> index c5128c229c52..ee2b550aa400 100644
> --- a/drivers/usb/gadget/udc/atmel_usba_udc.c
> +++ b/drivers/usb/gadget/udc/atmel_usba_udc.c
> @@ -2112,6 +2112,13 @@ static const struct of_device_id atmel_udc_dt_ids[] = {
>
> MODULE_DEVICE_TABLE(of, atmel_udc_dt_ids);
>
> +static const struct of_device_id atmel_pmc_dt_ids[] = {
> + { .compatible = "atmel,at91sam9g45-pmc" },
> + { .compatible = "atmel,at91sam9rl-pmc" },
> + { .compatible = "atmel,at91sam9x5-pmc" },
> + { /* sentinel */ }
> +};
> +
> static struct usba_ep * atmel_udc_of_init(struct platform_device *pdev,
> struct usba_udc *udc)
> {
> @@ -2128,13 +2135,17 @@ static struct usba_ep * atmel_udc_of_init(struct
> platform_device *pdev,
>
> udc_config = match->data;
> udc->errata = udc_config->errata;
> - udc->pmc = syscon_regmap_lookup_by_compatible("atmel,at91sam9g45-pmc");
> - if (IS_ERR(udc->pmc))
> - udc->pmc =
> syscon_regmap_lookup_by_compatible("atmel,at91sam9rl-pmc");
> - if (IS_ERR(udc->pmc))
> - udc->pmc =
> syscon_regmap_lookup_by_compatible("atmel,at91sam9x5-pmc");
> - if (udc->errata && IS_ERR(udc->pmc))
> - return ERR_CAST(udc->pmc);
> + if (udc->errata) {
> + pp = of_find_matching_node_and_match(NULL, atmel_pmc_dt_ids,
> + NULL);
> + if (!pp)
> + return ERR_PTR(-ENODEV);
> +
> + udc->pmc = syscon_node_to_regmap(pp);
> + of_node_put(pp);
> + if (IS_ERR(udc->pmc))
> + return ERR_CAST(udc->pmc);
> + }
This seems a bit not similar. I may had been wrong at the moment I wrote
this patch. Probably the best would be:
+ udc->pmc = ERR_PTR(-ENODEV);
+ pp = of_find_matching_node_and_match(NULL, atmel_pmc_dt_ids,
+ NULL);
+ if (pp) {
+ udc->pmc = syscon_node_to_regmap(pp);
+ of_node_put(pp);
+ }
+
+ if ((!pp || IS_ERR(udc->pmc)) && udc->errata)
+ return ERR_CAST(udc->pmc);
Thank you,
Claudiu Beznea
>
> udc->num_ep = 0;
>
>
