Re: [PATCH 7/8 RESEND] staging: dgap: introduce dgap_cleanup_nodes()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 07/15/2014 11:30 AM, Greg KH wrote:
On Tue, Jul 15, 2014 at 06:14:25PM +0900, Daeseok Youn wrote:
When a configration file is parsed with dgap_parsefile(),
makes nodes for saving configrations for board.

configuration files should not be parsed in the kernel at all.  That
logic should be removed as it should not be needed.

Mark, can you verify that this is not needed with your hardware anymore?


As far as I can see it is still needed. If no config file, no devices are created and configured. I had planned to remove that code such that all the different cards supported are still supported. Even though I only have 2 different types of cards. I Just haven't got to it yet. I realize "configuration files should not be parsed in the kernel" and I'm certainly not going to argue that. But the way it is, it allows user land to define board order and even device node names (additional links to the actual device). No matter where boards are installed and found by the kernel, you have persistent naming. I guess all that can be done with udev (well, sort of anyway).

There are also options in the config file that should be passed in as module parameters
and I planned to do that part before messing with the config file code.

We also have this "DIGI_EXPANDERS_SUPPORTED" thingy. I have none of these cards. These cards interrogate the expander to find out how many ports there are. Then configure accordingly. Should we just remove all that code and NOT support expanders since we don't currently have the hardware to test?

I will update from your current git tree and test this week. I've been swamped lately and unable to do much. I thought I'd be available to do some more work but things come up as you know.

Regards
Mark

_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel




[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux