> Date: Mon, 7 Sep 2020 19:59:13 +0200 > From: Marcus Glocker <[email protected]> > > On Mon, 7 Sep 2020 19:25:00 +0200 (CEST) > Mark Kettenis <[email protected]> wrote: > > > > Date: Mon, 7 Sep 2020 12:02:15 -0500 > > > From: joshua stein <[email protected]> > > > > > > On Mon, 07 Sep 2020 at 06:58:01 +0200, Marcus Glocker wrote: > > > > This is an initial driver for the Apple System Management > > > > Controller found in Intel based Apple computers. > > > > > > > > The driver is currently missing support for the Sudden Motion > > > > Sensor (SMS), light sensor, and keyboard backlight since I don't > > > > have that hardware available to develop on. > > > > > > > > On my iMac11,2 it can deliver fan and temperatures values: > > > > > > > > hw.sensors.acpiapplesmc0.temp0=24.00 degC (Airflow 1) > > > > hw.sensors.acpiapplesmc0.temp1=33.00 degC (CPU Core 0) > > > > hw.sensors.acpiapplesmc0.temp2=36.00 degC (CPU Heatsink) > > > > hw.sensors.acpiapplesmc0.temp3=40.00 degC (CPU Core 1) > > > > hw.sensors.acpiapplesmc0.temp4=47.00 degC (GPU) > > > > hw.sensors.acpiapplesmc0.temp5=45.00 degC (GPU Heatsink) > > > > hw.sensors.acpiapplesmc0.temp6=59.00 degC (PCH) > > > > hw.sensors.acpiapplesmc0.temp7=42.00 degC (Memory) > > > > hw.sensors.acpiapplesmc0.temp8=45.00 degC (Mainboard > > > > Proximity) hw.sensors.acpiapplesmc0.fan0=998 RPM > > > > hw.sensors.acpiapplesmc0.fan1=1132 RPM > > > > hw.sensors.acpiapplesmc0.fan2=1198 RPM > > > > > > > > Feedback, testers, OKs? > > > > > > Are there machines where asmc(4) will also attach? > > > > Good point. My old Macmini1,1 has: > > > > ... > > "APP0001" at acpi0 not configured > > ... > > asmc0 at isa0 port 0x300/32: rev 1.3f503, 137 keys > > ... > > > > So yes, I'd say there are. > > > > > > Having an acpi attachment is probably better than doing isa probes. > > But we probably should consolidate the drivers. > > D'oh! I wasn't even aware that we already have an asmc(4) driver in our > tree. Shame on me :-| > > Glancing over asmc(4) I don't think there is anything more that my > driver would support other than attaching over acpi(4). Would it be > possible to only write an acpi glue which attaches to asmc(4)?
I think we'd just want to turn it into an acpi(4) driver. Or maybe dump it in favour of your driver.
