Re: [PATCH] mc13xxx-ts: use zero as default value if no pdata was defined
On Tue, Aug 20, 2013 at 11:34:32AM +0200, Michael Grzeschik wrote: > On Tue, Aug 20, 2013 at 03:34:33AM +0200, Samuel Ortiz wrote: > > Hi Michael, > > > > On Tue, Aug 13, 2013 at 02:14:30PM +0200, Michael Grzeschik wrote: > > > In case of devicetree, we currently don't have a way to append pdata for > > > the touchscreen. The current approach is to bail out in that case. > > > This patch makes it possible to probe the touchscreen without pdata > > > and use zero as default values for the atox and ato adc conversion. > > Would that still make the touchscreen somehow functional ? > > Yes, it still works. It just defaults to zero values in no pdata case. > > > If that's the case, and if Dmitry is fine with the ts part of this > > patch, could you please separate the mfd part of this patch into a > > separate one ? > > Why? > > The purpose of this patch is to have both cases working, pdata and no > pdata. My concern with allowing defaults with missing pdata or device tree data is that it makes it easy for the integrator to miss the necessity of the parameters and then start piling on driver workarounds. I have seen a few examples of this happening, so I'd rather prefer an explicit values if possible. Thanks. -- Dmitry -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] mc13xxx-ts: use zero as default value if no pdata was defined
On Tue, Aug 20, 2013 at 11:50:48AM +0200, Samuel Ortiz wrote: > Hi Michael, > > On Tue, Aug 20, 2013 at 11:34:32AM +0200, Michael Grzeschik wrote: > > On Tue, Aug 20, 2013 at 03:34:33AM +0200, Samuel Ortiz wrote: > > > On Tue, Aug 13, 2013 at 02:14:30PM +0200, Michael Grzeschik wrote: > > > > In case of devicetree, we currently don't have a way to append pdata for > > > > the touchscreen. The current approach is to bail out in that case. > > > > This patch makes it possible to probe the touchscreen without pdata > > > > and use zero as default values for the atox and ato adc conversion. > > > Would that still make the touchscreen somehow functional ? > > > > Yes, it still works. It just defaults to zero values in no pdata case. > > > > > If that's the case, and if Dmitry is fine with the ts part of this > > > patch, could you please separate the mfd part of this patch into a > > > separate one ? > > > > Why? > To avoid cross tree commits as much as possible. > > > > The purpose of this patch is to have both cases working, pdata and no > > pdata. > I understood that part. But you can still split the patch in 2 as the ts > driver will not be probed without pdata until the mfd patch is merged > upstream. > > > This patch is actually fixing the patch of Michael Thalmeier. > > There we changed the mfd and touch in one patch as well. > There was a build time dependency between the ts driver and the MFD > changes, so the patch had to be merged atomically. > That is not the case with your patch. All right then. I will resend them and the codec mfd patch together with the oftree patches I am currently working on. Thanks, Michael -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0| Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917- | -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] mc13xxx-ts: use zero as default value if no pdata was defined
Hi Michael, On Tue, Aug 20, 2013 at 11:34:32AM +0200, Michael Grzeschik wrote: > On Tue, Aug 20, 2013 at 03:34:33AM +0200, Samuel Ortiz wrote: > > On Tue, Aug 13, 2013 at 02:14:30PM +0200, Michael Grzeschik wrote: > > > In case of devicetree, we currently don't have a way to append pdata for > > > the touchscreen. The current approach is to bail out in that case. > > > This patch makes it possible to probe the touchscreen without pdata > > > and use zero as default values for the atox and ato adc conversion. > > Would that still make the touchscreen somehow functional ? > > Yes, it still works. It just defaults to zero values in no pdata case. > > > If that's the case, and if Dmitry is fine with the ts part of this > > patch, could you please separate the mfd part of this patch into a > > separate one ? > > Why? To avoid cross tree commits as much as possible. > The purpose of this patch is to have both cases working, pdata and no > pdata. I understood that part. But you can still split the patch in 2 as the ts driver will not be probed without pdata until the mfd patch is merged upstream. > This patch is actually fixing the patch of Michael Thalmeier. > There we changed the mfd and touch in one patch as well. There was a build time dependency between the ts driver and the MFD changes, so the patch had to be merged atomically. That is not the case with your patch. Cheers, Samuel. -- Intel Open Source Technology Centre http://oss.intel.com/ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] mc13xxx-ts: use zero as default value if no pdata was defined
On Tue, Aug 20, 2013 at 03:34:33AM +0200, Samuel Ortiz wrote: > Hi Michael, > > On Tue, Aug 13, 2013 at 02:14:30PM +0200, Michael Grzeschik wrote: > > In case of devicetree, we currently don't have a way to append pdata for > > the touchscreen. The current approach is to bail out in that case. > > This patch makes it possible to probe the touchscreen without pdata > > and use zero as default values for the atox and ato adc conversion. > Would that still make the touchscreen somehow functional ? Yes, it still works. It just defaults to zero values in no pdata case. > If that's the case, and if Dmitry is fine with the ts part of this > patch, could you please separate the mfd part of this patch into a > separate one ? Why? The purpose of this patch is to have both cases working, pdata and no pdata. This patch is actually fixing the patch of Michael Thalmeier. There we changed the mfd and touch in one patch as well. http://lkml.indiana.edu/hypermail/linux/kernel/1201.1/01364.html Thanks, Michael -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0| Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917- | -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] mc13xxx-ts: use zero as default value if no pdata was defined
On Tue, Aug 20, 2013 at 03:34:33AM +0200, Samuel Ortiz wrote: Hi Michael, On Tue, Aug 13, 2013 at 02:14:30PM +0200, Michael Grzeschik wrote: In case of devicetree, we currently don't have a way to append pdata for the touchscreen. The current approach is to bail out in that case. This patch makes it possible to probe the touchscreen without pdata and use zero as default values for the atox and ato adc conversion. Would that still make the touchscreen somehow functional ? Yes, it still works. It just defaults to zero values in no pdata case. If that's the case, and if Dmitry is fine with the ts part of this patch, could you please separate the mfd part of this patch into a separate one ? Why? The purpose of this patch is to have both cases working, pdata and no pdata. This patch is actually fixing the patch of Michael Thalmeier. There we changed the mfd and touch in one patch as well. http://lkml.indiana.edu/hypermail/linux/kernel/1201.1/01364.html Thanks, Michael -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0| Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917- | -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] mc13xxx-ts: use zero as default value if no pdata was defined
Hi Michael, On Tue, Aug 20, 2013 at 11:34:32AM +0200, Michael Grzeschik wrote: On Tue, Aug 20, 2013 at 03:34:33AM +0200, Samuel Ortiz wrote: On Tue, Aug 13, 2013 at 02:14:30PM +0200, Michael Grzeschik wrote: In case of devicetree, we currently don't have a way to append pdata for the touchscreen. The current approach is to bail out in that case. This patch makes it possible to probe the touchscreen without pdata and use zero as default values for the atox and ato adc conversion. Would that still make the touchscreen somehow functional ? Yes, it still works. It just defaults to zero values in no pdata case. If that's the case, and if Dmitry is fine with the ts part of this patch, could you please separate the mfd part of this patch into a separate one ? Why? To avoid cross tree commits as much as possible. The purpose of this patch is to have both cases working, pdata and no pdata. I understood that part. But you can still split the patch in 2 as the ts driver will not be probed without pdata until the mfd patch is merged upstream. This patch is actually fixing the patch of Michael Thalmeier. There we changed the mfd and touch in one patch as well. There was a build time dependency between the ts driver and the MFD changes, so the patch had to be merged atomically. That is not the case with your patch. Cheers, Samuel. -- Intel Open Source Technology Centre http://oss.intel.com/ -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] mc13xxx-ts: use zero as default value if no pdata was defined
On Tue, Aug 20, 2013 at 11:50:48AM +0200, Samuel Ortiz wrote: Hi Michael, On Tue, Aug 20, 2013 at 11:34:32AM +0200, Michael Grzeschik wrote: On Tue, Aug 20, 2013 at 03:34:33AM +0200, Samuel Ortiz wrote: On Tue, Aug 13, 2013 at 02:14:30PM +0200, Michael Grzeschik wrote: In case of devicetree, we currently don't have a way to append pdata for the touchscreen. The current approach is to bail out in that case. This patch makes it possible to probe the touchscreen without pdata and use zero as default values for the atox and ato adc conversion. Would that still make the touchscreen somehow functional ? Yes, it still works. It just defaults to zero values in no pdata case. If that's the case, and if Dmitry is fine with the ts part of this patch, could you please separate the mfd part of this patch into a separate one ? Why? To avoid cross tree commits as much as possible. The purpose of this patch is to have both cases working, pdata and no pdata. I understood that part. But you can still split the patch in 2 as the ts driver will not be probed without pdata until the mfd patch is merged upstream. This patch is actually fixing the patch of Michael Thalmeier. There we changed the mfd and touch in one patch as well. There was a build time dependency between the ts driver and the MFD changes, so the patch had to be merged atomically. That is not the case with your patch. All right then. I will resend them and the codec mfd patch together with the oftree patches I am currently working on. Thanks, Michael -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0| Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917- | -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] mc13xxx-ts: use zero as default value if no pdata was defined
On Tue, Aug 20, 2013 at 11:34:32AM +0200, Michael Grzeschik wrote: On Tue, Aug 20, 2013 at 03:34:33AM +0200, Samuel Ortiz wrote: Hi Michael, On Tue, Aug 13, 2013 at 02:14:30PM +0200, Michael Grzeschik wrote: In case of devicetree, we currently don't have a way to append pdata for the touchscreen. The current approach is to bail out in that case. This patch makes it possible to probe the touchscreen without pdata and use zero as default values for the atox and ato adc conversion. Would that still make the touchscreen somehow functional ? Yes, it still works. It just defaults to zero values in no pdata case. If that's the case, and if Dmitry is fine with the ts part of this patch, could you please separate the mfd part of this patch into a separate one ? Why? The purpose of this patch is to have both cases working, pdata and no pdata. My concern with allowing defaults with missing pdata or device tree data is that it makes it easy for the integrator to miss the necessity of the parameters and then start piling on driver workarounds. I have seen a few examples of this happening, so I'd rather prefer an explicit values if possible. Thanks. -- Dmitry -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] mc13xxx-ts: use zero as default value if no pdata was defined
Hi Michael, On Tue, Aug 13, 2013 at 02:14:30PM +0200, Michael Grzeschik wrote: > In case of devicetree, we currently don't have a way to append pdata for > the touchscreen. The current approach is to bail out in that case. > This patch makes it possible to probe the touchscreen without pdata > and use zero as default values for the atox and ato adc conversion. Would that still make the touchscreen somehow functional ? If that's the case, and if Dmitry is fine with the ts part of this patch, could you please separate the mfd part of this patch into a separate one ? Cheers, Samuel. -- Intel Open Source Technology Centre http://oss.intel.com/ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] mc13xxx-ts: use zero as default value if no pdata was defined
Hi Michael, On Tue, Aug 13, 2013 at 02:14:30PM +0200, Michael Grzeschik wrote: In case of devicetree, we currently don't have a way to append pdata for the touchscreen. The current approach is to bail out in that case. This patch makes it possible to probe the touchscreen without pdata and use zero as default values for the atox and ato adc conversion. Would that still make the touchscreen somehow functional ? If that's the case, and if Dmitry is fine with the ts part of this patch, could you please separate the mfd part of this patch into a separate one ? Cheers, Samuel. -- Intel Open Source Technology Centre http://oss.intel.com/ -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] mc13xxx-ts: use zero as default value if no pdata was defined
Hi Dimitry, On Tue, Aug 13, 2013 at 09:46:09AM -0700, Dmitry Torokhov wrote: > Hi Michael, > > On Tue, Aug 13, 2013 at 02:14:30PM +0200, Michael Grzeschik wrote: > > In case of devicetree, we currently don't have a way to append pdata for > > the touchscreen. The current approach is to bail out in that case. > > This patch makes it possible to probe the touchscreen without pdata > > and use zero as default values for the atox and ato adc conversion. > > I'd rather you added the devicetree support to the driver. I know that we will need real devictree glue that generates pdata in the long run. I am working on that. Beside that, for now this patch makes sense anyway. Regards, Michael -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0| Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917- | -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] mc13xxx-ts: use zero as default value if no pdata was defined
Hi Michael, On Tue, Aug 13, 2013 at 02:14:30PM +0200, Michael Grzeschik wrote: > In case of devicetree, we currently don't have a way to append pdata for > the touchscreen. The current approach is to bail out in that case. > This patch makes it possible to probe the touchscreen without pdata > and use zero as default values for the atox and ato adc conversion. I'd rather you added the devicetree support to the driver. Thanks. -- Dmitry -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] mc13xxx-ts: use zero as default value if no pdata was defined
Hi Michael, On Tue, Aug 13, 2013 at 02:14:30PM +0200, Michael Grzeschik wrote: In case of devicetree, we currently don't have a way to append pdata for the touchscreen. The current approach is to bail out in that case. This patch makes it possible to probe the touchscreen without pdata and use zero as default values for the atox and ato adc conversion. I'd rather you added the devicetree support to the driver. Thanks. -- Dmitry -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH] mc13xxx-ts: use zero as default value if no pdata was defined
Hi Dimitry, On Tue, Aug 13, 2013 at 09:46:09AM -0700, Dmitry Torokhov wrote: Hi Michael, On Tue, Aug 13, 2013 at 02:14:30PM +0200, Michael Grzeschik wrote: In case of devicetree, we currently don't have a way to append pdata for the touchscreen. The current approach is to bail out in that case. This patch makes it possible to probe the touchscreen without pdata and use zero as default values for the atox and ato adc conversion. I'd rather you added the devicetree support to the driver. I know that we will need real devictree glue that generates pdata in the long run. I am working on that. Beside that, for now this patch makes sense anyway. Regards, Michael -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0| Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917- | -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/