From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1765348AbcINVCR (ORCPT ); Wed, 14 Sep 2016 17:02:17 -0400 Received: from mail-wm0-f41.google.com ([74.125.82.41]:37458 "EHLO mail-wm0-f41.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1765254AbcINVCM (ORCPT ); Wed, 14 Sep 2016 17:02:12 -0400 From: Pali =?utf-8?q?Roh=C3=A1r?= To: Dmitry Torokhov , Hans de Goede Subject: Re: [PATCH 0/5] Input: alps - cleanup Date: Wed, 14 Sep 2016 23:02:08 +0200 User-Agent: KMail/1.13.7 (Linux/3.13.0-95-generic; KDE/4.14.2; x86_64; ; ) Cc: Ben Gamari , linux-input@vger.kernel.org, linux-kernel@vger.kernel.org References: <1465212241-14702-1-git-send-email-pali.rohar@gmail.com> <201607091158.04008@pali> <20160809075607.GY30047@pali> In-Reply-To: <20160809075607.GY30047@pali> MIME-Version: 1.0 Content-Type: multipart/signed; boundary="nextPart15046757.8xEGElCDbr"; protocol="application/pgp-signature"; micalg=pgp-sha1 Content-Transfer-Encoding: 7bit Message-Id: <201609142302.08232@pali> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --nextPart15046757.8xEGElCDbr Content-Type: Text/Plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable On Tuesday 09 August 2016 09:56:07 Pali Roh=C3=A1r wrote: > On Saturday 09 July 2016 11:58:03 Pali Roh=C3=A1r wrote: > > On Friday 08 July 2016 23:37:54 Dmitry Torokhov wrote: > > > On Thu, Jul 07, 2016 at 01:41:01PM +0200, Pali Roh=C3=A1r wrote: > > > > On Tuesday 21 June 2016 13:27:30 Pali Roh=C3=A1r wrote: > > > > > On Monday 20 June 2016 17:31:13 Dmitry Torokhov wrote: > > > > > > Hi Pali, > > > > > >=20 > > > > > > On Mon, Jun 06, 2016 at 01:23:56PM +0200, Pali Roh=C3=A1r wrote: > > > > > > > This patch series cleanup usage of alps_model_data table. > > > > > > >=20 > > > > > > > Pali Roh=C3=A1r (5): > > > > > > > Input: alps - move ALPS_PROTO_V6 out of alps_model_data > > > > > > > table Input: alps - move ALPS_PROTO_V4 out of > > > > > > > alps_model_data table Input: alps - move ALPS_PROTO_V1 > > > > > > > out of alps_model_data table Input: alps - warn about > > > > > > > unsupported ALPS V9 touchpad Input: alps - cleanup > > > > > > > ALPS_PROTO_V2 detection > > > > > >=20 > > > > > > Frankly, I do not quite like this series. The rule of thumb > > > > > > we had: if we can use e7 data to identify the device it > > > > > > should go into table, if we need to have more elaborate > > > > > > logic - then implement it in __alps_indentify(). I would > > > > > > understand if we got rid of the table completely, but we > > > > > > didn't. > > > > >=20 > > > > > Hans and me agreed that alps_model_data array is for old > > > > > touchpads defined as quirks table. So in this patch series > > > > > I'm trying to eliminate using that array. And it is possible > > > > > for V1, V4 and V6 touchpads because each protocol has only > > > > > one entry in table. And last user is just V2 protocol which > > > > > is I think better... > > > > >=20 > > > > > So this is my motivation for this patch series. > > > >=20 > > > > Any suggestion how to rework it? And any agreement if we should > > > > remove V1/V4/V6 from alps_model_date or let it stay here? > > >=20 > > > As I mentioned below I am happy with removing ALPS_PROTO_V4 and > > > subsequently command_mode_resp from alps_model_info, while > > > leaving the rest in the table. > >=20 > > Now I'm not fully understand what you mean. This patch series does > > not remove ALPS_PROTO_V4 support. Just move ALPS_PROTO_V4 out of > > alps_model_info table structure (same as for V1 and V6). Field > > command_mode_resp is removed from alps_model_info, but that can be > > done only because command_mode_resp is used by ALPS_PROTO_V4 > > (which is moved out of alps_model_info). > >=20 > > So I do not understand why do you think moving ALPS_PROTO_V4 out of > > alps_model_info is good, but ALPS_PROTO_V6 or ALPS_PROTO_V1 not. >=20 > Hi Dmitry! If you do not agree with my changes, please can you let me > know what is needed to rework or change? Thanks. Hi! Can you clarify above? Or tell me what is wrong? Hans already acked this series, but if there are some problems I can try=20 to fix them... =2D-=20 Pali Roh=C3=A1r pali.rohar@gmail.com --nextPart15046757.8xEGElCDbr Content-Type: application/pgp-signature; name=signature.asc Content-Description: This is a digitally signed message part. -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.11 (GNU/Linux) iEYEABECAAYFAlfZutAACgkQi/DJPQPkQ1IVdwCdGwp2cVTAr/Pi29CjZIn+oRqS rdkAn1iHd8xOTBS5Pa8XB/POUtMr+ST1 =SbhQ -----END PGP SIGNATURE----- --nextPart15046757.8xEGElCDbr--