-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 On 7/29/14, 9:39 AM, Shawn W Dunn wrote:
So, what is the generally accepted procedure for kernel patches?
I have one to enable type cover keyboard support for the MS Surface Pro devices, and coolo suggested I ask here, instead of pushing it directly in OBS to the factory kernel
Hi Shawn - I did see your patch sent to me. Apologies, I was away over the weekend and hadn't gotten to it yet. Quick review is that the content is ok, but you need to add more documentation to it in order for it to be accepted. I don't mean in the form of comments in the code -- but documentation as to what it fixes and your Signed-off-by. The generally accepted practice is that a patch must be accepted into the upstream kernel or a 'maintainer' branch that is known to flow directly into the upstream kernel on a regular basis. To do otherwise would make the openSUSE kernel divergent from mainline rather than just a branch with backports[1]. Once your patch is upstream, we can include it into the openSUSE kernel. A good place to start would be to look at the SubmittingPatches[2] file in the upstream kernel source and the README[3] file in our own kernel-source repository. The former documents what a patch should generally look like for inclusion in the upstream kernel. The latter documents the specific rules that must be followed for us to accept it into the openSUSE kernel. Once you have everything set for the upstream kernel, all you'll need to do is open a bug report at bugzilla.novell.com, add a reference to it in your patch using the "References" tag, and add the "Patch-mainline" and "Git-commit" tags that reference where the commit landed upstream. There are examples of acceptable patch headers in the README file. Really, though, picking nearly any other patch already in the repository would be a good source to compare against. As for asking here instead of pushing directly to OBS, that's the right thing to do. The kernel-source package is maintained in a separate git repository[4] and patches against the OBS package will either be (correctly) declined or, if someone violated the rules, will be lost with the next sync from the git repository. I know this is cumbersome, but the documentation rules are mostly from the upstream kernel. They exist for long-term maintainability of the kernel. I know you just want to get your tablet's keyboard working, but everyone wants to just get /something/ working and we'd end up with a mess if we accepted patches outside of the rules. Unfortunately, it seems that the HID subsystem doesn't support the new_id mechanism that PCI and general USB devices do (HID is an abstraction layer above the USB bus). Otherwise, it would be possible to tell the driver about the new IDs at runtime until your kernel patch is accepted. I've added Jiri Kosina and Jiri Slaby, who are both active maintainers in the input/HID area, and may be able to help you once you get your patch in shape. - -Jeff [1] The SUSE "supported" patches and Xen are notable counterexamples here. The former will necessarily always be out-of-tree. There are several SUSE engineers working full-time on getting Xen reworked and merged upstream. [2] http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/Document... [3] http://kernel.opensuse.org/cgit/kernel-source/tree/README [4] http://kernel.opensuse.org/cgit/kernel-source/log/ - -- Jeff Mahoney SUSE Labs -----BEGIN PGP SIGNATURE----- Version: GnuPG/MacGPG2 v2.0.19 (Darwin) iQIcBAEBAgAGBQJT160vAAoJEB57S2MheeWyeJYP/AuCU+1rWsNEkSFr2n3dLtDb vFfxXGNdA7VlOTtQheEInA9AzeOCBETcLMtCRctYqAoLmn6/uS6lcEC2CAlAZa04 EiJNsVlE76E1aPXxhkgvq6OhN/luI8Ws8qIqOpgVfdvFvPfvwbdR5Q0TI0TuMqCh 7fulfB4yxaZdzgWegC7dbzo0d2NWLy4vhr5eyPdLLAzoBeVabWdFORUvTwo+LzoQ +z2iRZ5+oJ8HGYfQNu9W4dFa5afk7lXMb2uX398ebsM22WOFHQ5VeAegSOd8TQEh UurA8acna5aFNI2kfUjzPm55UTl4WrY1sr/xuGIAUVt6mPOF4cRWtmC8KnVgyfGo AnWrut6KLdO9A3iyPPQZcife7lxgiSZS5dIbV18tv/RQmo20vBSxDqHSEKSlePlN g/GGi0RFCg7f/7AfNvK7lT+mzdDcTYbzlOI2ixCaIMxmT6CSeIzpD+oY1z8bakpm Mca/hdir2Dd04TU/YlRXO1iKcAuGDLVHwmXCw7I3s736EORp4FcUugUZenDfa90W bs5iNSWSRvlNPiWzyNUMhE+0M46qh/mpJshCyxwZrkwM2Bmo7GePBD7BZ3ezU0uD phwftobKOU/PSIc9Ddh2FohgkxPD3mI7T1kRAsQd1PDstnRYysgP952A25NNTmb2 rIwaFlmNTStJEK5Albza =3n5p -----END PGP SIGNATURE----- -- To unsubscribe, e-mail: opensuse-kernel+unsubscribe@opensuse.org To contact the owner, e-mail: opensuse-kernel+owner@opensuse.org