[PATCH v2] ucc_uart: add support for Freescale QUICCEngine UART

Timur Tabi timur at freescale.com
Fri Jan 4 05:26:27 EST 2008


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



More information about the Linuxppc-dev mailing list