[PATCH] gpio: slg7xl45106: Add support for slg7xl45106 i2c gpo expander

Michal Simek michal.simek at xilinx.com
Tue Mar 1 16:29:40 CET 2022



On 3/1/22 15:58, Simon Glass wrote:
> Hi Michal,
> 
> On Wed, 23 Feb 2022 at 08:21, Michal Simek <michal.simek at xilinx.com> wrote:
>>
>> From: T Karthik Reddy <t.karthik.reddy at xilinx.com>
>>
>> slg7xl45106 is i2c based 8-bit gpo expander, gpo pins are set and get by
>> writing and reading corresponding gpo bit value into its data register.
>>
>> Signed-off-by: T Karthik Reddy <t.karthik.reddy at xilinx.com>
>> Signed-off-by: Michal Simek <michal.simek at xilinx.com>
>> ---
>>
>>   MAINTAINERS                     |   1 +
>>   drivers/gpio/Kconfig            |   8 +++
>>   drivers/gpio/Makefile           |   1 +
>>   drivers/gpio/gpio_slg7xl45106.c | 115 ++++++++++++++++++++++++++++++++
>>   4 files changed, 125 insertions(+)
>>   create mode 100644 drivers/gpio/gpio_slg7xl45106.c
> 
> You might consider reducing the length of the 'slg7xl45106_i2c_gpo' prefix.
> 
> Also did you know about dm_i2c_reg_clrset() ?

nope but logic there is a little bit different compare to what I need to do.
It can be done but this I can't see this as a problem.

> 
> Reviewed-by: Simon Glass <sjg at chromium.org>

Thanks,
Michal


More information about the U-Boot mailing list