linux-next: build warnings after merge of the tty.current tree

linux-next: build warnings after merge of the tty.current tree

[Stephen Rothwell]

[-- Attachment #1: Type: text/plain, Size: 819 bytes --]

Hi all,

After merging the tty.current tree, today's linux-next build (x86_64
allmodconfig) produced these warnings:

In file included from <command-line>:
./usr/include/linux/cyclades.h:6:2: warning: #warning "Support for features provided by this header has been removed" [-Wcpp]
    6 | #warning "Support for features provided by this header has been removed"
      |  ^~~~~~~
./usr/include/linux/cyclades.h:7:2: warning: #warning "Please consider updating your code" [-Wcpp]
    7 | #warning "Please consider updating your code"
      |  ^~~~~~~


Introduced by commit

  f23653fe6447 ("tty: Partially revert the removal of the Cyclades public API")

This is a bit annoying :-(

Can we exclude include/uapi/linux/cyclades.h from the headers checking,
please?

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: linux-next: build warnings after merge of the tty.current tree

[Greg KH]

On Thu, Jan 27, 2022 at 09:18:00AM +1100, Stephen Rothwell wrote:
> Hi all,
> 
> After merging the tty.current tree, today's linux-next build (x86_64
> allmodconfig) produced these warnings:
> 
> In file included from <command-line>:
> ./usr/include/linux/cyclades.h:6:2: warning: #warning "Support for features provided by this header has been removed" [-Wcpp]
>     6 | #warning "Support for features provided by this header has been removed"
>       |  ^~~~~~~
> ./usr/include/linux/cyclades.h:7:2: warning: #warning "Please consider updating your code" [-Wcpp]
>     7 | #warning "Please consider updating your code"
>       |  ^~~~~~~
> 
> 
> Introduced by commit
> 
>   f23653fe6447 ("tty: Partially revert the removal of the Cyclades public API")
> 
> This is a bit annoying :-(

Sorry, odd it doesn't show up in my builds, nor in 0-day.

> Can we exclude include/uapi/linux/cyclades.h from the headers checking,
> please?

Let me fix this up, thanks.

greg k-h

Re: linux-next: build warnings after merge of the tty.current tree

[Greg KH]

On Thu, Jan 27, 2022 at 08:22:56AM +0100, Greg KH wrote:
> On Thu, Jan 27, 2022 at 09:18:00AM +1100, Stephen Rothwell wrote:
> > Hi all,
> > 
> > After merging the tty.current tree, today's linux-next build (x86_64
> > allmodconfig) produced these warnings:
> > 
> > In file included from <command-line>:
> > ./usr/include/linux/cyclades.h:6:2: warning: #warning "Support for features provided by this header has been removed" [-Wcpp]
> >     6 | #warning "Support for features provided by this header has been removed"
> >       |  ^~~~~~~
> > ./usr/include/linux/cyclades.h:7:2: warning: #warning "Please consider updating your code" [-Wcpp]
> >     7 | #warning "Please consider updating your code"
> >       |  ^~~~~~~
> > 
> > 
> > Introduced by commit
> > 
> >   f23653fe6447 ("tty: Partially revert the removal of the Cyclades public API")
> > 
> > This is a bit annoying :-(
> 
> Sorry, odd it doesn't show up in my builds, nor in 0-day.

Ah, 0-day reported it on the original patch, but not once it was in my
tree.  odd.

Re: linux-next: build warnings after merge of the tty.current tree

[Greg KH]

On Thu, Jan 27, 2022 at 09:18:00AM +1100, Stephen Rothwell wrote:
> Hi all,
> 
> After merging the tty.current tree, today's linux-next build (x86_64
> allmodconfig) produced these warnings:
> 
> In file included from <command-line>:
> ./usr/include/linux/cyclades.h:6:2: warning: #warning "Support for features provided by this header has been removed" [-Wcpp]
>     6 | #warning "Support for features provided by this header has been removed"
>       |  ^~~~~~~
> ./usr/include/linux/cyclades.h:7:2: warning: #warning "Please consider updating your code" [-Wcpp]
>     7 | #warning "Please consider updating your code"
>       |  ^~~~~~~
> 
> 
> Introduced by commit
> 
>   f23653fe6447 ("tty: Partially revert the removal of the Cyclades public API")
> 
> This is a bit annoying :-(
> 
> Can we exclude include/uapi/linux/cyclades.h from the headers checking,
> please?

Fix sent:
	https://lore.kernel.org/r/20220127073304.42399-1-gregkh@linuxfoundation.org