All of lore.kernel.org
 help / color / mirror / Atom feed
* parse-integer support
@ 2015-09-16  3:37 Pierre-Louis Bossart
  2015-09-16  6:04 ` Johannes Berg
  0 siblings, 1 reply; 6+ messages in thread
From: Pierre-Louis Bossart @ 2015-09-16  3:37 UTC (permalink / raw)
  To: backports

Hi,
I've progressed a bit in my endeavor to backport the sound drivers with 
only a few compilation errors but I've reached a blocking point. The 
audio drivers were converted to use parse_integer this summer but I 
can't figure out how to back port this.

If I add include/linux/parse-integer to copy-list then there are tons of 
redeclations.
If I modify backport/backport-include/linux/kernel.h to add a 
conditional #include <linux/parse-integer.h> then I can't figure out how 
to add a subset of this header file from the backport-include directory.

It's only a matter of time before others are impacted so maybe smarter 
people than me have ideas on how to go about this.
Thanks,
-Pierre
--
To unsubscribe from this list: send the line "unsubscribe backports" in

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: parse-integer support
  2015-09-16  3:37 parse-integer support Pierre-Louis Bossart
@ 2015-09-16  6:04 ` Johannes Berg
  2015-09-16  8:12   ` Arend van Spriel
  0 siblings, 1 reply; 6+ messages in thread
From: Johannes Berg @ 2015-09-16  6:04 UTC (permalink / raw)
  To: Pierre-Louis Bossart, backports

Hi,

> I've progressed a bit in my endeavor to backport the sound drivers with 
> only a few compilation errors but I've reached a blocking point. The 
> audio drivers were converted to use parse_integer this summer but I 
> can't figure out how to back port this.

Err, I can't even figure out what "parse_integer" is? Such a function
doesn't exist in my kernel

> If I add include/linux/parse-integer to copy-list then there are tons of 
> redeclations.

Neither does a file called include/linux/parse-integer(.h)

If you can point out what you really meant here then I'm sure this is
easy to fix, but without looking at the code (which clearly I can't)
there's not much I can do.


johannes
--
To unsubscribe from this list: send the line "unsubscribe backports" in

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: parse-integer support
  2015-09-16  6:04 ` Johannes Berg
@ 2015-09-16  8:12   ` Arend van Spriel
  2015-09-16  8:17     ` Johannes Berg
  0 siblings, 1 reply; 6+ messages in thread
From: Arend van Spriel @ 2015-09-16  8:12 UTC (permalink / raw)
  To: Johannes Berg, Pierre-Louis Bossart, backports

On 09/16/2015 08:04 AM, Johannes Berg wrote:
> Hi,
>
>> I've progressed a bit in my endeavor to backport the sound drivers with
>> only a few compilation errors but I've reached a blocking point. The
>> audio drivers were converted to use parse_integer this summer but I
>> can't figure out how to back port this.
>
> Err, I can't even figure out what "parse_integer" is? Such a function
> doesn't exist in my kernel
>
>> If I add include/linux/parse-integer to copy-list then there are tons of
>> redeclations.
>
> Neither does a file called include/linux/parse-integer(.h)
>
> If you can point out what you really meant here then I'm sure this is
> easy to fix, but without looking at the code (which clearly I can't)
> there's not much I can do.

Probably it is not in because it was tagged as crap and not merged [1].

Regards,
Arend

[1] 
http://mid.gmane.org/CA+55aFxN7ebRPpayOQSznjJobz1UVQRQ5rMAKjxGvc3vW6fNYg@mail.gmail.com

--
To unsubscribe from this list: send the line "unsubscribe backports" in

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: parse-integer support
  2015-09-16  8:12   ` Arend van Spriel
@ 2015-09-16  8:17     ` Johannes Berg
  2015-09-16 14:00       ` Pierre-Louis Bossart
  0 siblings, 1 reply; 6+ messages in thread
From: Johannes Berg @ 2015-09-16  8:17 UTC (permalink / raw)
  To: Arend van Spriel, Pierre-Louis Bossart, backports

On Wed, 2015-09-16 at 10:12 +0200, Arend van Spriel wrote:
> 
> Probably it is not in because it was tagged as crap and not merged 


Oh, that thing. I actually saw that but didn't remember it was called
'parse-integer' :)

johannes
> 
--
To unsubscribe from this list: send the line "unsubscribe backports" in

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: parse-integer support
  2015-09-16  8:17     ` Johannes Berg
@ 2015-09-16 14:00       ` Pierre-Louis Bossart
  2015-09-16 14:09         ` Johannes Berg
  0 siblings, 1 reply; 6+ messages in thread
From: Pierre-Louis Bossart @ 2015-09-16 14:00 UTC (permalink / raw)
  To: Johannes Berg, Arend van Spriel, backports

On 9/16/15 3:17 AM, Johannes Berg wrote:
> On Wed, 2015-09-16 at 10:12 +0200, Arend van Spriel wrote:
>>
>> Probably it is not in because it was tagged as crap and not merged
>
>
> Oh, that thing. I actually saw that but didn't remember it was called
> 'parse-integer' :)

It is part of next-20150818, include/linux/parse-integer.h
It's also used by quite a few drivers now...

--
To unsubscribe from this list: send the line "unsubscribe backports" in

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: parse-integer support
  2015-09-16 14:00       ` Pierre-Louis Bossart
@ 2015-09-16 14:09         ` Johannes Berg
  0 siblings, 0 replies; 6+ messages in thread
From: Johannes Berg @ 2015-09-16 14:09 UTC (permalink / raw)
  To: Pierre-Louis Bossart, Arend van Spriel, backports

On Wed, 2015-09-16 at 09:00 -0500, Pierre-Louis Bossart wrote:
> On 9/16/15 3:17 AM, Johannes Berg wrote:
> > On Wed, 2015-09-16 at 10:12 +0200, Arend van Spriel wrote:
> > > 
> > > Probably it is not in because it was tagged as crap and not 
> > > merged
> > 
> > 
> > Oh, that thing. I actually saw that but didn't remember it was 
> > called
> > 'parse-integer' :)
> 
> It is part of next-20150818, include/linux/parse-integer.h
> It's also used by quite a few drivers now...

I guess you should wait until it's dropped from there ... or use a
snapshot from before when it was in. Perhaps it's already dropped,
since you're working on a month-old snapshot?

johannes
--
To unsubscribe from this list: send the line "unsubscribe backports" in

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2015-09-16 14:09 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-09-16  3:37 parse-integer support Pierre-Louis Bossart
2015-09-16  6:04 ` Johannes Berg
2015-09-16  8:12   ` Arend van Spriel
2015-09-16  8:17     ` Johannes Berg
2015-09-16 14:00       ` Pierre-Louis Bossart
2015-09-16 14:09         ` Johannes Berg

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.