Kernel/Upstreaming

From Openmoko

(Difference between revisions)
Jump to: navigation, search
 
(43 intermediate revisions by 5 users not shown)
Line 1: Line 1:
 
Random notes on SHR kernel patches (v2.6.39.4..shr-2.6.39-nodrm).
 
Random notes on SHR kernel patches (v2.6.39.4..shr-2.6.39-nodrm).
  
= [https://gitorious.org/shr/linux/commit/f58597e68f36da7d4f4dec08520d3e12258addf2 <nowiki>ARM: s3c24xx: Set ARCH_NR_GPIOS according to the selected SoC types.</nowiki>] =
+
= [https://gitorious.org/shr/linux/commit/f58597e68f36da7d4f4dec08520d3e12258addf2?format=patch <nowiki>ARM: s3c24xx: Set ARCH_NR_GPIOS according to the selected SoC types.</nowiki>] =
 
  lindi: bugfix/feature. sounds sensible. s3c specific
 
  lindi: bugfix/feature. sounds sensible. s3c specific
= [https://gitorious.org/shr/linux/commit/56ee57980e903f6fbcac9fd35d0410b7bb363ae5 <nowiki>s3c24xx: Fix level irqs on external interrupts.</nowiki>] =
+
Seems to be at least partially upstream since v3.2-rc1: http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=8a8ab2e64e09b56dc1324fd2f7da12346166cad1
 +
 
 +
= [https://gitorious.org/shr/linux/commit/56ee57980e903f6fbcac9fd35d0410b7bb363ae5?format=patch <nowiki>s3c24xx: Fix level irqs on external interrupts.</nowiki>] =
 
  lindi: bugfix. s3c specific
 
  lindi: bugfix. s3c specific
= [https://gitorious.org/shr/linux/commit/f3f81740815d9207356cb4fcddc3196c7cbadb72 <nowiki>mtd: s3c2410_nand: Add config option to disable hw ecc at runtime</nowiki>] =
+
= [https://gitorious.org/shr/linux/commit/f3f81740815d9207356cb4fcddc3196c7cbadb72?format=patch <nowiki>mtd: s3c2410_nand: Add config option to disable hw ecc at runtime</nowiki>] =
  lindi: feature. simple. I personally can live without nand.  
+
  lindi: feature. simple. I personally can live without nand.
= [https://gitorious.org/shr/linux/commit/25db4ec44e13d7934100eab7860a9e38d5644007 <nowiki>Add s3c24xx_serial_console_set_silence</nowiki>] =
+
lindi: Lars tried to upstream but it was rejected:
  lindi: What is this for? To allow using serial console and GSM on the same serial port? Who needs this anymore?
+
> Lars-Peter Clausen <lars@metafoo.de> writes:
= [https://gitorious.org/shr/linux/commit/d38596283fd7b1c92138ac44352f08bb24636e71 <nowiki>Add c fiq handler.</nowiki>] =
+
>> Not quite. I had for example a 5 liner rejected by a maintainer saying he can't
 +
>> accept any code which adds new platform code to ARM.
 +
lindi: however that appears to have been a misunderstanding and this can be resent
 +
 
 +
= [https://gitorious.org/shr/linux/commit/25db4ec44e13d7934100eab7860a9e38d5644007?format=patch <nowiki>Add s3c24xx_serial_console_set_silence</nowiki>] =
 +
  lindi: openmoko specific. only for gta01: to have serial console and GSM on the same port.
 +
= [https://gitorious.org/shr/linux/commit/d38596283fd7b1c92138ac44352f08bb24636e71?format=patch <nowiki>Add c fiq handler.</nowiki>] =
 
  lindi: feature. tricky! Used by battery hdq queries (temperature, current, capacity etc.) and vibrator. Should just rewrite them in assembler?
 
  lindi: feature. tricky! Used by battery hdq queries (temperature, current, capacity etc.) and vibrator. Should just rewrite them in assembler?
= [https://gitorious.org/shr/linux/commit/d3048278388b0908e179882587a8b6fcd95f2f50 <nowiki>Add glamo driver.</nowiki>] =
+
lindi: http://osdevnotes.blogspot.com/2011/10/fiq-debugger-redux.html mentions that google has something similar for their FIQ debugger.
 +
= [https://gitorious.org/shr/linux/commit/d3048278388b0908e179882587a8b6fcd95f2f50?format=patch <nowiki>Add glamo driver.</nowiki>] =
 
  lindi: feature. tricky!! This does not include the DRM/3D bits but it is enough for me.
 
  lindi: feature. tricky!! This does not include the DRM/3D bits but it is enough for me.
= [https://gitorious.org/shr/linux/commit/df2b549c2cd45742d7ea63b159b5a23cccd54c9e <nowiki>ARM: gta02: Add LED support</nowiki>] =
+
= [https://gitorious.org/shr/linux/commit/df2b549c2cd45742d7ea63b159b5a23cccd54c9e?format=patch <nowiki>ARM: gta02: Add LED support</nowiki>] =
 
  lindi: feature. openmoko specific. simple.
 
  lindi: feature. openmoko specific. simple.
= [https://gitorious.org/shr/linux/commit/ec9382726b6c58494210d0c361b33a0617a39316 <nowiki>gta02: Disable hardware ECC unless we get instructed to enable it</nowiki>] =
+
lindi: could this be the next candidate for upstreaming? Is there anything controversial here? has somebody already tried to upstream this?
 +
lindi: there's a lot of extra whitespace changes, probably because the patch has been rebased many times?
 +
 
 +
= [https://gitorious.org/shr/linux/commit/ec9382726b6c58494210d0c361b33a0617a39316?format=patch <nowiki>gta02: Disable hardware ECC unless we get instructed to enable it</nowiki>] =
 
  lindi: openmoko specific. nand. simple.
 
  lindi: openmoko specific. nand. simple.
= [https://gitorious.org/shr/linux/commit/caa1bc2b4331544a7b3835635e98b7f232dd5e59 <nowiki>gta02: Remove usage of pcf50633 gpio api</nowiki>] =
+
= [https://gitorious.org/shr/linux/commit/caa1bc2b4331544a7b3835635e98b7f232dd5e59?format=patch <nowiki>gta02: Remove usage of pcf50633 gpio api</nowiki>] =
 
  lindi: openmoko specific. cleanup.
 
  lindi: openmoko specific. cleanup.
= [https://gitorious.org/shr/linux/commit/4ed282d67df719b6c86ff8e5a180f826cbb7fff7 <nowiki>s3c2440: gta02: Configure pcf50633 gpios.</nowiki>] =
+
= [https://gitorious.org/shr/linux/commit/4ed282d67df719b6c86ff8e5a180f826cbb7fff7?format=patch <nowiki>s3c2440: gta02: Configure pcf50633 gpios.</nowiki>] =
 
  lindi: openmoko specific.
 
  lindi: openmoko specific.
= [https://gitorious.org/shr/linux/commit/146f7ec553fae1e7655cde17802d23b0c6e14a27 <nowiki>ARM: s3c2440: GTA02: Select SPARSE_IRQ</nowiki>] =
+
= [https://gitorious.org/shr/linux/commit/146f7ec553fae1e7655cde17802d23b0c6e14a27?format=patch <nowiki>ARM: s3c2440: GTA02: Select SPARSE_IRQ</nowiki>] =
 
  lindi: openmoko specific. simple.
 
  lindi: openmoko specific. simple.
= [https://gitorious.org/shr/linux/commit/9bd9379698b85d2baf7b98b5bf59f65a7e709b55 <nowiki>gta02: Add notify handler to probe device children</nowiki>] =
+
= [https://gitorious.org/shr/linux/commit/9bd9379698b85d2baf7b98b5bf59f65a7e709b55?format=patch <nowiki>gta02: Add notify handler to probe device children</nowiki>] =
 
  lindi: openmoko specific. feature.
 
  lindi: openmoko specific. feature.
= [https://gitorious.org/shr/linux/commit/0dace283ee622f2cdfa78dac1deee64038841309 <nowiki>gta02: Add bt power management device</nowiki>] =
+
= [https://gitorious.org/shr/linux/commit/0dace283ee622f2cdfa78dac1deee64038841309?format=patch <nowiki>gta02: Add bt power management device</nowiki>] =
 +
lindi: openmoko specific. simple. is this the right API to use? could we abuse rfkill? what do the androids/n900/N9 use?
 +
= [https://gitorious.org/shr/linux/commit/5a7eaa51c3e7298416d1faf17ba96678498b6a7e?format=patch <nowiki>gta02: Add gps power management device</nowiki>] =
 
  lindi: openmoko specific. simple.
 
  lindi: openmoko specific. simple.
= [https://gitorious.org/shr/linux/commit/5a7eaa51c3e7298416d1faf17ba96678498b6a7e <nowiki>gta02: Add gps power management device</nowiki>] =
+
= [https://gitorious.org/shr/linux/commit/c2742ff04d32db3d287c0bc4f7ab090fc957ee49?format=patch <nowiki>gta02: Add gsm power management device</nowiki>] =
 
  lindi: openmoko specific. simple.
 
  lindi: openmoko specific. simple.
= [https://gitorious.org/shr/linux/commit/c2742ff04d32db3d287c0bc4f7ab090fc957ee49 <nowiki>gta02: Add gsm power management device</nowiki>] =
+
= [https://gitorious.org/shr/linux/commit/ef27e9c481964789fa7ed703b9d7d53a00ad6036?format=patch <nowiki>gta02: Add wlan power management device</nowiki>] =
 
  lindi: openmoko specific. simple.
 
  lindi: openmoko specific. simple.
= [https://gitorious.org/shr/linux/commit/ef27e9c481964789fa7ed703b9d7d53a00ad6036 <nowiki>gta02: Add wlan power management device</nowiki>] =
+
= [https://gitorious.org/shr/linux/commit/2dc04b2d325cea146f4605cdbc885126ec8c0859?format=patch <nowiki>gta02: Add glamo device</nowiki>] =
lindi: openmoko specific. simple.
+
= [https://gitorious.org/shr/linux/commit/2dc04b2d325cea146f4605cdbc885126ec8c0859 <nowiki>gta02: Add glamo device</nowiki>] =
+
 
  lindi: openmoko specific. relatively simple, real trouble is in the glamo driver itself
 
  lindi: openmoko specific. relatively simple, real trouble is in the glamo driver itself
= [https://gitorious.org/shr/linux/commit/cf18c7ef4b91239ff51929d60173afa26627ba51 <nowiki>Add jbt device.</nowiki>] =
+
= [https://gitorious.org/shr/linux/commit/cf18c7ef4b91239ff51929d60173afa26627ba51?format=patch <nowiki>Add jbt device.</nowiki>] =
  lindi: openmoko specific. backlight?
+
  lindi: openmoko specific. LCD control?
= [https://gitorious.org/shr/linux/commit/3db6ef4d2bf156826a3c9ee7bd6be334d6696662 <nowiki>gta02: Add fiq handler</nowiki>] =
+
= [https://gitorious.org/shr/linux/commit/3db6ef4d2bf156826a3c9ee7bd6be334d6696662?format=patch <nowiki>gta02: Add fiq handler</nowiki>] =
 
  lindi: openmoko specific. depends on the tricky fiq support above.
 
  lindi: openmoko specific. depends on the tricky fiq support above.
= [https://gitorious.org/shr/linux/commit/d639c61f8338dcc221f88f350e2804e12e2dfaaf <nowiki>gta02: Add battery driver</nowiki>] =
+
= [https://gitorious.org/shr/linux/commit/d639c61f8338dcc221f88f350e2804e12e2dfaaf?format=patch <nowiki>gta02: Add battery driver</nowiki>] =
 
  lindi: openmoko specific. depends on the tricky fiq support above.
 
  lindi: openmoko specific. depends on the tricky fiq support above.
= [https://gitorious.org/shr/linux/commit/6afb7d88356ab738bb469850854df6c4c31f3f3f <nowiki>gta02: add support for platform_battery</nowiki>] =
+
lindi: mach-gta02.c:595: warning: ‘gta02_get_charger_online_status’ defined but not used
 +
 
 +
= [https://gitorious.org/shr/linux/commit/6afb7d88356ab738bb469850854df6c4c31f3f3f?format=patch <nowiki>gta02: add support for platform_battery</nowiki>] =
 
  lindi: openmoko specific. simple.
 
  lindi: openmoko specific. simple.
= [https://gitorious.org/shr/linux/commit/2111dc5154f0134b173ef122609d7b0e6fb0f6a0 <nowiki>Add atheros sdio ids.</nowiki>] =
+
= [https://gitorious.org/shr/linux/commit/2111dc5154f0134b173ef122609d7b0e6fb0f6a0?format=patch <nowiki>Add atheros sdio ids.</nowiki>] =
  lindi: trivial.
+
  lindi: trivial. why does AR6003 define this in drivers/net/wireless/ath/ath6kl/hif.h and not this common file?
= [https://gitorious.org/shr/linux/commit/81c7116b8418aa499ebea9755fc586e8bfc6d91e <nowiki>Add ar6000 wireless driver.</nowiki>] =
+
= [https://gitorious.org/shr/linux/commit/81c7116b8418aa499ebea9755fc586e8bfc6d91e?format=patch <nowiki>Add ar6000 wireless driver.</nowiki>] =
 
  lindi: tricky! probably can never be mainlined. Maybe we can build it as a separate module package in debian?
 
  lindi: tricky! probably can never be mainlined. Maybe we can build it as a separate module package in debian?
= [https://gitorious.org/shr/linux/commit/4e11662e7b95b1a8cdee1f5af9dc5ec49ce3c129 <nowiki>Add jbt6k74 display driver.</nowiki>] =
+
lindi: gcc seems to inline a lot and use a lot of stack here: ioctl.c:2531: warning: the frame size of 1888 bytes is larger than 1024 bytes
 +
 
 +
From #ath6kl on freenode:
 +
<pabs3> kvalo: does ath6kl support AR6001/AR6002?
 +
<kvalo> pabs3: no
 +
<pabs3> kvalo: do you think it is feasible to add support for them, or are they completely different?
 +
<kvalo> pabs3: I haven't even seen either of them so I'm not qualified to answer :)
 +
<pabs3> ok
 +
 
 +
= [https://gitorious.org/shr/linux/commit/4e11662e7b95b1a8cdee1f5af9dc5ec49ce3c129?format=patch <nowiki>Add jbt6k74 display driver.</nowiki>] =
 
  lindi: feature. quite essential. does not look too tricky.
 
  lindi: feature. quite essential. does not look too tricky.
= [https://gitorious.org/shr/linux/commit/82dfe46f68342b3139b4df33cf3f782ab6d52dd9 <nowiki>LEDS: leds-pwm: Add init, notfiy and exit callbacks</nowiki>] =
+
= [https://gitorious.org/shr/linux/commit/82dfe46f68342b3139b4df33cf3f782ab6d52dd9?format=patch <nowiki>LEDS: leds-pwm: Add init, notfiy and exit callbacks</nowiki>] =
 
  lindi: feature. not sure how easy this is to mainline
 
  lindi: feature. not sure how easy this is to mainline
= [https://gitorious.org/shr/linux/commit/8e99225f5a165b20e25611db3486058d7f4e9dc1 <nowiki>RTC: PCF50633: Don't request update IRQ</nowiki>] =
+
= [https://gitorious.org/shr/linux/commit/8e99225f5a165b20e25611db3486058d7f4e9dc1?format=patch <nowiki>RTC: PCF50633: Don't request update IRQ</nowiki>] =
  lindi:  
+
  lindi: safe cleanup?
= [https://gitorious.org/shr/linux/commit/0a304203ac7daf36070f8fecd8974bfaf84d9df6 <nowiki>pcf50633-gpio: Add gpiolib support.</nowiki>] =
+
lindi: seems to be upstream already in 3.0.0 as 3d62b8f59bb8bd08d8e3df3ab3174aab2971d6e6
= [https://gitorious.org/shr/linux/commit/22ce0c31d823e002859fe5ef50f17905ce90471b <nowiki>pcf50633: Move pcf50633-gpio driver to the gpio drivers folder</nowiki>] =
+
 
= [https://gitorious.org/shr/linux/commit/e583591037796f962d07e935b2a1bdd0f89a61fb <nowiki>MFD: pcf50633: Use mfd cells to register child devices</nowiki>] =
+
= [https://gitorious.org/shr/linux/commit/0a304203ac7daf36070f8fecd8974bfaf84d9df6?format=patch <nowiki>pcf50633-gpio: Add gpiolib support.</nowiki>] =
= [https://gitorious.org/shr/linux/commit/780da55e2ad7f9d934e7452fba4340a5f9575a32 <nowiki>MFD: pcf50633: Use the genirq for irq handling</nowiki>] =
+
lindi: feature. has some useless whitespace changes. gpio stuff which I don't really understand
= [https://gitorious.org/shr/linux/commit/3e77986a22550a8df84d9e78ddbc281488bf1120 <nowiki>power: implement platform battery driver</nowiki>] =
+
lindi: pcf50633-gpio.c:197: warning: ignoring return value of ‘gpiochip_remove’, declared with attribute warn_
= [https://gitorious.org/shr/linux/commit/d09e21824bacd5f9ac0285c14c6ddc2d79b96608 <nowiki>Define machine_is_neo1973_gta01()</nowiki>] =
+
 
= [https://gitorious.org/shr/linux/commit/a73311169fef3284647f3c80748c644abe3c72fb <nowiki>glamo-mci: revert changes for Per's patchset</nowiki>] =
+
= [https://gitorious.org/shr/linux/commit/22ce0c31d823e002859fe5ef50f17905ce90471b?format=patch <nowiki>pcf50633: Move pcf50633-gpio driver to the gpio drivers folder</nowiki>] =
= [https://gitorious.org/shr/linux/commit/833339cd47295e86e0ca8c838e8997a24ababf6c <nowiki>nand/s3c2410: add mising badblocksbits value</nowiki>] =
+
lindi: just moves a file
= [https://gitorious.org/shr/linux/commit/919ad8a33847d10bb2b6b4f9bb24702346455f1f <nowiki>Openmoko resume reason sysfs node ported from 2.6.29</nowiki>] =
+
= [https://gitorious.org/shr/linux/commit/e583591037796f962d07e935b2a1bdd0f89a61fb?format=patch <nowiki>MFD: pcf50633: Use mfd cells to register  
= [https://gitorious.org/shr/linux/commit/dd73aa36cb4d37d200aead95310df58cfec784d3 <nowiki>glamo-display: fix WSOD for 242 timming</nowiki>] =
+
child devices</nowiki>] =
= [https://gitorious.org/shr/linux/commit/5c7e36859e8e28f9b56a2cb3ceb0ab2c3d079f96 <nowiki>Enable powering off after 8s POWER press</nowiki>] =
+
lindi: cleanup? No idea how tricky this really is
= [https://gitorious.org/shr/linux/commit/ba207d46b7857f6cae1368f0e62e611eb773505c <nowiki>gta02: add support for usb host mode</nowiki>] =
+
= [https://gitorious.org/shr/linux/commit/780da55e2ad7f9d934e7452fba4340a5f9575a32?format=patch <nowiki>MFD: pcf50633: Use the genirq for irq handling</nowiki>] =
= [https://gitorious.org/shr/linux/commit/3066c8798168cb31af999e9d55bca2b445e27cdb <nowiki>ar6000_delay.patch</nowiki>] =
+
lindi: has again extra whitespace changes. I guess genirq is some more abstract way to handle IRQs? Can not really judge how tricky this is.
= [https://gitorious.org/shr/linux/commit/b3f60f3ed24f503bda64aabf94b62053379c9ca9 <nowiki>wm8753: use snd_soc_jack on neo1973</nowiki>] =
+
= [https://gitorious.org/shr/linux/commit/3e77986a22550a8df84d9e78ddbc281488bf1120?format=patch <nowiki>power: implement platform battery driver</nowiki>] =
= [https://gitorious.org/shr/linux/commit/d5d5ee9c07ee06174f06255ae3141c368ca48f6a <nowiki>Fix high power consumption in suspend</nowiki>] =
+
lindi: feature. we can live without this if bq27x00 is mainlined or vice versa.
= [https://gitorious.org/shr/linux/commit/084e5cd3284d499e3c4eb86f26042e7d72df2266 <nowiki>Force GPS power up on resume if it were powered up on suspend</nowiki>] =
+
= [https://gitorious.org/shr/linux/commit/d09e21824bacd5f9ac0285c14c6ddc2d79b96608?format=patch <nowiki>Define machine_is_neo1973_gta01()</nowiki>] =
= [https://gitorious.org/shr/linux/commit/edcc8410bc93dcc06b69d57e8b7b47756124cd1e <nowiki>lis302dl accelerometer driver</nowiki>] =
+
lindi: trivial
= [https://gitorious.org/shr/linux/commit/02e9ff6da8f66701739b5beac43200eb3e0acbd7 <nowiki>input: lis302dl: fix the resume path</nowiki>] =
+
= [https://gitorious.org/shr/linux/commit/a73311169fef3284647f3c80748c644abe3c72fb?format=patch <nowiki>glamo-mci: revert changes for Per's patchset</nowiki>] =
= [https://gitorious.org/shr/linux/commit/10b4b78dc2081caff1239f229e08a7e829ed3c7d <nowiki>lis302dl: use ABS events rather then REL events</nowiki>] =
+
lindi: glamo stuff, no idea
= [https://gitorious.org/shr/linux/commit/9668f02fe2af9dd880d5f83f95bd07bb2d963c56 <nowiki>Fix compile error for gta02 accelerometer.</nowiki>] =
+
= [https://gitorious.org/shr/linux/commit/833339cd47295e86e0ca8c838e8997a24ababf6c?format=patch <nowiki>nand/s3c2410: add mising badblocksbits value</nowiki>] =
= [https://gitorious.org/shr/linux/commit/f0e6a6317de2567383a543a545cf50e58b3caa3d <nowiki>tslib relies on ts pressures events so this hack is needed to get tslib stuff working</nowiki>] =
+
lindi: bugfix. trivial.
= [https://gitorious.org/shr/linux/commit/b05f074d5eb6f77cefcef35f7f2b2aaeb27af069 <nowiki>touchscreen: ignore unexpected interrupts</nowiki>] =
+
lindi: it seems this was broken in 2.6.36 and fixed again in 3.0.0 but differently: 26d9be11485ea8c1102c3e8eaa7667412eef4950
= [https://gitorious.org/shr/linux/commit/1f3750e2cf3895b269589495147416c61ba523c5 <nowiki>s3c2410_ts: jitter less touchscreen for glamo, version 4</nowiki>] =
+
= [https://gitorious.org/shr/linux/commit/919ad8a33847d10bb2b6b4f9bb24702346455f1f?format=patch <nowiki>Openmoko resume reason sysfs node ported from 2.6.29</nowiki>] =
= [https://gitorious.org/shr/linux/commit/74c481b35be9bf9b7d512de6942c7be345a7db47 <nowiki>wm8753: allow setting DAI mode even while pcm is active</nowiki>] =
+
lindi: feature. I personally need this a lot so that I can differentiate between RTC and GSM wakeups
= [https://gitorious.org/shr/linux/commit/37d242a32880093018a8665131e4b3e2b34d5dab <nowiki>Add .config from http://build.shr-project.org/tests/jama/kernel-nodrm/config-2.6.39.gz</nowiki>] =
+
= [https://gitorious.org/shr/linux/commit/dd73aa36cb4d37d200aead95310df58cfec784d3?format=patch <nowiki>glamo-display: fix WSOD for 242 timming</nowiki>] =
= [https://gitorious.org/shr/linux/commit/4f1ae92f97c4aaad16567a5101c8b668893a9c91 <nowiki>Workaround toolchain bug: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48783</nowiki>] =
+
lindi: openmoko specific. glamo hack. Who said 2-4-2 timings have no drawbacks? ;-)
= [https://gitorious.org/shr/linux/commit/098ed461036a5d9ba027b62c18a85622a055f63d <nowiki>First attempt at fixing openmoko bug #2478</nowiki>] =
+
gena2x: This is not original name for patch! And not true. I do not know why radek maned it with this way, but actually it JUST fixes WSOD with all kind of timings. Please rename it.
 +
 
 +
= [https://gitorious.org/shr/linux/commit/5c7e36859e8e28f9b56a2cb3ceb0ab2c3d079f96?format=patch <nowiki>Enable powering off after 8s POWER press</nowiki>] =
 +
lindi: openmoko specific. feature. simple. I personally don't really like this much however this should be easy to upstream.
 +
= [https://gitorious.org/shr/linux/commit/ba207d46b7857f6cae1368f0e62e611eb773505c?format=patch <nowiki>gta02: add support for usb host mode</nowiki>] =
 +
lindi: openmoko specific feature. looks quite simple.
 +
= [https://gitorious.org/shr/linux/commit/3066c8798168cb31af999e9d55bca2b445e27cdb?format=patch <nowiki>ar6000_delay.patch</nowiki>] =
 +
lindi: atheros wifi hack
 +
= [https://gitorious.org/shr/linux/commit/b3f60f3ed24f503bda64aabf94b62053379c9ca9?format=patch <nowiki>wm8753: use snd_soc_jack on neo1973</nowiki>] =
 +
lindi: openmoko specific. gta01-only?
 +
= [https://gitorious.org/shr/linux/commit/d5d5ee9c07ee06174f06255ae3141c368ca48f6a?format=patch <nowiki>Fix high power consumption in suspend</nowiki>] =
 +
lindi: openmoko specific. bugfix. simple. this should be a good candidate for upstreaming first?
 +
lindi: gena2x, this needs a better commit message
 +
 
 +
= [https://gitorious.org/shr/linux/commit/084e5cd3284d499e3c4eb86f26042e7d72df2266?format=patch <nowiki>Force GPS power up on resume if it were powered up on suspend</nowiki>] =
 +
lindi: openmoko specific. Is this needed at all with qi? (My guess is no, its resume path does not touch serial ports)
 +
= [https://gitorious.org/shr/linux/commit/edcc8410bc93dcc06b69d57e8b7b47756124cd1e?format=patch <nowiki>lis302dl accelerometer driver</nowiki>] =
 +
lindi: openmoko specific. feature. hard to say how tricky this is. I can personally live without accelerometers.
 +
Paul: upstream has a different driver for the hardware: http://lwn.net/Articles/304326/
 +
= [https://gitorious.org/shr/linux/commit/02e9ff6da8f66701739b5beac43200eb3e0acbd7?format=patch <nowiki>input: lis302dl: fix the resume path</nowiki>] =
 +
lindi: openmoko specific. bugfix to accelerometer support.
 +
= [https://gitorious.org/shr/linux/commit/10b4b78dc2081caff1239f229e08a7e829ed3c7d?format=patch <nowiki>lis302dl: use ABS events rather then REL events</nowiki>] =
 +
lindi: openmoko specific. bugfix to accelerometer support.
 +
= [https://gitorious.org/shr/linux/commit/9668f02fe2af9dd880d5f83f95bd07bb2d963c56?format=patch <nowiki>Fix compile error for gta02 accelerometer.</nowiki>] =
 +
lindi: openmoko specific. trivial bugfix to accelerometer support. Why not rebase this to the patch?
 +
= [https://gitorious.org/shr/linux/commit/f0e6a6317de2567383a543a545cf50e58b3caa3d?format=patch <nowiki>tslib relies on ts pressures events so this hack is needed to get tslib stuff working</nowiki>] =
 +
lindi: s3c specific. hack? I personally can live with evdev if this really only affects tslib.
 +
Paul: hm, worth investigating more: is not returning pressure at all a valid thing to do for a TS driver?
 +
Heiko Stübner <heiko@sntech.de>:
 +
  As I'm currently also working on a touchscreen driver I came upon a lot of
 +
  discussion and documentation about the handling of pressure values.
 +
  In the end a driver should not use the pressure property at all if it can't
 +
  provide meaningfull pressure data (i.e. more than 0 and 1) and tslib should be
 +
  "long fixed", if one does not use an ancient version.
 +
 
 +
= [https://gitorious.org/shr/linux/commit/b05f074d5eb6f77cefcef35f7f2b2aaeb27af069?format=patch <nowiki>touchscreen: ignore unexpected interrupts</nowiki>] =
 +
lindi: s3c specific. bugfix? hard to say how ready this is for mainlining.
 +
= [https://gitorious.org/shr/linux/commit/1f3750e2cf3895b269589495147416c61ba523c5?format=patch <nowiki>s3c2410_ts: jitter less touchscreen for glamo, version 4</nowiki>] =
 +
lindi: openmoko specific. quite essential. hack: read ts during vblank?
 +
gena2x: it slows down pixclock to read jitterless data. it does that while blank so screen has no artefacts. may be it is possible to do that while blank without slowing pixclock, even that we have no VSYNC-like interrupt we should be able to calculate exact time of VSYNC because pixclock is constant and we always know current position. But this doesn't solve only major concern of this patch - it introduces link (via callbacks) between glamo and ts.
 +
 
 +
= [https://gitorious.org/shr/linux/commit/74c481b35be9bf9b7d512de6942c7be345a7db47?format=patch <nowiki>wm8753: allow setting DAI mode even while pcm is active</nowiki>] =
 +
lindi: bugfix. should be easy to mainline. This basically fixes a bug introduced in 338ee25393a5627e8ded5819147f98b919656ce9 that was mainlined to 2.6.39
 +
lindi: is there some other driver that needs a similar fix? tlv320dac33.c seems to have a similar if (dac33->fifo_mode == ucontrol->value.integer.value[0]) return 0;
 +
lindi: why does this return 1?
 +
lindi: this seems to be called from snd_ctl_elem_write. If we return 1 then it calls snd_ctl_notify, what does this mean?
 +
lindi-_> gena2x_ptr: "return 1" is a bug in that patch, it needs to be "return 0"
 +
lindi-_> gena2x_ptr: easy to see
 +
lindi-_> gena2x_ptr: strace alsamixer
 +
lindi-_> gena2x_ptr: then while true; do alsactl restore -f somestate; done
 +
lindi-_> gena2x_ptr: alsamixer gets spammed by bogus DAI change events
 +
lindi-_> gena2x_ptr: no other alsa control sends notifications ion this test
 +
lindi: sent upstream: http://mailman.alsa-project.org/pipermail/alsa-devel/2011-November/046036.html
 +
lindi: ACCEPTED UPSTREAM: http://mailman.alsa-project.org/pipermail/alsa-devel/2011-November/046059.html
 +
 
 +
= [https://gitorious.org/shr/linux/commit/4f1ae92f97c4aaad16567a5101c8b668893a9c91?format=patch <nowiki>Workaround toolchain bug: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48783</nowiki>] =
 +
lindi: trivial. Occurs with gcc 4.4 but not 4.6 anymore. No need to mainline :-)
 +
= [https://gitorious.org/shr/linux/commit/098ed461036a5d9ba027b62c18a85622a055f63d?format=patch <nowiki>First attempt at fixing openmoko bug #2478</nowiki>] =
 +
lindi: openmoko specific. bugfix. not yet ready. See http://docs.openmoko.org/trac/ticket/2478
 +
 
 +
[[Category:Kernel]]

Latest revision as of 11:14, 25 January 2012

Random notes on SHR kernel patches (v2.6.39.4..shr-2.6.39-nodrm).

Contents

[edit] ARM: s3c24xx: Set ARCH_NR_GPIOS according to the selected SoC types.

lindi: bugfix/feature. sounds sensible. s3c specific
Seems to be at least partially upstream since v3.2-rc1: http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=8a8ab2e64e09b56dc1324fd2f7da12346166cad1

[edit] s3c24xx: Fix level irqs on external interrupts.

lindi: bugfix. s3c specific

[edit] mtd: s3c2410_nand: Add config option to disable hw ecc at runtime

lindi: feature. simple. I personally can live without nand.
lindi: Lars tried to upstream but it was rejected:
> Lars-Peter Clausen <lars@metafoo.de> writes:
>> Not quite. I had for example a 5 liner rejected by a maintainer saying he can't
>> accept any code which adds new platform code to ARM.
lindi: however that appears to have been a misunderstanding and this can be resent

[edit] Add s3c24xx_serial_console_set_silence

lindi: openmoko specific. only for gta01: to have serial console and GSM on the same port.

[edit] Add c fiq handler.

lindi: feature. tricky! Used by battery hdq queries (temperature, current, capacity etc.) and vibrator. Should just rewrite them in assembler?
lindi: http://osdevnotes.blogspot.com/2011/10/fiq-debugger-redux.html mentions that google has something similar for their FIQ debugger.

[edit] Add glamo driver.

lindi: feature. tricky!! This does not include the DRM/3D bits but it is enough for me.

[edit] ARM: gta02: Add LED support

lindi: feature. openmoko specific. simple.
lindi: could this be the next candidate for upstreaming? Is there anything controversial here? has somebody already tried to upstream this?
lindi: there's a lot of extra whitespace changes, probably because the patch has been rebased many times?

[edit] gta02: Disable hardware ECC unless we get instructed to enable it

lindi: openmoko specific. nand. simple.

[edit] gta02: Remove usage of pcf50633 gpio api

lindi: openmoko specific. cleanup.

[edit] s3c2440: gta02: Configure pcf50633 gpios.

lindi: openmoko specific.

[edit] ARM: s3c2440: GTA02: Select SPARSE_IRQ

lindi: openmoko specific. simple.

[edit] gta02: Add notify handler to probe device children

lindi: openmoko specific. feature.

[edit] gta02: Add bt power management device

lindi: openmoko specific. simple. is this the right API to use? could we abuse rfkill? what do the androids/n900/N9 use?

[edit] gta02: Add gps power management device

lindi: openmoko specific. simple.

[edit] gta02: Add gsm power management device

lindi: openmoko specific. simple.

[edit] gta02: Add wlan power management device

lindi: openmoko specific. simple.

[edit] gta02: Add glamo device

lindi: openmoko specific. relatively simple, real trouble is in the glamo driver itself

[edit] Add jbt device.

lindi: openmoko specific. LCD control?

[edit] gta02: Add fiq handler

lindi: openmoko specific. depends on the tricky fiq support above.

[edit] gta02: Add battery driver

lindi: openmoko specific. depends on the tricky fiq support above.
lindi: mach-gta02.c:595: warning: ‘gta02_get_charger_online_status’ defined but not used

[edit] gta02: add support for platform_battery

lindi: openmoko specific. simple.

[edit] Add atheros sdio ids.

lindi: trivial. why does AR6003 define this in drivers/net/wireless/ath/ath6kl/hif.h and not this common file?

[edit] Add ar6000 wireless driver.

lindi: tricky! probably can never be mainlined. Maybe we can build it as a separate module package in debian?
lindi: gcc seems to inline a lot and use a lot of stack here: ioctl.c:2531: warning: the frame size of 1888 bytes is larger than 1024 bytes
From #ath6kl on freenode:
<pabs3> kvalo: does ath6kl support AR6001/AR6002?
<kvalo> pabs3: no
<pabs3> kvalo: do you think it is feasible to add support for them, or are they completely different?
<kvalo> pabs3: I haven't even seen either of them so I'm not qualified to answer :)
<pabs3> ok

[edit] Add jbt6k74 display driver.

lindi: feature. quite essential. does not look too tricky.

[edit] LEDS: leds-pwm: Add init, notfiy and exit callbacks

lindi: feature. not sure how easy this is to mainline

[edit] RTC: PCF50633: Don't request update IRQ

lindi: safe cleanup?
lindi: seems to be upstream already in 3.0.0 as 3d62b8f59bb8bd08d8e3df3ab3174aab2971d6e6

[edit] pcf50633-gpio: Add gpiolib support.

lindi: feature. has some useless whitespace changes. gpio stuff which I don't really understand
lindi: pcf50633-gpio.c:197: warning: ignoring return value of ‘gpiochip_remove’, declared with attribute warn_

[edit] pcf50633: Move pcf50633-gpio driver to the gpio drivers folder

lindi: just moves a file

MFD: pcf50633: Use mfd cells to register child devices MFD: pcf50633: Use mfd cells to register child devices

lindi: cleanup? No idea how tricky this really is

[edit] MFD: pcf50633: Use the genirq for irq handling

lindi: has again extra whitespace changes. I guess genirq is some more abstract way to handle IRQs? Can not really judge how tricky this is.

[edit] power: implement platform battery driver

lindi: feature. we can live without this if bq27x00 is mainlined or vice versa.

[edit] Define machine_is_neo1973_gta01()

lindi: trivial

[edit] glamo-mci: revert changes for Per's patchset

lindi: glamo stuff, no idea

[edit] nand/s3c2410: add mising badblocksbits value

lindi: bugfix. trivial. 
lindi: it seems this was broken in 2.6.36 and fixed again in 3.0.0 but differently: 26d9be11485ea8c1102c3e8eaa7667412eef4950

[edit] Openmoko resume reason sysfs node ported from 2.6.29

lindi: feature. I personally need this a lot so that I can differentiate between RTC and GSM wakeups

[edit] glamo-display: fix WSOD for 242 timming

lindi: openmoko specific. glamo hack. Who said 2-4-2 timings have no drawbacks? ;-)
gena2x: This is not original name for patch! And not true. I do not know why radek maned it with this way, but actually it JUST fixes WSOD with all kind of timings. Please rename it.

[edit] Enable powering off after 8s POWER press

lindi: openmoko specific. feature. simple. I personally don't really like this much however this should be easy to upstream.

[edit] gta02: add support for usb host mode

lindi: openmoko specific feature. looks quite simple.

[edit] ar6000_delay.patch

lindi: atheros wifi hack

[edit] wm8753: use snd_soc_jack on neo1973

lindi: openmoko specific. gta01-only?

[edit] Fix high power consumption in suspend

lindi: openmoko specific. bugfix. simple. this should be a good candidate for upstreaming first?
lindi: gena2x, this needs a better commit message

[edit] Force GPS power up on resume if it were powered up on suspend

lindi: openmoko specific. Is this needed at all with qi? (My guess is no, its resume path does not touch serial ports)

[edit] lis302dl accelerometer driver

lindi: openmoko specific. feature. hard to say how tricky this is. I can personally live without accelerometers.
Paul: upstream has a different driver for the hardware: http://lwn.net/Articles/304326/

[edit] input: lis302dl: fix the resume path

lindi: openmoko specific. bugfix to accelerometer support.

[edit] lis302dl: use ABS events rather then REL events

lindi: openmoko specific. bugfix to accelerometer support.

[edit] Fix compile error for gta02 accelerometer.

lindi: openmoko specific. trivial bugfix to accelerometer support. Why not rebase this to the patch?

[edit] tslib relies on ts pressures events so this hack is needed to get tslib stuff working

lindi: s3c specific. hack? I personally can live with evdev if this really only affects tslib.
Paul: hm, worth investigating more: is not returning pressure at all a valid thing to do for a TS driver?
Heiko Stübner <heiko@sntech.de>: 
 As I'm currently also working on a touchscreen driver I came upon a lot of
 discussion and documentation about the handling of pressure values.
 In the end a driver should not use the pressure property at all if it can't
 provide meaningfull pressure data (i.e. more than 0 and 1) and tslib should be
 "long fixed", if one does not use an ancient version.

[edit] touchscreen: ignore unexpected interrupts

lindi: s3c specific. bugfix? hard to say how ready this is for mainlining.

[edit] s3c2410_ts: jitter less touchscreen for glamo, version 4

lindi: openmoko specific. quite essential. hack: read ts during vblank?
gena2x: it slows down pixclock to read jitterless data. it does that while blank so screen has no artefacts. may be it is possible to do that while blank without slowing pixclock, even that we have no VSYNC-like interrupt we should be able to calculate exact time of VSYNC because pixclock is constant and we always know current position. But this doesn't solve only major concern of this patch - it introduces link (via callbacks) between glamo and ts.

[edit] wm8753: allow setting DAI mode even while pcm is active

lindi: bugfix. should be easy to mainline. This basically fixes a bug introduced in 338ee25393a5627e8ded5819147f98b919656ce9 that was mainlined to 2.6.39
lindi: is there some other driver that needs a similar fix? tlv320dac33.c seems to have a similar if (dac33->fifo_mode == ucontrol->value.integer.value[0]) return 0;
lindi: why does this return 1?
lindi: this seems to be called from snd_ctl_elem_write. If we return 1 then it calls snd_ctl_notify, what does this mean?
lindi-_> gena2x_ptr: "return 1" is a bug in that patch, it needs to be "return 0"
lindi-_> gena2x_ptr: easy to see
lindi-_> gena2x_ptr: strace alsamixer
lindi-_> gena2x_ptr: then while true; do alsactl restore -f somestate; done
lindi-_> gena2x_ptr: alsamixer gets spammed by bogus DAI change events
lindi-_> gena2x_ptr: no other alsa control sends notifications ion this test
lindi: sent upstream: http://mailman.alsa-project.org/pipermail/alsa-devel/2011-November/046036.html
lindi: ACCEPTED UPSTREAM: http://mailman.alsa-project.org/pipermail/alsa-devel/2011-November/046059.html

[edit] Workaround toolchain bug: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48783

lindi: trivial. Occurs with gcc 4.4 but not 4.6 anymore. No need to mainline :-)

[edit] First attempt at fixing openmoko bug #2478

lindi: openmoko specific. bugfix. not yet ready. See http://docs.openmoko.org/trac/ticket/2478
Personal tools

Random notes on SHR kernel patches (v2.6.39.4..shr-2.6.39-nodrm).

ARM: s3c24xx: Set ARCH_NR_GPIOS according to the selected SoC types.

lindi: bugfix/feature. sounds sensible. s3c specific

s3c24xx: Fix level irqs on external interrupts.

lindi: bugfix. s3c specific

mtd: s3c2410_nand: Add config option to disable hw ecc at runtime

lindi: feature. simple. I personally can live without nand. 

Add s3c24xx_serial_console_set_silence

lindi: What is this for? To allow using serial console and GSM on the same serial port? Who needs this anymore?

Add c fiq handler.

lindi: feature. tricky! Used by battery hdq queries (temperature, current, capacity etc.) and vibrator. Should just rewrite them in assembler?

Add glamo driver.

lindi: feature. tricky!! This does not include the DRM/3D bits but it is enough for me.

ARM: gta02: Add LED support

lindi: feature. openmoko specific. simple.

gta02: Disable hardware ECC unless we get instructed to enable it

lindi: openmoko specific. nand. simple.

gta02: Remove usage of pcf50633 gpio api

lindi: openmoko specific. cleanup.

s3c2440: gta02: Configure pcf50633 gpios.

lindi: openmoko specific.

ARM: s3c2440: GTA02: Select SPARSE_IRQ

lindi: openmoko specific. simple.

gta02: Add notify handler to probe device children

lindi: openmoko specific. feature.

gta02: Add bt power management device

lindi: openmoko specific. simple.

gta02: Add gps power management device

lindi: openmoko specific. simple.

gta02: Add gsm power management device

lindi: openmoko specific. simple.

gta02: Add wlan power management device

lindi: openmoko specific. simple.

gta02: Add glamo device

lindi: openmoko specific. relatively simple, real trouble is in the glamo driver itself

Add jbt device.

lindi: openmoko specific. backlight?

gta02: Add fiq handler

lindi: openmoko specific. depends on the tricky fiq support above.

gta02: Add battery driver

lindi: openmoko specific. depends on the tricky fiq support above.

gta02: add support for platform_battery

lindi: openmoko specific. simple.

Add atheros sdio ids.

lindi: trivial.

Add ar6000 wireless driver.

lindi: tricky! probably can never be mainlined. Maybe we can build it as a separate module package in debian?

Add jbt6k74 display driver.

lindi: feature. quite essential. does not look too tricky.

LEDS: leds-pwm: Add init, notfiy and exit callbacks

lindi: feature. not sure how easy this is to mainline

RTC: PCF50633: Don't request update IRQ

lindi: 

pcf50633-gpio: Add gpiolib support.

pcf50633: Move pcf50633-gpio driver to the gpio drivers folder

MFD: pcf50633: Use mfd cells to register child devices

MFD: pcf50633: Use the genirq for irq handling

power: implement platform battery driver

Define machine_is_neo1973_gta01()

glamo-mci: revert changes for Per's patchset

nand/s3c2410: add mising badblocksbits value

Openmoko resume reason sysfs node ported from 2.6.29

glamo-display: fix WSOD for 242 timming

Enable powering off after 8s POWER press

gta02: add support for usb host mode

ar6000_delay.patch

wm8753: use snd_soc_jack on neo1973

Fix high power consumption in suspend

Force GPS power up on resume if it were powered up on suspend

lis302dl accelerometer driver

input: lis302dl: fix the resume path

lis302dl: use ABS events rather then REL events

Fix compile error for gta02 accelerometer.

tslib relies on ts pressures events so this hack is needed to get tslib stuff working

touchscreen: ignore unexpected interrupts

s3c2410_ts: jitter less touchscreen for glamo, version 4

wm8753: allow setting DAI mode even while pcm is active

Add .config from http://build.shr-project.org/tests/jama/kernel-nodrm/config-2.6.39.gz

Workaround toolchain bug: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48783

First attempt at fixing openmoko bug #2478