From: Rob Herring <robh+dt@kernel.org> To: "H. Nikolaus Schaller" <hns@goldelico.com> Cc: "Pawel Moll" <pawel.moll@arm.com>, "Mark Rutland" <mark.rutland@arm.com>, "Ian Campbell" <ijc+devicetree@hellion.org.uk>, "Kumar Gala" <galak@codeaurora.org>, "Benoît Cousson" <bcousson@baylibre.com>, "Tony Lindgren" <tony@atomide.com>, "Russell King" <linux@arm.linux.org.uk>, "Dmitry Torokhov" <dmitry.torokhov@gmail.com>, "Hans Verkuil" <hans.verkuil@cisco.com>, "Mauro Carvalho Chehab" <mchehab@osg.samsung.com>, "Sebastian Reichel" <sre@kernel.org>, "Haibo Chen" <haibo.chen@freescale.com>, "Andrey Gelman" <andrey.gelman@compulab.co.il>, "Igor Grinberg" <grinberg@compulab.co.il>, "Aaron Sierra" <asierra@xes-inc.com>, "Krzysztof Kozlowski" <k.kozlowski@samsung.com>, "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, linux-omap <linux-omap@vger.kernel.org>, "List for communicating with real GTA04 owners" <gta04-owner@goldelico.com>, "linux-input@vger.kernel.org" <linux-input@vger.kernel.org> Subject: Re: [PATCH v2 6/8] drivers:input:ads7846(+tsc2046): recognise old binding for coordinate flipping Date: Sat, 14 Nov 2015 20:19:06 -0600 [thread overview] Message-ID: <CAL_JsqKXN1J-vkZm1f-4en4oa8Dz3c=xN_POCnDvzbAWqsPGfA@mail.gmail.com> (raw) In-Reply-To: <f036e0d2b019ae9463e108f48643c16a80936ddb.1447446957.git.hns@goldelico.com> On Fri, Nov 13, 2015 at 2:35 PM, H. Nikolaus Schaller <hns@goldelico.com> wrote: > By this patch we still recognise the old binding ti,swap-xy in parallel to > the common binding touchscreen-swapped-x-y. This keeps compatibility > to older (out-of-tree) device tree binaries. > > We do this in a separate patch so that it can be easily reverted in the > future to retire the old API. A notice is printed to remind developers > of using old API. > > We also fix the bindings name for all in-tree device tree sources in > a separate patch. This one and patch 5 should be combined, so the series is bisectable. Rob > > Signed-off-by: H. Nikolaus Schaller <hns@goldelico.com> > --- > drivers/input/touchscreen/ads7846.c | 6 +++++- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/drivers/input/touchscreen/ads7846.c b/drivers/input/touchscreen/ads7846.c > index 4525f00..b9896fd 100644 > --- a/drivers/input/touchscreen/ads7846.c > +++ b/drivers/input/touchscreen/ads7846.c > @@ -1259,7 +1259,11 @@ static const struct ads7846_platform_data *ads7846_probe_dt(struct device *dev) > of_property_read_u16(node, "ti,vref-mv", &pdata->vref_mv); > pdata->keep_vref_on = of_property_read_bool(node, "ti,keep-vref-on"); > > - pdata->swap_xy = of_property_read_bool(node, "touchscreen-swapped-x-y"); > + pdata->swap_xy = of_property_read_bool(node, "ti,swap-xy"); > + if (pdata->swap_xy) > + dev_notice(dev, "please update device tree to use touchscreen-swapped-x-y"); > + pdata->swap_xy |= of_property_read_bool(node, > + "touchscreen-swapped-x-y"); > > of_property_read_u16(node, "ti,settle-delay-usec", > &pdata->settle_delay_usecs); > -- > 2.5.1 >
WARNING: multiple messages have this Message-ID (diff)
From: Rob Herring <robh+dt@kernel.org> To: "H. Nikolaus Schaller" <hns@goldelico.com> Cc: "Pawel Moll" <pawel.moll@arm.com>, "Mark Rutland" <mark.rutland@arm.com>, "Ian Campbell" <ijc+devicetree@hellion.org.uk>, "Kumar Gala" <galak@codeaurora.org>, "Benoît Cousson" <bcousson@baylibre.com>, "Tony Lindgren" <tony@atomide.com>, "Russell King" <linux@arm.linux.org.uk>, "Dmitry Torokhov" <dmitry.torokhov@gmail.com>, "Hans Verkuil" <hans.verkuil@cisco.com>, "Mauro Carvalho Chehab" <mchehab@osg.samsung.com>, "Sebastian Reichel" <sre@kernel.org>, "Haibo Chen" <haibo.chen@freescale.com>, "Andrey Gelman" <andrey.gelman@compulab.co.il>, "Igor Grinberg" <grinberg@compulab.co.il>, "Aaron Sierra" <asierra@xes-inc.com>, "Krzysztof Kozlowski" <k.kozlowski@samsung.com>, "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, linux-omap <linux-omap@vger.kernel.org> Subject: Re: [PATCH v2 6/8] drivers:input:ads7846(+tsc2046): recognise old binding for coordinate flipping Date: Sat, 14 Nov 2015 20:19:06 -0600 [thread overview] Message-ID: <CAL_JsqKXN1J-vkZm1f-4en4oa8Dz3c=xN_POCnDvzbAWqsPGfA@mail.gmail.com> (raw) In-Reply-To: <f036e0d2b019ae9463e108f48643c16a80936ddb.1447446957.git.hns@goldelico.com> On Fri, Nov 13, 2015 at 2:35 PM, H. Nikolaus Schaller <hns@goldelico.com> wrote: > By this patch we still recognise the old binding ti,swap-xy in parallel to > the common binding touchscreen-swapped-x-y. This keeps compatibility > to older (out-of-tree) device tree binaries. > > We do this in a separate patch so that it can be easily reverted in the > future to retire the old API. A notice is printed to remind developers > of using old API. > > We also fix the bindings name for all in-tree device tree sources in > a separate patch. This one and patch 5 should be combined, so the series is bisectable. Rob > > Signed-off-by: H. Nikolaus Schaller <hns@goldelico.com> > --- > drivers/input/touchscreen/ads7846.c | 6 +++++- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/drivers/input/touchscreen/ads7846.c b/drivers/input/touchscreen/ads7846.c > index 4525f00..b9896fd 100644 > --- a/drivers/input/touchscreen/ads7846.c > +++ b/drivers/input/touchscreen/ads7846.c > @@ -1259,7 +1259,11 @@ static const struct ads7846_platform_data *ads7846_probe_dt(struct device *dev) > of_property_read_u16(node, "ti,vref-mv", &pdata->vref_mv); > pdata->keep_vref_on = of_property_read_bool(node, "ti,keep-vref-on"); > > - pdata->swap_xy = of_property_read_bool(node, "touchscreen-swapped-x-y"); > + pdata->swap_xy = of_property_read_bool(node, "ti,swap-xy"); > + if (pdata->swap_xy) > + dev_notice(dev, "please update device tree to use touchscreen-swapped-x-y"); > + pdata->swap_xy |= of_property_read_bool(node, > + "touchscreen-swapped-x-y"); > > of_property_read_u16(node, "ti,settle-delay-usec", > &pdata->settle_delay_usecs); > -- > 2.5.1 >
next prev parent reply other threads:[~2015-11-15 2:19 UTC|newest] Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-11-13 20:35 [PATCH v2 0/8] drivers: touchscreen: tsc2007 and ads7846/tsc2046 improvements (use common touchscreen bindings, pre-calibration, spi fix and provide iio raw values) H. Nikolaus Schaller 2015-11-13 20:35 ` [PATCH v2 1/8] drivers:input:tsc2007: add new common binding names, pre-calibration, flipping and rotation H. Nikolaus Schaller 2015-11-13 20:35 ` H. Nikolaus Schaller 2015-11-15 2:14 ` Rob Herring 2015-11-15 11:34 ` H. Nikolaus Schaller 2015-11-13 20:35 ` [PATCH v2 2/8] drivers:input:tsc2007: send pendown and penup only once like ads7846(+tsc2046) driver does H. Nikolaus Schaller 2015-11-13 20:35 ` [PATCH v2 3/8] drivers:input:tsc2007: add iio interface to read external ADC input, temperature and raw conversion values H. Nikolaus Schaller 2015-11-13 20:35 ` [PATCH v2 4/8] DT:omap3+tsc2007: use new common touchscreen bindings H. Nikolaus Schaller 2015-11-13 20:35 ` H. Nikolaus Schaller 2015-11-13 20:35 ` [PATCH v2 5/8] drivers:input:ads7846(+tsc2046): add new common binding names, pre-calibration and flipping H. Nikolaus Schaller 2015-11-13 22:29 ` Sebastian Reichel 2015-11-13 20:35 ` [PATCH v2 6/8] drivers:input:ads7846(+tsc2046): recognise old binding for coordinate flipping H. Nikolaus Schaller 2015-11-15 2:19 ` Rob Herring [this message] 2015-11-15 2:19 ` Rob Herring 2015-11-15 10:54 ` H. Nikolaus Schaller 2015-11-15 10:54 ` H. Nikolaus Schaller 2015-11-13 20:35 ` [PATCH v2 7/8] drivers:input:ads7846(+tsc2046): fix spi module table H. Nikolaus Schaller 2015-11-13 20:35 ` [PATCH v2 8/8] DT:omap3+ads7846: use new common touchscreen bindings H. Nikolaus Schaller 2015-11-16 14:37 ` Grazvydas Ignotas 2015-11-16 14:37 ` Grazvydas Ignotas 2015-11-16 15:31 ` H. Nikolaus Schaller 2015-11-16 15:31 ` H. Nikolaus Schaller
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to='CAL_JsqKXN1J-vkZm1f-4en4oa8Dz3c=xN_POCnDvzbAWqsPGfA@mail.gmail.com' \ --to=robh+dt@kernel.org \ --cc=andrey.gelman@compulab.co.il \ --cc=asierra@xes-inc.com \ --cc=bcousson@baylibre.com \ --cc=devicetree@vger.kernel.org \ --cc=dmitry.torokhov@gmail.com \ --cc=galak@codeaurora.org \ --cc=grinberg@compulab.co.il \ --cc=gta04-owner@goldelico.com \ --cc=haibo.chen@freescale.com \ --cc=hans.verkuil@cisco.com \ --cc=hns@goldelico.com \ --cc=ijc+devicetree@hellion.org.uk \ --cc=k.kozlowski@samsung.com \ --cc=linux-input@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-omap@vger.kernel.org \ --cc=linux@arm.linux.org.uk \ --cc=mark.rutland@arm.com \ --cc=mchehab@osg.samsung.com \ --cc=pawel.moll@arm.com \ --cc=sre@kernel.org \ --cc=tony@atomide.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.