Re: [PATCH 02/14] dwc/otg: Structure declaration for shared data
I will send you those patches. But, how to go about it? Will you send next revision of your patces soon, and will also include comments for current review? Or, I replly all the comments from community and get agreed about modifications first. Then I will release next revision with all these modifications and get it included. Once these patches are included then we can keep on sending further patches over those included. This will shorten review time and will be easier for inclusion. Regards Pratyush On Wed, Aug 31, 2011 at 10:47 PM, Tirumala Marri tma...@apm.com wrote: Tirumala, If you agree , then I can send you modifications which I did over your patches(v13) separately, and then you can decide the final inclusion of only these modifications. [Tirumala Marri] Sounds like a plan. Could you send the changes, I will take A look at the changes. --marri ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
Re: [PATCH 02/14] dwc/otg: Structure declaration for shared data
On Thu, Sep 1, 2011 at 4:06 AM, Greg KH g...@kroah.com wrote: On Tue, Aug 30, 2011 at 05:27:49PM +0530, Pratyush Anand wrote: There are some DWC OTG parameters which might be passed by a platform. Declaration for structure of those parameters have been provided in this include file. Signed-off-by: Pratyush Anand pratyush.an...@st.com --- include/linux/usb/dwc_otg.h | 274 +++ 1 files changed, 274 insertions(+), 0 deletions(-) create mode 100644 include/linux/usb/dwc_otg.h diff --git a/include/linux/usb/dwc_otg.h b/include/linux/usb/dwc_otg.h Shouldn't this be in include/linux/platform/ instead? do you mean linux/platform_data/? seems better place. Regards Pratyush -- To unsubscribe from this list: send the line unsubscribe linux-usb in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
Re: [PATCH 02/14] dwc/otg: Structure declaration for shared data
В Fri, 2 Sep 2011 17:12:17 +0530 Pratyush Anand pratyush.an...@gmail.com пишет: I will send you those patches. But, how to go about it? Will you send next revision of your patces soon, and will also include comments for current review? Or, I replly all the comments from community and get agreed about modifications first. Then I will release next revision with all these modifications and get it included. Once these patches are included then we can keep on sending further patches over those included. This will shorten review time and will be easier for inclusion. What do you think about making a public git repository on github, for example? I'm using dwc_otg on several MIPS boards and would be happy to contribute too (at least, fix things pointed out by reviewers to speed up inclusion). -- Alexander signature.asc Description: PGP signature ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
RE: [PATCH 02/14] dwc/otg: Structure declaration for shared data
I will send you those patches. But, how to go about it? Will you send next revision of your patces soon, and will also include comments for current review? Or, I replly all the comments from community and get agreed about modifications first. Then I will release next revision with all these modifications and get it included. Once these patches are included then we can keep on sending further patches over those included. This will shorten review time and will be easier for inclusion. [Tirumala Marri] We did 13 submission before and not expecting acceptance any time soon. Let us see what we can include From your changes. Regards, Marri ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
RE: [PATCH 02/14] dwc/otg: Structure declaration for shared data
Pratyush Anand pratyush.an...@gmail.com пишет: I will send you those patches. But, how to go about it? Will you send next revision of your patces soon, and will also include comments for current review? Or, I replly all the comments from community and get agreed about modifications first. Then I will release next revision with all these modifications and get it included. Once these patches are included then we can keep on sending further patches over those included. This will shorten review time and will be easier for inclusion. What do you think about making a public git repository on github, for example? I'm using dwc_otg on several MIPS boards and would be happy to contribute too (at least, fix things pointed out by reviewers to speed up inclusion). [Tirumala Marri] I can take a look at this and setup a public git repository. We started the initial dwc_otg submissions and planning to continue it. Regards, Marri ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
Re: [PATCH 02/14] dwc/otg: Structure declaration for shared data
On Fri, Sep 02, 2011 at 05:19:17PM +0530, Pratyush Anand wrote: On Thu, Sep 1, 2011 at 4:06 AM, Greg KH g...@kroah.com wrote: On Tue, Aug 30, 2011 at 05:27:49PM +0530, Pratyush Anand wrote: There are some DWC OTG parameters which might be passed by a platform. Declaration for structure of those parameters have been provided in this include file. Signed-off-by: Pratyush Anand pratyush.an...@st.com --- include/linux/usb/dwc_otg.h | 274 +++ 1 files changed, 274 insertions(+), 0 deletions(-) create mode 100644 include/linux/usb/dwc_otg.h diff --git a/include/linux/usb/dwc_otg.h b/include/linux/usb/dwc_otg.h Shouldn't this be in include/linux/platform/ instead? do you mean linux/platform_data/? Yes, sorry. ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
RE: [PATCH 02/14] dwc/otg: Structure declaration for shared data
Tirumala, If you agree , then I can send you modifications which I did over your patches(v13) separately, and then you can decide the final inclusion of only these modifications. [Tirumala Marri] Sounds like a plan. Could you send the changes, I will take A look at the changes. --marri ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
Re: [PATCH 02/14] dwc/otg: Structure declaration for shared data
On Tue, Aug 30, 2011 at 05:27:49PM +0530, Pratyush Anand wrote: There are some DWC OTG parameters which might be passed by a platform. Declaration for structure of those parameters have been provided in this include file. Signed-off-by: Pratyush Anand pratyush.an...@st.com --- include/linux/usb/dwc_otg.h | 274 +++ 1 files changed, 274 insertions(+), 0 deletions(-) create mode 100644 include/linux/usb/dwc_otg.h diff --git a/include/linux/usb/dwc_otg.h b/include/linux/usb/dwc_otg.h Shouldn't this be in include/linux/platform/ instead? ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
Re: [PATCH 02/14] dwc/otg: Structure declaration for shared data
On Tue, Aug 30, 2011 at 08:29:21AM -0700, Tirumala Marri wrote: -Original Message- From: Pratyush Anand [mailto:pratyush.an...@st.com] Sent: Tuesday, August 30, 2011 4:58 AM To: linux-...@vger.kernel.org Cc: tma...@apm.com; linuxppc-dev@lists.ozlabs.org; fc...@apm.com; mmiesf...@apm.com; shiraz.has...@st.com; deepak.si...@st.com; vipulkumar.sa...@st.com; rajeev-dlh.ku...@st.com; vipin.ku...@st.com; bhupesh.sha...@st.com; viresh.ku...@st.com; amit.vi...@st.com; Pratyush Anand Subject: [PATCH 02/14] dwc/otg: Structure declaration for shared data There are some DWC OTG parameters which might be passed by a platform. Declaration for structure of those parameters have been provided in this include file. Signed-off-by: Pratyush Anand pratyush.an...@st.com --- include/linux/usb/dwc_otg.h | 274 [Tirumala Marri] I am not sure how to separate your changes. But we need More time as our initial patches are still pending. pending where? Have you submitted them for inclusion? confused, greg k-h ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
Re: [PATCH 02/14] dwc/otg: Structure declaration for shared data
On Wed, Aug 31, 2011 at 6:36 PM, Greg KH g...@kroah.com wrote: On Tue, Aug 30, 2011 at 08:29:21AM -0700, Tirumala Marri wrote: -Original Message- From: Pratyush Anand [mailto:pratyush.an...@st.com] Sent: Tuesday, August 30, 2011 4:58 AM To: linux-...@vger.kernel.org Cc: tma...@apm.com; linuxppc-dev@lists.ozlabs.org; fc...@apm.com; mmiesf...@apm.com; shiraz.has...@st.com; deepak.si...@st.com; vipulkumar.sa...@st.com; rajeev-dlh.ku...@st.com; vipin.ku...@st.com; bhupesh.sha...@st.com; viresh.ku...@st.com; amit.vi...@st.com; Pratyush Anand Subject: [PATCH 02/14] dwc/otg: Structure declaration for shared data There are some DWC OTG parameters which might be passed by a platform. Declaration for structure of those parameters have been provided in this include file. Signed-off-by: Pratyush Anand pratyush.an...@st.com --- include/linux/usb/dwc_otg.h | 274 [Tirumala Marri] I am not sure how to separate your changes. But we need More time as our initial patches are still pending. pending where? Have you submitted them for inclusion? 13 times in fact. Each time, more comments and fixes need. Par for the course with a vendor driver, but in this case Pratyush thought the APM guys had let it die and tried to carry it forward. I guess APM has been sitting on it for more than 4 months now. Anyway, hope that clears up some of the confusion. josh ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
Re: [PATCH 02/14] dwc/otg: Structure declaration for shared data
On Wed, Aug 31, 2011 at 08:18:12PM -0400, Josh Boyer wrote: On Wed, Aug 31, 2011 at 6:36 PM, Greg KH g...@kroah.com wrote: On Tue, Aug 30, 2011 at 08:29:21AM -0700, Tirumala Marri wrote: -Original Message- From: Pratyush Anand [mailto:pratyush.an...@st.com] Sent: Tuesday, August 30, 2011 4:58 AM To: linux-...@vger.kernel.org Cc: tma...@apm.com; linuxppc-dev@lists.ozlabs.org; fc...@apm.com; mmiesf...@apm.com; shiraz.has...@st.com; deepak.si...@st.com; vipulkumar.sa...@st.com; rajeev-dlh.ku...@st.com; vipin.ku...@st.com; bhupesh.sha...@st.com; viresh.ku...@st.com; amit.vi...@st.com; Pratyush Anand Subject: [PATCH 02/14] dwc/otg: Structure declaration for shared data There are some DWC OTG parameters which might be passed by a platform. Declaration for structure of those parameters have been provided in this include file. Signed-off-by: Pratyush Anand pratyush.an...@st.com --- include/linux/usb/dwc_otg.h | 274 [Tirumala Marri] I am not sure how to separate your changes. But we need More time as our initial patches are still pending. pending where? Have you submitted them for inclusion? 13 times in fact. Each time, more comments and fixes need. Par for the course with a vendor driver, but in this case Pratyush thought the APM guys had let it die and tried to carry it forward. I guess APM has been sitting on it for more than 4 months now. Anyway, hope that clears up some of the confusion. It does, thanks. greg k-h ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
RE: [PATCH 02/14] dwc/otg: Structure declaration for shared data
-Original Message- From: Pratyush Anand [mailto:pratyush.an...@st.com] Sent: Tuesday, August 30, 2011 4:58 AM To: linux-...@vger.kernel.org Cc: tma...@apm.com; linuxppc-dev@lists.ozlabs.org; fc...@apm.com; mmiesf...@apm.com; shiraz.has...@st.com; deepak.si...@st.com; vipulkumar.sa...@st.com; rajeev-dlh.ku...@st.com; vipin.ku...@st.com; bhupesh.sha...@st.com; viresh.ku...@st.com; amit.vi...@st.com; Pratyush Anand Subject: [PATCH 02/14] dwc/otg: Structure declaration for shared data There are some DWC OTG parameters which might be passed by a platform. Declaration for structure of those parameters have been provided in this include file. Signed-off-by: Pratyush Anand pratyush.an...@st.com --- include/linux/usb/dwc_otg.h | 274 [Tirumala Marri] I am not sure how to separate your changes. But we need More time as our initial patches are still pending. --marri ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
Re: [PATCH 02/14] dwc/otg: Structure declaration for shared data
On Tue, Aug 30, 2011 at 8:59 PM, Tirumala Marri tma...@apm.com wrote: -Original Message- From: Pratyush Anand [mailto:pratyush.an...@st.com] Sent: Tuesday, August 30, 2011 4:58 AM To: linux-...@vger.kernel.org Cc: tma...@apm.com; linuxppc-dev@lists.ozlabs.org; fc...@apm.com; mmiesf...@apm.com; shiraz.has...@st.com; deepak.si...@st.com; vipulkumar.sa...@st.com; rajeev-dlh.ku...@st.com; vipin.ku...@st.com; bhupesh.sha...@st.com; viresh.ku...@st.com; amit.vi...@st.com; Pratyush Anand Subject: [PATCH 02/14] dwc/otg: Structure declaration for shared data There are some DWC OTG parameters which might be passed by a platform. Declaration for structure of those parameters have been provided in this include file. Signed-off-by: Pratyush Anand pratyush.an...@st.com --- include/linux/usb/dwc_otg.h | 274 [Tirumala Marri] I am not sure how to separate your changes. But we need More time as our initial patches are still pending. --marri Tirumala, If you agree , then I can send you modifications which I did over your patches(v13) separately, and then you can decide the final inclusion of only these modifications. Regards Pratyush -- To unsubscribe from this list: send the line unsubscribe linux-usb in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
