Hi, On 7/30/2009 7:26 PM, Tobias Lorenz wrote: > Hi, > >> I send the radio-si470x patches worked on http://linuxtv.org/hg/v4l-dvb. >> The patches is updated to version 2. > > The patchset looks good. I'll give my feedback in the following mails. > >> Tobias informed me the base code for seperating at >> http://linuxtv.org/hg/~tlorenz/v4l-dvb of Tobias repository in above >> mail, i based on it, but it cannot find now at Tobias repository. > > Before sending a pull request, I usually clean up the archive from any other > patches. > But nevertheless, you and me still have the I2C patches. They now reached a > quality to finally bring them in the kernel. > Good work. >
Thanks. I am concerned about one thing. I cannot test the si470x usb radio driver because i don't have the si470x usb radio device, so i believe you would have probably tested it. >> The patch 1/4 is for separating common and usb code. >> The patch 2/4 is about using dev_* macro instead of printk. >> The patch 3/4 is about adding disconnect check function for i2c interface. >> The patch 4/4 is for supporting si470x i2c interface. > > Bye, > Toby > -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
