From:   Chester Lin <clin@suse.com>
To:     Linus Walleij <linus.walleij@linaro.org>,
        Andy Shevchenko <andy.shevchenko@gmail.com>
Cc:     Chester Lin <clin@suse.com>, NXP S32 Linux Team <s32@nxp.com>,
        linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org,
        Ghennadi Procopciuc <Ghennadi.Procopciuc@oss.nxp.com>,
        Andrei Stefanescu <andrei.stefanescu@nxp.com>,
        Radu Pirea <radu-nicolae.pirea@nxp.com>,
        =?UTF-8?q?Andreas=20F=C3=A4rber?= <afaerber@suse.de>,
        Matthias Brugger <mbrugger@suse.com>
Subject: [PATCH v2 0/4] pinctrl: s32: driver improvements and generic struct use
Date:   Tue, 21 Mar 2023 00:38:19 +0800
Message-Id: <20230320163823.886-1-clin@suse.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?0/JkugyoDtCHCy0Q3oNlK6KztS6jI7RHuEi+t8phK9NplEneLQeEmYKEPIj3?=
 =?us-ascii?Q?scXVzkn+JeNvMp1vqK7L1GgKxKb64bhWAd1vUsmMYDZ5wNeyfxWqI4JzlWA7?=
 =?us-ascii?Q?PIYoe8P/ljCVPJ6d4CM/Ytk4kzLSlZ7ys5yi+dYKaQxLmc2XB/rTL6PxpAOa?=
 =?us-ascii?Q?ERccH3ozSne4boPhEIxTAB9p0xNueWJ1T9iqO/jmZHurv3ptSVPoqozd1XZ/?=
 =?us-ascii?Q?67IeQe717sYlmazQVhGLER+cuI3ZJRaHPB68NC9nbAQZ65kgutQrBCCfCmvh?=
 =?us-ascii?Q?a5ZIzx+e4TICF8HGiLjh0MrsxATvjTU3LKraNEPu2QmwcdGNy4MT+fQS4jHW?=
 =?us-ascii?Q?1S/VRWawXDHCLr8eY2Uri+Bes116WHgZ3ic4K0sevZdwUJjlYF+iydozo7xp?=
 =?us-ascii?Q?4vFCUObsi/DEM+Flalsg5awxbf/5R55wpf1bIXUsQBmEekiO4m3+A1afg9jD?=
 =?us-ascii?Q?qxNdPzn6DMWkXPFHPEPV+9iACwUxd8tsMA59Z6AO3iSCg/nh5cbuw4+yiSKT?=
 =?us-ascii?Q?O2aCslbTXrihqjaDPb0uco1L1Y8vUPvm5ddxxD//VhEZ2eEB3uOvmfQdTDsQ?=
 =?us-ascii?Q?Cf+arg0BydefVROVrTxNe7F/2p50KLXe8uwgP9s38bnJfwXWhh7yvsq7nn0h?=
 =?us-ascii?Q?1EPsOfh4B3MVrmQ9PHN3NEr9ZKnpF3iTe2ovM9bM4+RK0lFE+gSpCIiyV3YD?=
 =?us-ascii?Q?UW+E0FFmN6P5Um6H6K71cDwart7ZwGTHNmaZt6ary7meT4QkjVsWS8qUORgW?=
 =?us-ascii?Q?HCRvFPnVC373FeZPgJmOadh8KLPZk8pbSX08UZWA74D2e1tCAfdCcKuoeqtN?=
 =?us-ascii?Q?BQHKbRAT2BpIOsBSigYA1/ubCZd62eiQF5Mp2WPDZ6G2UM/jGjIP71G17Drr?=
 =?us-ascii?Q?/qFdvgXRKV8IKYWhQNyEK86KCoRaOR600e2FuvrqkMuKw0wmmY4KgtfnFlmq?=
 =?us-ascii?Q?CMPrD/c+JB1F5JilWpHstbEf+X6jG7JUDBIVOdnQZupaFklGkbGqZ3NI5aMZ?=
 =?us-ascii?Q?/0RwTUniqt3K1uLH2qsRem35X5pXLybADGzCx8fCcBwgCTxolRgkRfdjgXh9?=
 =?us-ascii?Q?yPrXnkWP7c58uqFujJysq4AIuK0McDVK33B0Mm0noUf5Bm3bDyt92q9xiITW?=
 =?us-ascii?Q?afJQBjeRKj0j++6wMrbTQiDRz5DfmCpNEfl53eeHHH9NpReuMSRY9YwTiSpr?=
 =?us-ascii?Q?Q56WWlF8wP5JNbFqv3osG6fZu73hcEl9f0QrsKcLRqfCXFs0xyepQcVCEwPy?=
 =?us-ascii?Q?510MAJALKyX65iRRUN6NfsNHJgC0vj1gBU5enZlvohEeZJkVC3GkWiH6lWtU?=
 =?us-ascii?Q?2AoggwugGS4ZsTUgGu6bHDHZnVJGRaKOVJoBWzhbx2jBg5K5jq+S68w96H2O?=
 =?us-ascii?Q?MhrFtOf6iNKr+4bQd4hjWXAPe/0hlqPX5PDispNZNWtyE7W72kA1gPK5Ly9l?=
 =?us-ascii?Q?lNufcrq0kUcR0U91/4ufxmyBKD1cmjIbo++C4pNJq89eoGoLMIjGKERXqZz2?=
 =?us-ascii?Q?DPfbk/sF0aYZMhp65tib2nlXLuadzNpPe5pdXu7poYfTn98FQ3y9KydzOPo2?=
 =?us-ascii?Q?PFPl/eBS6oQtWVF2OnQ=3D?=
X-OriginatorOrg: suse.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 1c9ab4c1-0ac6-4b31-7743-08db2961973e
X-MS-Exchange-CrossTenant-AuthSource: VI1PR0402MB3439.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 20 Mar 2023 16:38:53.1892
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: f7a17af6-1c5c-4a36-aa8b-f5be247aa4ba
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: N9V+sOZbR/xz2HXgnM/AAu0jFlMrUb8UebwC8jvODEFvmWbBhCldAlrLXRFD0fGR
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DU0PR04MB9274
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hello,

This patch series contains some improvements for s32 pinctrl drivers suggested
by upstream[1], such as
  - Fix error shadowings and improve return value handlings.
  - Fix print format.
  - Remove unnecessary blanks.
  - Use proper macros and helpers to simplify codes.
  - Refactor config param parsing and remove config arguments that are never used.
  - Use generic struct pingroup and struct pinfunction to describe pin data.

Regards,
Chester

[1] https://lore.kernel.org/all/20230220023320.3499-1-clin@suse.com/

Changes in v2:
- Use of_device_get_match_data() to get matched of_device_id data.
- Enhance sizeof() arguments.
- Fix blanks and remove unnecessary parentheses.
- Drop unnecessary marcos and s32_pin_config() implemented in v1 and set/clear
  mask/config values transparently.
- Put pull-function related cases together in s32_pin_set_pull().
- Simply use generic 'struct pinfunction' rather than having extra 'struct
  s32_pmx_func'.

Chester Lin (4):
  pinctrl: s32: use of_device_get_match_data() to get device data
  pinctrl: s32: refine error/return/config checks and simplify driver
    codes
  pinctrl: s32cc: refactor pin config parsing
  pinctrl: s32cc: embed generic struct pingroup and pinfunction

 drivers/pinctrl/nxp/pinctrl-s32.h   |  26 +--
 drivers/pinctrl/nxp/pinctrl-s32cc.c | 261 +++++++++++++++-------------
 drivers/pinctrl/nxp/pinctrl-s32g2.c |  14 +-
 3 files changed, 152 insertions(+), 149 deletions(-)

-- 
2.37.3