Re: [PATCH v2 10/10] drivers: PL011: add support for the ARM SBSA generic UART

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Mar 12, 2015 at 01:43:17PM +0000, Andre Przywara wrote:
> Hi Russel,
> 
> thanks a lot for looking at the patches!
> 
> On 12/03/15 10:52, Russell King - ARM Linux wrote:
> > On Wed, Mar 04, 2015 at 05:59:54PM +0000, Andre Przywara wrote:
> >> +module_platform_driver(arm_sbsa_uart_platform_driver);
> > 
> > No need to open code the initialisation, rather than using the
> > module_*_driver() helper macros to avoid the problem which Dave mentioned.
> > 
> > These macros are only there to avoid having to write out the same boiler
> > plate in loads of simple drivers.  As soon as a driver has more than one
> > device driver structure in it, it needs to be open coded.
> 
> Actually I prepared this already for the ACPI guys, which want to stuff
> their ACPI table match function in there - I think then we need the open
> coded version. So if you don't mind too much, I'd like to keep it like
> this and hope for someone to actually use it ;-)

Either your statement is ambiguous, or I'm not understanding you.

You can't "keep it like this" where "this" is the above code.  The
above will fail if the driver is built as a module, and cause a build
time error.  That is not acceptable.

-- 
FTTC broadband for 0.8mile line: currently at 10.5Mbps down 400kbps up
according to speedtest.net.
--
To unsubscribe from this list: send the line "unsubscribe linux-serial" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux PPP]     [Linux FS]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Linmodem]     [Device Mapper]     [Linux Kernel for ARM]

  Powered by Linux