[PATCH 3/3] [v6] pinctrl: qcom: qdf2xxx: add support for new ACPI HID QCOM8002
Timur Tabi
timur at codeaurora.org
Thu Jan 4 07:46:18 PST 2018
On 12/21/2017 07:46 PM, Stephen Boyd wrote:
> Ok. That's testable with acpi_match_device_ids() though. I can
> add that into pinctrl-msm.c so we don't have to pass info about
> available gpios from ACPI specific driver into the pinctrl-msm
> core driver. That's why I'm trying to avoid doing it in the ACPI
> specific driver. Do it close to where the gpiochip is created
> instead.
I guess we're just going to have to agree to disagree. I see the DSDs
as being SOC-specific, and therefore belong in the SOC driver. I'm
still going to need an SOC driver to define the TLMM register layout.
But as I said earlier, I've already spent way too much time working on a
driver that, in all likelihood, never be used in any production system
anyway.
I look forward to reviewing the next version of your patch.
> Maybe future HIDs could follow the DT design and then we can look
> for the same device property name in both firmwares.
DSDs generally don't have the vendor prefix that DT properties do.
> Parsing
> ranges is simpler.
I'm not sure I agree with that.
--
Qualcomm Datacenter Technologies, Inc. as an affiliate of Qualcomm
Technologies, Inc. Qualcomm Technologies, Inc. is a member of the
Code Aurora Forum, a Linux Foundation Collaborative Project.
More information about the linux-arm-kernel
mailing list