linux-kernel - Re: [PATCH 06/06] serial8250-em: Add Emma Mobile UART driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CANqRtoR7rtJW65jY3urnTcaS5S6swY84C-Xa8NMQkZP66m+==w@mail.gmail.com>
Date:	Thu, 3 May 2012 21:19:40 +0900
From:	Magnus Damm <magnus.damm@...il.com>
To:	Greg KH <gregkh@...uxfoundation.org>
Cc:	Paul Gortmaker <paul.gortmaker@...driver.com>,
	linux-serial@...r.kernel.org, horms@...ge.net.au, arnd@...db.de,
	linux-sh@...r.kernel.org, swarren@...dotorg.org,
	linux-kernel@...r.kernel.org, rjw@...k.pl, lethal@...ux-sh.org,
	olof@...om.net, dan.j.williams@...el.com, alan@...ux.intel.com
Subject: Re: [PATCH 06/06] serial8250-em: Add Emma Mobile UART driver

On Thu, May 3, 2012 at 6:22 AM, Greg KH <gregkh@...uxfoundation.org> wrote:
> On Wed, May 02, 2012 at 10:41:12AM -0400, Paul Gortmaker wrote:
>> On 12-05-02 08:47 AM, Magnus Damm wrote:
>> > From: Magnus Damm <damm@...nsource.se>
>> >
>> > Add a platform driver for Emma Mobile 8250 hardware.
>>
>>
>> [...]
>>
>> > +
>> > +config SERIAL_8250_EM
>> > +   tristate "Support for Emma Mobile intergrated serial port"
>> > +   depends on SERIAL_8250
>>
>> Should we depend on arch=ARM or something similar?  Even if the
>> driver is written to be arch independent, if the Emma itself is
>> physically tied to ARM platforms, we probably don't need to be
>> building it for sparc allyesconfig etc.
>
> Yeah, it doesn't build on x86-64, I get the following build problems:
>        ERROR: "clk_get_rate" [drivers/tty/serial/8250/8250_em.ko] undefined!
>        ERROR: "clk_enable" [drivers/tty/serial/8250/8250_em.ko] undefined!
>        ERROR: "clk_get" [drivers/tty/serial/8250/8250_em.ko] undefined!
>        ERROR: "clk_put" [drivers/tty/serial/8250/8250_em.ko] undefined!
>        ERROR: "clk_disable" [drivers/tty/serial/8250/8250_em.ko] undefined!
>
> So it should depend on some arch that has clk at the least.
>
> Magnus, care to redo just this one patch that fixes this up?  I've
> applied the first 5 already.

Done!

I decided to go with SERIAL_8250 && ARM && HAVE_CLK to limit the
exposure a bit but not hinder any single ARM kernel image work. Hope
this balance is making everyone happy.

It is not impossible that this driver can be used on older MIPS based
SoCs from NEC, but that can easily be dealt with later if needed.

Thanks for your help!

/ magnus
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/