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
| ||
|
Date: Thu, 14 Nov 2019 11:47:47 +0800 From: Greg Kroah-Hartman <gregkh@...uxfoundation.org> To: Sudip Mukherjee <sudipm.mukherjee@...il.com> Cc: Jiri Slaby <jslaby@...e.com>, linux-kernel <linux-kernel@...r.kernel.org>, linux-serial@...r.kernel.org Subject: Re: [PATCH] tty: serial: samsung: rename to fix build warning On Mon, Nov 11, 2019 at 03:18:38PM +0000, Sudip Mukherjee wrote: > Hi Greg, > > On Mon, Nov 4, 2019 at 4:43 PM Greg Kroah-Hartman > <gregkh@...uxfoundation.org> wrote: > > > > On Fri, Oct 18, 2019 at 08:47:07PM +0100, Sudip Mukherjee wrote: > > > The build of arm allmodconfig gives a warning: > > > > > > warning: same module names found: > > > drivers/tty/serial/samsung.ko > > > drivers/mtd/nand/onenand/samsung.ko > > > > > > Rename drivers/tty/serial/samsung.c to drivers/tty/serial/samsung_tty.c > > > to fix the warning. > > > > <snip> > > > > > > What are you going to break if you rename this module? > > hopefully nothing. > > > > > What configs enable both of these other than allmodconfig? Why rename > > the tty driver and not the mtd driver? Why not both? > > But, there is no other config defined which has both enabled. Though I can > make one, but since it is not defined and no one else has reported this, I think > its better if you discard this. ( I think you already have :) ) or if > you want I can > send a v2 renaming both. Let's rename both and cause equal pain :) thanks, greg k-h
Powered by blists - more mailing lists