From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 32389 invoked by uid 1002); 11 May 2003 19:02:17 -0000 Mailing-List: contact gentoo-dev-help@gentoo.org; run by ezmlm Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-Id: Gentoo Linux mail X-BeenThere: gentoo-dev@gentoo.org Received: (qmail 940 invoked from network); 11 May 2003 19:02:17 -0000 Date: Sun, 11 May 2003 15:03:35 -0400 From: Peter Johanson To: Eric Noack Cc: gentoo-dev@gentoo.org, chadh@gentoo.org Message-ID: <20030511190311.GA3255@gonzo.peterjohanson.com> Mail-Followup-To: Eric Noack , gentoo-dev@gentoo.org, chadh@gentoo.org References: <20030511203623.72716711.eric.noack@gmx.de> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20030511203623.72716711.eric.noack@gmx.de> User-Agent: Mutt/1.5.4i Subject: Re: [gentoo-dev] which contribution prefered - to gentoo or to project itself? X-Archives-Salt: ca9e34b0-291b-42c8-9890-cb8c94c82c42 X-Archives-Hash: 4e23347649704b8adf6d8c847c79b1c4 On Sun, May 11, 2003 at 08:36:23PM +0200, Eric Noack wrote: > however unfortunately i needed support for an isdn pcmcia card, and > pcmcia-cs has no isdn support or drivers at all. ): > > on the other side, adding the support to pcmcia_cs is as easy as copying a single driver file > (something_cs.c) from linux/drivers/isdn/hisax/ into the pcmcia-cs source tree > and add it to the make file, if only the kernel hisax driver has been compiled with support > for the pcmcia chip > > (which is possible without enabling kernel pcmcia, only the carddriver itself > cant be compiled without) > > so it would be easy to write a patch to pcmcia_cs that would check whether the kernel has > isdn and hisax support, and which pcmcia-chip-protocols have been compiled into the hisax driver, > and copy & compile the needed *_cs.c from the hisax dir The check would definitely be good, but i'm hesitant to blindly copy files over from /usr/src/linux (which may contain any bastardization of the linux kernel with unknown patches, etc) and attempting to compile it. i think it'd be better to push a packaged tarball to gentoo mirrors that hold the necessary driver pieces. That way, we can test that things actually work. If we copy over from /usr/src/linux, we have no way of verifying things will work. > > however this "patch" could be completely provided by a modified ebuild, maybe with support for an > "isdn" USEflag or such, too, eventually more easily than modifying the pcmcia_cs configure script i definitely don't think this warrants a new USE flag, unless there's quite a few more ebuilds that would benefit from it. As long as we can verify that no compilation errors will occur, i see no problems with doing this by default (given the above tests pass of course). > > so before writing this modification as either a patch to pcmcia_cs or the ebuild or both (ebuild that adds the patch) If we have a seperate tarball, We can do the testing in the ebuild and apply a patch to the pcmcia directory including the configure edit and the new *_cs.c files. also, i haven't scoured the pcmcia-cs m/l for this particular topic, but does david hinds intend to add this support eventually? chad, what do you think about this approach? any other ideas or implementations you see as being superior? -pete -- Peter Johanson -- gentoo-dev@gentoo.org mailing list