* [PATCH 1/2] serial: sc16is7xx: depend on I2C
@ 2014-04-25 19:53 jon
2014-04-25 19:53 ` [PATCH 2/2] serial: sc16is7xx: fix implicit decl of func copy_{to,from}_user jon
2014-04-25 20:23 ` [PATCH 1/2] serial: sc16is7xx: depend on I2C Greg KH
0 siblings, 2 replies; 7+ messages in thread
From: jon @ 2014-04-25 19:53 UTC (permalink / raw)
To: linux-kernel, linux-serial; +Cc: gregkh, shc_work, Jon Ringle
From: Jon Ringle <jringle@gridpoint.com>
Signed-off-by: Jon Ringle <jringle@gridpoint.com>
---
drivers/tty/serial/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/tty/serial/Kconfig b/drivers/tty/serial/Kconfig
index c3e2b32..64c5659 100644
--- a/drivers/tty/serial/Kconfig
+++ b/drivers/tty/serial/Kconfig
@@ -1180,6 +1180,7 @@ config SERIAL_SCCNXP_CONSOLE
config SERIAL_SC16IS7XX
tristate "SC16IS7xx serial support"
+ depends on I2C
select SERIAL_CORE
select REGMAP_I2C if I2C
help
--
1.8.5.4
^ permalink raw reply related [flat|nested] 7+ messages in thread* [PATCH 2/2] serial: sc16is7xx: fix implicit decl of func copy_{to,from}_user
2014-04-25 19:53 [PATCH 1/2] serial: sc16is7xx: depend on I2C jon
@ 2014-04-25 19:53 ` jon
2014-04-25 20:23 ` [PATCH 1/2] serial: sc16is7xx: depend on I2C Greg KH
1 sibling, 0 replies; 7+ messages in thread
From: jon @ 2014-04-25 19:53 UTC (permalink / raw)
To: linux-kernel, linux-serial; +Cc: gregkh, shc_work, Jon Ringle
From: Jon Ringle <jringle@gridpoint.com>
Fix by including linux/uaccess.h:
drivers/tty/serial/sc16is7xx.c: In function 'sc16is7xx_ioctl':
>> drivers/tty/serial/sc16is7xx.c:861:3: error: implicit declaration of function 'copy_from_user' [-Werror=implicit-function-declaration]
>> drivers/tty/serial/sc16is7xx.c:867:3: error: implicit declaration of function 'copy_to_user' [-Werror=implicit-function-declaration]
Signed-off-by: Jon Ringle <jringle@gridpoint.com>
---
drivers/tty/serial/sc16is7xx.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/tty/serial/sc16is7xx.c b/drivers/tty/serial/sc16is7xx.c
index ed139f5..7206a64 100644
--- a/drivers/tty/serial/sc16is7xx.c
+++ b/drivers/tty/serial/sc16is7xx.c
@@ -25,6 +25,7 @@
#include <linux/serial.h>
#include <linux/tty.h>
#include <linux/tty_flip.h>
+#include <linux/uaccess.h>
#define SC16IS7XX_NAME "sc16is7xx"
--
1.8.5.4
^ permalink raw reply related [flat|nested] 7+ messages in thread* Re: [PATCH 1/2] serial: sc16is7xx: depend on I2C
2014-04-25 19:53 [PATCH 1/2] serial: sc16is7xx: depend on I2C jon
2014-04-25 19:53 ` [PATCH 2/2] serial: sc16is7xx: fix implicit decl of func copy_{to,from}_user jon
@ 2014-04-25 20:23 ` Greg KH
2014-04-25 20:26 ` Alexander Shiyan
1 sibling, 1 reply; 7+ messages in thread
From: Greg KH @ 2014-04-25 20:23 UTC (permalink / raw)
To: jon; +Cc: linux-kernel, linux-serial, shc_work, Jon Ringle
On Fri, Apr 25, 2014 at 03:53:09PM -0400, jon@ringle.org wrote:
> From: Jon Ringle <jringle@gridpoint.com>
>
> Signed-off-by: Jon Ringle <jringle@gridpoint.com>
> ---
> drivers/tty/serial/Kconfig | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/tty/serial/Kconfig b/drivers/tty/serial/Kconfig
> index c3e2b32..64c5659 100644
> --- a/drivers/tty/serial/Kconfig
> +++ b/drivers/tty/serial/Kconfig
> @@ -1180,6 +1180,7 @@ config SERIAL_SCCNXP_CONSOLE
>
> config SERIAL_SC16IS7XX
> tristate "SC16IS7xx serial support"
> + depends on I2C
> select SERIAL_CORE
> select REGMAP_I2C if I2C
> help
> --
> 1.8.5.4
Better, but I still have the build warnings :(
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH 1/2] serial: sc16is7xx: depend on I2C
2014-04-25 20:23 ` [PATCH 1/2] serial: sc16is7xx: depend on I2C Greg KH
@ 2014-04-25 20:26 ` Alexander Shiyan
2014-04-25 20:35 ` Greg KH
0 siblings, 1 reply; 7+ messages in thread
From: Alexander Shiyan @ 2014-04-25 20:26 UTC (permalink / raw)
To: Greg KH; +Cc: linux-kernel, linux-serial, Jon Ringle, jon
Fri, 25 Apr 2014 13:23:15 -0700 от Greg KH <gregkh@linuxfoundation.org>:
> On Fri, Apr 25, 2014 at 03:53:09PM -0400, jon@ringle.org wrote:
> > From: Jon Ringle <jringle@gridpoint.com>
> >
> > Signed-off-by: Jon Ringle <jringle@gridpoint.com>
> > ---
> > drivers/tty/serial/Kconfig | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/drivers/tty/serial/Kconfig b/drivers/tty/serial/Kconfig
> > index c3e2b32..64c5659 100644
> > --- a/drivers/tty/serial/Kconfig
> > +++ b/drivers/tty/serial/Kconfig
> > @@ -1180,6 +1180,7 @@ config SERIAL_SCCNXP_CONSOLE
> >
> > config SERIAL_SC16IS7XX
> > tristate "SC16IS7xx serial support"
> > + depends on I2C
> > select SERIAL_CORE
> > select REGMAP_I2C if I2C
> > help
> > --
> > 1.8.5.4
>
> Better, but I still have the build warnings :(
I hope not same as before?
---
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/2] serial: sc16is7xx: depend on I2C
2014-04-25 20:26 ` Alexander Shiyan
@ 2014-04-25 20:35 ` Greg KH
2014-04-25 20:39 ` Alexander Shiyan
0 siblings, 1 reply; 7+ messages in thread
From: Greg KH @ 2014-04-25 20:35 UTC (permalink / raw)
To: Alexander Shiyan; +Cc: linux-kernel, linux-serial, Jon Ringle, jon
On Sat, Apr 26, 2014 at 12:26:16AM +0400, Alexander Shiyan wrote:
> Fri, 25 Apr 2014 13:23:15 -0700 от Greg KH <gregkh@linuxfoundation.org>:
> > On Fri, Apr 25, 2014 at 03:53:09PM -0400, jon@ringle.org wrote:
> > > From: Jon Ringle <jringle@gridpoint.com>
> > >
> > > Signed-off-by: Jon Ringle <jringle@gridpoint.com>
> > > ---
> > > drivers/tty/serial/Kconfig | 1 +
> > > 1 file changed, 1 insertion(+)
> > >
> > > diff --git a/drivers/tty/serial/Kconfig b/drivers/tty/serial/Kconfig
> > > index c3e2b32..64c5659 100644
> > > --- a/drivers/tty/serial/Kconfig
> > > +++ b/drivers/tty/serial/Kconfig
> > > @@ -1180,6 +1180,7 @@ config SERIAL_SCCNXP_CONSOLE
> > >
> > > config SERIAL_SC16IS7XX
> > > tristate "SC16IS7xx serial support"
> > > + depends on I2C
> > > select SERIAL_CORE
> > > select REGMAP_I2C if I2C
> > > help
> > > --
> > > 1.8.5.4
> >
> > Better, but I still have the build warnings :(
>
> I hope not same as before?
Exactly the same.
--
To unsubscribe from this list: send the line "unsubscribe linux-serial" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/2] serial: sc16is7xx: depend on I2C
2014-04-25 20:35 ` Greg KH
@ 2014-04-25 20:39 ` Alexander Shiyan
2014-04-25 20:42 ` Randy Dunlap
0 siblings, 1 reply; 7+ messages in thread
From: Alexander Shiyan @ 2014-04-25 20:39 UTC (permalink / raw)
To: Greg KH; +Cc: linux-kernel, linux-serial, Jon Ringle, jon
Fri, 25 Apr 2014 13:35:28 -0700 от Greg KH <gregkh@linuxfoundation.org>:
> On Sat, Apr 26, 2014 at 12:26:16AM +0400, Alexander Shiyan wrote:
> > Fri, 25 Apr 2014 13:23:15 -0700 от Greg KH <gregkh@linuxfoundation.org>:
> > > On Fri, Apr 25, 2014 at 03:53:09PM -0400, jon@ringle.org wrote:
> > > > From: Jon Ringle <jringle@gridpoint.com>
> > > >
> > > > Signed-off-by: Jon Ringle <jringle@gridpoint.com>
> > > > ---
> > > > drivers/tty/serial/Kconfig | 1 +
> > > > 1 file changed, 1 insertion(+)
> > > >
> > > > diff --git a/drivers/tty/serial/Kconfig b/drivers/tty/serial/Kconfig
> > > > index c3e2b32..64c5659 100644
> > > > --- a/drivers/tty/serial/Kconfig
> > > > +++ b/drivers/tty/serial/Kconfig
> > > > @@ -1180,6 +1180,7 @@ config SERIAL_SCCNXP_CONSOLE
> > > >
> > > > config SERIAL_SC16IS7XX
> > > > tristate "SC16IS7xx serial support"
> > > > + depends on I2C
> > > > select SERIAL_CORE
> > > > select REGMAP_I2C if I2C
> > > > help
> > > > --
> > > > 1.8.5.4
> > >
> > > Better, but I still have the build warnings :(
> >
> > I hope not same as before?
>
> Exactly the same.
Hmm...
In the source:
#ifdef CONFIG_REGMAP_I2C
Should be of course #ifdef CONFIG_I2C
But this should not provide such warnings...
---
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/2] serial: sc16is7xx: depend on I2C
2014-04-25 20:39 ` Alexander Shiyan
@ 2014-04-25 20:42 ` Randy Dunlap
0 siblings, 0 replies; 7+ messages in thread
From: Randy Dunlap @ 2014-04-25 20:42 UTC (permalink / raw)
To: Alexander Shiyan, Greg KH; +Cc: linux-kernel, linux-serial, Jon Ringle, jon
On 04/25/14 13:39, Alexander Shiyan wrote:
> Fri, 25 Apr 2014 13:35:28 -0700 от Greg KH <gregkh@linuxfoundation.org>:
>> On Sat, Apr 26, 2014 at 12:26:16AM +0400, Alexander Shiyan wrote:
>>> Fri, 25 Apr 2014 13:23:15 -0700 от Greg KH <gregkh@linuxfoundation.org>:
>>>> On Fri, Apr 25, 2014 at 03:53:09PM -0400, jon@ringle.org wrote:
>>>>> From: Jon Ringle <jringle@gridpoint.com>
>>>>>
>>>>> Signed-off-by: Jon Ringle <jringle@gridpoint.com>
>>>>> ---
>>>>> drivers/tty/serial/Kconfig | 1 +
>>>>> 1 file changed, 1 insertion(+)
>>>>>
>>>>> diff --git a/drivers/tty/serial/Kconfig b/drivers/tty/serial/Kconfig
>>>>> index c3e2b32..64c5659 100644
>>>>> --- a/drivers/tty/serial/Kconfig
>>>>> +++ b/drivers/tty/serial/Kconfig
>>>>> @@ -1180,6 +1180,7 @@ config SERIAL_SCCNXP_CONSOLE
>>>>>
>>>>> config SERIAL_SC16IS7XX
>>>>> tristate "SC16IS7xx serial support"
>>>>> + depends on I2C
>>>>> select SERIAL_CORE
>>>>> select REGMAP_I2C if I2C
>>>>> help
>>>>> --
>>>>> 1.8.5.4
>>>>
>>>> Better, but I still have the build warnings :(
>>>
>>> I hope not same as before?
>>
>> Exactly the same.
>
> Hmm...
>
> In the source:
> #ifdef CONFIG_REGMAP_I2C
> Should be of course #ifdef CONFIG_I2C
> But this should not provide such warnings...
Can I see the warnings and .config file that causes them?
--
~Randy
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2014-04-25 20:42 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-04-25 19:53 [PATCH 1/2] serial: sc16is7xx: depend on I2C jon
2014-04-25 19:53 ` [PATCH 2/2] serial: sc16is7xx: fix implicit decl of func copy_{to,from}_user jon
2014-04-25 20:23 ` [PATCH 1/2] serial: sc16is7xx: depend on I2C Greg KH
2014-04-25 20:26 ` Alexander Shiyan
2014-04-25 20:35 ` Greg KH
2014-04-25 20:39 ` Alexander Shiyan
2014-04-25 20:42 ` Randy Dunlap
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).