linux-kernel - Re: [RFC PATCH 09/10] i2c: Remove I2C

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAMuHMdXLM-eeAa9=YwkU6gcwmD60Wba5v=F6PU20QMqzxRbM4w@mail.gmail.com>
Date: Thu, 14 Nov 2024 15:37:15 +0100
From: Geert Uytterhoeven <geert@...ux-m68k.org>
To: Michael Ellerman <mpe@...erman.id.au>
Cc: linuxppc-dev@...ts.ozlabs.org, linux-kernel@...r.kernel.org, arnd@...db.de, 
	Linux I2C <linux-i2c@...r.kernel.org>
Subject: Re: [RFC PATCH 09/10] i2c: Remove I2C_HYDRA

CC linux-i2c

On Thu, Nov 14, 2024 at 2:11 PM Michael Ellerman <mpe@...erman.id.au> wrote:
> The i2c-hydra driver depends on PPC_CHRP which has now been removed,
> remove the driver also.
>
> Signed-off-by: Michael Ellerman <mpe@...erman.id.au>

Acked-by: Geert Uytterhoeven <geert@...ux-m68k.org>

> ---
>  drivers/i2c/busses/Kconfig     |  12 ---
>  drivers/i2c/busses/Makefile    |   1 -
>  drivers/i2c/busses/i2c-hydra.c | 150 ---------------------------------
>  3 files changed, 163 deletions(-)
>  delete mode 100644 drivers/i2c/busses/i2c-hydra.c
>
> diff --git a/drivers/i2c/busses/Kconfig b/drivers/i2c/busses/Kconfig
> index 6b3ba7e5723a..cbd072955e45 100644
> --- a/drivers/i2c/busses/Kconfig
> +++ b/drivers/i2c/busses/Kconfig
> @@ -373,18 +373,6 @@ endif # ACPI
>  comment "Mac SMBus host controller drivers"
>         depends on PPC_CHRP || PPC_PMAC
>
> -config I2C_HYDRA
> -       tristate "CHRP Apple Hydra Mac I/O I2C interface"
> -       depends on PCI && PPC_CHRP
> -       select I2C_ALGOBIT
> -       help
> -         This supports the use of the I2C interface in the Apple Hydra Mac
> -         I/O chip on some CHRP machines (e.g. the LongTrail).  Say Y if you
> -         have such a machine.
> -
> -         This support is also available as a module.  If so, the module
> -         will be called i2c-hydra.
> -
>  config I2C_POWERMAC
>         tristate "Powermac I2C interface"
>         depends on PPC_PMAC
> diff --git a/drivers/i2c/busses/Makefile b/drivers/i2c/busses/Makefile
> index ecc07c50f2a0..ab366ce6f15c 100644
> --- a/drivers/i2c/busses/Makefile
> +++ b/drivers/i2c/busses/Makefile
> @@ -32,7 +32,6 @@ obj-$(CONFIG_I2C_VIAPRO)      += i2c-viapro.o
>  obj-$(CONFIG_I2C_ZHAOXIN)      += i2c-viai2c-zhaoxin.o i2c-viai2c-common.o
>
>  # Mac SMBus host controller drivers
> -obj-$(CONFIG_I2C_HYDRA)                += i2c-hydra.o
>  obj-$(CONFIG_I2C_POWERMAC)     += i2c-powermac.o
>
>  # Embedded system I2C/SMBus host controller drivers
> diff --git a/drivers/i2c/busses/i2c-hydra.c b/drivers/i2c/busses/i2c-hydra.c
> deleted file mode 100644

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@...ux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds