* patches for staging:media drivers @ 2022-11-06 10:52 Deepak R Varma 2022-11-07 17:29 ` Neil Armstrong 0 siblings, 1 reply; 10+ messages in thread From: Deepak R Varma @ 2022-11-06 10:52 UTC (permalink / raw) To: Neil Armstrong, Mauro Carvalho Chehab, Greg Kroah-Hartman, Kevin Hilman, Jerome Brunet, Martin Blumenstingl, linux-media, linux-amlogic, linux-staging, linux-arm-kernel, linux-kernel Greetings to all. My name is Deepak R Varma and I was part of the recent Outreachy internship round. I learned during this program that patches for the media drivers were prohibited to be included in the Outreachy effort due to lack of bandwidth for the maintainers to respond on those. The Outreachy round is now over but I am continuing to work on code improvement and changes. I would like to know if it is okay for me to now identify opportunities in the media drivers and send patch proposal for your kind consideration. I intend to send couple of Coccinelle identified code improvements as patches. Please let me know. Thank you, ./drv _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: patches for staging:media drivers 2022-11-06 10:52 patches for staging:media drivers Deepak R Varma @ 2022-11-07 17:29 ` Neil Armstrong 2022-11-07 18:57 ` Deepak R Varma 2022-11-08 8:10 ` Dan Carpenter 0 siblings, 2 replies; 10+ messages in thread From: Neil Armstrong @ 2022-11-07 17:29 UTC (permalink / raw) To: Deepak R Varma, Mauro Carvalho Chehab, Greg Kroah-Hartman, Kevin Hilman, Jerome Brunet, Martin Blumenstingl, linux-media, linux-amlogic, linux-staging, linux-arm-kernel, linux-kernel Hi, On 06/11/2022 11:52, Deepak R Varma wrote: > Greetings to all. > My name is Deepak R Varma and I was part of the recent Outreachy internship > round. I learned during this program that patches for the media drivers were > prohibited to be included in the Outreachy effort due to lack of bandwidth for > the maintainers to respond on those. > > The Outreachy round is now over but I am continuing to work on code improvement > and changes. I would like to know if it is okay for me to now identify > opportunities in the media drivers and send patch proposal for your kind > consideration. I intend to send couple of Coccinelle identified code > improvements as patches. > > Please let me know. I won't say no but since the media maintainers will need to apply them, the problem is the same. Neil > > Thank you, > ./drv > > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: patches for staging:media drivers 2022-11-07 17:29 ` Neil Armstrong @ 2022-11-07 18:57 ` Deepak R Varma 2022-11-08 8:10 ` Dan Carpenter 1 sibling, 0 replies; 10+ messages in thread From: Deepak R Varma @ 2022-11-07 18:57 UTC (permalink / raw) To: Neil Armstrong Cc: Mauro Carvalho Chehab, Greg Kroah-Hartman, Kevin Hilman, Jerome Brunet, Martin Blumenstingl, linux-media, linux-amlogic, linux-staging, linux-arm-kernel, linux-kernel On Mon, Nov 07, 2022 at 06:29:30PM +0100, Neil Armstrong wrote: > Hi, > > On 06/11/2022 11:52, Deepak R Varma wrote: > > Greetings to all. > > My name is Deepak R Varma and I was part of the recent Outreachy internship > > round. I learned during this program that patches for the media drivers were > > prohibited to be included in the Outreachy effort due to lack of bandwidth for > > the maintainers to respond on those. > > > > The Outreachy round is now over but I am continuing to work on code improvement > > and changes. I would like to know if it is okay for me to now identify > > opportunities in the media drivers and send patch proposal for your kind > > consideration. I intend to send couple of Coccinelle identified code > > improvements as patches. > > > > Please let me know. > > I won't say no but since the media maintainers will need to apply them, > the problem is the same. Hello Neil, Thank you for responding. I will send in the patches. I am sure they will be reviewed at an appropriate time and responded accordingly. Have a good day! ./drv > > Neil > > > > > Thank you, > > ./drv > > > > > > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: patches for staging:media drivers 2022-11-07 17:29 ` Neil Armstrong 2022-11-07 18:57 ` Deepak R Varma @ 2022-11-08 8:10 ` Dan Carpenter 2022-11-08 9:35 ` Hans Verkuil 2022-11-08 20:27 ` Andrew Lunn 1 sibling, 2 replies; 10+ messages in thread From: Dan Carpenter @ 2022-11-08 8:10 UTC (permalink / raw) To: Neil Armstrong Cc: Deepak R Varma, Mauro Carvalho Chehab, Greg Kroah-Hartman, Kevin Hilman, Jerome Brunet, Martin Blumenstingl, linux-media, linux-amlogic, linux-staging, linux-arm-kernel, linux-kernel On Mon, Nov 07, 2022 at 06:29:30PM +0100, Neil Armstrong wrote: > Hi, > > On 06/11/2022 11:52, Deepak R Varma wrote: > > Greetings to all. > > My name is Deepak R Varma and I was part of the recent Outreachy internship > > round. I learned during this program that patches for the media drivers were > > prohibited to be included in the Outreachy effort due to lack of bandwidth for > > the maintainers to respond on those. > > > > The Outreachy round is now over but I am continuing to work on code improvement > > and changes. I would like to know if it is okay for me to now identify > > opportunities in the media drivers and send patch proposal for your kind > > consideration. I intend to send couple of Coccinelle identified code > > improvements as patches. > > > > Please let me know. > > I won't say no but since the media maintainers will need to apply them, > the problem is the same. Yeah. Me either. Those rules are for Outreachy so they don't apply to anyone else, but at the same time no one is going to Ack your patches without seeing them and especially for a subsystem they don't maintain. I always try to think about if maintainers will want my patches. Like I sometimes see people returning -1 instead of -ENOMEM but, you know, unless it reaches user space it's not technically a bug. You can't go around fixing everyone's bad taste. But everyone likes bug fixes. regards, dan carpenter _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: patches for staging:media drivers 2022-11-08 8:10 ` Dan Carpenter @ 2022-11-08 9:35 ` Hans Verkuil 2022-11-08 9:46 ` Dan Carpenter 2022-11-08 20:27 ` Andrew Lunn 1 sibling, 1 reply; 10+ messages in thread From: Hans Verkuil @ 2022-11-08 9:35 UTC (permalink / raw) To: Deepak R Varma Cc: Mauro Carvalho Chehab, Greg Kroah-Hartman, Kevin Hilman, Jerome Brunet, Martin Blumenstingl, linux-media, linux-amlogic, linux-staging, linux-arm-kernel, linux-kernel, Dan Carpenter, Neil Armstrong Hi Deepak, On 11/8/22 09:10, Dan Carpenter wrote: > On Mon, Nov 07, 2022 at 06:29:30PM +0100, Neil Armstrong wrote: >> Hi, >> >> On 06/11/2022 11:52, Deepak R Varma wrote: >>> Greetings to all. >>> My name is Deepak R Varma and I was part of the recent Outreachy internship >>> round. I learned during this program that patches for the media drivers were >>> prohibited to be included in the Outreachy effort due to lack of bandwidth for >>> the maintainers to respond on those. >>> >>> The Outreachy round is now over but I am continuing to work on code improvement >>> and changes. I would like to know if it is okay for me to now identify >>> opportunities in the media drivers and send patch proposal for your kind >>> consideration. I intend to send couple of Coccinelle identified code >>> improvements as patches. >>> >>> Please let me know. >> >> I won't say no but since the media maintainers will need to apply them, >> the problem is the same. > > Yeah. Me either. Those rules are for Outreachy so they don't apply to > anyone else, but at the same time no one is going to Ack your patches > without seeing them and especially for a subsystem they don't maintain. > > I always try to think about if maintainers will want my patches. Like > I sometimes see people returning -1 instead of -ENOMEM but, you know, > unless it reaches user space it's not technically a bug. You can't go > around fixing everyone's bad taste. But everyone likes bug fixes. > > regards, > dan carpenter > I don't mind taking patches for media staging drivers as long as: 1) they are not for atomisp, and 2) they are not for deprecated drivers (drivers/staging/media/deprecated/) Regards, Hans _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: patches for staging:media drivers 2022-11-08 9:35 ` Hans Verkuil @ 2022-11-08 9:46 ` Dan Carpenter 0 siblings, 0 replies; 10+ messages in thread From: Dan Carpenter @ 2022-11-08 9:46 UTC (permalink / raw) To: Hans Verkuil, Greg Kroah-Hartman Cc: Deepak R Varma, Mauro Carvalho Chehab, Kevin Hilman, Jerome Brunet, Martin Blumenstingl, linux-media, linux-amlogic, linux-staging, linux-arm-kernel, linux-kernel, Neil Armstrong On Tue, Nov 08, 2022 at 10:35:54AM +0100, Hans Verkuil wrote: > > I don't mind taking patches for media staging drivers as long as: > > 1) they are not for atomisp, and > 2) they are not for deprecated drivers (drivers/staging/media/deprecated/) > This is unrelated but I somehow misread an email and thought we were going to create a drivers/deprecated/ directory. I still think that's a good idea for future deprecated drivers. We normally put them in staging/ but it's not really the ideal place for them. regards, dan carpenter _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: patches for staging:media drivers 2022-11-08 8:10 ` Dan Carpenter 2022-11-08 9:35 ` Hans Verkuil @ 2022-11-08 20:27 ` Andrew Lunn 2022-11-09 6:40 ` Deepak R Varma 1 sibling, 1 reply; 10+ messages in thread From: Andrew Lunn @ 2022-11-08 20:27 UTC (permalink / raw) To: Dan Carpenter Cc: Neil Armstrong, Deepak R Varma, Mauro Carvalho Chehab, Greg Kroah-Hartman, Kevin Hilman, Jerome Brunet, Martin Blumenstingl, linux-media, linux-amlogic, linux-staging, linux-arm-kernel, linux-kernel On Tue, Nov 08, 2022 at 11:10:21AM +0300, Dan Carpenter wrote: > On Mon, Nov 07, 2022 at 06:29:30PM +0100, Neil Armstrong wrote: > > Hi, > > > > On 06/11/2022 11:52, Deepak R Varma wrote: > > > Greetings to all. > > > My name is Deepak R Varma and I was part of the recent Outreachy internship > > > round. I learned during this program that patches for the media drivers were > > > prohibited to be included in the Outreachy effort due to lack of bandwidth for > > > the maintainers to respond on those. > > > > > > The Outreachy round is now over but I am continuing to work on code improvement > > > and changes. I would like to know if it is okay for me to now identify > > > opportunities in the media drivers and send patch proposal for your kind > > > consideration. I intend to send couple of Coccinelle identified code > > > improvements as patches. > > > > > > Please let me know. > > > > I won't say no but since the media maintainers will need to apply them, > > the problem is the same. > > Yeah. Me either. Those rules are for Outreachy so they don't apply to > anyone else, but at the same time no one is going to Ack your patches > without seeing them and especially for a subsystem they don't maintain. > > I always try to think about if maintainers will want my patches. Like > I sometimes see people returning -1 instead of -ENOMEM but, you know, > unless it reaches user space it's not technically a bug. You can't go > around fixing everyone's bad taste. But everyone likes bug fixes. I cannot say anything about the media subsystem, but for networking patches, i will be much more willing to help somebody going from Outreach to being a full kernel developers if they decide to take on something more than just some Coccinelle changes. If it looks like you have the hardware and want to make it better, have a vision where the driver should go, then you are likely to get more help from me. So if media drivers are what you interested in, go buy some hardware which uses one of the drivers in staging, look at the TODO file, and submit bigger patches than just -1 to -ENOMEM. Andrew _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: patches for staging:media drivers 2022-11-08 20:27 ` Andrew Lunn @ 2022-11-09 6:40 ` Deepak R Varma 2022-11-09 13:49 ` Andrew Lunn 0 siblings, 1 reply; 10+ messages in thread From: Deepak R Varma @ 2022-11-09 6:40 UTC (permalink / raw) To: Andrew Lunn Cc: Dan Carpenter, Neil Armstrong, Mauro Carvalho Chehab, Greg Kroah-Hartman, Kevin Hilman, Jerome Brunet, Martin Blumenstingl, linux-media, linux-amlogic, linux-staging, linux-arm-kernel, linux-kernel On Tue, Nov 08, 2022 at 09:27:32PM +0100, Andrew Lunn wrote: > On Tue, Nov 08, 2022 at 11:10:21AM +0300, Dan Carpenter wrote: > > I cannot say anything about the media subsystem, but for networking > patches, i will be much more willing to help somebody going from > Outreach to being a full kernel developers if they decide to take on > something more than just some Coccinelle changes. If it looks like you > have the hardware and want to make it better, have a vision where the > driver should go, then you are likely to get more help from me. So if > media drivers are what you interested in, go buy some hardware which > uses one of the drivers in staging, look at the TODO file, and submit > bigger patches than just -1 to -ENOMEM. Hello Andrew, Thank you so much for your kind advice and willingness to help a newbie become a valued kernel developer. I am committed to acquire necessary knowledge and skills to contribute more than current cleanup and minor correction patches. I am sure that the upcoming Coccinelle based project work [if I am shortlisted] will be much more complex and meaningful. I will explore the staging directory to identify a hardware I can afford. I would welcome suggestion on a driver that is not much expensive and complex to start with. Anything for the networking subsystem??? Thank you once again! ./drv > > Andrew > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: patches for staging:media drivers 2022-11-09 6:40 ` Deepak R Varma @ 2022-11-09 13:49 ` Andrew Lunn 2022-11-11 17:11 ` Deepak R Varma 0 siblings, 1 reply; 10+ messages in thread From: Andrew Lunn @ 2022-11-09 13:49 UTC (permalink / raw) To: Deepak R Varma Cc: Dan Carpenter, Neil Armstrong, Mauro Carvalho Chehab, Greg Kroah-Hartman, Kevin Hilman, Jerome Brunet, Martin Blumenstingl, linux-media, linux-amlogic, linux-staging, linux-arm-kernel, linux-kernel On Wed, Nov 09, 2022 at 12:10:37PM +0530, Deepak R Varma wrote: > On Tue, Nov 08, 2022 at 09:27:32PM +0100, Andrew Lunn wrote: > > On Tue, Nov 08, 2022 at 11:10:21AM +0300, Dan Carpenter wrote: > > > > I cannot say anything about the media subsystem, but for networking > > patches, i will be much more willing to help somebody going from > > Outreach to being a full kernel developers if they decide to take on > > something more than just some Coccinelle changes. If it looks like you > > have the hardware and want to make it better, have a vision where the > > driver should go, then you are likely to get more help from me. So if > > media drivers are what you interested in, go buy some hardware which > > uses one of the drivers in staging, look at the TODO file, and submit > > bigger patches than just -1 to -ENOMEM. > > Hello Andrew, > Thank you so much for your kind advice and willingness to help a newbie > become a valued kernel developer. I am committed to acquire necessary > knowledge and skills to contribute more than current cleanup and minor > correction patches. I am sure that the upcoming Coccinelle based project > work [if I am shortlisted] will be much more complex and meaningful. > > I will explore the staging directory to identify a hardware I can afford. > I would welcome suggestion on a driver that is not much expensive and > complex to start with. Anything for the networking subsystem??? Staging drivers for network tend to be USB dongles for WiFi. Some of those drivers need throwing away and starting again, others need less work. If you are interested in that, i would suggest asking on linux-wireless@vger.kernel.org. Another option might be pi433? The problem with that is, i don't know how it fits into the rest of the network stack? Could it maybe fit under ieee802154? Or is it just unique and needs to do its own thing? Andrew _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: patches for staging:media drivers 2022-11-09 13:49 ` Andrew Lunn @ 2022-11-11 17:11 ` Deepak R Varma 0 siblings, 0 replies; 10+ messages in thread From: Deepak R Varma @ 2022-11-11 17:11 UTC (permalink / raw) To: Andrew Lunn Cc: Dan Carpenter, Neil Armstrong, Mauro Carvalho Chehab, Greg Kroah-Hartman, Kevin Hilman, Jerome Brunet, Martin Blumenstingl, linux-media, linux-amlogic, linux-staging, linux-arm-kernel, linux-kernel On Wed, Nov 09, 2022 at 02:49:27PM +0100, Andrew Lunn wrote: > On Wed, Nov 09, 2022 at 12:10:37PM +0530, Deepak R Varma wrote: > > On Tue, Nov 08, 2022 at 09:27:32PM +0100, Andrew Lunn wrote: > > Staging drivers for network tend to be USB dongles for WiFi. Some of > those drivers need throwing away and starting again, others need less > work. If you are interested in that, i would suggest asking on > linux-wireless@vger.kernel.org. > > Another option might be pi433? The problem with that is, i don't know > how it fits into the rest of the network stack? Could it maybe fit > under ieee802154? Or is it just unique and needs to do its own thing? Hello Andrew, I looked at the pi433 driver and it appears to be a good starting point. I have started studying more about the hardware, the RF technology etc. There is a TODO file with reasonable challenges. Thank you for the recommendation. Have a nice weekend! ./drv > > Andrew > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2022-11-11 17:12 UTC | newest] Thread overview: 10+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2022-11-06 10:52 patches for staging:media drivers Deepak R Varma 2022-11-07 17:29 ` Neil Armstrong 2022-11-07 18:57 ` Deepak R Varma 2022-11-08 8:10 ` Dan Carpenter 2022-11-08 9:35 ` Hans Verkuil 2022-11-08 9:46 ` Dan Carpenter 2022-11-08 20:27 ` Andrew Lunn 2022-11-09 6:40 ` Deepak R Varma 2022-11-09 13:49 ` Andrew Lunn 2022-11-11 17:11 ` Deepak R Varma
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).