Re: [PATCH 04/22] input/ti_am33x_tsc: Order of TSC wires, made configurable
On 7/4/2013 7:20 PM, Sebastian Andrzej Siewior wrote: > On 07/04/2013 03:39 PM, Sekhar Nori wrote: >> Yes, I noticed that after sending the mail. To me it does not make sense >> to make changes to accept something as platform data only to remove >> platform data itself later. > > The patches were made earlier and it was easier that way to take > everything and simple remove the platform part later. Okay, then. If the maintainers do not have objection, I am fine too! Regards, Sekhar -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 04/22] input/ti_am33x_tsc: Order of TSC wires, made configurable
On 07/04/2013 03:39 PM, Sekhar Nori wrote: > Yes, I noticed that after sending the mail. To me it does not make sense > to make changes to accept something as platform data only to remove > platform data itself later. The patches were made earlier and it was easier that way to take everything and simple remove the platform part later. > May be reorder the series to move this to after platform data removal - > that way any platform data related changes in the patch will have to go > away. > > Thanks, > Sekhar Sebastian -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 04/22] input/ti_am33x_tsc: Order of TSC wires, made configurable
On 7/4/2013 5:03 PM, Sebastian Andrzej Siewior wrote: > On 07/04/2013 01:14 PM, Sekhar Nori wrote: >> >> On 6/11/2013 5:00 PM, Sebastian Andrzej Siewior wrote: >>> From: "Patil, Rachna" >>> >>> The current driver expected touchscreen input >>> wires(XP,XN,YP,YN) to be connected in a particular order. >>> Making changes to accept this as platform data >> >> The platform data part of this driver will never get used since it is >> used on DT-only platforms (and future platforms will all be DT-only). >> You should get rid of it as it will save you some code. > > If you follow the series you will notice that the platform bits are > removed later. Should I have overlooked something please say so. Yes, I noticed that after sending the mail. To me it does not make sense to make changes to accept something as platform data only to remove platform data itself later. May be reorder the series to move this to after platform data removal - that way any platform data related changes in the patch will have to go away. Thanks, Sekhar -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 04/22] input/ti_am33x_tsc: Order of TSC wires, made configurable
On 07/04/2013 01:14 PM, Sekhar Nori wrote: > > On 6/11/2013 5:00 PM, Sebastian Andrzej Siewior wrote: >> From: "Patil, Rachna" >> >> The current driver expected touchscreen input >> wires(XP,XN,YP,YN) to be connected in a particular order. >> Making changes to accept this as platform data > > The platform data part of this driver will never get used since it is > used on DT-only platforms (and future platforms will all be DT-only). > You should get rid of it as it will save you some code. If you follow the series you will notice that the platform bits are removed later. Should I have overlooked something please say so. > > Thanks, > Sekhar > Sebastian -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 04/22] input/ti_am33x_tsc: Order of TSC wires, made configurable
On 6/11/2013 5:00 PM, Sebastian Andrzej Siewior wrote: > From: "Patil, Rachna" > > The current driver expected touchscreen input > wires(XP,XN,YP,YN) to be connected in a particular order. > Making changes to accept this as platform data The platform data part of this driver will never get used since it is used on DT-only platforms (and future platforms will all be DT-only). You should get rid of it as it will save you some code. Thanks, Sekhar -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 04/22] input/ti_am33x_tsc: Order of TSC wires, made configurable
On 06/11/2013 04:23 PM, Samuel Ortiz wrote: > Hi Sebastian, Hi Samuel, > On Tue, Jun 11, 2013 at 01:30:50PM +0200, Sebastian Andrzej Siewior wrote: >> diff --git a/include/linux/mfd/ti_am335x_tscadc.h >> b/include/linux/mfd/ti_am335x_tscadc.h >> index eeead15..2d78af8 100644 >> --- a/include/linux/mfd/ti_am335x_tscadc.h >> +++ b/include/linux/mfd/ti_am335x_tscadc.h >> @@ -116,6 +113,13 @@ >> #define CNTRLREG_8WIRE CNTRLREG_AFE_CTRL(3) >> #define CNTRLREG_TSCENB BIT(7) >> >> +#define XPP STEPCONFIG_XPP >> +#define XNP STEPCONFIG_XNP >> +#define XNN STEPCONFIG_XNN >> +#define YPP STEPCONFIG_YPP >> +#define YPN STEPCONFIG_YPN >> +#define YNN STEPCONFIG_YNN > What is that for ? Isn't STEPCONFIG_XPP explicit enough ? Yeah :P It was added by the original author of the patch, I have no problem getting rid of it. > > Cheers, > Samuel. Sebastian -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 04/22] input/ti_am33x_tsc: Order of TSC wires, made configurable
Hi Sebastian, On Tue, Jun 11, 2013 at 01:30:50PM +0200, Sebastian Andrzej Siewior wrote: > diff --git a/include/linux/mfd/ti_am335x_tscadc.h > b/include/linux/mfd/ti_am335x_tscadc.h > index eeead15..2d78af8 100644 > --- a/include/linux/mfd/ti_am335x_tscadc.h > +++ b/include/linux/mfd/ti_am335x_tscadc.h > @@ -71,8 +71,6 @@ > #define STEPCONFIG_INM_ADCREFM STEPCONFIG_INM(8) > #define STEPCONFIG_INP_MASK (0xF << 19) > #define STEPCONFIG_INP(val) ((val) << 19) > -#define STEPCONFIG_INP_AN2 STEPCONFIG_INP(2) > -#define STEPCONFIG_INP_AN3 STEPCONFIG_INP(3) > #define STEPCONFIG_INP_AN4 STEPCONFIG_INP(4) > #define STEPCONFIG_INP_ADCREFM STEPCONFIG_INP(8) > #define STEPCONFIG_FIFO1 BIT(26) > @@ -94,7 +92,6 @@ > #define STEPCHARGE_INM_AN1 STEPCHARGE_INM(1) > #define STEPCHARGE_INP_MASK (0xF << 19) > #define STEPCHARGE_INP(val) ((val) << 19) > -#define STEPCHARGE_INP_AN1 STEPCHARGE_INP(1) > #define STEPCHARGE_RFM_MASK (3 << 23) > #define STEPCHARGE_RFM(val) ((val) << 23) > #define STEPCHARGE_RFM_XNUR STEPCHARGE_RFM(1) > @@ -116,6 +113,13 @@ > #define CNTRLREG_8WIRE CNTRLREG_AFE_CTRL(3) > #define CNTRLREG_TSCENB BIT(7) > > +#define XPP STEPCONFIG_XPP > +#define XNP STEPCONFIG_XNP > +#define XNN STEPCONFIG_XNN > +#define YPP STEPCONFIG_YPP > +#define YPN STEPCONFIG_YPN > +#define YNN STEPCONFIG_YNN What is that for ? Isn't STEPCONFIG_XPP explicit enough ? Cheers, Samuel. -- Intel Open Source Technology Centre http://oss.intel.com/ -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html