Scott Wood wrote:

>>>> +static struct of_platform_driver ucc_uart_of_driver = {
>>>> +  .owner          = THIS_MODULE,
>>>> +  .name           = "ucc_uart",
>>> Maybe better fsl,ucc_uart?
> 
> fsl,qe-uart is defined by Documentation/powerpc/booting-without-of.txt.

Wait, I'm confused.  What does the of_platform_driver.name string have to do 
with the compatible field in the device tree?  Like I said earlier, I'm just 
following the example of the other QE device drivers.  If you want me to break 
that example, I'm going to need an explanation why the other drivers do it 
wrong.


-- 
Timur Tabi
Linux Kernel Developer @ Freescale
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev

Reply via email to