**** BEGIN LOGGING AT Mon Apr 04 02:59:58 2016 Apr 04 12:36:03 hi, bradfa, Do you have time. I added some implementation details and information about McSPI slave patch. Apr 04 12:36:37 bradfa: link is here: https://docs.google.com/document/d/1QGxnjvGWkyppsN7H8Hrc7XFoltBzVDaxKx9nXnQ1fBY/edit?usp=sharing Apr 04 12:39:06 bradfa: I corrected Development timeline. I would like you to comment on some changes in the drafts. Apr 04 15:14:40 pmezydlo: ok, I'll take a look, thanks! :) Apr 04 15:21:21 bradfa: Now I'm reading about the DMA, I think DMA is more difficult part in project. Apr 04 15:25:23 pmezydlo: doing the DMA probably should be a reach goal, just doing PIO is probably fine, so long as it works. DMA will make some kinds of transactions less CPU bound but it could be a bunch of work, yes Apr 04 15:32:09 bradfa: for low speed clk should be ok Apr 04 17:51:02 bradfa: I corrected my application and i resolved all of your comments. Should I add or change something else? Apr 04 17:55:37 bradfa:Unfortunately, I can't find more SPI slave patch. Apr 04 18:02:20 alexhiam, You there? Apr 04 18:09:45 Any mentor who could help me a bit with makefile? Apr 04 18:14:55 not if you don't ask your question... Apr 04 18:15:39 Hey nerdboy ! Apr 04 18:16:19 I have written a driver along with an userland API that performs ioctls. Apr 04 18:16:51 But I am not able to understand the makefile part of it. Could you please take a look? Apr 04 18:16:59 https://github.com/chanakya-vc/BBB-Gsoctry/tree/master/ioctlsdriver Apr 04 18:22:09 chanakya_vc: module: Apr 04 18:22:09 $(MAKE) -C $(KERNEL_SOURCE) SUBDIRS=$(PWD) modules Apr 04 18:22:45 this is link for linux headers Apr 04 18:25:13 obj-m += query_ioctldriver.o It means that the file compiles Apr 04 18:25:54 it is my Makefile for LKM: https://github.com/pmezydlo/BBB_driver/blob/master/Makefile Apr 04 18:28:04 it is good to make clean Apr 04 18:29:01 Okay so issue of whitespace? Apr 04 18:29:37 Maybe you could do a pull request?It would help me better understand what is wrong. Apr 04 18:34:14 chanakya_vc: I'm writing for you Makefile Apr 04 18:35:19 pmezydlo, Thanks a lot.Would help me a lot in understanding it. Apr 04 18:39:58 whitespace is mainly spaces or tabs for definitions, but tab for each set of target commands Apr 04 18:40:32 for kernel makefile you shouldn't need to define any new targets Apr 04 18:40:42 keep it simple as above Apr 04 18:40:52 nerdboy, Okay. Apr 04 18:41:17 nerdboy, I need to compile the driver and the userland application together. Apr 04 18:42:04 So what pmzydlo suggested above would only compile the driver? Apr 04 18:44:09 Also there is a third file called query_ioctls.h,in which I define the macros like IOR,IOW. And that is included in both the driver and the userland application. Apr 04 18:45:47 I am assuming that using make,it would automatically link that file as well? Apr 04 18:49:28 probably need an include directive for that Apr 04 18:49:36 as in -I./ Apr 04 18:49:55 or wherever your includes are Apr 04 18:50:59 chanayka_vc: Makefile for your's code: obj-m+=query_ioctldriver.o Apr 04 18:50:59 all: Apr 04 18:50:59 make -C /lib/modules/$(shell uname -r)/build/ M=$(PWD) modules Apr 04 18:50:59 $(CC) App_to_perform_ioctls.c -o App_to_perform_ioctls Apr 04 18:51:00 clean: Apr 04 18:51:00 make -C /lib/modules/$(shell uname -r)/build/ M=$(PWD) clean Apr 04 18:51:04 sorry Apr 04 18:51:17 is here: http://pastebin.com/7UWENyN8 Apr 04 18:51:36 i don't copy link Apr 04 18:52:12 you need to correct code. Apr 04 18:55:23 pmezydlo, Thanks a lot!!. Apr 04 18:55:47 pmezydlo, Yes I would have to correct it. Apr 04 18:56:46 nerdboy, I will look into it.Meanwhile could you please tell all the mentors to look through the code? Apr 04 18:58:00 And tell me if anything is wrong?Like except the makefile and some careless mistakes in the syntax. Apr 04 18:59:21 makefile must be named "Makefile" Apr 04 19:01:11 pmezydlo, I know :) Apr 04 19:01:59 Thanks for the help :) Apr 04 19:14:17 pmezydlo, On running the makefile,I get the following error: *** commands commence before first target. Stop. Apr 04 19:22:45 which line? Apr 04 19:24:08 https://github.com/sarnold/BeagleSat/blob/master/LKM/inoutlkm/Makefile <= keep it simple Apr 04 19:24:48 your app source should be separate dir/Makefiles Apr 04 19:25:17 make a top-level makefile that calls the other ones Apr 04 19:25:58 commands should be tabbed and part of a make target Apr 04 19:51:34 pmezydlo, $(CC) App_to_perform_ioctls.c -o App_to_perform_ioctl Apr 04 19:52:02 nerdboy, will go through it.Thanks! Apr 04 19:54:02 yes, you must add directory Apr 04 21:29:32 who elinux has bb wiki edit perms? http://elinux.org/Beagleboard:BeagleBoneBlack_Debian Apr 04 21:29:38 *has even Apr 04 21:29:52 some of the new image links are broken Apr 04 21:31:07 ... bone-debian-8.4-machinekit-armhf-2016-04-03-4gb.img.xz.img.xz <= remove one ".img.xz" please **** ENDING LOGGING AT Tue Apr 05 02:59:58 2016