From mboxrd@z Thu Jan 1 00:00:00 1970 From: Uwe =?iso-8859-1?Q?Kleine-K=F6nig?= Subject: Re: [PATCH v3] Input: rotary_encoder - support binary encoding of states Date: Fri, 8 Apr 2016 08:40:58 +0200 Message-ID: <20160408064058.GC10108@pengutronix.de> References: <1460055528-13814-1-git-send-email-u.kleine-koenig@pengutronix.de> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Content-Disposition: inline In-Reply-To: Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Rob Herring Cc: "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Dmitry Torokhov , "kernel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org" , "linux-input-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" List-Id: linux-input@vger.kernel.org Hello Rob, On Thu, Apr 07, 2016 at 05:45:00PM -0500, Rob Herring wrote: > On Thu, Apr 7, 2016 at 1:58 PM, Uwe Kleine-K=F6nig > wrote: > > It's not advisable to use this encoding, but to support existing de= vices > > add support for this to the driver. > > > > Signed-off-by: Uwe Kleine-K=F6nig > > --- >=20 > Ack for the binding, but... Thanks. Actually I already had your ack for v2 and only dropped it to make you look over the patch again :-) =20 > > @@ -213,6 +222,17 @@ static int rotary_encoder_probe(struct platfor= m_device *pdev) > > encoder->rollover =3D > > device_property_read_bool(dev, "rotary-encoder,roll= over"); > > > > + err =3D device_property_read_string(dev, "rotary-encoder,en= coding", > > + &encoding); > > + if (!err && !strcmp(encoding, "binary")) { > > + encoder->encoding =3D ROTENC_BINARY; > > + } else if (err || !strcmp(encoding, "gray")) { > > + encoder->encoding =3D ROTENC_GRAY; > > + } else { > > + dev_err(dev, "unknown encoding setting"); > > + return -EINVAL; >=20 > device_property_match_string() here instead. With the added downside that the property is parsed at least twice from dt, right? This is not a hot path, but still I wonder if it simplyfies the driver code enough to be worth the effort? Also this seems to have a different semantic and allows the property to be an array. Best regards Uwe --=20 Pengutronix e.K. | Uwe Kleine-K=F6nig = | Industrial Linux Solutions | http://www.pengutronix.de/= | -- To unsubscribe from this list: send the line "unsubscribe devicetree" i= n the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html