[Bug 300515] New: AMD/ATI SB600/700/800 use same SMBus controller devID
https://bugzilla.novell.com/show_bug.cgi?id=300515 Summary: AMD/ATI SB600/700/800 use same SMBus controller devID Product: openSUSE 10.3 Version: Beta 1 Platform: All OS/Version: openSUSE 10.3 Status: NEW Severity: Major Priority: P5 - None Component: Kernel AssignedTo: kernel-maintainers@forge.provo.novell.com ReportedBy: shane.huang@amd.com QAContact: qa@suse.de CC: joachim.deguara@gmail.com, shane.huang@amd.com Depends on: 284195 Found By: --- We find that SB700 and SB800 use the same SMBus device ID as SB600, which is 0x4385, instead of the already submitted 0x4395. We submitted an error SB700 SMBus patch in the bugzilla# 284195 Please apply the attached new SMBus patch, which can remove the wrong applied SB700 SMBus patch submitted before. Besides removing the SB700 wrong patch, the attached patch add SB800 support to kernel, by renaming the PCI_DEVICE_ID_ATI_IXP600_SMBUS into PCI_DEVICE_ID_ATI_SBX00_SMBUS. Would you please help to apply this new patch ASAP before the release of openSUSE10.3? Because the error device ID 0x4395 will be used in SB800 SATA controller. I have sent this new patch to kernel.org, I will send you the upstream commit after they apply it. The patch is generated on the base of linux-2.6.23-rc3, so maybe you need do some small backport to it when you apply it. Thanks -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=300515#c1 --- Comment #1 from Shane Huang <shane.huang@amd.com> 2007-08-14 21:11:13 MST --- Created an attachment (id=157554) --> (https://bugzilla.novell.com/attachment.cgi?id=157554) SB600/SB700/SB800 use same SMBus controll device ID Besides removing the SB700 wrong patch, the attached patch add SB800 support to kernel as well as SB700. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=300515#c2 Greg Kroah-Hartman <gregkh@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |NEEDINFO Info Provider| |shane.huang@amd.com --- Comment #2 from Greg Kroah-Hartman <gregkh@novell.com> 2007-08-14 22:51:27 MST --- Should this also be sent to the stable@kernel.org developers also so they can fix the 2.6.22.y releases? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=300515#c3 --- Comment #3 from Shane Huang <shane.huang@amd.com> 2007-08-14 23:00:27 MST --- I have sent this patch to i2c maintainer Jean Delvare<khali@linux-fr.org> and i2c@lm-sensors.org before I created this bugzilla item. I think they will add this patch into the linux kernel source tree after their review. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=300515#c4 --- Comment #4 from Shane Huang <shane.huang@amd.com> 2007-08-14 23:04:54 MST --- I think this patch cound NOT appear in 2.6.22.y kernel version, it may appear in 2.6.23 or later. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=300515#c5 --- Comment #5 from Shane Huang <shane.huang@amd.com> 2007-08-16 03:39:53 MST --- This patch has been accepted by linux i2c maintainer, and will appear from linux-2.6.23-rc4 most probably. I will present the git link of this patch after I get it. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=300515#c6 --- Comment #6 from Shane Huang <shane.huang@amd.com> 2007-08-16 05:02:27 MST --- I find that the original SB600 patch is NOT in openSUSE10.2 kernel source: http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=... I'm not sure whether the original SB600 patch has been applied to openSUSE10.3 kernel. There are altogether 3 SB600/SB700 SMBus related patches(in time order): 1. SB600 original patch: http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=... 2. SB700 SMBus patch (useless): http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=... 3. this patch attachment in this bugzilla, which remove SB700 error patch and add support to SB800 SMBus. So please apply these three patches one by one. Thanks -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=300515#c7 Shane Huang <shane.huang@amd.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |NEW Info Provider|shane.huang@amd.com | --- Comment #7 from Shane Huang <shane.huang@amd.com> 2007-08-20 20:13:57 MST --- My comments above have provided the needed information. thanks. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=300515#c8 Jeff Mahoney <jeffm@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jeffm@novell.com AssignedTo|kernel-maintainers@forge.provo.novell.com |jdelvare@novell.com --- Comment #8 from Jeff Mahoney <jeffm@novell.com> 2007-08-25 09:40:20 MST --- Jean, this is your playground. Care to comment and/or apply if you consider it safe for 10.3? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=300515#c9 Jean Delvare <jdelvare@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |NEEDINFO Info Provider| |jeffm@novell.com --- Comment #9 from Jean Delvare <jdelvare@novell.com> 2007-08-27 01:41:33 MST --- Support for the ATI SB600 SMBus is already included in openSuse 10.3 (it was added upstream in 2.6.21 and openSuse 10.3 is based on 2.6.22.) The bug (wrong PCI ID listed) which the attached patch fixes, was introduced upstream in 2.6.23-rc1. This means that openSuse 10.3 is not affected, and neither is 2.6.22 upstream, so there's nothing to fix there. Given that the SB700 and SB800 SMBus use the same PCI ID as the SB600 SMBus (and I thank AMD for this), they are already supported properly in openSuse 10.3. It's just not documented, but I don't think it's a problem. After all, the whole point of reusing the PCI IDs where possible is to avoid backporting patches, isn't it? So I think that this bug can be closed as invalid with no further action. Jeff, is this OK with you? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=300515#c10 Jeff Mahoney <jeffm@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |RESOLVED Info Provider|jeffm@novell.com | Resolution| |INVALID --- Comment #10 from Jeff Mahoney <jeffm@novell.com> 2007-08-27 07:56:42 MST --- Thanks Jean, that works for me. Shane, we've seen a number of reports from you recently that either add hardware support to an already shipped release or fix problems which don't exist in our kernel simply because the problems were introduced in a version after 2.6.22 or fixed before 2.6.22. I appreciate the effort in keeping us updated, but the former is a violation of our patch acceptance policy, and the latter could be avoided by checking the X-Git-Tag header for the version in the raw version of the git patch to ensure it's a patch that is needed for our kernel. Either way, unnecessary effort could be avoided with a little due diligence. Closing as INVALID. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=300515#c11 --- Comment #11 from Shane Huang <shane.huang@amd.com> 2007-08-27 19:10:38 MST --- Hi Jeff, sorry for the unnecessary work to you. Usually when we create a kernel patch, we will submit it to kernel.org and many linux distributions to be released very soon through bugzillas. We will check it more carefully in the future to see whether the bugzilla is really valid to this distribution. Thanks. Shane -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
participants (1)
-
bugzilla_noreply＠novell.com