Hello

Kevin Hilman wrote:

Hi Sergei,

[...]
 void __iomem *da8xx_syscfg0_base;
 void __iomem *da8xx_syscfg1_base;
+EXPORT_SYMBOL_GPL(da8xx_syscfg0_base);
Would it be an overkill to pass as a resource and ioremap in
usb_hcd_da8xx_probe() instead?
  Passing a resource is certainly an overkill.

Do you really want me to pass the single CFGCHIP2 register as a resource?! Since the rest of the system config. registers don't belong to USB...

Disagree.

I would much rather see this as a resource than as a global variable.

  But it's already a global variable!

What happens if/when this HW block is used on a non-da8xx device?

  I don't know. Why are you asking?

Kevin

WBR, Sergei

_______________________________________________
Davinci-linux-open-source mailing list
[email protected]
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source

Reply via email to