* [PATCH 0/4] Add Samsung system manager and secure service
[not found] <CGME20220713045143epcms1p5483300a3cf9ed97308a1bcfe5b889a02@epcms1p5>
@ 2022-07-13 4:51 ` Dongjin Yang
2022-07-13 7:42 ` Krzysztof Kozlowski
0 siblings, 1 reply; 2+ messages in thread
From: Dongjin Yang @ 2022-07-13 4:51 UTC (permalink / raw)
To: jesper.nilsson, lars.persson, robh+dt, krzk+dt, lee.jones
Cc: javierm, bjorn.andersson, linux-arm-kernel, devicetree,
linux-kernel, Moon-Ki Jun, Sang Min Kim, Wangseok Lee,
Dongjin Yang
Add Samsung System manager driver and Secure Service code.
Samsung System manager is for SoCs produced by Samsung Foundry
to provide system read/write request by System Manager API.
Samsung Secure service is for SoCs produced by Samsung Foundry
to provide secure monitor service using Trusted Foundations.
Dongjin Yang (4):
dt-bindings: firmware: Add bindings for Samsung smc
firmware: Samsung: Add secure monitor driver
dt-bindings: mfd: Add bindings for Samsung SysMgr
mfd: Samsung: Add Samsung sysmgr driver
.../bindings/firmware/samsung,smccc-svc.yaml | 31 ++++
.../devicetree/bindings/mfd/samsung,sys-mgr.yaml | 42 ++++++
MAINTAINERS | 6 +
drivers/firmware/Kconfig | 11 ++
drivers/firmware/Makefile | 1 +
drivers/firmware/samsung-smc-svc.c | 154 +++++++++++++++++++
drivers/mfd/Kconfig | 11 ++
drivers/mfd/Makefile | 1 +
drivers/mfd/samsung-sysmgr.c | 167 +++++++++++++++++++++
include/linux/firmware/samsung-smc-svc.h | 59 ++++++++
include/linux/mfd/samsung-sysmgr.h | 30 ++++
11 files changed, 513 insertions(+)
create mode 100644 Documentation/devicetree/bindings/firmware/samsung,smccc-svc.yaml
create mode 100644 Documentation/devicetree/bindings/mfd/samsung,sys-mgr.yaml
create mode 100644 drivers/firmware/samsung-smc-svc.c
create mode 100644 drivers/mfd/samsung-sysmgr.c
create mode 100644 include/linux/firmware/samsung-smc-svc.h
create mode 100644 include/linux/mfd/samsung-sysmgr.h
--
2.9.5
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH 0/4] Add Samsung system manager and secure service
2022-07-13 4:51 ` [PATCH 0/4] Add Samsung system manager and secure service Dongjin Yang
@ 2022-07-13 7:42 ` Krzysztof Kozlowski
0 siblings, 0 replies; 2+ messages in thread
From: Krzysztof Kozlowski @ 2022-07-13 7:42 UTC (permalink / raw)
To: dj76.yang, jesper.nilsson, lars.persson, robh+dt, krzk+dt, lee.jones
Cc: javierm, bjorn.andersson, linux-arm-kernel, devicetree,
linux-kernel, Moon-Ki Jun, Sang Min Kim, Wangseok Lee
On 13/07/2022 06:51, Dongjin Yang wrote:
> Add Samsung System manager driver and Secure Service code.
> Samsung System manager is for SoCs produced by Samsung Foundry
> to provide system read/write request by System Manager API.
> Samsung Secure service is for SoCs produced by Samsung Foundry
> to provide secure monitor service using Trusted Foundations.
>
1. You need to provide users of these drivers. For the
syscon-regmap-export driver (better name than manager which does not
manager anything), having a user would answer the question why such
weird driver is needed. Since there is no user and driver's behavior
breaks design choices, it has to be NAKed.
2. Thread your submissions properly. Git send email does it
automatically for you. You can use other tools, but then be sure your
patches are properly threaded. This patchset was not which makes review
more difficult than it should.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2022-07-13 7:42 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <CGME20220713045143epcms1p5483300a3cf9ed97308a1bcfe5b889a02@epcms1p5>
2022-07-13 4:51 ` [PATCH 0/4] Add Samsung system manager and secure service Dongjin Yang
2022-07-13 7:42 ` Krzysztof Kozlowski
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.