From mboxrd@z Thu Jan 1 00:00:00 1970 From: Takashi Iwai Subject: Re: Remaining compatibility patches for NetBSD Date: Thu, 03 Mar 2016 14:54:09 +0100 Message-ID: References: <20160303095852.GH8492@danbala.tuwien.ac.at> <20160303101005.GI8492@danbala.tuwien.ac.at> <20160303111840.GJ8492@danbala.tuwien.ac.at> <20160303113921.GK8492@danbala.tuwien.ac.at> <20160303120242.GL8492@danbala.tuwien.ac.at> <20160303134641.GB26041@danbala.tuwien.ac.at> Mime-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mx2.suse.de (mx2.suse.de [195.135.220.15]) by alsa0.perex.cz (Postfix) with ESMTP id EB3B2260535 for ; Thu, 3 Mar 2016 14:54:10 +0100 (CET) In-Reply-To: <20160303134641.GB26041@danbala.tuwien.ac.at> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: alsa-devel-bounces@alsa-project.org To: Thomas Klausner Cc: alsa-devel@alsa-project.org List-Id: alsa-devel@alsa-project.org On Thu, 03 Mar 2016 14:46:41 +0100, Thomas Klausner wrote: > > On Thu, Mar 03, 2016 at 02:35:06PM +0100, Takashi Iwai wrote: > > On Thu, 03 Mar 2016 13:02:42 +0100, > > Thomas Klausner wrote: > > > > > > Here are the remaining patches to make the build succeed on NetBSD. > > > More patches are needed for FreeBSD and Solaris, but let's take it one > > > at a time :-) > > > > > > I think the first two (0002 and 0003) should be uncontroversial. > > > > > > For patch 0004 it would perhaps be good to have versionsort/alphasort > > > in a central place instead of two. Do you have a suggestion where? > > > > > > I'm not sure what to do about 0005 since you said you don't want to > > > modify include/sound/asound.h. > > > > Right. If we need to modify, it must be applied to Linux kernel tree > > at first, then copy back to alsa-lib. > > I haven't done this before. > > I see on https://www.kernel.org/doc/linux/MAINTAINERS > > SOUND > M: Jaroslav Kysela > M: Takashi Iwai > L: alsa-devel@alsa-project.org (moderated for non-subscribers) > W: http://www.alsa-project.org/ > T: git git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git > T: git git://git.alsa-project.org/alsa-kernel.git > Q: http://patchwork.kernel.org/project/alsa-devel/list/ > S: Maintained > F: Documentation/sound/ > F: include/sound/ > F: include/uapi/sound/ > F: sound/ > > Against which of the two git trees should I prepare the patch? Use my sound git tree. > > > endian.h seems to be unneeded on src/topology/tplg_local.h; if it's > > > needed, I can move it inside the #ifdef. > > > > src/topology/parser.c contains the code depending on the endian. > > The include can be moved there, too. > > local.h already handles endianness: > > #ifdef HAVE_ENDIAN_H > #include > #elif defined(HAVE_SYS_ENDIAN_H) > #include > #ifndef __BYTE_ORDER > #define __BYTE_ORDER BYTE_ORDER > #endif > #ifndef __LITTLE_ENDIAN > #define __LITTLE_ENDIAN LITTLE_ENDIAN > #endif > #ifndef __BIG_ENDIAN > #define __BIG_ENDIAN BIG_ENDIAN > #endif > #else > #error Header defining endianness not defined > #endif > > So I think the endian.h include is not necessary and can just be removed. Yes, looks so. Takashi