Modify

Opened 6 years ago

Closed 6 years ago

Last modified 4 years ago

#10893 closed defect (obsolete)

[trunk] Minor issues with Net 5501 config files.

Reported by: j.scruggs@… Owned by: developers
Priority: normal Milestone: Barrier Breaker 14.07
Component: packages Version: Trunk
Keywords: Cc:

Description

I originally was going to post a complete fix up for the 2.6.x series of kernels, but you beat me to it by updating everything to 3.2.x. However, there are still some small things that need adjusting.

First, the default CFLAGS. I have been trying in vain to compile an image for my Net 5501 that worked. Most of the time Grub would not even load. Then using the lowest version of Binutils and GCC I got something that booted really slowly. The redraw of Grub was insanely slow, and this is over a 115200 BAUD serial connection. I did some research and I believe the -march=K6-2 flag is the culprit. I think it was creating code that was just incompatible with the Geode LX CPU since things are being changed in the later GCCs. I have a patch for new CFLAGS that change the flags to something that all Geode LX CPUs should use on GGC versions 4.3 and up. These flags made a bootable system first time with no troubles.

Second, the new config-default file in the net5501 directory has the symbol:
CONFIG_NET5501
which does not exist in the 3.2.x kernels. The symbol:
CONFIG_LEDS_NET5501
does exist. However, it is dependent on the symbol:
CONFIG_GPIO_CS5535
being marked as "y". However, this symbol is marked as "m" by the options in the file target.mk.
There is a patch at this link:
http://patchwork.openwrt.org/patch/1667/
which creates the CONFIG_NET5501 symbol and which the later updated patch was based off of that was added to the OpenWRT trunk. Looking at that older patch, it seems to do more than just configure LEDs, maybe that patch does more than what was merged into mainline kernel, because they are only giving it the symbol LEDS_NET5501 in mainline???
I don't have time to study the difference, but I did find that there is an existing kmod-leds-net5501 in OpenWrt, so I added that missing kmod to the target.mk patch.

I hope this info proves useful to you.

Jon

Attachments (2)

net5501-target-mk.patch (952 bytes) - added by j.scruggs@… 6 years ago.
This patches the CFLAGS for Geode LX CPUs for 4.3 and new GGC versions. This creates more compatible code and allowed me to boot my system first time with no troubles. Also, the missing kmod is added in.
kernel-modules-leds-net5501-depend-fix.patch (562 bytes) - added by j.scruggs@… 6 years ago.
This fixes the dependency for kmod-leds-net5501.

Download all attachments as: .zip

Change History (6)

Changed 6 years ago by j.scruggs@…

This patches the CFLAGS for Geode LX CPUs for 4.3 and new GGC versions. This creates more compatible code and allowed me to boot my system first time with no troubles. Also, the missing kmod is added in.

comment:1 Changed 6 years ago by jow

  • Resolution set to fixed
  • Status changed from new to closed

Comitted in r30013, thanks!

comment:2 Changed 6 years ago by j.scruggs@…

  • Resolution fixed deleted
  • Status changed from closed to reopened

I found another minor issue with the Net 5501 series. The definition in leds.mk for the kmod-leds-net5501 depends on the old "+kmod-gpio-cs5535", which defines dependencies on the 2.6.x series of kernels, thus not allowing this config option to compile the led support for net 5501 or even select it in doing a make menuconfig. It needs the new gpio defined as "+kmod-gpio-cs5535-new". I made a patch for that.

Hopefully this is the last issue. :P Thank you for responding to this bug post really quickly! I thought I had time to attach this new patch before it was dealt with. :P

Thanks,
Jon

Changed 6 years ago by j.scruggs@…

This fixes the dependency for kmod-leds-net5501.

comment:3 Changed 6 years ago by jow

  • Resolution set to obsolete
  • Status changed from reopened to closed

This patch has been obsoleted by r30020 and r30021.

comment:4 Changed 4 years ago by jow

  • Milestone changed from Attitude Adjustment 12.09 to Barrier Breaker 14.07

Milestone Attitude Adjustment 12.09 deleted

Add Comment

Modify Ticket

Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.